search

embeddedartistry / athena-bootloader

An AVR Arduino bootloader that supports uploads over Ethernet (via TFTP) and Serial.
GNU Lesser General Public License v2.1
12 stars 5 forks source link

update git clone command #20

Closed AkhileshThorat closed 4 years ago

AkhileshThorat commented 4 years ago

There are issues with git clone git@github.com:embeddedartistry/ariadne-bootloader.git

Following message is displayed on running the command on all Windows, Linux and MacOS

Cloning into 'ariadne-bootloader'...
The authenticity of host 'github.com (13.234.210.38)' can't be established.
RSA key fingerprint is SHA256:nThbg6kXUpJWGl7E1IGOCspRomTxdCARLviKw6E5SY8.
Are you sure you want to continue connecting (yes/no/[fingerprint])? y
Please type 'yes', 'no' or the fingerprint: yes
Warning: Permanently added 'github.com,13.234.210.38' (RSA) to the list of known hosts.
git@github.com: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

git clone https://github.com/embeddedartistry/ariadne-bootloader works without any issues

phillipjohnston commented 4 years ago

Somehow I missed this one... didn't get the email from GitHub!

phillipjohnston commented 4 years ago

Do you have SSH keys configured with GitHub?

I think that's what this is complaining about:

git@github.com: Permission denied (publickey).

phillipjohnston commented 4 years ago

Either way, will get this merged today.

AkhileshThorat commented 4 years ago

Do you have SSH keys configured with GitHub?

I think that's what this is complaining about:

git@github.com: Permission denied (publickey).

I didn't noticed the SSH config in the first place. Now I configured SSH and I'm able to clone the repository successfully. Maybe a reminder or note should be added regarding SSH configuration

phillipjohnston commented 4 years ago

Yeah definitely... going to have to update all of my project readmes with that info :)

phillipjohnston commented 4 years ago

Updated approach in #22, closing this PR.