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
zeronet: init at 0.6.0 #31910
zeronet: init at 0.6.0 #31910
Conversation
With the latest commit this works, if the user has sufficient rights on the directories. As I want to turn this into a nixos service, this shouldn't be an issue... DO NOT MERGE!!! |
Added service file - so now, as always: Please tell me how to properly test this! |
data_dir = ${cfg.dataDir} | ||
log_dir = ${cfg.logDir} | ||
'' | ||
+ (ifNotNull cfg.uiIP |
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.
Its not needed to implemented all configuration options. Considering many do not have a description I suggest just using extraConfig
for them.
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.
Removed some, feel free to suggest more... I will provide descriptions for the rest then.
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 is one of those packages where the authors think its a good idea to bundle everything and do updating from within the application, see HelloZeroNet/ZeroNet#382.
This is likely going to need quite some patching to get rid of vendored dependencies. They also do some sys.path
. manipulation in zeronet.py
I saw.
|
||
propagatedBuildInputs = with pythonPackages; [ gevent msgpack ]; | ||
|
||
phases = [ "unpackPhase" "installPhase" "fixupPhase" ]; |
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.
format = "other";
buildPhase = ":";
The phases
should typically not be touched as you may loose phases that are relevant.
enable = true; | ||
after = [ "network.target" ]; | ||
wantedBy = [ "multi-user.target" ]; | ||
serviceConfig.ExecStart = "${pkgs.zeronet}/bin/zeronet.py --conf_file ${configFile}"; |
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.
Executables should not have an extension, and especially in the case of Python where it would create bytecode.
Any progress? |
Closing in favor of #40883 since this seems stalled. |
Motivation for this change
Closing #31566
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)As upstream does not ship a
setup.py
in the repository, we have to copy the sources "by hand" to$out/bin
. Unfortunately, that somehow breaks the code and it fails with(There's also no configuration as it seems)