[python-updates] 2023/12/02 major updates (python312Packages, pythonRuntimeDepsCheckHook)#271586
[python-updates] 2023/12/02 major updates (python312Packages, pythonRuntimeDepsCheckHook)#271586mweinelt merged 818 commits intostaging-nextfrom
Conversation
| metadata = get_metadata(args.wheel) | ||
| tests = [test_requirement(requirement) for requirement in metadata.requires_dist] | ||
|
|
||
| if not all(tests): |
There was a problem hiding this comment.
We shoud list some of the possible options for people here like buildNpmPackage
There was a problem hiding this comment.
Please take some time to make your review comments more legible.
I think you are asking for more actionable feedback to the user. I can do that, but buildNpmPackage offers these 3 hints, because the way to resolve things are much more unclear.
The only thing I would recommend is pythonRelaxDepsHook, if version constraints are too tight.
There was a problem hiding this comment.
You got exactly what I wanted to say, no need to write romans. :)
We could print something like
Please try to use pythonRelaxDepsHook to release the version constraints if the library allows it and tests continue to pass.
If the hook does not work for some reason, please try to fall back to manually patching in postPatch.
There was a problem hiding this comment.
You got exactly what I wanted to say, no need to write romans. :)
It took me more than a minute to grok though, and that happens more often than you think.
There was a problem hiding this comment.
hmmmm, I try to remember that in the future but that's just my normal writing style. I am not leaving things out on purpose.
e5af2a4 to
fa00e71
Compare
fa00e71 to
ca33ecd
Compare
ecb5694 to
b89d6c9
Compare
c859349 to
0151fcd
Compare
|
Report for the latest eval on my private hydra, if someone wants to dig in. |
5bbba79 to
2f42d7c
Compare
558efe4 to
72d817c
Compare
|
Bisect says 536b7d1 |
|
On mobile right now, but a fix was referenced in the staging-next PR. |
|
It looks like |
|
|
|
@mweinelt , I found this commit merged as a part of this PR updating BTW, I am one of the maintainers of this package and I haven't seen any notification about the change. |
This should have only been included if they didn't mark that tag as a |
|
although, looked at that branch... and it's from 9 years ago, not sure why it did get included.... the release metadata should be getting fetched from github. |
|
We arrive at the following using https://api.github.com/repos/rasterio/rasterio/releases and because we determine the highest version in |
|
xformers upgrade is broken here: Version 0.03 exists, but this is a typo in upstream repository: And in fact, xformers version is downgraded here |
This was added in NixOS#271586, but never used due to the spelling mistake of a double s.
This was added in NixOS#271586, but never used due to the spelling mistake of a double s.


This is a bulk update of python packages, that includes major version bumps.
TODO
pythonRuntimDepsCheckHook(see python3.pkgs.pythonRuntimeDepsCheckHook: init #270457)python312Packages, stop recursing intopython310PackagesSETUPTOOLS_SCM_PRETEND_VERSIONOnce the evaluation has finished, jobs will start building and breakage can be reviewed over at https://hydra.nixos.org/jobset/nixpkgs/python-updates.
Additionally, a review report will appear at https://malob.github.io/nix-review-tools-reports.
Known Issues:
Latest eval report
https://shells.darmstadt.ccc.de/~hexa/93.html