Re: Review Request 60892: Fixed the hard-coded forward slash which breaks windows docker container task launching in DC/OS path

2017-07-14 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60892/#review180624 --- Ship it! This is a small fix, so I'm going to point out the

Re: Review Request 60892: Fixed the hard-coded forward slash which breaks windows docker container task launching in DC/OS path

2017-07-14 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60892/#review180621 --- Fix it, then Ship it! Other than style, this patch looks good.

Re: Review Request 60892: Fixed the hard-coded forward slash which breaks windows docker container task launching in DC/OS path

2017-07-14 Thread Li Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60892/ --- (Updated July 15, 2017, 1:13 a.m.) Review request for mesos, Andrew

Re: Review Request 60892: Fixed the hard-coded forward slash which breaks windows docker container task launching in DC/OS path

2017-07-14 Thread Li Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60892/ --- (Updated July 15, 2017, 1:13 a.m.) Review request for mesos, Andrew

Re: Review Request 60892: Fixed the hard-coded forward slash which breaks windows docker container task launching in DC/OS path

2017-07-14 Thread Li Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60892/ --- (Updated July 15, 2017, 1:11 a.m.) Review request for mesos, Andrew

Review Request 60892: Fixed the hard-coded forward slash which breaks windows docker container task launching in DC/OS path

2017-07-14 Thread Li Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60892/ --- Review request for mesos, Andrew Schwartzmeyer and Joseph Wu. Bugs: MESOS-7797