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
Dependencies for realms-wiki #21873
Dependencies for realms-wiki #21873
Conversation
I will put all packages into separate files. |
38661ab
to
4cc68d7
Compare
@FRidh I also move out packages out of python-packages.nix here. Can you take a look at some packages and tell me whether you agree on the way I pull in python dependencies into the scope into external packages. |
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 for moving out those packages Just one more change is needed. Aside from that it looks good to me!
@@ -0,0 +1,26 @@ | |||
{ stdenv, buildPythonPackage, fetchurl | |||
, pkgs, gevent, geventhttpclient, mock, fastimport }: |
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.
not the package set pkgs
but individual packages
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.
How can I avoid shadowing of packages from pkgs
by python packages with the same name in this case?
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.
You can explicitly pass the package to callPackage
, e.g. inherit (pkgs) git;
,
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.
the reasoning behind passing individual packages is to make it easier to overwrite them?
4cc68d7
to
17c457e
Compare
@FRidh is specifying python packages in seperate files the prefered syle now? |
The file is getting way too big, and the history is also not useful, so yes. |
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/
)