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_data_to_graph.sh still errors out for older curl versions #252

Closed
1 task done
alyssadai opened this issue Jan 8, 2024 · 0 comments · Fixed by #253
Closed
1 task done

add_data_to_graph.sh still errors out for older curl versions #252

alyssadai opened this issue Jan 8, 2024 · 0 comments · Fixed by #253
Labels
quick fix Minimal planning and/or implementation work required.

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Jan 8, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Expected Behavior

No response

Current Behavior

  • [FIX] Remove --no-progress-meter #227 switched to an older syntax for hiding the progress meter that works on curl < 7.67, but this change was not made everywhere in the script that currently uses --no-progress-meter. As a result, the script currently still errors out on older curl versions

Error message

CLEARING EXISTING DATA FROM repositories/my_db...
curl: option --no-progress-meter: is unknown
curl: try 'curl --help' or 'curl --manual' for more information
./add_data_to_graph.sh: line 160: ((: < 200 ||  >= 300 : syntax error: operand expected (error token is "< 200 ||  >= 300 ")
...

Environment

  • OS:
  • Python/Node version:

How to reproduce

No response

Anything else?

@surchs surchs added this to Neurobagel Jan 8, 2024
@alyssadai alyssadai added flag:schedule Flag issue that should go on the roadmap or backlog. quick fix Minimal planning and/or implementation work required. labels Jan 8, 2024
@alyssadai alyssadai moved this to Backlog in Neurobagel Jan 9, 2024
@alyssadai alyssadai removed the flag:schedule Flag issue that should go on the roadmap or backlog. label Jan 9, 2024
@alyssadai alyssadai moved this from Backlog to Specify - Done in Neurobagel Jan 10, 2024
@alyssadai alyssadai moved this from Specify - Done to Implement - Active in Neurobagel Jan 10, 2024
@alyssadai alyssadai moved this from Implement - Active to Implement - Done in Neurobagel Jan 10, 2024
@surchs surchs moved this from Implement - Done to Review - Active in Neurobagel Jan 11, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quick fix Minimal planning and/or implementation work required.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant