From: Eric Wong <e@80x24.org>
Date: Fri, 26 Apr 2024 11:29:41 +0000 (+0000)
Subject: test_common: don't needlessly rebuild C++ Xapian helper
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=fac2a191fd211fda9309dd63fac4eae0e6f03589;p=thirdparty%2Fpublic-inbox.git

test_common: don't needlessly rebuild C++ Xapian helper

We should almost always be calling `check_build' instead of
`build'.  Using ccache masked some of the overhead from
this, but various linker implementations are still slow.
---

diff --git a/lib/PublicInbox/TestCommon.pm b/lib/PublicInbox/TestCommon.pm
index b8b7b827c..708fa698a 100644
--- a/lib/PublicInbox/TestCommon.pm
+++ b/lib/PublicInbox/TestCommon.pm
@@ -726,7 +726,7 @@ SKIP: {
 	require File::Path;
 	eval { # use XDG_CACHE_HOME, first:
 		require PublicInbox::XapHelperCxx;
-		PublicInbox::XapHelperCxx::build();
+		PublicInbox::XapHelperCxx::check_build();
 	};
 	local %ENV = %ENV;
 	delete $ENV{XDG_DATA_HOME};