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
qtwebengine: Use system ffmpeg on aarch64 #45122
Conversation
Maybe it could be done on all platforms? Having the build time of qtwebengine go down would be pretty useful. |
Yes, I wondered about doing it on all platforms. I decided (at least initially) to err on the side of caution and not risk breaking things which currently work. |
@@ -117,7 +118,9 @@ EOF | |||
fi | |||
''; | |||
|
|||
qmakeFlags = optional enableProprietaryCodecs "-- -proprietary-codecs"; |
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.
This is causing a rebuild on other platforms (I think) because the number of list elements changes. Until we decide to use -system-ffmpeg
on all platforms, could you change this so that it does not rebuild on other platforms?
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.
I've just noticed I also made a mistake in my addition to propagatedBuildInputs, which means ffmpeg gets added on other platforms, when it shouldn't.
8488313
to
f59d67d
Compare
I've made a few changes:
|
On aarch64, linking against the vendored ffmpeg fails. Including ffmpeg as a dependency and passing -system-ffmpeg to qmake fixes this. Slightly odd conditional in qmakeFlags to avoid altering the list on non-arm platforms, so that the change doesn't trigger an unneccessary rebuild.
f59d67d
to
e01c613
Compare
Fixed mistake: Replaced |
On aarch64, linking against the vendored ffmpeg fails. Including ffmpeg
as a dependency and passing -system-ffmpeg to qmake fixes this.
Motivation for this change
QtWebengine fails to build on aarch64 due to problems linking against the bundled ffmpeg.
Things done
Added a qmake flag
-system-ffmpeg
when building for Aarch64 or Aarch32Added ffmpeg as an optional dependency
Note: So far, I have only tested Qt 5.11's qtwebengine.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @matthewbauer