Removed 'source activate' from new CLI wrapper scripts. Having these command in these scripts was only necessary if you wanted to run them from outside the virtual environment they were meant to run in. Since we will soon be allowing people to create this virtual environment anywhere on their filesystem, it doesn't make sense to assume they have done it in the 'src/python/cli_new' directory anymore.
Since we can't assume that, we will just have to mandate that the virtualenv is actually set up before invoking these scripts. Review: https://reviews.apache.org/r/65585/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/237a121c Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/237a121c Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/237a121c Branch: refs/heads/master Commit: 237a121c47221cadcaa7836a48b5ed0371bf03f3 Parents: 2f7f03f Author: Armand Grillet <agril...@mesosphere.io> Authored: Thu Apr 12 14:43:54 2018 +0200 Committer: Kevin Klues <klue...@gmail.com> Committed: Thu Apr 12 14:55:23 2018 +0200 ---------------------------------------------------------------------- src/python/cli_new/bin/mesos | 1 - src/python/cli_new/bin/mesos-cli-tests | 1 - 2 files changed, 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/237a121c/src/python/cli_new/bin/mesos ---------------------------------------------------------------------- diff --git a/src/python/cli_new/bin/mesos b/src/python/cli_new/bin/mesos index c5152a2..aac5400 100755 --- a/src/python/cli_new/bin/mesos +++ b/src/python/cli_new/bin/mesos @@ -2,5 +2,4 @@ CURRDIR="$(cd "$(dirname "$0")" && pwd)" -source ${CURRDIR}/../activate python ${CURRDIR}/main.py "$@" http://git-wip-us.apache.org/repos/asf/mesos/blob/237a121c/src/python/cli_new/bin/mesos-cli-tests ---------------------------------------------------------------------- diff --git a/src/python/cli_new/bin/mesos-cli-tests b/src/python/cli_new/bin/mesos-cli-tests index 07659e0..9157d49 100755 --- a/src/python/cli_new/bin/mesos-cli-tests +++ b/src/python/cli_new/bin/mesos-cli-tests @@ -2,5 +2,4 @@ CURRDIR="$(cd "$(dirname "$0")" && pwd)" -source ${CURRDIR}/../activate python ${CURRDIR}/../tests/main.py "$@"