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
python3Packages.tokenizers: 0.9.4 -> 0.10.0 #109532
Conversation
seems to fail for me locally:
|
@@ -105,6 +107,11 @@ in rustPlatform.buildRustPackage rec { | |||
pipInstallPhase | |||
''; | |||
|
|||
disabledTests = [ |
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.
disabledTests = [ | |
preCheck = '' | |
HOME=$TMPDIR | |
''; | |
disabledTests = [ |
/nix/store/yi9a1fafijn0nf5m8qkmnl955vi2mncm-python3-3.8.6/lib/python3.8/os.py:213: in makedirs
makedirs(head, exist_ok=exist_ok)
/nix/store/yi9a1fafijn0nf5m8qkmnl955vi2mncm-python3-3.8.6/lib/python3.8/os.py:223: in makedirs
mkdir(name, mode)
E PermissionError: [Errno 13] Permission denied: '/homeless-shelter'
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.
Thanks, and sorry for the delay (busy week). Should be fixed now.
Result of 4 packages failed to build:
|
Result of 4 packages failed to build:
|
168998d
to
cca0b01
Compare
this breaks python3Packages.transformers. Presumably because of overly restrictive pinning, but as maintainer you probably know better. |
Indeed, transformers pins tokenizers to a specific version. We do relax the bounds in transformers, but it was too specific. Now also fixed as part of #109505 . Thanks! |
Motivation for this change
Changelog:
https://github.com/huggingface/tokenizers/releases/tag/python-v0.10.0
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)