New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
libsForQt5.qtwebengine: use ffmpeg instead of ffmpeg_3 #120646
Conversation
@@ -159,7 +159,7 @@ qtModule { | |||
|
|||
libevent | |||
] ++ optionals (stdenv.hostPlatform.isAarch32 || stdenv.hostPlatform.isAarch64) [ | |||
ffmpeg_3 | |||
ffmpeg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does anyone know why this is only used on ARM?
Do we use a bundled Ffmpeg on other platforms? If so, we shouldn't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at qmakeFlags above:
qmakeFlags = if stdenv.hostPlatform.isAarch32 || stdenv.hostPlatform.isAarch64
then [ "--" "-system-ffmpeg" ] ++ optional enableProprietaryCodecs "-proprietary-codecs"
else optional enableProprietaryCodecs "-- -proprietary-codecs";
Maybe we shouldn't? Looks like it was changed in e01c613, but only done for aarch64 to avoid a rebuild.
It's interesting that OfBorg claims this causes 0 rebuilds...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we shouldn't? Looks like it was changed in e01c613, but only done for aarch64 to avoid a rebuild.
Yes, let's use system ffmpeg on all platforms.
It's interesting that OfBorg claims this causes 0 rebuilds...
That's because ofborg only checks on x86_64.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, ofborg checks aarch64-linux too. The ^.^
check includes the list of changed paths.
The reported count, though, might be x86_64 only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reported count, though, might be x86_64 only.
That's what I meant.
Can someone explain why we're using |
I can play everything on https://tekeye.uk/html/html5-video-test-page in qutebrowser on x86_64-linux. Any objections to merging this? |
This still bugs me. |
Motivation for this change
#94003 and #120372
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)I did not test this cause I don't have an aarch64 machine that's powerful enough.
cc @NixOS/qt-kde