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 installing ponyc dependency #258

Merged
merged 1 commit into from
Nov 25, 2023
Merged

Fix installing ponyc dependency #258

merged 1 commit into from
Nov 25, 2023

Conversation

SeanTAllen
Copy link
Member

Curling against cloudsmith isn't working. It downloads nothing. wget is working.

Curling against cloudsmith isn't working. It downloads nothing.
wget is working.
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Nov 25, 2023
@SeanTAllen SeanTAllen merged commit e62b0f9 into main Nov 25, 2023
11 checks passed
@SeanTAllen SeanTAllen deleted the curl-doesnt-work branch November 25, 2023 01:46
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Nov 25, 2023
@lskillen
Copy link

Hey @SeanTAllen - We changed the latest endpoint to redirect to the download file, rather than just serving it immediately. The reason for it was, static file caching (3600 seconds for static files) meant the latest endpoint was slow for users to update, so subsequent updates might be missed by users. This way, there's no issues with caching anymore. Adding -L to curl would make it follow the redirect, but otherwise, wget is a good substitute too. :)

@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Nov 25, 2023
@SeanTAllen
Copy link
Member Author

@lskillen thanks for the info. that makes sense. it being a holiday weekend, i didn't investigate very deeply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss during sync Should be discussed during an upcoming sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants