launchpad-reviewers team mailing list archive

[Colin Watson <mp+442018@xxxxxxxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:build-tarball-yarn into launchpad:master.

Commit message:
Include download-cache/yarn/ in deployment tarballs

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/442018

This adds about 25 MiB to the size of those tarballs, but it makes it possible for us to run `make build` in a charmed deployment, which is needed for deploying our frontend assets using Juju.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:build-tarball-yarn into launchpad:master.

diff --git a/utilities/build-tarball b/utilities/build-tarball
index 22cba4a..afacd81 100755
--- a/utilities/build-tarball
+++ b/utilities/build-tarball
@@ -30,6 +30,7 @@ mkdir -p "$output_dir"
     # Most of download-cache is unnecessary since we include a wheelhouse
     # instead, but JavaScript-enabled builds need yarn.
     find ./download-cache/dist/ -name yarn-\*.tar.gz
+    find ./download-cache/yarn/ -type f
     echo ./version-info.py
     echo ./version-info.txt
     find ./wheels/ -name \*.whl -print