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
Fixes the lldpd service not starting on NixOS 18.09 #46367
Conversation
@@ -10,6 +10,15 @@ stdenv.mkDerivation rec { | |||
sha256 = "0lgiappbjm95r1m0xyxb6gzz4izcjixknbzq3s7pbqbsmhm642s5"; | |||
}; | |||
|
|||
remove_backported_patch = version != "1.0.1"; |
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.
Thats a clever pattern, but I guess it could lead to confusion. Won't the patch fail to apply on future versions anyways?
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 don't know, is that to be expected? I don't have previous experience backporting patches.
Seeing how it has been broken since #38718 five months ago without anyone noticing I figured something was needed. Nixos tests would be nice of course.
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.
Since you've backported a patch that is already included upsteram (right?) the patch will fail to apply after the next update, since the relevant code is already changed.
Thanks for your effort in fixing this :)
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.
OK, good. I'll remove the extra cruft. Indeed this is from upstream and confirmed to be in the next release.
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.
Great :) Don't forget to ping me once you've removed it as github doesn't always send notifications on code changes.
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.
Pushed new commit, should be all good now.
remove_backported_patch = version != "1.0.1"; | ||
patches = [ | ||
# Should be removed in the version after 1.0.1 as this will be included. | ||
(fetchpatch { |
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.
Please add a short comment describing what this patch is for and/or a name.
3d2b5d0
to
646877d
Compare
Backports a patch that will be included in later upstream versions. The patch removes a call to /bin/mkdir.
646877d
to
8a03ec0
Compare
@GrahamcOfBorg build lldpd |
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.
Great, thank you!
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: lldpd Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: lldpd Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: lldpd Partial log (click to expand)
|
Backports a patch that will be included in later versions, no longer calling /bin/mkdir.
Fixes #44507.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)