> On June 4, 2018, 1:50 p.m., Joseph Wu wrote: > > src/Makefile.am > > Lines 218 (patched) > > <https://reviews.apache.org/r/67064/diff/7/?file=2034659#file2034659line218> > > > > Won't need this line when linking against the bundled libarchive since > > we'll be adding the library to libmesos further down the makefile.
Ah, nevermind. This becomes necessary as the build rules for libmesos and mesos-fetcher are slightly separate in the automake build. - Joseph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67064/#review204069 ----------------------------------------------------------- On June 4, 2018, 12:11 p.m., John Kordich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67064/ > ----------------------------------------------------------- > > (Updated June 4, 2018, 12:11 p.m.) > > > Review request for mesos, Akash Gupta, Andrew Schwartzmeyer, Chun-Hung Hsiao, > Eric Mumau, Jie Yu, Joseph Wu, Li Li, and Radhika Jandhyala. > > > Bugs: MESOS-8064 > https://issues.apache.org/jira/browse/MESOS-8064 > > > Repository: mesos > > > Description > ------- > > These libraries being available will allow stout to be changed such that > stout can invoke libarchive instead of shelling out to tar to extract > archives. On Windows, tar usually doesn't exist, and even if it does, it > rarely supports most compression formats. On Windows, we will build > libarchive to support each of those compression formats. > > > Diffs > ----- > > 3rdparty/CMakeLists.txt ecb6946401d9b81c6610cf9f33dcf2caa9ff0f04 > 3rdparty/Makefile.am 062df18643cadb4f8f5bcbd56c1dad63a7b8f894 > 3rdparty/bzip2-1.0.6.patch PRE-CREATION > 3rdparty/cmake/Versions.cmake 2828acd2aa00c8778c6c462bc594716b2b6289ba > 3rdparty/libarchive-3.3.2.patch PRE-CREATION > 3rdparty/versions.am ada998d62d012a45b2cf17256c1ae16b92d611f2 > 3rdparty/xz-5.2.3.patch PRE-CREATION > configure.ac 03d333d65bcab2e46cff0b1329e5ad7bb26da697 > src/Makefile.am b7184ceccef5f2e985d905c155156f95c7a7c7b4 > > > Diff: https://reviews.apache.org/r/67064/diff/8/ > > > Testing > ------- > > I've built on Windows with CMake and have run all tests, which all pass. > I've also built on Linux with both the autotools build and cmake build and > have run all tests, which all pass. > > > Thanks, > > John Kordich > >