-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
changeme: init at 1.2.3 #325188
base: master
Are you sure you want to change the base?
changeme: init at 1.2.3 #325188
Conversation
@fabaff any fixes for cerberus in this scenario. |
Do you use this? |
I currently do |
Sorry, i can't follow. Can you please be more specific? |
526bb56
to
8a52b9e
Compare
pkgs/by-name/ch/changeme/package.nix
Outdated
checkPhase = '' | ||
# allow to find the module helper during the test run | ||
export PYTHONPATH=$PYTHONPATH:$PWD/tests | ||
py.test changeme/tests |
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.
Please rely on pytestCheckHook and move the export to preCheck
mkdir -p $out/{bin,share/changeme} | ||
cp -R * $out/share/changeme |
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.
Are those files not installed by python?
870f67b
to
05dfa5b
Compare
d63560a
to
003b23e
Compare
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.
Since this package doesn't follow the standard Python packaging, we want to ensure it works using testers.testVersion
or similar.
build-system = with python3Packages; [ setuptools ]; | ||
|
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-system = with python3Packages; [ setuptools ]; |
There is no pyproject.toml
or setup.py
.
logutils | ||
lxml | ||
netaddr | ||
nose |
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.
nose |
nose
is a test dependency, and it should not be added to inputs.
see #326513
makeWrapper "${python3Packages.python.interpreter}" "$out/bin/changeme" \ | ||
--set PYTHONPATH "$out/share/changeme/changeme.py" \ | ||
--add-flags "$out/share/changeme/changeme.py" |
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.
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.