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
pythonXXPackages.httpserver: init at 1.1.0 #25256
Conversation
@P-E-Meunier, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer. |
pkgs/top-level/python-packages.nix
Outdated
@@ -2964,6 +2964,33 @@ in { | |||
}; | |||
}; | |||
|
|||
httpserver = buildPythonPackage rec { |
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 move expression to pkgs/development/python-modules/httpserver/default.nix
pkgs/top-level/python-packages.nix
Outdated
pname = "httpserver"; | ||
version = "1.1.0"; | ||
buildInputs = with self; [ docopt ]; | ||
doCheck = false; |
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.
why do you disable the tests? If you disable the tests, always include a comment.
pkgs/top-level/python-packages.nix
Outdated
version = "1.1.0"; | ||
buildInputs = with self; [ docopt ]; | ||
doCheck = false; | ||
src = pkgs.fetchurl { |
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.
fetchPypi
pkgs/top-level/python-packages.nix
Outdated
@@ -22154,6 +22181,7 @@ in { | |||
}; | |||
}; | |||
|
|||
|
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.
this shouldn't be included
Don't merge, rebase instead. If you need help with it or like me to do it let me know. |
Please go ahead! If this PR requires me to start fiddling with git, I'm willing to abandon this it. |
there it is 5db271c |
Awesome! Thanks a lot! |
Motivation for this change
SimpleHTTPServer is a python package that starts an HTTP server serving files in the local directory.
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/
)