Skip to content

dconf: enable cross compilation#275732

Merged
Mindavi merged 1 commit intoNixOS:masterfrom
uninsane:pr-dconf-ed-cross
Feb 13, 2024
Merged

dconf: enable cross compilation#275732
Mindavi merged 1 commit intoNixOS:masterfrom
uninsane:pr-dconf-ed-cross

Conversation

@uninsane
Copy link
Contributor

@uninsane uninsane commented Dec 20, 2023

Description of changes

combined with #267550, this enables gnome.dconf-editor to cross compile.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@uninsane uninsane requested a review from jtojnar as a code owner December 20, 2023 22:30
@github-actions github-actions bot added the 6.topic: GNOME GNOME desktop environment and its underlying platform label Dec 20, 2023
@uninsane uninsane changed the title gnome.dconf-editor: enable cross compilation dconf: enable cross compilation Dec 20, 2023
@github-actions github-actions bot removed the 6.topic: GNOME GNOME desktop environment and its underlying platform label Dec 20, 2023
@ofborg ofborg bot added 6.topic: cross-compilation Building packages on a different platform than they will be used on 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Dec 21, 2023
Copy link
Contributor

@Mindavi Mindavi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the right approach to me, and good to get rid of the vala and introspection hacks.

@Mindavi Mindavi merged commit e4a5140 into NixOS:master Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: cross-compilation Building packages on a different platform than they will be used on 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments