Skip to content

Remove git extension in coolprop download link#1900

Merged
pcarruscag merged 2 commits intosu2code:developfrom
davidscn:coolprop-fetch
Jan 26, 2023
Merged

Remove git extension in coolprop download link#1900
pcarruscag merged 2 commits intosu2code:developfrom
davidscn:coolprop-fetch

Conversation

@davidscn
Copy link
Contributor

Proposed Changes

While trying to install SU2 7.5.0 on ubuntu, the CoolProp auto download was failing (although CoolProp was disabled in the meson options, why is the download triggered at all?). The reason, however, was the .git extension in the download link. Removing the link fixed the problem for me.

PR Checklist

  • I am submitting my contribution to the develop branch.

Copy link
Member

@pcarruscag pcarruscag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks

@pcarruscag pcarruscag merged commit d83c371 into su2code:develop Jan 26, 2023
@pr-triage pr-triage bot added the PR: merged label Jan 26, 2023
@davidscn davidscn deleted the coolprop-fetch branch January 26, 2023 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants