Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 750260 - media-video/ffmpeg-4.3.1: USE=vulkan doesn't enable vulkan filters
Summary: media-video/ffmpeg-4.3.1: USE=vulkan doesn't enable vulkan filters
Status: UNCONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal with 1 vote (vote)
Assignee: media-video herd
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-10-19 22:36 UTC by Gregory Beauregard
Modified: 2020-12-02 20:12 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gregory Beauregard 2020-10-19 22:36:57 UTC
ffmpeg now has some new vulkan filters, but the +vulkan USE flag in the current ebuild that does --enable-vulkan is not sufficient to enable them, as --enable-libglslang is also needed

See for example scale_vulkan in the configure file:
scale_vulkan_filter_deps="vulkan libglslang"

https://github.com/FFmpeg/FFmpeg/blob/master/configure
Comment 1 Gregory Beauregard 2020-11-24 04:07:57 UTC
Apparently there's some weirdness with glslang not supporting pkgconfig. Upstream referred me to reverting commit 98ea1a662ee172961feaa374fe8d26078838d250 as fixing building with --enable-libglslang on debian, and that worked for me on Gentoo as well. Will probably need to be inspected what this does and included as a .patch/some other method to fixup the glslang detection for our specific packaging whenever this gets looked at.
Comment 2 Gregory Beauregard 2020-11-28 00:28:20 UTC
(In reply to Gregory Beauregard from comment #1)
> Apparently there's some weirdness with glslang not supporting pkgconfig.
> Upstream referred me to reverting commit
> 98ea1a662ee172961feaa374fe8d26078838d250 as fixing building with
> --enable-libglslang on debian, and that worked for me on Gentoo as well.
> Will probably need to be inspected what this does and included as a
> .patch/some other method to fixup the glslang detection for our specific
> packaging whenever this gets looked at.

So as far as I know, a patch that reverts this commit (it just removes a couple things from a line), a dev-util/glslang dependency, and a dev-util/spirv-tools dependency are what's needed here. The spirv-tools dependency isn't obvious, but it's required.