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

Put quotes around curled url and filename #666

Closed
wants to merge 3 commits into from

Conversation

audrow
Copy link
Member

@audrow audrow commented Jun 1, 2022

This allows a ros2.repos url to have special characters. This is useful for a tool that I'm working on https:/audrow/next-repos-ci, which creates a repo file from the url.

Before this PR, this CI failed.

Signed-off-by: Audrow Nash <[email protected]>

Co-authored-by: Steven! Ragnarök <[email protected]>
@audrow audrow force-pushed the audrow/add-quotes-to-curl branch from bb50fe3 to 06ce190 Compare June 1, 2022 18:13
@audrow
Copy link
Member Author

audrow commented Jun 1, 2022

Linux seems to be working (https://ci.ros2.org/job/ci_linux/16766/), but I'm getting Windows failures; eg, https://ci.ros2.org/job/ci_windows/17164/console#console-section-1.

I'll look into this later. Do let me know if anyone has suggestions of how to resolve these.

@audrow audrow force-pushed the audrow/add-quotes-to-curl branch 2 times, most recently from 11e0e70 to 06ce190 Compare June 1, 2022 22:09
@clalancette clalancette marked this pull request as draft June 16, 2022 17:08
@audrow
Copy link
Member Author

audrow commented Jul 20, 2022

I'm not exactly sure how to get past Windows. I'm going to close this for now. I may try again in the future.

@audrow audrow closed this Jul 20, 2022
@clalancette clalancette deleted the audrow/add-quotes-to-curl branch September 3, 2024 16:24
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.

3 participants