Skip to content

Commit

Permalink
Stop forcing /usr/bin to the front of the path
Browse files Browse the repository at this point in the history
Summary:
This messes with python version selection. At least based on
the comments, it is no longer needed, as /opt/local/bin appears to be
empty on my devserver and on a sandcastle host.

D32337004 revealed the problem.  T105920600 has some more context.  I couldn't repro locally, so D32451858 is a control (which I expect to fail tests) to confirm that this fixes the test failures.

#utd-hermes

Reviewed By: neildhar

Differential Revision: D32451851

fbshipit-source-id: 4c9c04571154d9ce4e5a3059d98e043865b76f78
  • Loading branch information
mhorowitz authored and facebook-github-bot committed Nov 17, 2021
1 parent 1e7da82 commit d123a6f
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions ReactCommon/hermes/inspector/tools/sandcastle/setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,6 @@ set -o pipefail
THIS_DIR=$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)
ROOT_DIR=$(cd "$THIS_DIR" && hg root)

# Buck by default uses clang-3.6 from /opt/local/bin.
# Override it to use system clang.
export PATH="/usr/bin:$PATH"

# Enter xplat
cd "$ROOT_DIR"/xplat || exit 1

Expand Down

0 comments on commit d123a6f

Please sign in to comment.