Skip to content

libpulseaudio: fix aarch64-darwin build#152371

Merged
bobby285271 merged 1 commit intoNixOS:masterfrom
mikroskeem:darwin-fix-libpulseaudio-aarch64
Jan 2, 2022
Merged

libpulseaudio: fix aarch64-darwin build#152371
bobby285271 merged 1 commit intoNixOS:masterfrom
mikroskeem:darwin-fix-libpulseaudio-aarch64

Conversation

@mikroskeem
Copy link
Member

Removes darwin Libc buildInput, which does not appear to be needed on
x86_64, and which does not support aarch64 at all (too old)

Motivation for this change
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • 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.

Removes darwin Libc buildInput, which does not appear to be needed on
x86_64, and which does not support aarch64 at all (too old)
@mikroskeem mikroskeem force-pushed the darwin-fix-libpulseaudio-aarch64 branch from 1b6592d to e9f4ecd Compare December 27, 2021 20:52
@mikroskeem mikroskeem changed the title pulseaudio: fix aarch64-darwin build libpulseaudio: fix aarch64-darwin build Dec 27, 2021
@mikroskeem mikroskeem requested review from lovek323 and toonn December 28, 2021 14:40
Copy link
Contributor

@toonn toonn left a comment

Choose a reason for hiding this comment

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

LGTM

Result of nixpkgs-review pr 152371 run on x86_64-darwin 1

1 package built:
  • libpulseaudio

@ofborg ofborg bot added 6.topic: darwin Running or building packages on Darwin 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 2501-5000 This PR causes many rebuilds on Linux and should target the staging branches. labels Dec 29, 2021
@bobby285271 bobby285271 merged commit f4ace5e into NixOS:master Jan 2, 2022
@bobby285271
Copy link
Member

bobby285271 commented Jan 2, 2022

Wait... I think this should target staging...?

Edit: Reverted in #153176, @mikroskeem I am very sorry I didn't check that. Would you like to cherry-pick this and open a new PR that target staging? I think the reason for the mass rebuild to happen on linux is similar with #142091 (NIX_CFLAGS_COMPILE = null not equivalent to NIX_CFLAGS_COMPILE = "").

@mikroskeem
Copy link
Member Author

Sure thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: darwin Running or building packages on Darwin 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 2501-5000 This PR causes many rebuilds on Linux and should target the staging branches.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants