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
python36Packages.ipython: 7.17 -> 7.16.1 (downgrade on Python 3.6 for compatibility) #98024
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.
To comply with CONTRIBUTING.md please have the commit message name be of the format
<pkg-name>: <subject-line>
for more examples, please look at https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes
in your case, the commit message should be:
python36Packages: disable
cc @FRidh
facb696
to
f746462
Compare
Argh, sorry about that. I've now changed the commit message to
to match your commit ec1dc42 from last year. |
Thanks. We should probably duplicate the expression for 7.16 for Python 3.6. |
Makes sense. Should we also have IPython 7.9 for compatibility with Python 3.5? It looks like we never actually had IPython 7.9, since we jumped from 7.8 to 7.10 in 1f7ee3e. |
3.5 has been dropped from Nixpkgs because its EOL soon |
IPython versions 7.17 and higher don't support Python 3.6 anymore, so this commit adds back the old expression for 7.16.1 from before ba1038a for use in python36Packages only. nix-repl> :b python36.withPackages (ps: [ ps.ipython ]) builder for '/nix/store/q8v4f89xwv35a3idb9z345z6n3nzfycb-python3.6-ipython-7.17.0.drv' failed with exit code 1; last 10 log lines: Python 3.5 was supported with IPython 7.0 to 7.9. Python 3.6 was supported with IPython up to 7.16. See IPython `README.rst` file for more information: https://github.com/ipython/ipython/blob/master/README.rst Python sys.version_info(major=3, minor=6, micro=12, releaselevel='final', serial=0) detected. cannot build derivation '/nix/store/mhpdarp18z6skzswrl7sbgzv8hr4gwih-python3-3.6.12-env.drv': 1 dependencies couldn't be built [0 built (1 failed)] error: build of '/nix/store/mhpdarp18z6skzswrl7sbgzv8hr4gwih-python3-3.6.12-env.drv' failed
f746462
to
c3618b0
Compare
Ah, I didn't notice that, thanks. (I still see I've added a duplicate expression for 7.16 and plugged it into |
Motivation for this change
IPython 7.17 and higher don't support Python 3.6 anymore.
Things 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)