Skip to content
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

meson: apply patch fixing exception during cross-compilation #168753

Merged
merged 1 commit into from
Apr 21, 2022

Conversation

Mindavi
Copy link
Contributor

@Mindavi Mindavi commented Apr 15, 2022

Fixes a traceback where you get the following error:

  • AttributeError: 'NoneType' object has no attribute 'get_command'

This happened when cross-compiling e.g. harfbuzz.

I did release this on my hydra and didn't see any new regressions with it, so I'm pretty certain it will only improve meson and not cause further issues.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux (cross-compiled harfbuzz)
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 22.05 Release Notes (or backporting 21.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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

Copy link
Member

@NickCao NickCao left a comment

Choose a reason for hiding this comment

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

168 newly succeeding and zero regression on my hydra.

@risicle
Copy link
Contributor

risicle commented Apr 17, 2022

meson builds for me, macos 10.15

Fixes a traceback where you get the following error:
- AttributeError: 'NoneType' object has no attribute 'get_command'

This happened when cross-compiling e.g. harfbuzz.
@Mindavi Mindavi force-pushed the meson/cross-fix-gnome-staging branch from d57723f to e91229b Compare April 19, 2022 14:53
@ofborg ofborg bot requested a review from jtojnar April 19, 2022 16:16
@FRidh FRidh added this to Needs review in Staging Apr 19, 2022
Staging automation moved this from Needs review to Ready Apr 21, 2022
@jtojnar jtojnar merged commit fa6aa9e into NixOS:staging Apr 21, 2022
Staging automation moved this from Ready to Done Apr 21, 2022
@Mindavi Mindavi deleted the meson/cross-fix-gnome-staging branch April 24, 2022 20:01
@ghost ghost mentioned this pull request Apr 29, 2022
9 tasks
@r-burns r-burns mentioned this pull request Apr 30, 2022
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Staging
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

6 participants