search

prefix-dev / rip

Solve and install Python packages quickly with rip (pip in Rust)
https://prefix.dev
BSD 3-Clause "New" or "Revised" License
643 stars 21 forks source link

fix: correctly limits the concurrency, by limiting the `tokio::spawn`'s in all cases #215

Closed tdejager closed 7 months ago

tdejager commented 7 months ago

In the first implmentation the concurrency could potentially be unbounded. Because, when resolving sdists a new DependencyProvider was created. This would imply a new semaphore, and because the building is recursive this could still lead to too many tasks.