----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67066/#review203366 -----------------------------------------------------------
src/tests/fetcher_tests.cpp Lines 1023-1026 (original), 1013-1019 (patched) <https://reviews.apache.org/r/67066/#comment285538> Should we just delete this part of the tests then, as it's obselete? - Andrew Schwartzmeyer On May 15, 2018, 11:30 a.m., John Kordich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67066/ > ----------------------------------------------------------- > > (Updated May 15, 2018, 11:30 a.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 > ------- > > Modified the fetcher to use libarchive and added associated tests. > > > Diffs > ----- > > src/launcher/fetcher.cpp 4cff7273fdc9e3897074da4da7dad02483086c2d > src/tests/fetcher_tests.cpp 311be6a36f137819ee91187a16df2b8e61def478 > > > Diff: https://reviews.apache.org/r/67066/diff/4/ > > > 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 > >
