Limit concurrency with a semaphore

pearkes / gethub

A utility for cloning and fetching your remote git repositories from GitHub.

MIT License

217 stars 266 forks source link

Limit concurrency with a semaphore #10

Closed vinc closed 11 years ago

vinc commented 11 years ago

This fix may resolve #9.

pearkes commented 11 years ago

Ah yea! That fixes it. Much cleaner then #9.

For speeds sake, should we maybe raise the limit from 8 to, maybe, 20 or 25? I only noticed it with > 50 repos.

vinc commented 11 years ago

Yes, the number 8 was chosen arbitrary because I could not reproduce the issue on my computer. We could raise it to 32 and wait to see if the problem still occurs.

pearkes commented 11 years ago

32 sounds good to me!

pearkes commented 11 years ago

:+1: