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

Commit

Permalink
Revert "removed piping of stderr in collect targets"
Browse files Browse the repository at this point in the history
Summary:
This reverts commit 9a4b66c.
Turns out that piping of stderr is necessary, because we want the output to be a JSON. The correct fix would be to only print out stderr if there is an error, but since we are moving this logic to Changes client, let's not spend more time working on this.

Test Plan: N/A

Reviewers: anupc

Reviewed By: anupc

Subscribers: kylec

Differential Revision: https://tails.corp.dropbox.com/D225079
  • Loading branch information
Naphat Sanguansin committed Aug 31, 2016
1 parent 0f28880 commit fbd6c51
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion changes/utils/bazel_setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
(sudo apt-get -y update || sudo apt-get -y update) >/dev/null 2>&1
sudo apt-get install -y --force-yes {bazel_apt_pkgs} python >/dev/null 2>&1
python -c "{script}" "{bazel_root}" "{bazel_targets}" "{bazel_exclude_tags}" "{max_jobs}"
python -c "{script}" "{bazel_root}" "{bazel_targets}" "{bazel_exclude_tags}" "{max_jobs}" 2> /dev/null
""".strip()


Expand Down

0 comments on commit fbd6c51

Please sign in to comment.