-
Notifications
You must be signed in to change notification settings - Fork 131
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
Generate GIT-REPO#version versions in bower.json for packages that are not found in Bower. #324
Conversation
… Bower complain about it
…#version style bower dependency.
src/Spago/Bower.hs
Outdated
deps <- Packages.getDirectDeps config | ||
|
||
for deps $ \(PackageName{..}, Package{..}) -> do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was wondering if we should do this in parallel, as from what I've seen bower info
can be quite slow. We could use an async pool like we do here, or just mapTasks
as we don't need the fancy error handling
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(this is not a requirement for merging, just noting that it might a bottleneck on larger packages)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah - done. Bower info is pretty slow (it takes a couple of seconds per call here, but I do have very high latency internet).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Dretch, good work! (especially with the Windows stuff 😄)
…because it is slow (~2s per call on my machine)
This is a follow-on from #303.
Checklist: