svn commit: r1931635 - subversion/trunk

[email protected]
Newsgroups gmane.comp.version-control.subversion.svn
Message-ID <176989781821.3250458.12818335787945961879@svn02-us-east.apache.org>
Author: rinrab
Date: Sat Jan 31 22:16:57 2026
New Revision: 1931635

Log:
cmake: Make sure that serf is checked as a REQUIRED dependency.

- Tested that current trunk will successfully configure when using serf though
  pkcconfig even when removing the serf-1.pc itself. Errors will only reveal
  after executing the build. After this change, an explicit error message is
  shown.

(now i have to restore it...)

* CMakeLists.txt
  (serf): Call pkg_search_module() with REQUIRED argument and remove
   unnecessary condition.

Reviewed by: dsahlberg

Modified:
   subversion/trunk/CMakeLists.txt

Modified: subversion/trunk/CMakeLists.txt
==============================================================================
--- subversion/trunk/CMakeLists.txt	Sat Jan 31 20:10:02 2026	(r1931634)
+++ subversion/trunk/CMakeLists.txt	Sat Jan 31 22:16:57 2026	(r1931635)
@@ -374,11 +374,8 @@ endif()
 ### Serf
 if (SVN_ENABLE_RA_SERF)
   if(SVN_USE_PKG_CONFIG)
-    pkg_search_module(serf IMPORTED_TARGET serf-2 serf-1)
-
-    if(serf_FOUND)
-      add_library(external-serf ALIAS PkgConfig::serf)
-    endif()
+    pkg_search_module(serf IMPORTED_TARGET REQUIRED serf-2 serf-1)
+    add_library(external-serf ALIAS PkgConfig::serf)
   else()
     find_package(Serf REQUIRED)
     add_library(external-serf ALIAS Serf::Serf)
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.