----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10491/#review19225 -----------------------------------------------------------
src/launcher/launcher.cpp <https://reviews.apache.org/r/10491/#comment39818> not yours, but can you fix this? s/+ command/ << command/ src/launcher/launcher.cpp <https://reviews.apache.org/r/10491/#comment39817> s/+ command/ << command/ src/launcher/launcher.cpp <https://reviews.apache.org/r/10491/#comment39820> s/if/ If/ Also, we use line comments ("//") instead of block comments ("/*") src/launcher/launcher.cpp <https://reviews.apache.org/r/10491/#comment39819> s/+ command/ << command/ - Vinod Kone On April 15, 2013, 9:20 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10491/ > ----------------------------------------------------------- > > (Updated April 15, 2013, 9:20 p.m.) > > > Review request for mesos. > > > Description > ------- > > Support bzip2 and XZ (LZMA) by launcher. > > > Diffs > ----- > > src/launcher/launcher.cpp bc50c5f > > Diff: https://reviews.apache.org/r/10491/diff/ > > > Testing > ------- > > Used in production at Airbnb. > > > Thanks, > > Brenden Matthews > >
