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
pypy: 5.6 -> 5.8 #27636
pypy: 5.6 -> 5.8 #27636
Conversation
this commit includes a patch, backported from master, to skip a failing test
@@ -3,13 +3,13 @@ | |||
, makeWrapper, callPackage, self, gdbm, db | |||
, python-setup-hook | |||
# For the Python package set | |||
, pkgs, packageOverrides ? (self: super: {}) | |||
, pkgs, pythonPackages, packageOverrides ? (self: super: {}) |
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.
The packages set is an attribute of the interpreter, python
. Please use that instead.
substituteInPlace "lib-python/2.7/lib-tk/Tix.py" --replace "os.environ.get('TIX_LIBRARY')" "os.environ.get('TIX_LIBRARY') or '${tix}/lib'" | ||
''; | ||
|
||
buildInputs = [ bzip2 openssl pkgconfig python libffi ncurses expat sqlite tk tcl xlibsWrapper libX11 makeWrapper gdbm db ] | ||
++ (with pythonPackages; [ pycparser ]) |
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.
Maybe its good to use python.withPackages
here so that its clear that pycparser
is for that instance of Python.
thanks for the feedback |
Thinking about this, it's actually a regression because now pypy can't be built by pypy anymore ... |
That's fine. We still need to fix
Could you clarify? I do not see any regression. |
e.g. if you wanted to build pypy with pypy: |
Could you clarify? |
You now wrote
Yesterday I committed ea4121d. |
oh, cool, so now I'm able to use |
Yes indeed.
…On Aug 2, 2017 15:21, "Herwig Hochleitner" ***@***.***> wrote:
oh, cool, so now I'm able to use .interpreter on a withPackages - wrapped
python?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#27636 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACB87xFwRNMD_HIIxlmqxZsF54hqL-xPks5sUHfRgaJpZM4OiVmL>
.
|
updated |
Builds fine for me. |
@FRidh Any reason not to merge this? |
this commit includes a patch, backported from master, to skip a
failing test
Motivation for this change
maintenance
Things done
Please check what applies. Note that these are not hard requirements but mereley serve as information for reviewers.
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)