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

Delete --x-use-aria2. #1582

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Delete --x-use-aria2. #1582

merged 1 commit into from
Jan 27, 2025

Conversation

BillyONeal
Copy link
Member

@BillyONeal BillyONeal commented Jan 22, 2025

  • Huge parts of the vcpkg product do not respect this setting. In particular, nothing in asset caching or anything vcpkg the exe tries to download are hooked up to it.
  • z_vcpkg_download_distfile_via_aria() uses malformed arguments to aria2 download command vcpkg#30079 shows it has been entirely broken for more than a year.
  • It's an --x switch and we never claimed full support for it.
  • We have never tested that it ever actually works.
  • It creates catch-22 problems where we have to download aria2 with something in order to use it.

It could be OK to bring this back someday by teaching x-download and friends to use it directly, but given how long it's been broken and ~2 people noticed or cared, that doesn't seem worth it.

Users also have available workarounds if they've arranged aria2c to be present already with the x-script asset caching mechanism as a last resort.

Related: microsoft/vcpkg-docs#441

* Huge parts of the vcpkg product do not respect this setting
* microsoft/vcpkg#30079 shows it has been entirely broken for an extremely long time.
* It's an --x switch and we never claimed full support for it.
* We have never tested that it ever actually works.

Related: microsoft/vcpkg-docs#441
@BillyONeal BillyONeal merged commit c79ccaa into microsoft:main Jan 27, 2025
6 checks passed
@BillyONeal BillyONeal deleted the delete-aria2 branch January 27, 2025 19:33
jcamiel added a commit to Orange-OpenSource/hurl that referenced this pull request Feb 12, 2025
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.

2 participants