> On Aug. 6, 2014, 10:44 p.m., Benjamin Hindman wrote: > > mpi/mpiexec-mesos.in, line 32 > > <https://reviews.apache.org/r/24264/diff/2/?file=652335#file652335line32> > > > > Can we throw a comment above this block to explain what it is that > > you're doing? It's starting to get more and more complicated.
Looking at it, there's a way to simplify this whole thing significantly. - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24264/#review49824 ----------------------------------------------------------- On Aug. 7, 2014, 7 p.m., Thomas Rampelberg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24264/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2014, 7 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Bugs: MESOS-899 > https://issues.apache.org/jira/browse/MESOS-899 > > > Repository: mesos-git > > > Description > ------- > > Because eggs are impossible to uninstall, pip + wheel has been added as a > packaging format. The wheels are generated in addition to eggs and then > subsequently installed/uninstalled. > > Note that the building of eggs has not been removed. This is because wheels > cannot be run in place like eggs are via. modification of PYTHONPATH. The eggs > are needed for all the test scripts to run correctly without actually > installing anything locally. > > > Diffs > ----- > > 3rdparty/Makefile.am 70b45fe8b846a2a3fda599c0b5b7cfa5eb7e78e0 > 3rdparty/pip-1.5.6.tar.gz PRE-CREATION > 3rdparty/versions.am cd7c1cf087dbcfd385ac33145a562764e426c5e5 > 3rdparty/wheel-0.24.0.tar.gz PRE-CREATION > configure.ac 8fb0a3a794db4d3671243d06ff45232eae53c27b > mpi/mpiexec-mesos.in 8812ee28c1f845bc3de40ffbf9e9d18033e450f2 > src/Makefile.am 39af0365e429b8d08addadb09ee18080a19625f8 > src/examples/python/test-containerizer.in > f71828db98a90f455c88d90cb4e3320b7e8c9e9e > src/examples/python/test-executor.in > b22e7a7dc0c26f805eb63c7139066ce7dc830636 > src/examples/python/test-framework.in > 64fb1ddc1a0e5772c12d7497dfc1cf6ca2a7dceb > > Diff: https://reviews.apache.org/r/24264/diff/ > > > Testing > ------- > > `make distcheck -j6` > > > Thanks, > > Thomas Rampelberg > >