RFR (trivial): 8235833: PosixPlatform.cpp should not include sysctl.h
suenaga at oss.nttdata.com
Thu Dec 12 13:23:41 UTC 2019
I can see same error whether --disable-precompiled-headers or not.
So I filed it to JBS and uploaded webrev.
Could you review it?
It works fine on Fedora 31.
I've pushed it to submit repo. I believe it passes all tests.
On 2019/12/12 19:25, Andrew Haley wrote:
> On 12/12/19 9:31 AM, Yasumasa Suenaga wrote:
>> AFAICS sysctl.h provides `sysctl()` and some macros, but they are not used in PosixPlatform.cpp .
>> In fact, I did not see any error when I remove `#include <sys/sysctl.h>` from it.
> You probably won't see an error unless you configure with
> --disable-precompiled-headers. Please try that, and we can then
> consider removing that #include.
More information about the core-libs-dev