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

Cyclic dependencies hang bower if install is run twice #1169

Closed
dfreedm opened this issue Mar 12, 2014 · 12 comments
Closed

Cyclic dependencies hang bower if install is run twice #1169

dfreedm opened this issue Mar 12, 2014 · 12 comments
Labels

Comments

@dfreedm
Copy link

dfreedm commented Mar 12, 2014

In bower 1.3.1, cyclic dependencies seem to hang if bower install is called more than once.

No errors are emitted even with the verbose flag set.

I've set up two dummy packages for testing: http://github.com/azakus/bower-foo and http://github.com/azakus/bower-bar. Both depend on each other.

Steps to Reproduce:

  1. bower install azakus/bower-foo
  2. bower install

Notes

bower update seems to work just fine

@sindresorhus
Copy link
Contributor

@SevInf

@sheerun
Copy link
Contributor

sheerun commented Apr 25, 2014

It was caused by @satazor by removing true parameter from walkTree here: 6125c04#diff-8b1ae93eaec3db41f7a6da1e4a77583bL64

Could you fix it?

@sheerun sheerun added the bug label Apr 25, 2014
@ebidel
Copy link

ebidel commented May 28, 2014

@satazor any word? This is still an issue with 1.3.3

@silenceisgolden
Copy link

#1324 needs to close, duplicate. Any update? Supposedly this is fixed in "master" branch but not "available" yet. I'm paraphrasing without full understanding here.

@enriquemorenotent
Copy link

Is there any word on when will this be available or at least any workaround until that time arrives?

@wibblymat
Copy link
Member

I believe this was fixed by #1306, pushing a release v1.3.4 in the next 30 minutes that includes it, I'm sorry a new version wasn't pushed when the fix first landed.

@enriquemorenotent
Copy link

Lovely! Thank you :)

@sofiushko
Copy link

Unfortunately, not fixed

@sheerun
Copy link
Contributor

sheerun commented Jun 5, 2014

@sofiushko Works for me. 1.3.4

@ebidel
Copy link

ebidel commented Jun 5, 2014

Me too
On Jun 5, 2014 8:54 AM, "Adam Stankiewicz" notifications@github.com wrote:

@sofiushko https://github.com/sofiushko Works for me. 1.3.4


Reply to this email directly or view it on GitHub
#1169 (comment).

@guotechfin
Copy link

me too

@kaminskypavel
Copy link

didnt work for me.

v1.3.9

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

No branches or pull requests

10 participants