[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#27220: [PATCH 3/3] gnu: openldap: Provide path to cyrus-sasl for lib
From: |
Leo Famulari |
Subject: |
bug#27220: [PATCH 3/3] gnu: openldap: Provide path to cyrus-sasl for libtool. |
Date: |
Sun, 4 Jun 2017 12:44:50 -0400 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Sun, Jun 04, 2017 at 03:38:29PM +0200, Marius Bakke wrote:
> Leo Famulari <address@hidden> writes:
> > * gnu/packages/openldap.scm (openldap)[arguments]: Add 'patch-sasl-path'
> > phase..
> > + (add-after 'install 'patch-sasl-path
> > + ;; Give -L arguments for cyrus-sasl to avoid propagation.
> > + (lambda* (#:key inputs outputs #:allow-other-keys)
> > + (let ((out (assoc-ref outputs "out"))
> > + (sasl (assoc-ref inputs "cyrus-sasl")))
> > + (substitute* (map (lambda (f) (string-append out "/" f))
> > + '("lib/libldap.la" "lib/libldap_r.la"))
> > + (("-lsasl2" lib)
> > + (string-append "-L" sasl "/lib " lib)))
> > + #t))))))
>
> Good find. LGTM, but I wonder if it would be better to remove the static
> library (and associated .la files) and instead provide an explicit
> static-package. This mostly for grafting purposes.
True. I found this while trying to build the latest ncmpcpp, which added a
dependency on curl, which depends on openldap. I copied the solution from the
hwloc package in (gnu packages mpi).
> However, I think that's a journey for another core-updates.
Yes :)
signature.asc
Description: PGP signature