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
opencollada: include pcre.h (KhronosGroup/OpenCOLLADA#570) #44126
opencollada: include pcre.h (KhronosGroup/OpenCOLLADA#570) #44126
Conversation
attached patch fails against latest master
Looking at it, there seem to be some superflous whitespace changes in the attached patch. Maybe the merge conflict can be resolved, by just adding the PCRE include? Could you link the original ticket(s), for reference? |
90bbd16
to
e9198c8
Compare
Does the new commit work? I guess I successfully isolated the hunk, though if I try to change the file endings in the hunk, it fails to get patched on mine, which results in the patch having unix line endings for the file but dos line endings in the hunk. (Original Issue: KhronosGroup/OpenCOLLADA#570) |
This comment has been minimized.
This comment has been minimized.
@GrahamcOfBorg build opencollada |
Failure on x86_64-darwin (full log) Attempted: opencollada Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: opencollada Partial log (click to expand)
|
Does the build on darwin build without this PR? Don't have a machine with macOS currently installed so can't really test darwin builds. |
Yeah macOS is broken too so it won't be a blocker. |
@@ -21,6 +21,8 @@ stdenv.mkDerivation rec { | |||
|
|||
enableParallelBuilding = true; | |||
|
|||
patches = [ ./pcre.patch ]; | |||
|
|||
patchPhase = lib.optionalString stdenv.isDarwin '' |
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.
@eadwu Change this to postPatch and it should fix macOS building.
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.
Oh didn't see patchPhase, just added it to patchPhase
Success on x86_64-linux (full log) Attempted: opencollada Partial log (click to expand)
|
e9198c8
to
6f62bf3
Compare
6f62bf3
to
2df3073
Compare
I just pushed a commit showing you what is preferred. It's a little bit hard to explain but usually using @GrahamcOfBorg build opencollada |
Success on x86_64-linux (full log) Attempted: opencollada Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: opencollada Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: opencollada Partial log (click to expand)
|
2df3073
to
6f62bf3
Compare
Thanks! |
Motivation for this change
Fixes build (or at least on nixOS)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)