Merge pull request #2153 from weaveworks/mike/track-build-external

Fix ownership issues with client/build-external
This commit is contained in:
Mike Lang
2017-01-27 15:25:52 -08:00
committed by GitHub
2 changed files with 4 additions and 0 deletions

View File

@@ -174,6 +174,9 @@ ui-build-pkg:
clean:
$(GO) clean ./...
# build-external was accidentially created as owned by root in earlier versions. This is fixed now,
# but we need to fix up old checkouts, since git doesn't track ownership.
if [ "$$(stat -c %%U client/build-external)" == "root" ]; then sudo chown "$$USER:$$USER" client/build-external; fi
# Don't actually rmi the build images - rm'ing the .uptodate files is enough to ensure
# we rebuild the images, and rmi'ing the images causes us to have to redownload a lot of stuff.
# $(SUDO) docker rmi $(SCOPE_UI_BUILD_IMAGE) $(SCOPE_BACKEND_BUILD_IMAGE) >/dev/null 2>&1 || true

1
client/build-external/.gitignore vendored Normal file
View File

@@ -0,0 +1 @@
# this file is only present to force git to create this directory in the working tree