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

fetcher gets stuck on https://github.com/npm-ml/merlin-extend.git #595

Closed
bestander opened this issue Oct 11, 2016 · 11 comments
Closed
Assignees
Labels

Comments

@bestander
Copy link
Member

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

bug - regression

What is the current behavior?

package.json

"dependencies": {
    "merlin-extend": "https://github.com/npm-ml/merlin-extend.git#e14fbeab5301dcd3f710eb5d2ff9c391ec410b61"
  }
$ yarn install
yarnpkg install v0.14.0
info No lockfile found.
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
███░░ 3/5

Please mention your node.js, yarn and operating system version.
Node 6.7
Mac OS
latest master yarn 0.14.0

@bestander
Copy link
Member Author

I'll investigate more

@bestander
Copy link
Member Author

This worked in 0.13.6 (latest on 0.13-stable branch)
Stopped working in 0.14.3 (latest 0.14-stable branch)

@sebmck
Copy link
Contributor

sebmck commented Oct 11, 2016

I switched from clone to fetch recently to fix a few bugs around ssh permissions with git. Most likely the cause.

@bestander
Copy link
Member Author

Yeah, that might be the cause

@sebmck
Copy link
Contributor

sebmck commented Oct 11, 2016

Managed to fix this locally by killing the Yarn cache, rm -rf ~/.yarn ~/.yarn-cache.

@bestander
Copy link
Member Author

Tried that, did not help me

@bestander
Copy link
Member Author

Reverting 1c8f2c1?diff=unified does not help

@bestander bestander self-assigned this Oct 11, 2016
@bestander
Copy link
Member Author

bestander commented Oct 11, 2016

So far I found that extractor.on("end") is never called for git archive command at https://github.com/yarnpkg/yarn/blob/master/src/util/git.js#L189

Quite odd

@bestander
Copy link
Member Author

Looks like this might be a known issue for some apps not to flush stdout when exit nodejs/node#2972.

@sebmck sebmck added the cat-bug label Oct 11, 2016
@palmerj3
Copy link

@bestander that repo 404's on github. Do you have an updated location or repository that we could use to reproduce this issue?

@bestander
Copy link
Member Author

Ok, let's close this one then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants