----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52707/#review152114 -----------------------------------------------------------
Patch looks great! Reviews applied: [52707] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On Oct. 10, 2016, 9:42 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52707/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2016, 9:42 p.m.) > > > Review request for mesos, Artem Harutyunyan, Kevin Klues, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > A locally installed virtualenv does not always show up on the `PATH`, > which fails `which virtualenv`. This adds an extra search location > based on the user's site package install directory. > > > Diffs > ----- > > src/cli_new/bootstrap 74a8568daef4e0cb93911352cc73c165a2fbea1d > > Diff: https://reviews.apache.org/r/52707/diff/ > > > Testing > ------- > > pip uninstall virtualenv -y # Global > pip uninstall virtualenv -y # Local > > src/cli_new/bootstrap # Fails > > pip install --user virtualenv > > src/cli_new/bootstrap # Succeeds > > pip uninstall virtualenv -y # Local > pip install virtualenv # Global > > src/cli_new/bootstrap # Succeeds > > > Thanks, > > Joseph Wu > >
