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
Add the tychus application #51047
Add the tychus application #51047
Conversation
I locally built with sandbox enabled on Sandbox failures mostly happen when the actual build implicitly depends on your setup. The sandbox ensures a pure-as-possible environment (using |
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.
Thanks for the contribution to nixpkgs.
I left some comment for you to consider.
Wow, I'd not expected to get feedback so quickly! Thank you. I'll try to address them later today. Nix is amazing so far |
As for the Darwin issue, you might solve it by adding CoreFoundation as an optional build input when building on Darwin. it would look like: { stdenv, fetchFromGitHub, buildGoPackage, CoreFoundation }:
# blahcode
buildInputs = stdenv.lib.optionals stdenv.hostPlatform.isDarwin [ CoreFoundation ]; |
@c0bw3b Thank you for the suggestions. I tried them out, but it still isn't able to find CoreFoundation |
@dawehner as you only filled out the I recommend to ask in |
@Ma27 Yeah I'm planning to use this on both darwin and on my linux machine (Nix is really nice for this hybrid setup world!). |
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.
Don't worry about sandboxing in this case, darwin + frameworks currently don't work properly when sandboxing is enabled. (Builds that don't depend on those work fine tho)
@GrahamcOfBorg build tychus |
Success on aarch64-linux (full log) Attempted: tychus Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: tychus Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: tychus Partial log (click to expand)
|
Motivation for this change
This PR adds the tychus package which is a really neat go program which restarts servers during development. Its main trick is to keep around http requests as long the server restarts, so "reloads" always work.
Things done
sandbox
innix.conf
on non-NixOS)Fails with
though.
nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)