impermanence icon indicating copy to clipboard operation
impermanence copied to clipboard

Remove /etc/nix/id_rsa from example

Open jakobkukla opened this issue 2 years ago • 3 comments

I couldn't find any mention of this file except sources related with this repository. Furthermore the example changes the permissions of the /etc/nix directory. This breaks the new nix command for non-root users because the nix config file isn't readable anymore.

jakobkukla avatar Nov 05 '22 22:11 jakobkukla

The purpose of this file is just to serve as an example of how to use the options, not as a default config. It's the ssh key I use for remote building in my setup. I'd prefer to replace this example with something else if it's confusing.

talyz avatar Nov 17 '22 07:11 talyz

I was aware that this only served as a usage example. Still it's rather confusing since it's next to sensible defaults like /etc/NetworkManager/system-connections and /etc/machine-id. How about we use something like /etc/ssh/ssh_known_hosts instead, to avoid that users who blindly copy the example end up with a broken nix command?

jakobkukla avatar Nov 17 '22 19:11 jakobkukla

AFAICT, the global known hosts file would mainly be used for declared known hosts and shouldn't need to be persisted, so it wouldn't be a good example either. Maybe we could use the ssh host keys as examples?

talyz avatar Apr 16 '23 09:04 talyz

Closing as #151 fixed this.

jakobkukla avatar Jun 03 '24 16:06 jakobkukla