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
pykms: init at 20170719 and corresponding NixOS module #30050
Conversation
propagatedBuildInputs = with python2Packages; [ argparse pytz ]; | ||
|
||
prePatch = '' | ||
siteDir=$(toPythonPath $out) |
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.
siteDir=$out/${python.sitePackages}
?
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.
What's the difference?
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.
For the record, I have no issue at all with your version but I'm genuinely curious about the 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.
I personally prefer to use Nix as much as possible. This is also more common I think.
homepage = "https://forums.mydigitallife.info/threads/50234-Emulated-KMS-Servers-on-non-Windows-platforms/page2?p=840410&viewfull=1#post840410"; | ||
license = licenses.mit; | ||
maintainers = with maintainers; [ peterhoeg ]; | ||
platforms = platforms.unix; |
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 needed
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.
I would merge this 472e24429a47feec7dd7da681683f84819ee8597 commit into one of already existing.
All, thanks for your input! |
Motivation for this change
Have been running it here for testing purposes on 17.03 and 17.09 without problems.
While it works, I don't feel particularly proud of how the binaries are being wrapped as well as the handling of the DB creation script, so I'm very open to feedback on that.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)