From 3e7d928ddd42767e956907cd647eb37c2c04b3e3 Mon Sep 17 00:00:00 2001 From: Kohei Takahashi Date: Wed, 18 Oct 2017 08:09:30 +0900 Subject: [PATCH 1/6] Remove unused env value. --- .travis.yml | 8 -------- 1 file changed, 8 deletions(-) diff --git a/.travis.yml b/.travis.yml index 842a89f0..f9f3d672 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,15 +16,7 @@ branches: - master - develop -env: - matrix: - - BOGUS_JOB=true - matrix: - - exclude: - - env: BOGUS_JOB=true - include: - os: linux compiler: g++ From 8243b3088bce6c184f32b2e6078bd23386d97f42 Mon Sep 17 00:00:00 2001 From: Kohei Takahashi Date: Wed, 18 Oct 2017 08:35:15 +0900 Subject: [PATCH 2/6] Remove notification --- .travis.yml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/.travis.yml b/.travis.yml index f9f3d672..6edea7a0 100644 --- a/.travis.yml +++ b/.travis.yml @@ -548,7 +548,3 @@ script: - |- echo "using $TOOLSET : : $COMPILER : -std=$CXXSTD ;" > ~/user-config.jam - ./b2 libs/fusion/test toolset=$TOOLSET - -notifications: - email: - on_success: always From 99699ede5db54ac9fdc97c305f8e5e3a93868abf Mon Sep 17 00:00:00 2001 From: Kohei Takahashi Date: Wed, 18 Oct 2017 22:43:08 +0900 Subject: [PATCH 3/6] Added define to determine where running on. --- .travis.yml | 2 +- appveyor.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 6edea7a0..4c2e095d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -547,4 +547,4 @@ install: script: - |- echo "using $TOOLSET : : $COMPILER : -std=$CXXSTD ;" > ~/user-config.jam - - ./b2 libs/fusion/test toolset=$TOOLSET + - ./b2 libs/fusion/test toolset=$TOOLSET define=RUNNING_ON_TRAVIS=1 diff --git a/appveyor.yml b/appveyor.yml index 82c1bedf..ba83a8c1 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -71,4 +71,4 @@ install: build: off test_script: - - b2 libs/fusion/test toolset=%TOOLSET% + - b2 libs/fusion/test toolset=%TOOLSET% define=RUNNING_ON_APPVEYOR=1 From e992658ce350b1245cb3ba730178a2ec67536cb5 Mon Sep 17 00:00:00 2001 From: Kohei Takahashi Date: Wed, 18 Oct 2017 23:09:27 +0900 Subject: [PATCH 4/6] Skipping some tests due to compiler bug. It is better to skip such tests on AppVeyor to detect regression. --- test/sequence/define_struct_inline_move.cpp | 3 +++ test/sequence/define_tpl_struct_inline_move.cpp | 3 +++ 2 files changed, 6 insertions(+) diff --git a/test/sequence/define_struct_inline_move.cpp b/test/sequence/define_struct_inline_move.cpp index 13d37931..6cf2d053 100644 --- a/test/sequence/define_struct_inline_move.cpp +++ b/test/sequence/define_struct_inline_move.cpp @@ -51,6 +51,8 @@ int main() BOOST_TEST(y.w.value == 42); } + // Older MSVCs don't generate move ctor by default. +#if !(defined(RUNNING_ON_APPVEYOR) && BOOST_WORKAROUND(BOOST_MSVC, < 1900)) { ns::value x; ns::value y(std::move(x)); // move @@ -68,6 +70,7 @@ int main() BOOST_TEST(x.w.value == 0); BOOST_TEST(y.w.value == 0); } +#endif // !(appveyor && msvc < 14.0) return boost::report_errors(); } diff --git a/test/sequence/define_tpl_struct_inline_move.cpp b/test/sequence/define_tpl_struct_inline_move.cpp index 9917fea2..27be0573 100644 --- a/test/sequence/define_tpl_struct_inline_move.cpp +++ b/test/sequence/define_tpl_struct_inline_move.cpp @@ -51,6 +51,8 @@ int main() BOOST_TEST(y.w.value == 42); } + // Older MSVCs don't generate move ctor by default. +#if !(defined(RUNNING_ON_APPVEYOR) && BOOST_WORKAROUND(BOOST_MSVC, < 1900)) { ns::value x; ns::value y(std::move(x)); // move @@ -68,6 +70,7 @@ int main() BOOST_TEST(x.w.value == 0); BOOST_TEST(y.w.value == 0); } +#endif // !(appveyor && msvc < 14.0) return boost::report_errors(); } From cbda55b962a509b22d40c58d8f96ff33706e159c Mon Sep 17 00:00:00 2001 From: Kohei Takahashi Date: Wed, 18 Oct 2017 23:35:48 +0900 Subject: [PATCH 5/6] Added repo local failure markups. --- meta/explicit-failures-markup.xml | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) create mode 100644 meta/explicit-failures-markup.xml diff --git a/meta/explicit-failures-markup.xml b/meta/explicit-failures-markup.xml new file mode 100644 index 00000000..ffed9d81 --- /dev/null +++ b/meta/explicit-failures-markup.xml @@ -0,0 +1,20 @@ + + + + + + + + + + + + The compiler doesn't generate defaulted move ctor/assgin thus + perform copy construction/assginment. The `inline` versions + don't provide move ctor/assign to preserve standard layout + constraint. Since that is not documented behaviour, it might + be changed in future release. + + + + From 320386dcf11d4c4f91964426851027b76022ea41 Mon Sep 17 00:00:00 2001 From: Kohei Takahashi Date: Wed, 18 Oct 2017 23:50:10 +0900 Subject: [PATCH 6/6] Revert "Remove unused env value." This reverts commit 3e7d928ddd42767e956907cd647eb37c2c04b3e3. This env value is needed to prevent run job with default compiler. --- .travis.yml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.travis.yml b/.travis.yml index 4c2e095d..b834af88 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,7 +16,15 @@ branches: - master - develop +env: + matrix: + - BOGUS_JOB=true + matrix: + + exclude: + - env: BOGUS_JOB=true + include: - os: linux compiler: g++