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
bcachefs/update-7 #67696
bcachefs/update-7 #67696
Conversation
I'm having some overheating issues and no compressed air on hand, so I'll have to test this at a later date. |
046c5ff
to
9007daf
Compare
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.
Got some compressed air!
Tested with nix-build nixos/tests/bcachefs.nix
, all good!
@GrahamcOfBorg build linux_testing_bcachefs |
Yeah sorry, I don't think I have permissions for that, we'll have to wait until someone with them can build/test/merge with the build bot. |
9007daf
to
d9d8ef2
Compare
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.
Tested latest with nix-build nixos/tests/bcachefs.nix
, all good!
I think the checks are due to a hydra failure.
d9d8ef2
to
6eb5915
Compare
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.
Tested again as above.
Still good.
@GrahamcOfBorg test bcachefs |
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.
LGTM
not tested tho
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Thanks all! |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @