-
-
Notifications
You must be signed in to change notification settings - Fork 14.9k
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
nix-editor: init at 0.3.0 #340033
base: master
Are you sure you want to change the base?
nix-editor: init at 0.3.0 #340033
Conversation
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.
Some feedback below. Also isn't rnix dead?
595c240
to
b898055
Compare
Thanks for the review. Fiy: No rnix is not dead. It is a pretty good quality rust library. That is used in a lot of tools we use in nixpkgs. |
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.
Fiy: No rnix is not dead.
I asked because of no activity in 2+ years, but alright. In any case
seems like there's a hash mismatch here when I build locally
hash = "sha256-Ne9NG7x45a8aJyAN+yYWbr/6mQHBVVkwZZ72EZHHRqw="; | ||
}; | ||
|
||
cargoHash = "sha256-ZWSLW+mFT+Qls2GeVPamispL8ObieXT4tDc1Go5cjL4="; |
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.
cargoHash = "sha256-ZWSLW+mFT+Qls2GeVPamispL8ObieXT4tDc1Go5cjL4="; | |
cargoHash = "sha256-1OErQ+bKKInz7pQQCKW82BGODzbD9EFImBnwbiojJgk="; |
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.
Or I wonder is this #308089 rearing its head?
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.
No, I get the same hash mismatch on Linux.
|
||
cargoHash = "sha256-ZWSLW+mFT+Qls2GeVPamispL8ObieXT4tDc1Go5cjL4="; | ||
|
||
buildInputs = lib.optionals stdenv.isDarwin [ darwin.Security ]; |
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.
buildInputs = lib.optionals stdenv.isDarwin [ darwin.Security ]; | |
buildInputs = lib.optionals stdenv.hostPlatform.isDarwin [ darwin.Security ]; |
hash = "sha256-Ne9NG7x45a8aJyAN+yYWbr/6mQHBVVkwZZ72EZHHRqw="; | ||
}; | ||
|
||
cargoHash = "sha256-ZWSLW+mFT+Qls2GeVPamispL8ObieXT4tDc1Go5cjL4="; |
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.
No, I get the same hash mismatch on Linux.
Description of changes
A simple rnix-ast based programm to modify NixOS configuration files.
It allows to edit atomic nix values while preserving the overall structure such as formatting and comments.
It has some known bugs but generally works really nice.
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.