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)
