Re: Review Request 25355: Minor update to include package config file

2014-09-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25355/#review52448 --- Patch looks great! Reviews applied: [25355] All tests passed. - M

Re: Review Request 25355: Minor update to include package config file

2014-09-04 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25355/#review52398 --- Ship it! Ship It! - Vinod Kone On Sept. 4, 2014, 9 p.m., Timothy

Re: Review Request 25355: Minor update to include package config file

2014-09-04 Thread Timothy St. Clair
> On Sept. 4, 2014, 9:19 p.m., Vinod Kone wrote: > > Makefile.am, lines 27-28 > > > > > > i'm likely missing context. what is this for? A while back we added a package config file, or .pc file. This mod installs it i

Re: Review Request 25355: Minor update to include package config file

2014-09-04 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25355/#review52358 --- Makefile.am

Review Request 25355: Minor update to include package config file

2014-09-04 Thread Timothy St. Clair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25355/ --- Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone. Bugs: MESOS