----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66753/ -----------------------------------------------------------
Review request for mesos, Akash Gupta, Chun-Hung Hsiao, and Joseph Wu. Repository: mesos Description ------- When `-DENABLE_SSL=TRUE`, then `USE_SSL_SOCKET` is defined, and in this file in a section guarded by the definition, the construct `os::Permissions` is used. However, this is defined in `stout/os/permissions.hpp`, which is implicitly included on POSIX, but not on Windows. The fix is to IWYU and explicitly include it. Diffs ----- src/local/local.cpp 9c5b45b7c4e7705861f1e7926847711dc16f1264 Diff: https://reviews.apache.org/r/66753/diff/1/ Testing ------- Built on Windows with `-DENABLE_SSL`. Previously it failed due to `os::Permissions` being undefined. Thanks, Andrew Schwartzmeyer
