Re: Review Request 47843: Agent: Added Windows support for launcher/fetcher.cpp.

2016-05-27 Thread Alex Clemmer
> On May 25, 2016, 9:02 p.m., Joris Van Remoortere wrote: > > src/launcher/fetcher.cpp, lines 76-80 > > > > > > can we use the updated `paths::join` here to simplify this? > > > > Can you add a JIRA to add

Re: Review Request 47843: Agent: Added Windows support for launcher/fetcher.cpp.

2016-05-27 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47843/ --- (Updated May 28, 2016, 2:19 a.m.) Review request for mesos, Daniel Pravat,

Re: Review Request 47843: Agent: Added Windows support for launcher/fetcher.cpp.

2016-05-25 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47843/#review134833 --- src/launcher/fetcher.cpp (lines 76 - 80)

Review Request 47843: Agent: Added Windows support for launcher/fetcher.cpp.

2016-05-25 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47843/ --- Review request for mesos, Daniel Pravat, Artem Harutyunyan, Joris Van