Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-09 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/#review91223 --- Ship it! Ship It! - Benjamin Hindman On July 8, 2015, 8:23

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/#review90883 --- Ship it! Great, looks good to me now. How about other files like

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/ --- (Updated July 8, 2015, 8:23 p.m.) Review request for mesos and Benjamin

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/#review90983 --- Bad patch! Reviews applied: [23784] Failed command:

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/#review91022 --- Patch looks great! Reviews applied: [36226] All tests passed. -

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/ --- (Updated July 8, 2015, 6:50 p.m.) Review request for mesos and Benjamin

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Isabel Jimenez
On July 8, 2015, 12:13 p.m., Till Toenshoff wrote: Great, looks good to me now. How about other files like e.g. 3rdparty/libprocess/configure.ac? Just added to review 23783, thanks! - Isabel --- This is an automatically generated

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-08 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/ --- (Updated July 8, 2015, 6:51 p.m.) Review request for mesos and Benjamin

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-07 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/#review90832 --- 3rdparty/libprocess/3rdparty/Makefile.am (lines 1 - 7)

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-07 Thread Isabel Jimenez
On July 7, 2015, 11:43 p.m., Till Toenshoff wrote: 3rdparty/libprocess/3rdparty/Makefile.am, lines 1-7 https://reviews.apache.org/r/36226/diff/1/?file=1000612#file1000612line1 This should be the “Apache License Version 2.0” header instead which applies to libprocess and stout,

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-07 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/ --- (Updated July 8, 2015, 12:12 a.m.) Review request for mesos and Benjamin

Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-06 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/ --- Review request for mesos and Benjamin Hindman. Repository: mesos-incubating

Re: Review Request 36226: Missing Apache headers for libprocess 3rdparty

2015-07-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36226/#review90621 --- Patch looks great! Reviews applied: [36226] All tests passed. -