Skip to content
This repository has been archived by the owner on Dec 15, 2018. It is now read-only.

Commit

Permalink
Don't run bazel version during setup.
Browse files Browse the repository at this point in the history
Summary: Since we're using `--output_user_root=/tmp/somewhere`, this content is discarded after setup anyway (not preserved in a snaphot)

Reviewers: naphat, kylec

Reviewed By: kylec

Subscribers: changesbot, wwu

Differential Revision: https://tails.corp.dropbox.com/D233195
  • Loading branch information
Anup Chenthamarakshan committed Sep 29, 2016
1 parent 96dda8f commit 842343b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions changes/utils/bazel_setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
# Assume that apt sources have been set up correctly up front, and `apt-get update` run already.
BASH_BAZEL_SETUP = """#!/bin/bash -eux
sudo apt-get install -y --force-yes {bazel_apt_pkgs}
/usr/bin/bazel --nomaster_blazerc --blazerc=/dev/null --output_user_root={bazel_root} --batch version
""".strip()

# We run setup again because changes does not run setup before collecting tests, but we need bazel
Expand Down
2 changes: 0 additions & 2 deletions tests/changes/models/test_job.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ def test_autogenerated_commands(self, get_config):

bazel_setup_expected = """#!/bin/bash -eux
sudo apt-get install -y --force-yes bazel
/usr/bin/bazel --nomaster_blazerc --blazerc=/dev/null --output_user_root=/bazel/root/path --batch version
""".strip()

sync_encap_expected = """
Expand Down

0 comments on commit 842343b

Please sign in to comment.