From 4bfc66b3a955ebc82779f8eaa7b4c56a52d4b4c0 Mon Sep 17 00:00:00 2001 From: Evan Martin Date: Tue, 17 Jul 2012 09:34:10 -0700 Subject: [PATCH] subprocess_test: make SetWithLots run on all non-Windows platforms I think it might have been an oversight that this was marked Linux-specific. --- src/subprocess_test.cc | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/subprocess_test.cc b/src/subprocess_test.cc index 0c10250..7dbaf97 100644 --- a/src/subprocess_test.cc +++ b/src/subprocess_test.cc @@ -16,9 +16,11 @@ #include "test.h" -#ifdef linux -// required by SubprocessTest +#ifndef _WIN32 +// SetWithLots need setrlimit. +#include #include +#include #endif namespace { @@ -147,7 +149,7 @@ TEST_F(SubprocessTest, SetWithMulti) { } } -#ifdef linux +#ifndef _WIN32 TEST_F(SubprocessTest, SetWithLots) { // Arbitrary big number; needs to be over 1024 to confirm we're no longer // hostage to pselect. @@ -174,4 +176,4 @@ TEST_F(SubprocessTest, SetWithLots) { } ASSERT_EQ(kNumProcs, subprocs_.finished_.size()); } -#endif // linux +#endif // _WIN32 -- 2.7.4