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

Support handling multiple arguments #583

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

Alstruit
Copy link

@Alstruit Alstruit commented Jan 9, 2025

Resolves #582

Depends on #581.

@yaakov-h
Copy link
Member

Reading through this quickly, tThis doesn't looks like it brings anything to the table that wouldn't be solved already by running DepotDownloader multiple time.

Particularly, the requirement for -app and -depot to line up with perfectly matching argument pairs is quite unusual and counter-intuitive.

Mutating the static config also seems like an easy source of bugs either now or in future.

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

Successfully merging this pull request may close these issues.

Support for multiple arguments
2 participants