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

Add aria2 as a recommended dependency #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juanjonol
Copy link

For most users, aria2 is a vast improvement to xcodes usage, so I think it should be added as a recommended dependency to xcodes when installed using Homebrew, so it'll be automatically installed with xcodes unless the --without-aria2 flag is used.

This also has the added advantage of marking aria2 as a dependency of xcodes, so it will be listed with brew leaves --installed-as-dependency and it can be easily deleted if xcodes is deleted with brew autoremove.

This closes #4.

For most users, `aria2` is a vast improvement to `xcodes` usage, so I think it should be added as a [recommended dependency](https://docs.brew.sh/Formula-Cookbook#specifying-other-formulae-as-dependencies) to `xcodes` when installed using Homebrew, so it'll be automatically installed with `xcodes` unless the `--without-aria2` flag is used.
 
This also has the added advantage of marking `aria2` as a dependency of `xcodes`, so it will be listed with `brew leaves --installed-as-dependency` and it can be easily deleted if `xcodes` is deleted with `brew autoremove`.

This closes RobotsAndPencils#4.
@juanjonol
Copy link
Author

juanjonol commented Sep 17, 2022

I created #4 to see if you considered this useful before making a Pull Request, but the change is so small that I think making the PR directly makes more sense.

Also, please note that even if this is a small change, I don't know how to test Homebrew formulas. Maybe there's something wrong with this, or another change is needed...

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.

Add aria2 as a recommended dependency
1 participant