-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
graphite: make systemd create /run directory for us #44392
Conversation
@GrahamcOfBorg eval |
What about using |
@jtojnar you mean something like:
how is that more declarative than using |
|
@GrahamcOfBorg test graphite |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.graphite Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: tests.graphite Partial log (click to expand)
|
f856f00
to
069a34a
Compare
@GrahamcOfBorg test graphite |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.graphite Partial log (click to expand)
|
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: tests.graphite Partial log (click to expand)
|
@GrahamcOfBorg test graphite |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.graphite Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: tests.graphite Partial log (click to expand)
|
The test throw:
this looks like something else is incorrect in the module. |
Strange, on my system the test completes successfully:
|
Test succeeds on my local machine as well (on this branch). |
@GrahamcOfBorg test graphite |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.graphite Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: tests.graphite Partial log (click to expand)
|
I'm late to the party (we just had kids!), but this is great. Thanks for pushing this past the goal line! |
@peterhoeg congrats on the kids! |
Motivation for this change
Part of #41092.
@peterhoeg I tested your commit and it seems to work fine. If you approve the PR I'll merge it in.
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)