Fix: tap.h conflicts with Clang C++ headers

Always include tap.h last since it defines a 'fail' macro that clashes
with some version of the clang++ system headers.

Change-Id: I20938697045d00b467b12ca1e6dae159c634720a
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/tests/regression/tools/live/live_test.cpp b/tests/regression/tools/live/live_test.cpp
index 559b9a7b1e8298885ae574090a26683e6058eb82..08eb8197cddc29f1676f2ef9183f5f5630e1c5da 100644 (file)
--- a/tests/regression/tools/live/live_test.cpp
+++ b/tests/regression/tools/live/live_test.cpp
@@ -20,7 +20,6 @@
 #include <sys/mman.h>
 #include <sys/stat.h>
 
-#include <tap/tap.h>
 #include <lttng/lttng.h>
 
 #include <urcu/list.h>
@@ -32,6 +31,8 @@
 #include <common/compat/errno.hpp>
 #include <common/compat/endian.hpp>
 
+#include <tap/tap.h>
+
 #define SESSION1 "test1"
 #define RELAYD_URL "net://localhost"
 #define LIVE_TIMER 2000000

diff --git a/tests/unit/test_event_rule.cpp b/tests/unit/test_event_rule.cpp
index a6bf6180a3bb007081d06ac06e59acd59ba81345..c3fd01a81837011293444addf13b61f62b3581e3 100644 (file)
--- a/tests/unit/test_event_rule.cpp
+++ b/tests/unit/test_event_rule.cpp
@@ -12,8 +12,6 @@
 #include <string.h>
 #include <unistd.h>
 
-#include <tap/tap.h>
-
 #include <common/payload-view.hpp>
 #include <common/payload.hpp>
 #include <lttng/domain.h>
@@ -38,6 +36,8 @@
 #include <lttng/userspace-probe.h>
 #include "bin/lttng/loglevel.hpp"
 
+#include <tap/tap.h>
+
 /* For error.h. */
 int lttng_opt_quiet = 1;
 int lttng_opt_verbose;

diff --git a/tests/unit/test_relayd_backward_compat_group_by_session.cpp b/tests/unit/test_relayd_backward_compat_group_by_session.cpp
index 4b794faf7c7081660cc1317dbdc575c999f8cfbb..602376c4376588f9be54888488a047d3e79c9b4c 100644 (file)
--- a/tests/unit/test_relayd_backward_compat_group_by_session.cpp
+++ b/tests/unit/test_relayd_backward_compat_group_by_session.cpp
@@ -9,12 +9,13 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <tap/tap.h>
 
 #include <common/time.hpp>
 
 #include "backward-compatibility-group-by.hpp"
 
+#include <tap/tap.h>
+
 /* Number of TAP tests in this file */
 #define NUM_TESTS_PER_TEST 1
 

diff --git a/tests/unit/test_utils_compat_poll.cpp b/tests/unit/test_utils_compat_poll.cpp
index 697440963c58279514129e0515b00d425d75eb61..49cd4a1eb52af272b9cf679f4a11c3728ce56f95 100644 (file)
--- a/tests/unit/test_utils_compat_poll.cpp
+++ b/tests/unit/test_utils_compat_poll.cpp
@@ -16,13 +16,13 @@
 #include <sys/types.h>
 #include <sys/wait.h>
 
-#include <tap/tap.h>
-
 #include <common/compat/poll.hpp>
 #include <common/readwrite.hpp>
 #include <common/pipe.hpp>
 #include <common/dynamic-array.hpp>
 
+#include <tap/tap.h>
+
 /* Verification without trashing test order in the child process */
 #define childok(e, test, ...) do { \
        if (!(e)) { \

diff --git a/tests/unit/test_utils_expand_path.cpp b/tests/unit/test_utils_expand_path.cpp
index 9892727bad86f7e615e9a644c43236cb78208191..9fe1619ee9bf031a71989487e4edc6cac62edf5f 100644 (file)
--- a/tests/unit/test_utils_expand_path.cpp
+++ b/tests/unit/test_utils_expand_path.cpp
@@ -13,12 +13,12 @@
 #include <sys/stat.h>
 #include <sys/types.h>
 
-#include <tap/tap.h>
-
 #include <common/utils.hpp>
 #include <common/path.hpp>
 #include <common/common.hpp>
 
+#include <tap/tap.h>
+
 /* For error.h */
 int lttng_opt_quiet = 1;
 int lttng_opt_verbose = 3;

diff --git a/tests/unit/test_uuid.cpp b/tests/unit/test_uuid.cpp
index 07a2f4afb221d393f5fc9c5dfb095520e5927d05..3ead2a254360a95db5d2d82ff291ede82eb57e4d 100644 (file)
--- a/tests/unit/test_uuid.cpp
+++ b/tests/unit/test_uuid.cpp
@@ -9,10 +9,10 @@
 #include <stdbool.h>
 #include <string.h>
 
-#include <tap/tap.h>
-
 #include "common/uuid.hpp"
 
+#include <tap/tap.h>
+
 #define NR_TESTS 21
 
 static const char valid_str_1[] = "3d260c88-75ea-47b8-a7e2-d6077c0378d9";