Skip to content

default.xml: Fetch from GitHub over https, push over ssh - #12

Closed
lucianogdittgen wants to merge 2 commits into
OSSystemsEmbeddedLinux:masterfrom
lucianogdittgen:manifest/https-fetch-ssh-push
Closed

lucianogdittgen wants to merge 2 commits into
OSSystemsEmbeddedLinux:masterfrom
lucianogdittgen:manifest/https-fetch-ssh-push

Conversation

@lucianogdittgen

Copy link
Copy Markdown

Fetching over ssh needs a GitHub key, so anonymous users could not repo init/repo sync these public projects. Fetch now uses https, and a pushurl keeps pushes on ssh, as bb064da intended. The README's repo init example uses https too.

Tested: anonymous (GIT_SSH_COMMAND=false) repo init from this branch, then a sync of the projects. remote.github.url is https and the pushurl is ssh.

Merge last, after the layer PRs.

Layer PRs in this release update

The manifest tracks master, so the merged layer changes need no manifest change. Draft until the open layer PRs merge.

Fetching over ssh needs a GitHub key, so anonymous users could not repo
sync these public projects. A pushurl keeps pushes on ssh, as bb064da
intended. Tested: anonymous repo init and sync of the manifest.

Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
The ssh URL needs a GitHub key, so anonymous users failed at repo init.

Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
@lucianogdittgen

Copy link
Copy Markdown
Author

Opened by mistake; held until the layer PRs merge.

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