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
Update scrapy & dependencies #22593
Update scrapy & dependencies #22593
Conversation
cc @jozko |
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.
haven't tested it but it looks good
@drewkett you ok with these changes? |
Looks fine to me, though I'd probably add a comment to the nix file explaining why the permissions patch is needed rather than having that information only be in a commit message. I'm not using this package at the moment, so feel free to take over as maintainer if you'd like. |
I left you in as the maintainer for now - I'm not directly using them either, had some unrelated code that needed a newer scrapy. Updated according to your comment. |
@FRidh - I think this is ready to merge. |
@@ -0,0 +1,36 @@ | |||
{ self, buildPythonPackage, pkgs, lib }: |
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.
- never pass in
pkgs
- python libraries are passed the individual python packages, not pythonPackages or self
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.
done, PTAL
…code. Scrapy is usually installed via pip where copying all permissions makes sense. In Nix the files copied are owned by root and readonly. As a consequence scrapy can't edit the project templates so scrapy startproject fails.
thanks |
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)