Skip to content

Enable uploading manifest to GitHub.#7898

Merged
jgraham merged 1 commit intomasterfrom
manifest_upload
Oct 19, 2017
Merged

Enable uploading manifest to GitHub.#7898
jgraham merged 1 commit intomasterfrom
manifest_upload

Conversation

@jgraham
Copy link
Contributor

@jgraham jgraham commented Oct 19, 2017

This uploads a MANIFEST.json.gz file for every commit to master, so
that we can build tooling that attempts to download that file as an
initial manifest that can then have a rapid update rather than a slow
build.


This change is Reviewable

WPT_ROOT=$(readlink -f $SCRIPT_DIR/../..)
cd $WPT_ROOT

mkdir -p ~/meta
Copy link
Member

Choose a reason for hiding this comment

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

I don't understand the use of ~ here, where will the file actually end up? Is it in the wpt repo itself?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is generated in the $HOME/meta directory on the travis machine, and then finally gets uploaded to the "releases" part of the GitHub repo (if it works correctly :). But since it's not tagged I don't think it will show up as an actual release. But I hope we can still download it.

Copy link
Member

Choose a reason for hiding this comment

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

OK, as long as it's not in the wpt repo itself, I have no concerns. Let's see what happens :)

This uploads a MANIFEST.json.gz file for every commit to master, so
that we can build tooling that attempts to download that file as an
initial manifest that can then have a rapid update rather than a slow
build.
@jgraham jgraham merged commit 1551e85 into master Oct 19, 2017
@gsnedders gsnedders deleted the manifest_upload branch October 19, 2017 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants