-
-
Notifications
You must be signed in to change notification settings - Fork 12.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
python3Packages.httpie-ntlm: init at 1.0.2 #169302
Conversation
Result of 1 package failed to build:
1 package built:
|
6a6532e
to
0f03775
Compare
Result of 1 package built:
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Result of 1 package built:
|
pname = "httpie-ntlm"; | ||
version = "1.0.2"; | ||
format = "setuptools"; | ||
disabled = pythonAtLeast "3.10"; |
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.
disabled = pythonAtLeast "3.10"; |
Why shouldn't it work with python 3.11? if we use httpie from the same pythonPackages it will always have the same python version.
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.
Hi @SuperSandro2000, thank you for good questions. httpie
is a python application, it is built with default python version in nixpkgs, at the moment python39
(https://github.com/NixOS/nixpkgs/blob/nixos-21.11/pkgs/tools/networking/httpie/default.nix)
Since httpie
is a dependency to httpie-ntlm
and httpie
only exist for python39 in nixpkgs. httpie-ntlm
will fail if trying to build in any another python version. E.g nixpkgs-review
(#169302 (comment))
This is way I proposed to use disabled = ! isPy39;
, to only build for python39.
You proposed to use pythonAtLeast
or pythonOlder
. So I used disabled = pythonAtLeast "3.10";
to avoid python310 og bigger. But it will allow python38 that fails to build.
At the moment I have reverted back to disabled = ! isPy39;
, which I think is the cleanest solution. Do you have a proposal to a better solution?
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.
We don't want this. If we bump python3Packages to 3.10 it will silently break for no good reason.
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.
@SuperSandro2000 thanks for follow up :-) Maybe I have misunderstood, correct me if wrong. I have now removed the disable line. So now the package can be built as long httpie
exist for given python version and fail if not. I can see that all PR checks have passed. But nixpkgs-review
fail for python 3.10 as expected.
Result of 1 package built:
|
Result of 1 package failed to build:
1 package built:
|
Now I know what the problem is: You are using httpie which is outside of python3Packages. You cannot draw httpie in like that. I think the best solution is to move the plugin out of python3Packages or httpie back in. |
Result of 5 packages built:
|
@SuperSandro2000 Thanks for you patience :-) I have now made |
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.
that should be it
Description of changes
Add missing python package httpie-ntlm, NTLM auth plugin for HTTPie.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes