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
numpy: fix distutils patch for 1.19.0 #92112
Conversation
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.
nothing looks darwin specific.
but diff LGTM
@jonringer The patch is only applied when python is built with clang. |
Ah, I suspected it was a "darwin-only" code path. but the diff looks good so, I don't think it would be any harm |
@GrahamcOfBorg build python2Packages.numpy |
af2194d
to
bafc8e1
Compare
Changed target to staging-next for #91090 |
@@ -26,6 +25,6 @@ index 6ed5eec..82a88b5 100644 | |||
+ self.spawn(self.compiler_so + cc_args + [src, '-o', obj] + deps + | |||
+ extra_postargs, display = display) | |||
+ | |||
except DistutilsExecError: | |||
msg = str(get_exception()) | |||
except DistutilsExecError as e: |
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.
hope they don't run pylint as part of the tests because pylint doesn't like variables this short ;)
except DistUtilsExecError as exc:
raise CompileError from exc
Any plans for |
Motivation for this change
The
python3Packages.numpy
build is broken on darwin after update to 1.19.0Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)