Skip to content

libcaca: fix build on Darwin#347014

Merged
reckenrode merged 1 commit intoNixOS:masterfrom
reckenrode:push-mnntwmkpwqvw
Oct 8, 2024
Merged

libcaca: fix build on Darwin#347014
reckenrode merged 1 commit intoNixOS:masterfrom
reckenrode:push-mnntwmkpwqvw

Conversation

@reckenrode
Copy link
Contributor

@reckenrode reckenrode commented Oct 7, 2024

libcaca tries to access a private API that is no longer available. It worked before the SDK changes in #346043 because no Darwin frameworks were available in the build environment, but once an SDK is always available, libcaca tries to build Cocoa code that no longer works.

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.11 Release Notes (or backporting 23.11 and 24.05 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.

libcaca tries to access a private API that is no longer available. It
worked before the SDK changes because no Darwin frameworks were
available in the build environment, but there is now always an SDK
available, causing it to try to build Cocoa code that no longer works.
@ofborg ofborg bot added the 6.topic: darwin Running or building packages on Darwin label Oct 7, 2024
@ofborg ofborg bot requested a review from AndersonTorres October 7, 2024 06:12
@ofborg ofborg bot added 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Oct 7, 2024
@AndersonTorres
Copy link
Member

Off-topic: which tool are you using that generates this name push-mnntwmkpwqvw on the fork?

@wegank wegank added 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Oct 7, 2024
@emilazy
Copy link
Member

emilazy commented Oct 7, 2024

Off-topic: which tool are you using that generates this name push-mnntwmkpwqvw on the fork?

It’s Jujutsu.

@reckenrode
Copy link
Contributor Author

Jujutsu can generate a branch name from the change id when you push. For small PRs like this, it’s reduces the friction to creating them if one doesn’t have to think of a name for each one. (There were 30+ follow ups I had to create to #346043.)

@reckenrode reckenrode merged commit 4fc3efa into NixOS:master Oct 8, 2024
@reckenrode reckenrode deleted the push-mnntwmkpwqvw branch October 8, 2024 00:27
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: 0 This PR does not cause any packages to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants