----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66002/#review199478 -----------------------------------------------------------
PASS: Mesos patch 66002 was successfully built and tested. Reviews applied: `['66002']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/66002 - Mesos Reviewbot Windows On March 19, 2018, 6:20 p.m., John Kordich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66002/ > ----------------------------------------------------------- > > (Updated March 19, 2018, 6:20 p.m.) > > > Review request for mesos, Akash Gupta, Andrew Schwartzmeyer, Jeff Coffler, > and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > Many mesos frameworks assume that path separators are forward slashes, > like dcos-ui and marathon-ui. dcos-cli also assumes this. Previous to > this patch, if a forward slash was given in the path variable to an HTTP > API function call to the mesos agent on a Windows system, the Windows > API would fail at recognizing the path, because the Windows API accepts > only backslashes as path separators. To remedy this issue, we will now > convert all forward slashes passed as a path to the HTTP API to back > slashes on Windows agents by using the path::from_uri function. > > > Diffs > ----- > > src/files/files.cpp 5f92d2af722c3201200fa1d6a75dc0b87fdc6078 > src/tests/files_tests.cpp c703cae03345112715aeab83cb0a74abe3e12469 > > > Diff: https://reviews.apache.org/r/66002/diff/3/ > > > Testing > ------- > > Tested on both Windows and Linux, all tests pass, including the two newly > enabled tests on Windows. > > > Thanks, > > John Kordich > >
