> On Aug. 5, 2014, 4:11 p.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [24264] > > > > Failed command: git apply --index 24264.patch > > > > Error: > > error: missing binary patch data for '3rdparty/pip-1.5.6.tar.gz' > > error: binary patch does not apply to '3rdparty/pip-1.5.6.tar.gz' > > error: 3rdparty/pip-1.5.6.tar.gz: patch does not apply > > error: missing binary patch data for '3rdparty/wheel-0.24.0.tar.gz' > > error: binary patch does not apply to '3rdparty/wheel-0.24.0.tar.gz' > > error: 3rdparty/wheel-0.24.0.tar.gz: patch does not apply > > > > Thomas Rampelberg wrote: > The files are in the patch. Did I miss something? > > Vinod Kone wrote: > i suspect it's because of the binary files not being present in the diff. > i remember we had this issue before. @benh: do you remember how to upload > binary files to the diff?
I did a `git diff --full-index --binary master` and it looks like the patch includes the actual bytes. We'll see if reviewbot likes that or not. - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24264/#review49607 ----------------------------------------------------------- On Aug. 5, 2014, 8:10 p.m., Thomas Rampelberg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24264/ > ----------------------------------------------------------- > > (Updated Aug. 5, 2014, 8:10 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 a9cc6df5869616e9fe17fd68b88ee23e9a5ada15 > mpi/mpiexec-mesos.in 8812ee28c1f845bc3de40ffbf9e9d18033e450f2 > src/Makefile.am c741cdc9d59ad2c4e3132b080a8fcdd517942aa5 > 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 > >