diff options
| author | jessicag <jessicag@chromium.org> | 2016-02-23 10:53:09 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-02-23 18:54:20 +0000 |
| commit | 0c26c3e28214d5460a511817c6a69bd91b3afc18 (patch) | |
| tree | 1ffc2f1a197f8b004a14074c796bbc6cc66705fc /blimp | |
| parent | 391becb772b2f2006aa814d98c0fd2cbf1a02777 (diff) | |
| download | chromium_src-0c26c3e28214d5460a511817c6a69bd91b3afc18.zip chromium_src-0c26c3e28214d5460a511817c6a69bd91b3afc18.tar.gz chromium_src-0c26c3e28214d5460a511817c6a69bd91b3afc18.tar.bz2 | |
Ensure files in Blimp engine bundle are group readable.
Some build environments (like the buildbots) output files that aren't group readable. This can cause a problem when doing a docker build because docker may not be able to read the files once it untars the bundle. Makes sure all files in the bundle are group readable and are group executable if user executable.
BUG=577309
R=maniscalco@chromium.org
Review URL: https://codereview.chromium.org/1710423003
Cr-Commit-Position: refs/heads/master@{#377028}
Diffstat (limited to 'blimp')
| -rwxr-xr-x | blimp/tools/bundle-engine.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/blimp/tools/bundle-engine.py b/blimp/tools/bundle-engine.py index 4c3aa04..e94cf22 100755 --- a/blimp/tools/bundle-engine.py +++ b/blimp/tools/bundle-engine.py @@ -66,6 +66,10 @@ def main(): subprocess.check_output( ["tar", "-zcf", args.output, + # Ensure tarball content group permissions are appropriately set for + # use as part of a "docker build". That is group readable with + # executable files also being group executable. + "--mode=g+rX", "-C", dockerfile_dirname, dockerfile_basename, "-C", startup_script_dirname, startup_script_basename, "-C", args.build_dir] + deps, |