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

cURL download double quotes #7999

Merged
merged 1 commit into from
May 26, 2022
Merged

cURL download double quotes #7999

merged 1 commit into from
May 26, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 26, 2022

Attempt to resolve Windows issues

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved file download reliability in YOLOv5's utility script.

πŸ“Š Key Changes

  • Modified the command used to download files with curl, changing single quotes to double quotes around parameters.

🎯 Purpose & Impact

  • The change is intended to prevent potential issues with the previous quote usage when downloading files, ensuring better compatibility and stability.
  • Users might experience more robust downloads, especially in environments where the single-quote syntax could lead to errors.

Attempt to resolve Windows issues
@glenn-jocher glenn-jocher linked an issue May 26, 2022 that may be closed by this pull request
1 task
@glenn-jocher glenn-jocher merged commit a6e99e4 into master May 26, 2022
@glenn-jocher glenn-jocher deleted the update/curl branch May 26, 2022 18:37
tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
Attempt to resolve Windows issues
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
Attempt to resolve Windows issues
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.

visDrone.yaml download script not working
1 participant