From 6502bca2773bfa3fe1d343c2ae071dd5d1ced8d0 Mon Sep 17 00:00:00 2001 From: Dave Watson Date: Thu, 15 Jan 2015 13:25:02 -0800 Subject: [PATCH] 'fix' threadlocal fork test Summary: configure seems to think ubuntu 14.04 doesn't have pthread_atfork, which seems wrong?! But this test seems to fail if we don't have it on some platform anyway. Not sure the right fix, just disable it for now? Test Plan: watch https://ci-builds.fb.com/job/folly/ Reviewed By: hans@fb.com Subscribers: doug, folly-diffs@ FB internal diff: D1783046 Signature: t1:1783046:1421348575:d279877abcc597a04c285092a5e5e53406922d5a Blame Revision: D1768669 --- folly/test/ThreadLocalTest.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/folly/test/ThreadLocalTest.cpp b/folly/test/ThreadLocalTest.cpp index 5c92d40c..f2a48862 100644 --- a/folly/test/ThreadLocalTest.cpp +++ b/folly/test/ThreadLocalTest.cpp @@ -437,6 +437,7 @@ int totalValue() { } // namespace +#ifdef FOLLY_HAVE_PTHREAD_ATFORK TEST(ThreadLocal, Fork) { EXPECT_EQ(1, ptr->value()); // ensure created EXPECT_EQ(1, totalValue()); @@ -506,6 +507,7 @@ TEST(ThreadLocal, Fork) { EXPECT_EQ(1, totalValue()); } +#endif struct HoldsOneTag2 {}; -- 2.34.1