Re: Review Request 39383: [OOPS, SORRY, I PUSHED THIS ONE BY ACCIDENT] Windows: Moved `os::mkdir` to its own file, `stout/os/mkdir`.

2015-10-16 Thread Alex Clemmer
ut of the dependency chain though because I wanted to make it explicit that this review should not be slurped up when Joris applies them. - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Review Request 39377: Windows: Add Windows support to `process/socket.hpp`.

2015-10-16 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39377/#review102974 ------- On Oct. 16, 2015, 7:50 p.m., Alex Clemmer wrote: > > -

Re: Review Request 39380: Windows: Added support for `stout/flags/flags.hpp`.

2015-10-17 Thread Alex Clemmer
d e-mail. To reply, visit: https://reviews.apache.org/r/39380/#review102978 ------- On Oct. 16, 2015, 7:50 p.m., Alex Clemmer wrote: > > --- > This is an automatically generat

Re: Review Request 39382: Windows: Moved `os::rm` to its own file, `stout/os/rm.hpp`.

2015-10-17 Thread Alex Clemmer
/stout/windows/os.hpp 2b0966889af73238a08e29f1136d0ce286a0ffda Diff: https://reviews.apache.org/r/39382/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39382: Windows: Moved `os::rm` to its own file, `stout/os/rm.hpp`.

2015-10-17 Thread Alex Clemmer
led to apply patch Looks like someone pushed to master after I rebased. Sorry! - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39382/#review102993 ---------

Re: Review Request 39219: Windows: Added support for `slave/state.cpp`.

2015-10-18 Thread Alex Clemmer
`make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39262: Windows:[1/3] Moved `os::environ` -> `os::raw::environment`.

2015-10-18 Thread Alex Clemmer
stout/windows/os.hpp 2b0966889af73238a08e29f1136d0ce286a0ffda 3rdparty/libprocess/3rdparty/stout/tests/os_tests.cpp ee2a7a79617612ed448e650cc77608c4962fe5a5 Diff: https://reviews.apache.org/r/39262/diff/ Testing --- `make check` on Ubuntu 15, OS X 10.10, and running the `check` project on Windows

Re: Review Request 39220: CMake: Added `slave/state.cpp` to Windows builds.

2015-10-19 Thread Alex Clemmer
ran `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39375: Windows: Introduced socket flag interop.

2015-10-19 Thread Alex Clemmer
`make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39378: Windows: Added support for `process/subprocess.hpp`.

2015-10-19 Thread Alex Clemmer
. Thanks, Alex Clemmer

Re: Review Request 39375: Windows: Introduced socket flag interop.

2015-10-19 Thread Alex Clemmer
check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39377: Windows: Add Windows support to `process/socket.hpp`.

2015-10-19 Thread Alex Clemmer
--- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39379: Windows: Prepared agent for Windows changes to `stout/flags/flags.hpp`.

2015-10-19 Thread Alex Clemmer
10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39376: Windows: Prepared agent for Windows support of `process/socket.hpp`.

2015-10-19 Thread Alex Clemmer
10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39381: CMake: Added protobuf and `slave/flags.cpp` to Windows builds.

2015-10-19 Thread Alex Clemmer
c85bddbda3a17b45d7e31517ddf70358ebde1b41 Diff: https://reviews.apache.org/r/39381/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39382: Windows: Moved `os::rm` to its own file, `stout/os/rm.hpp`.

2015-10-19 Thread Alex Clemmer
/stout/windows/os.hpp eaeed7393b1c44f04737e18cd90a6fc292aa061d Diff: https://reviews.apache.org/r/39382/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39380: Windows: Added support for `stout/flags/flags.hpp`.

2015-10-19 Thread Alex Clemmer
/3rdparty/stout/include/stout/flags/parse.hpp e34ed78386130a1e757be1b1f1a2dbf794b7cd37 Diff: https://reviews.apache.org/r/39380/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex

Re: Review Request 39378: Windows: Added support for `process/subprocess.hpp`.

2015-10-19 Thread Alex Clemmer
, Alex Clemmer

Re: Review Request 39220: CMake: Added subset of agent target in Windows builds.

2015-10-19 Thread Alex Clemmer
` (or equivalent) on Windows 10, Ubuntu 15, OS X 10.10. Built autotools solution and ran `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39382: Windows: Moved `os::rm` to its own file, `stout/os/rm.hpp`.

2015-10-20 Thread Alex Clemmer
/os.hpp eaeed7393b1c44f04737e18cd90a6fc292aa061d Diff: https://reviews.apache.org/r/39382/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39381: CMake: Added protobuf and `slave/flags.cpp` to Windows builds.

2015-10-20 Thread Alex Clemmer
Diff: https://reviews.apache.org/r/39381/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39220: CMake: Added subset of agent target in Windows builds.

2015-10-20 Thread Alex Clemmer
://reviews.apache.org/r/39220/diff/ Testing --- Built CMake solution and ran `make check` (or equivalent) on Windows 10, Ubuntu 15, OS X 10.10. Built autotools solution and ran `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39220: CMake: Added subset of agent target in Windows builds.

2015-10-21 Thread Alex Clemmer
--- Built CMake solution and ran `make check` (or equivalent) on Windows 10, Ubuntu 15, OS X 10.10. Built autotools solution and ran `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39019: [WORK IN PROGRESS, NOT IMMEDIATELY NECESSARY] Windows: Added dirent compat code for non-Unix systems.

2015-10-21 Thread Alex Clemmer
` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39537: Windows: Moved `os::utime` to its own file, `stout/os/utime.hpp`.

2015-10-21 Thread Alex Clemmer
` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39538: Windows: Moved `os::touch` to its own file, `stout/os/touch.hpp`.

2015-10-21 Thread Alex Clemmer
/libprocess/3rdparty/stout/include/stout/windows/os.hpp 3e6f2aafd0f541f512025dfa683ab4178701f7c4 Diff: https://reviews.apache.org/r/39538/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex

Review Request 39540: Windows: Moved `os::getcwd` to its own file, `stout/os/getcwd.hpp`.

2015-10-21 Thread Alex Clemmer
Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39541: Windows: Moved `os::chdir` to its own file, `stout/os/chdir.hpp`.

2015-10-21 Thread Alex Clemmer
Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39539: Windows: Moved `os::mktemp` to its own file, `stout/os/mktemp.hpp`.

2015-10-21 Thread Alex Clemmer
://reviews.apache.org/r/39539/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39539: Windows: Moved `os::mktemp` to its own file, `stout/os/mktemp.hpp`.

2015-10-21 Thread Alex Clemmer
right? - Alex Clemmer On Oct. 22, 2015, 3:35 a.m., Alex Clemmer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 39219: Windows: Added support for `slave/state.cpp`.

2015-10-22 Thread Alex Clemmer
cialization though! That blows my mind. - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39219/#review103062 -------

Re: Review Request 39537: Windows: Moved `os::utime` to its own file, `stout/os/utime.hpp`.

2015-10-22 Thread Alex Clemmer
rts clarity, because it is not obvious. - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39537/#review103599 -------

Re: Review Request 39383: Windows: Moved `os::mkdir` to its own file, `stout/os/mkdir`.

2015-10-22 Thread Alex Clemmer
/stout/os/mkdir.hpp PRE-CREATION Diff: https://reviews.apache.org/r/39383/diff/ Testing --- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39383: Windows: Moved `os::mkdir` to its own file, `stout/os/mkdir`.

2015-10-22 Thread Alex Clemmer
--- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-22 Thread Alex Clemmer
/stout/posix/os.hpp b6afe0e76366d0bc68d37097ced83a1e14828d84 3rdparty/libprocess/3rdparty/stout/include/stout/windows/os.hpp 3e6f2aafd0f541f512025dfa683ab4178701f7c4 Diff: https://reviews.apache.org/r/39559/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39219: Windows: Added support for `slave/state.cpp`.

2015-10-22 Thread Alex Clemmer
and ran `make check` on Ubuntu 15. Thanks, Alex Clemmer

Review Request 39560: CMake: Add state.cpp, flags.cpp to Windows agent build.

2015-10-22 Thread Alex Clemmer
Wu. Repository: mesos Description --- CMake: Add state.cpp, flags.cpp to Windows agent build. Diffs - src/CMakeLists.txt e6169a0e3ad34dd0e4c3430a6532bd48c4bd04fd Diff: https://reviews.apache.org/r/39560/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-22 Thread Alex Clemmer
an't commit this stuff with tests" is "yes, I agree, but I have to write this before we can open up the tests!" :) - Alex Clemmer On Oct. 22, 2015, 6:23 p.m., Alex Clemmer wrote: > > --- > This is an automat

Re: Review Request 39383: Windows: Moved `os::mkdir` to its own file, `stout/os/mkdir`.

2015-10-22 Thread Alex Clemmer
he answer is yes. Also, you might be leery of committing this without tests. That's ok! We need this to pull the FS tests out of `os_tests.cpp` though. The last function we will need to port to get those tests functional on Windows is `rmdir`, and I'm almost done. - Alex Clemme

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-22 Thread Alex Clemmer
> On Oct. 22, 2015, 6:33 p.m., Alex Clemmer wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/mkdtemp.hpp, > > line 31 > > <https://reviews.apache.org/r/39559/diff/1/?file=1103501#file1103501line31> > > > > Just so I catch it be

Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-23 Thread Alex Clemmer
on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39583: Windows: Added `WindowsError` to parallel `ErrnoError`.

2015-10-23 Thread Alex Clemmer
, Alex Clemmer

Re: Review Request 39583: Windows: Added `WindowsError` to parallel `ErrnoError`.

2015-10-23 Thread Alex Clemmer
https://reviews.apache.org/r/39583/#review103746 ------- On Oct. 23, 2015, 8:57 a.m., Alex Clemmer wrote: > > --- > This is an automatically generated e-m

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-23 Thread Alex Clemmer
autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39540: Windows: Moved `os::getcwd` to its own file, `stout/os/getcwd.hpp`.

2015-10-23 Thread Alex Clemmer
views.apache.org/r/39540/#review103800 --- On Oct. 22, 2015, 3:36 a.m., Alex Clemmer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39540/ >

Review Request 39622: CMake: Pointed Stout test linker flags at correct gtest directory.

2015-10-24 Thread Alex Clemmer
--- `make check` from CMake on OS X 10.10. `make check` from Autotools on OS X 10.10. Ran `check` project in VS on Windows 10. PLEASE NOTE: I am on a terrible network with proxy problems and I can't SSH into my Ubuntu box to test this from Ubuntu Thanks, Alex Clemmer

Review Request 39621: CMake: Corrected linking path for gmock libraries on Windows builds.

2015-10-24 Thread Alex Clemmer
--- `make check` from CMake on OS X 10.10. `make check` from Autotools on OS X 10.10. Ran `check` project in VS on Windows 10. PLEASE NOTE: I am on a terrible network with proxy problems and I can't SSH into my Ubuntu box to test this from Ubuntu Thanks, Alex Clemmer

Review Request 39623: CMake: Added protobuf compilation to Windows builds.

2015-10-24 Thread Alex Clemmer
Autotools on OS X 10.10. Ran `check` project in VS on Windows 10. PLEASE NOTE: I am on a terrible network with proxy problems and I can't SSH into my Ubuntu box to test this from Ubuntu Thanks, Alex Clemmer

Review Request 39620: Windows: Included headers to make `stout/windows.hpp` standalone.

2015-10-24 Thread Alex Clemmer
--- `make check` from CMake on OS X 10.10. `make check` from Autotools on OS X 10.10. Ran `check` project in VS on Windows 10. PLEASE NOTE: I am on a terrible network with proxy problems and I can't SSH into my Ubuntu box to test this from Ubuntu Thanks, Alex Clemmer

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

2015-10-24 Thread Alex Clemmer
--- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

2015-10-24 Thread Alex Clemmer
` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

2015-10-24 Thread Alex Clemmer
; code. Yes, I admit it's a super nitpicky thing to do. - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39019/#review101663 ---------

Re: Review Request 39537: Windows: Moved `os::utime` to its own file, `stout/os/utime.hpp`.

2015-10-26 Thread Alex Clemmer
check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39539: Windows: Moved `os::mktemp` to its own file, `stout/os/mktemp.hpp`.

2015-10-26 Thread Alex Clemmer
7a8819541506f57f70a9b577dc97a76fc26ebaa8 Diff: https://reviews.apache.org/r/39539/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39540: Windows: Moved `os::getcwd` to its own file, `stout/os/getcwd.hpp`.

2015-10-26 Thread Alex Clemmer
/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39383: Windows: Moved `os::mkdir` to its own file, `stout/os/mkdir`.

2015-10-26 Thread Alex Clemmer
--- CMake `make check` on Ubuntu 15, OS X 10.10, and ran `check` target in VS2015 on Windows 10. Autotools `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-26 Thread Alex Clemmer
: https://reviews.apache.org/r/39559/#review103815 ------- On Oct. 22, 2015, 6:23 p.m., Alex Clemmer wrote: > > --- > This is an automatically generated e-

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-27 Thread Alex Clemmer
eviews.apache.org/r/39559/#review103815 --- On Oct. 22, 2015, 6:23 p.m., Alex Clemmer wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-27 Thread Alex Clemmer
3rdparty/libprocess/3rdparty/stout/include/stout/posix/os.hpp b6afe0e76366d0bc68d37097ced83a1e14828d84 3rdparty/libprocess/3rdparty/stout/include/stout/windows/os.hpp 3e6f2aafd0f541f512025dfa683ab4178701f7c4 Diff: https://reviews.apache.org/r/39559/diff/ Testing --- Thanks, Alex

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-27 Thread Alex Clemmer
> On Oct. 23, 2015, 6:45 p.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/mkdtemp.hpp, > > line 34 > > <https://reviews.apache.org/r/39559/diff/1/?file=1103501#file1103501line34> > > > > `strlen()` might be better

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-27 Thread Alex Clemmer
;re not using snake case. Have I missed something important? - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39559/#review103815 ------- On Oct. 27, 2015, 8:14 a.m., Alex Clemmer wrote: > &

Re: Review Request 39583: Windows: Added `WindowsError` to parallel `ErrnoError`.

2015-10-27 Thread Alex Clemmer
. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-27 Thread Alex Clemmer
autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-27 Thread Alex Clemmer
- Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39584/#review104054 --- On Oct. 27, 2015, 8:23 a.m., Ale

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-27 Thread Alex Clemmer
/rmdir.hpp (line 138) <https://reviews.apache.org/r/39584/#comment162433> note that actually `::rmdir` here will deal appropriately with Unix-style paths, so there's no real need to normalize them. - Alex Clemmer On Oct. 27, 2015, 8:23 a.m., Alex Cl

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-27 Thread Alex Clemmer
autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-27 Thread Alex Clemmer
autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

2015-10-29 Thread Alex Clemmer
3rdparty/libprocess/3rdparty/stout/include/stout/posix/os.hpp e26df59d9b837e1f4a4b92577f0a3de4b9076cb4 3rdparty/libprocess/3rdparty/stout/include/stout/windows/os.hpp edf17d5ad8efbc988e909bfb8ffa5a015ecdc89d Diff: https://reviews.apache.org/r/39559/diff/ Testing --- Thanks, Alex

Re: Review Request 39219: Windows: Added support for `slave/state.cpp`.

2015-10-29 Thread Alex Clemmer
and ran `make check` on Ubuntu 15. Thanks, Alex Clemmer

Re: Review Request 39560: CMake: Add state.cpp, flags.cpp to Windows agent build.

2015-10-29 Thread Alex Clemmer
--- Thanks, Alex Clemmer

Re: Review Request 39583: Windows: Added `WindowsError` to parallel `ErrnoError`.

2015-10-29 Thread Alex Clemmer
. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39621: CMake: Corrected linking path for gmock libraries on Windows builds.

2015-10-29 Thread Alex Clemmer
hanks, Alex Clemmer

Re: Review Request 39620: Windows: Included headers to make `stout/windows.hpp` standalone.

2015-10-29 Thread Alex Clemmer
hanks, Alex Clemmer

Re: Review Request 39622: CMake: Pointed Stout test linker flags at correct gtest directory.

2015-10-29 Thread Alex Clemmer
buntu Thanks, Alex Clemmer

Re: Review Request 39623: CMake: Added protobuf compilation to Windows builds.

2015-10-29 Thread Alex Clemmer
check` from CMake on OS X 10.10. `make check` from Autotools on OS X 10.10. Ran `check` project in VS on Windows 10. PLEASE NOTE: I am on a terrible network with proxy problems and I can't SSH into my Ubuntu box to test this from Ubuntu Thanks, Alex Clemmer

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

2015-10-29 Thread Alex Clemmer
` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

2015-10-29 Thread Alex Clemmer
autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39802: Windows: Implemented `stout/os/windows/ls.hpp`.

2015-10-29 Thread Alex Clemmer
, Alex Clemmer

Review Request 39804: Windows: Moved `os::find` to its own file, `stout/os/find.hpp`.

2015-10-31 Thread Alex Clemmer
, Alex Clemmer

Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

2015-10-31 Thread Alex Clemmer
1a7037d64afeedc340258c92067e95d1d3caa027 Diff: https://reviews.apache.org/r/39803/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39805: Moved filesystems tests to their own file.

2015-10-31 Thread Alex Clemmer
://reviews.apache.org/r/39805/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39834: Made `path_tests.cpp` standalone.

2015-10-31 Thread Alex Clemmer
Remoortere, and Joseph Wu. Repository: mesos Description --- Made `path_tests.cpp` standalone. Diffs - 3rdparty/libprocess/3rdparty/stout/tests/path_tests.cpp ad9ce324eaf940f68d04c6db7ba37d05efb1216a Diff: https://reviews.apache.org/r/39834/diff/ Testing --- Thanks, Alex

Re: Review Request 39805: Moved filesystems tests to their own file.

2015-10-31 Thread Alex Clemmer
/os_tests.cpp 5a1da57f7e27cf8154f0d5f6efd47dcee8a430ff Diff: https://reviews.apache.org/r/39805/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Re: Review Request 39805: Moved filesystems tests to their own file.

2015-10-31 Thread Alex Clemmer
tps://reviews.apache.org/r/39805/#comment162892> Oh, interesting, I thought that the lineup here was identical to that of `git diff`. I guess not. - Alex Clemmer On Nov. 1, 2015, 1:22 a.m., Alex Clemmer wrote: > > ---

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

2015-10-31 Thread Alex Clemmer
/3rdparty/stout/include/stout/windows.hpp 1a7037d64afeedc340258c92067e95d1d3caa027 Diff: https://reviews.apache.org/r/39803/diff/ Testing --- `make check` from autotools on Ubuntu 15. `make check` from CMake on OS X 10.10. Ran `check` project in VS on Windows 10. Thanks, Alex Clemmer

Review Request 39850: Process: Added headers to make `process/mime.hpp` standalone.

2015-11-02 Thread Alex Clemmer
--- Thanks, Alex Clemmer

Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
--- Thanks, Alex Clemmer

Review Request 39852: Windows: Replaced global `GetMessage` macro with inline function.

2015-11-02 Thread Alex Clemmer
/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
://reviews.apache.org/r/39851/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
://reviews.apache.org/r/39851/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
tps://reviews.apache.org/r/39851/#comment162948> Ah. VisualStudio tricked me into believing this was ok! It claimed this built correctly, but now when I `make clean` it doesn't. I'll have to revisit tomorrow, as it's 3:30 AM. - Alex Clemmer On Nov. 2, 2015, 11:06 a.m

Re: Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
://reviews.apache.org/r/39851/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
://reviews.apache.org/r/39851/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39834: Made `path_tests.cpp` standalone.

2015-11-02 Thread Alex Clemmer
/path_tests.cpp ad9ce324eaf940f68d04c6db7ba37d05efb1216a Diff: https://reviews.apache.org/r/39834/diff/ Testing --- Thanks, Alex Clemmer

Re: Review Request 39230: Added prevention of SASL deprecation warnings all around its invocations on OS X.

2015-11-02 Thread Alex Clemmer
230/#comment162998> Sorry I'm super late to the party. Does it make sense to add these to the CMake build system as well? - Alex Clemmer On Oct. 22, 2015, 10:02 a.m., Till Toenshoff wrote: > > --- > This is an automatica

Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

2015-11-02 Thread Alex Clemmer
g/r/39634/#comment162999> If it's not too much trouble, it would be great to see this logic added also the `cmake/CompilationCOnfigure.cmake`. It should only be a couple lines of code. - Alex Clemmer On Oct. 30, 2015, 5:05 a.m., David For

Re: Review Request 39719: Fixed marking mounts as slave in ubuntu.

2015-11-02 Thread Alex Clemmer
719/#comment163000> It looks like it might make sense to add this to `src/CMakeLists.txt` also? - Alex Clemmer On Nov. 2, 2015, 6:52 p.m., Timothy Chen wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 39005: stout: Added thread-safe replacement for strerror.

2015-11-02 Thread Alex Clemmer
s to its own tests file so that we can run it on Windows too? `os_test.cpp` has is too big and full of unimplemented stuff to add Windows support right now. - Alex Clemmer On Oct. 23, 2015, 7:49 a.m., Benjamin Bannier wrote: > > --

Re: Review Request 39851: Windows: fixed ambiguousity error in `process/owned.hpp`.

2015-11-02 Thread Alex Clemmer
://reviews.apache.org/r/39851/diff/ Testing --- Thanks, Alex Clemmer

Review Request 39888: Windows: Added compatibility code for `grp.h` and `pwd.h`.

2015-11-02 Thread Alex Clemmer
/diff/ Testing --- Thanks, Alex Clemmer

<    4   5   6   7   8   9   10   11   >