Skip to content
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

Start to manage ssh config with home-manager #263

Merged
merged 5 commits into from
Aug 9, 2023
Merged

Start to manage ssh config with home-manager #263

merged 5 commits into from
Aug 9, 2023

Conversation

kachick
Copy link
Owner

@kachick kachick commented Aug 8, 2023

May relate to #262

macos

Host github.com
  AddKeysToAgent yes
  UseKeychain yes
  IdentityFile ~/.ssh/id_ed25519

Comment on lines 60 to 66
'' + (
if pkgs.stdenv.isDarwin then
''
# In Linux, this config makes error as "Bad configuration option: usekeychain"
UseKeychain yes
''
else ""
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kachick added 3 commits August 9, 2023 12:39
In macOS, starting ssh-agent is still /usr/bin/ssh-agent even added the nixpkgs
@kachick
Copy link
Owner Author

kachick commented Aug 9, 2023

@kachick
Copy link
Owner Author

kachick commented Aug 9, 2023

Integrating git sign with ssh will be done in another PR

@kachick kachick marked this pull request as ready for review August 9, 2023 16:00
@kachick kachick merged commit 01f9c1d into main Aug 9, 2023
@kachick kachick deleted the ssh-and-git branch August 9, 2023 16:01
kachick added a commit that referenced this pull request Aug 10, 2023
1. [x] Manage git with home-manager
2. [x] Enable ssh commit sign with *.pub by #263
3. [x] Resolves #265

May relate to #262 in future

---------

Signed-off-by: Kenichi Kamiya <[email protected]>
kachick added a commit to kachick/kachick.github.io that referenced this pull request Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant