Mailing-List: contact mesos-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: mesos-dev@incubator.apache.org
Content-Type: multipart/alternative;
	boundary="===============1301713616873575703=="
MIME-Version: 1.0
Subject: Re: Review Request: Remove almost all remaining uses of MESOS_HOME
From: "Charles Reiss" <woggling@gmail.com>
To: "Benjamin Hindman" <benh@berkeley.edu>
Date: Fri, 23 Mar 2012 06:04:06 -0000
Message-ID: <20120323060406.32682.79142@reviews.apache.org>
Cc: "mesos" <mesos-dev@incubator.apache.org>,
	"Charles Reiss" <woggling@gmail.com>
In-Reply-To: <20120321190207.32682.42960@reviews.apache.org>
References: <20120321190207.32682.42960@reviews.apache.org>

--===============1301713616873575703==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4434/
-----------------------------------------------------------

(Updated 2012-03-23 06:04:06.888041)


Review request for mesos and Benjamin Hindman.


Summary
-------

Removes most remaining uses of MESOS_HOME from the tree.

Includes updates to the deploy scripts, but does not fix the deploy scripts=
 (since mesos-getconf still won't exist).

After this patch, the remaining uses of MESOS_HOME I'm aware of are in the =
EC2 scripts.


This addresses bug MESOS-85.
    https://issues.apache.org/jira/browse/MESOS-85


Diffs (updated)
-----

  src/common/logging.cpp 9e346f2 =

  src/configurator/configurator.hpp a3a104f =

  src/configurator/configurator.cpp 2666e50 =

  src/deploy/mesos-daemon 2e75d04 =

  src/deploy/mesos-env.sh 9f93a41 =

  src/launcher/launcher.hpp 221094f =

  src/launcher/launcher.cpp 98a4847 =

  src/launcher/main.cpp 5c55ace =

  src/local/main.cpp d4b68a9 =

  src/master/main.cpp a8fadc9 =

  src/mesos/main.cpp bbdb981 =

  src/sched/sched.cpp 16770a1 =

  src/slave/lxc_isolation_module.cpp 8c25dd4 =

  src/slave/main.cpp ac780c4 =

  src/slave/process_based_isolation_module.cpp e0f3ee8 =

  src/slave/slave.cpp 9332caa =

  src/slave/webui.cpp 9f45442 =

  src/tests/configurator_tests.cpp 662e01f =

  src/tests/external_tests.cpp 21fa705 =


Diff: https://reviews.apache.org/r/4434/diff


Testing
-------

make check


Thanks,

Charles


--===============1301713616873575703==--