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
pythonPackages.faker: fix build #61110
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.
Build has a requirement for freezegun
and why the build seems to be now failing.
Though this did fix the rm: missing operand
.
@worldofpeace yep I noticed that after a little bit, I've fixed that locally but have now run into a problem that I don't know how to solve. Pushing what I have now. |
You could try feeding the
and if |
I'm partial to just patching this away because it's just for tests. Also note we've already patched the pin for Edit: the pinning for And we use |
patched to ignore the version specifier (with some notes). @worldofpeace PTAL. |
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.
Perfect 👍
Builds again for all interpreters.
Please squash your commits and we can merge.
Not sure why, but the `__pycache__` folder no longer exists by the time `postPatch` runs which now causes the `rm` to error and fail the build.
done @worldofpeace thanks for help! |
You're very welcome. Thanks for contributing ✨ |
Motivation for this change
pythonPackages.faker no longer builds
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)