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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
ntfs2btrfs: init at 20240115 #297152
ntfs2btrfs: init at 20240115 #297152
Conversation
Reviewed points
Possible improvements
CommentsI did this review because of the recommendation in the PR template. If you want to and find some time you could also look at my PR: #297245 |
squashed the two commits, and changed the commit message to the PR title! i also changed a bit of the wording, please tell me if added thanks for the review, please tell me if there is anything else i need to fix. |
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.
See my above comments
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3661 |
also added |
Yes, I've seen it :) |
this is quoted straight from the first link you sent! |
Ah, you're right, my bad 馃槄 |
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.
built successfully. looks pretty good!
it does output to /sbin
also, maybe that should be deleted in a postInstall hook?
ideally (at least, in my own testing) you probably should invoke |
Description of changes
Ntfs2btrfs is a tool which does in-place conversion of Microsoft's NTFS filesystem to the open-source filesystem Btrfs, much as btrfs-convert does for ext2.
(original GitHub page: https://github.com/maharmstone/ntfs2btrfs)
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 馃憤 reaction to pull requests you find important.