From ab14b152507ec7c7a8e3c1b60c40d81f1244e428 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Thu, 27 Feb 2025 00:08:19 +0000 Subject: [PATCH] t/xap_helper: bail on build failure if TEST_XH_CXX_ONLY We shouldn't transparently fall back to the Perl bindings test on build failures if the tester wants to explicitly test the C++ implementation. --- t/xap_helper.t | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/t/xap_helper.t b/t/xap_helper.t index e87c9da8e..da26b9a86 100644 --- a/t/xap_helper.t +++ b/t/xap_helper.t @@ -189,7 +189,10 @@ SKIP: { require PublicInbox::XapHelperCxx; PublicInbox::XapHelperCxx::cmd(); }; - skip "XapHelperCxx build: $@", 1 if $@; + if ($@) { + xbail "C++ build failed: $@" if $ENV{TEST_XH_CXX_ONLY}; + skip "XapHelperCxx build: $@", 1; + } @NO_CXX = $ENV{TEST_XH_CXX_ONLY} ? (0) : (0, 1); my $ar = $test->(@$cmd, '-j0'); -- 2.47.3