mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <and...@schwartzmeyer.com>
Subject Review Request 66753: Windows: Fixed build with OpenSSL due to missing header.
Date Fri, 20 Apr 2018 23:52:32 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message