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
added macOS compatibility to portmidi-3.17 #24855
Conversation
@plumps, thanks for your PR! By analyzing the history of the files in this pull request, we identified @aszlig, @wkennington and @fpletz to be potential reviewers. |
url = "https://raw.github.com/Rogentos/argent-gentoo/master/media-libs/" | ||
+ "portmidi/files/portmidi-217-cmake-libdir-java-opts.patch"; | ||
sha256 = "1jbjwan61iqq9fqfpq2a4fd30k3clg7a6j0gfgsw87r8c76kqf6h"; | ||
}); | ||
}) ++ optional stdenv.isDarwin ./darwin.patch; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you forgot to commit this file
error: getting status of ‘/nix/store/hljk2wdrny0rzdx9krl1zrqh2disv0ha-git-export/pkgs/development/libraries/portmidi/darwin.patch’: No such file or directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you are so right o.O., fixed that
substituteInPlace ./${path}/CMakeLists.txt --replace "CoreMIDI.framework" \ | ||
"${CoreMIDI}/Library/Frameworks/CoreMIDI.framework" | ||
substituteInPlace ./${path}/CMakeLists.txt --replace "CoreServices.framework" \ | ||
"${CoreServices}/Library/Frameworks/CoreServices.framework" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to replace the same things as the patch, why not replace it directly here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To clarify, I think something like this should make most of the patch file redundant.
substituteInPlace pm_dylib/CMakeLists.txt \
--replace '${FRAMEWORK_PATH}/CoreAudio.framework' "${CoreAudio}/Library/Frameworks/CoreAudio.framework"
...
@plumps are you still working on this? |
@c0bw3b nope |
@NixOS/darwin-maintainers does someone feel like resuming/completing this or should we close for now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See @LnL7's suggestions
Stalled |
Motivation for this change
Needed portmidi for pygame. Haven't changed a thing about the missing java builds.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)