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

Github Actions uploads built artifacts #83

Merged
merged 5 commits into from
Dec 15, 2019
Merged

Conversation

pchickey
Copy link
Collaborator

@pchickey pchickey commented Nov 25, 2019

This changes the build scripts to put the artifacts we use for releases in dist/, and has Github Actions upload those artifacts to Github.

I tried doing some yaml programming to automate putting those artifacts into a release, but I wasn't having a great time of it, but this should make it much easier to produce releases by manually downloading artifacts and then attaching them to a release, rather than build them locally.

@pchickey pchickey force-pushed the pch/gh_actions_upload branch from c4515c4 to d75e4a4 Compare November 27, 2019 22:09
@pchickey pchickey changed the title Use Github Actions to upload built artifacts Github Actions to uploads built artifacts Nov 27, 2019
@pchickey pchickey changed the title Github Actions to uploads built artifacts Github Actions uploads built artifacts Nov 27, 2019
@pchickey pchickey marked this pull request as ready for review December 2, 2019 17:31
@pchickey pchickey requested a review from sbc100 December 13, 2019 19:17
else
export VERSION=`./version.sh`
OUTDIR=$PWD/build
Copy link
Member

Choose a reason for hiding this comment

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

Default to "$PWD/dist"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@@ -40,3 +40,7 @@ jobs:
- run: sudo make -j4 package
shell: bash
name: Build
- uses: actions/upload-artifact@v1
with:
name: ${{ format( 'packages-{0}.tgz', matrix.os) }}
Copy link
Member

Choose a reason for hiding this comment

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

Can you explain this line?

What is packages-?

Is there any way to test this? Don't we want to limit this to only uploading for tags?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I renamed it dist to be more consistent, and added a comment.

The upload appears in the "Actions" tab on GH, and its not actually a release. It just allows us to download the Actions-built contents of dist, untar that into the actual built artifacts, and then upload those to a tag. I gave up on programming that sequence in yml, so we can do that by hand until someone dles.

Copy link
Member

Choose a reason for hiding this comment

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

Sweet! So we can get a copy of the sdk for each pull request. Nice. Out of interest do you know are disk space limits enforced on this kind of resource?

@pchickey pchickey merged commit 2bfea0c into master Dec 15, 2019
@pchickey pchickey deleted the pch/gh_actions_upload branch December 15, 2019 18:31
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.

None yet

2 participants