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
qtcreator: enable on aarch64 and armv7 #45312
Conversation
Patch (from https://bugreports.qt.io/browse/QTCREATORBUG-8107) to allow botan to build on arm.
@GrahamcOfBorg build qtcreator |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: qtcreator Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: qtcreator Partial log (click to expand)
|
Timed out, unknown build status on aarch64-linux (full log) Attempted: qtcreator Partial log (click to expand)
|
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.
Sounds OK to me. Build by hand on aarch64.nixos.community
succeeded in 70–80 minutes.
@@ -23,7 +23,7 @@ stdenv.mkDerivation rec { | |||
|
|||
nativeBuildInputs = [ qmake makeWrapper ]; | |||
|
|||
patches = optional (stdenv.isAarch64 || stdenv.isAarch32) ./0001-Fix-Allow-qt-creator-to-build-on-arm-aarch32-and-aar.patch; | |||
patches = optional stdenv.hostPlatform.isArm ./0001-Fix-Allow-qt-creator-to-build-on-arm-aarch32-and-aar.patch; |
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.
It looks like isArm is a synonym for isAarch32, so this doesn't apply the patch on aarch64 when it should.
stdenv.hostPlatform.isArm is a synonym for stdenv.hostPlatform.isAarch32, so it doesn't apply the patch on aarch64 and the build fails: https://hydra.nixos.org/build/80107778 As this has been merged, should I open a new pull request to fix it? |
Right, I'm sorry for the mistake, thanks. |
Patch (from https://bugreports.qt.io/browse/QTCREATORBUG-8107) to allow botan to build on arm.
Motivation for this change
QtCreator can be built and run on aarch64 and aarch32, but one of its components, botan, requires patching. Patch taken from https://codereview.qt-project.org/#/c/79728/.
Things done
Added patch to fix qtcreator build on aarch64 and aarch32. Added these architectures to the supported platforms.
N.B. Believe it should work on aarch32, but have only tested on aarch64
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @akaWolf
I haven't yet looked into whether qtcreator's copy of botan differs significantly from the botan in pkgs/development/libraries/botan. It might make sense to patch that and make qtcreator use system botan instead of its bundled botan.