Skip to content

git: handle exit code 2 for git remote url retrieval#6422

Merged
kit-ty-kate merged 4 commits intoocaml:masterfrom
rjbou:git-exit-code-2
Oct 16, 2025
Merged

git: handle exit code 2 for git remote url retrieval#6422
kit-ty-kate merged 4 commits intoocaml:masterfrom
rjbou:git-exit-code-2

Conversation

@rjbou
Copy link
Copy Markdown
Collaborator

@rjbou rjbou commented Mar 14, 2025

And update some messages : remote branch not found & skipping local pins

TODO:

  • add a test

@rjbou rjbou changed the title git: handle exit code 2 for git remote url retrieval git: handle exit code 2 for git remote url retrieval Mar 14, 2025
@rjbou rjbou requested a review from kit-ty-kate August 25, 2025 15:43
@rjbou rjbou added this to the 2.5.0~alpha1 milestone Aug 25, 2025
@rjbou rjbou marked this pull request as ready for review August 25, 2025 15:43
Comment on lines +107 to +108
[ERROR] Can't retrieve remote informations for dep-nip.dev
[WARNING] Dependency dep-nip.dev is pinned to local target git+file://${BASEDIR}/dep-nip#master
Copy link
Copy Markdown
Member

@kit-ty-kate kit-ty-kate Aug 25, 2025

Choose a reason for hiding this comment

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

Shouldn't this require --keep-local to be used?
I also find the error message confusing. It sounds like an unexpected error occurred like a lack of internet access or something, but it's simply that the repository doesn't have a main remote (origin).

@kit-ty-kate
Copy link
Copy Markdown
Member

Thanks!

@kit-ty-kate kit-ty-kate merged commit 2ecabc5 into ocaml:master Oct 16, 2025
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants