-
Notifications
You must be signed in to change notification settings - Fork 63
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
vitasdk-update: Add a script to update an existing vitasdk installation #25
Conversation
This is to be installed into vitasdk by buildscripts. The installation flow remains compatible (bootstrap-vitasdk, then install-all) but now the user can run vitasdk-update to update the SDK and packages.
$DIR/../vdpm $1 | ||
} | ||
|
||
install_packages() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maintaining two identical lists is annoying, maybe do a git checkout
on https://github.com/vitasdk/packages/blob/master/travis_packages.sh?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can install this in buildscripts, but then it won't work out of the vdpm
checkout (I'm not sure what you mean by do a git checkout
of it)
also, given vitasdk/packages#28 (comment) we might want to build some stuff twice eventually, this shouldn't matter much now but I'm thinking how much the two lists will diverge eventually
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest to do
git archive --remote=ssh://[email protected]/vitasdk/packages.git HEAD travis_packages.sh | tar -x
but apparently GitHub doesn't support git-archive
.
Maybe source https://raw.githubusercontent.com/vitasdk/packages/master/travis_packages.sh
?
#!/bin/sh | ||
|
||
get_download_link () { | ||
curl "https://api.github.com/repos/vitasdk/autobuilds/releases" | grep "master" | grep "browser_download_url" | grep $1 | head -n 1 | cut -d '"' -f 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GitHub API's keeps being rate-limited from Travis, use this instead: https://github.com/vitasdk/vita-headers/blob/master/.travis.d/download_sdk.sh
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this would add a dep on python
exit 1 | ||
fi | ||
|
||
. $DIR/include/install-vitasdk.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that renamed headers and stubs will be duplicated when this script is executed.
vitasdk/vdpm#25 makes problems
vitasdk/vdpm#25 makes problems
vitasdk/vdpm#25 makes problems
This is to be installed into vitasdk by buildscripts.
The installation flow remains compatible (bootstrap-vitasdk, then install-all)
but now the user can run vitasdk-update to update the SDK and packages.