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
pythonPackages.gateone: needs tornado_4 #51435
Conversation
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.
commit msg should be pythonPackages.gateone: needs tornado_4
@worldofpeace updated |
Python modules are not allowed to rely on a different version of a package in order to prevent multiple versions in an environment. |
Please elaborate on what change is needed? |
This is actually an executable... Not sure who'd use gateone as a library. |
In that case, this should be moved out of |
I think this derivation predates the splitting between libraries and applications. |
Well, I'd appreciate it if someone could do that then, I'm not familiar enough with Python. |
I can do this for you, I'll push the required changes to this branch in a bit. |
Thanks :) |
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.
Still functions, thanks for the cleanup @manveru
Also use `buildPythonApplication`.
f185930
to
75b5e7d
Compare
@tomberek With the NixOS service? |
@worldofpeace : Non-NixOS worked fine when just running the package. Ran into issues with the service on NixOS:
|
Are there any updates on this pull request, please? |
Since this seems to be outdated, and I'm not planning on using this package. I'll close it. |
Should we mark as broken? |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)