-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
feedbackd: support cross-compilation #175998
Conversation
Disables introspection for now.
So yeah, this all seems correct, but the wrapGAppsHook produces a binary for x86-64... 😞. See #175045. Guess we can still merge this anyway, but another fix is needed to actually make it fully work. |
So it looks like cross-compilation and gobject introspection are mutually exclusive, until #72868 is fixed. The pro case for the status quo of keeping introspection:
Cross-compilation is important to have, but is it worth losing the pros of keeping introspection? FWIW, I think it's common for mobile NixOS users to compile natively (either on an existing aarch64 device, or via binfmt emulation). |
To me it wouldn't make sense that the option of disabling introspection would be available if it broke everything, but that may be naive. I agree that keeping simple would be nice. I don't agree that it aligns with upstream if they provide the option. Some progress is being made on improving the gir situation! #176464 So we may be able to remove the disables in the near future. Personally I like to be able to cross-compile, even if it reduces functionality a bit. I do try it out from time to time and am aiming to get a full-blown image (with DM) working at some point, just because I like to prove it's possible and to be able to use my own hardware to build the image. But of course it would be nice to have more functionality. Thanks for keeping the discussion alive btw, it's good that we think about these choices. |
SGTM. I see low risk of breaking usecases, and since feedbackd is typically used on aarch64, I can see how cross-compilation would be convenient. Have you tried compilation via binfmt? That should work, even without cross-compilation. |
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
Successfully created backport PR #176942 for |
Disables introspection for now.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)