search

clawpack / apps

Repository of applications of Clawpack and related codes
Other
11 stars 40 forks source link

Bypassing 'Permission denied' errors' while fetching submodules #85

Closed Dirivian closed 7 years ago

Dirivian commented 7 years ago

The README is modified to suggest adding ssh keys to avoid 'Permission denied' errors encountered when using HTTPS urls.

mandli commented 7 years ago

Can you add an additional comment to the "one liner" mentioning you need to run that from $CLAW rather than inside of the apps repository?

Dirivian commented 7 years ago

I reran the two commands from within the apps repository.

By one-liner, do you mean the commit message or the modification to the README file ?

mandli commented 7 years ago

I meant the command including the recursive command.

Dirivian commented 7 years ago

I added a few more lines to explain it in a bit more detail. Does this seem enough/alright ?

rjleveque commented 7 years ago

I think if you clone recursively then you don't need to do the submodule init and update, that's only if you already have the apps repository and want to get the submodules?

I don't understand why using the https address gives permission errors? If you have an ssh key set up then you might be able to clone git@github.com:clawpack/apps.git, but I didn't think ssh keys are used at all when using https?

At any rate it seems like users should be able to clone without needing ssh keys at all.

@mandli: I think one problem is that apps/.gitmodules contains an ssh address for one of the two submodules. Can you fix this and test?

rjleveque commented 7 years ago

I suggest #86 instead.