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

Fix flags on downloading with wget. #881

Closed
wants to merge 1 commit into from

Conversation

cdmatters
Copy link

The wget -o flag puts the standard out to the file, -O actually writes the content to the file.

Description

I just tried to install miniconda and hit this bug (looks like it was introduced recently).

Screenshot 2023-08-23 at 12 21 08 Screenshot 2023-08-23 at 12 22 00

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

The wget  `-o` flag puts the standard out to the file, `-O` actually writes the content to the file.
@conda-bot
Copy link
Contributor

We require contributors to sign our Contributor License Agreement and we don't have one on file for @condnsdmatters.

In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature, merge the PR (conda/infrastructure#807), and ping the bot to refresh the PR.

@cdmatters
Copy link
Author

Don't know if I can sign that, legally.

@kathatherine
Copy link
Contributor

Apologies for overlooking this PR. I pushed a fix without checking the Pull Request list after seeing a issue for the problem. Thank you for your contribution!

Closed as duplicate of #886.

@github-actions github-actions bot added the locked [bot] locked due to inactivity label Aug 25, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked [bot] locked due to inactivity
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants