Most likely this user just has a bunch of libraries in that environment 
variable, and this links the ra layers to quite a few dependencies it doesn't 
need, potentially loading things like BDB in cases where it Is absolutely not 
necessary.

There are other better ways to add the depencies to specific libraries, like 
that libs line right above this. This line links the fake serf target, which 
does have the dependencies for serf... And that one would need fixing if the 
user has a problem.

-----Original Message-----
From: Bert Huijben <b...@qqmail.nl> 
Sent: Monday, July 4, 2022 4:07 PM
To: dev@subversion.apache.org; comm...@subversion.apache.org
Subject: RE: svn commit: r1902364 - /subversion/trunk/build.conf

> ==============================================================================
> --- subversion/trunk/build.conf (original)
> +++ subversion/trunk/build.conf Thu Jun 30 08:10:48 2022
> @@ -340,6 +340,7 @@ type = ra-module
>  path = subversion/libsvn_ra_serf
>  install = serf-lib
>  libs = libsvn_delta libsvn_subr aprutil apriconv apr serf zlib
> +add-install-deps = $(SVN_FS_LIB_INSTALL_DEPS)
>  msvc-static = yes
>  
>  # Accessing repositories via SVN

Why add this dependency to libsvn_ra_serf?

I think you have a different problem in your build if you need the fs libraries 
for building the serf ra layer.

        Bert


Reply via email to