Re: Review Request 25447: Update deploy_dir from localstate to sysconf

2014-09-09 Thread Timothy St. Clair
> On Sept. 9, 2014, 12:55 a.m., Vinod Kone wrote: > > This probably needs a call out in the CHANGELOG so that people are not > > surprised by it. Also, do you know of anyone using/depending on it? If yes, > > this probably needs to go through a deprecation cycle. > > Timothy St. Clair wrote: >

Re: Review Request 25447: Update deploy_dir from localstate to sysconf

2014-09-09 Thread Vinod Kone
> On Sept. 9, 2014, 12:55 a.m., Vinod Kone wrote: > > This probably needs a call out in the CHANGELOG so that people are not > > surprised by it. Also, do you know of anyone using/depending on it? If yes, > > this probably needs to go through a deprecation cycle. > > Timothy St. Clair wrote: >

Re: Review Request 25447: Update deploy_dir from localstate to sysconf

2014-09-09 Thread Timothy St. Clair
> On Sept. 9, 2014, 12:55 a.m., Vinod Kone wrote: > > This probably needs a call out in the CHANGELOG so that people are not > > surprised by it. Also, do you know of anyone using/depending on it? If yes, > > this probably needs to go through a deprecation cycle. Are there any mechanics to aut

Re: Review Request 25447: Update deploy_dir from localstate to sysconf

2014-09-08 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25447/#review52659 --- Ship it! This probably needs a call out in the CHANGELOG so that pe

Re: Review Request 25447: Update deploy_dir from localstate to sysconf

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

Review Request 25447: Update deploy_dir from localstate to sysconf

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