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

Ensure getting real releases, only master branch, fix headers folder name #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

andrebrait
Copy link

@andrebrait andrebrait commented Aug 14, 2020

This ensures it'll always get a release with "prerelease": false, even if there's a newer prerelease
Ensures the release will always be from the master branch (by name, not the strongest guarantee, but better than nothing)
It also avoids saving the file to disk by piping the output from curl directly to tar
Add -L parameter to curl so it follows redirects (necessary for fetching the archive).

I also renamed the includes folder to include, which is the standard way of naming the folders containing header files, and I got rid of the . root directory on the tar archive. Now installing it to the VitaSDK folder is just a tar xz -C "${VITASDK}/arm-vita-eabi" and the headers will go into the right directory.

This ensures it'll always get a release with `"prerelease": false`, even if there's a newer prerelease
Ensures the release will always be from the master branch (by name, not the strongest guarantee, but better than nothing)
It also avoids saving the file to disk and by piping the output from `curl` directly to `tar` 
Add -L parameter to curl so it follows redirects (necessary for fetching the archive)
.travis.yml Outdated Show resolved Hide resolved
@andrebrait andrebrait changed the title Ensure getting real releases, only master branch Ensure getting real releases, only master branch, fix headers folder name Aug 15, 2020
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.

1 participant