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

git+ssh install protocol is not used correctly #1397

Closed
sateffen opened this issue Oct 24, 2016 · 3 comments
Closed

git+ssh install protocol is not used correctly #1397

sateffen opened this issue Oct 24, 2016 · 3 comments

Comments

@sateffen
Copy link

sateffen commented Oct 24, 2016

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Yarn is not capable of installing git dependencies

If the current behavior is a bug, please provide the steps to reproduce.

  1. Add a git dependency to package.json (looking like "myMod": "git+ssh://git@github.....git" (important is git+ssh)
  2. type yarn to install all packages
  3. See yarn erroring, because it's trying git clone ssh://git@....git, without git+

What is the expected behavior?
To install the dependency with the correct protocol (git+ssh), just like NPM or git itself

Please mention your node.js, yarn and operating system version.
Windows 7, yarn v0.16.1, node v6.6.0

Additional Note
The Git URL is an internal, so I had to shrink them down. Internally we use gitlab on a special port, which is not present in the git url (so it's something like git+ssh://internal.gitlab.com:group/project.git. Gitlab itself runs on a special port, but neither git, nor npm have any problems with that. The main difference is the protocol used for git clone (git and npm using git+ssh, yarn using ssh).

If you got any futher questions, please ask, I try to reproduce it on a different system is necessary. I didn't find a bug mentioning THIS problem, so I opened this issue, if there is any, sorry for the extra work,

Cheers

@wyze
Copy link
Member

wyze commented Oct 24, 2016

Looks like #1081 is open to resolve this.

@samccone
Copy link
Member

👍 thanks @wyze

@nightlyop
Copy link

nightlyop commented Oct 25, 2016

Windows: 10
Yarn: 0.16.1
Node: 6.2.1
Git: 2.7.3.windows1

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

No branches or pull requests

4 participants