[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#52578] [PATCH] updating openldap and adding service definition
From: |
zimoun |
Subject: |
[bug#52578] [PATCH] updating openldap and adding service definition |
Date: |
Sat, 18 Dec 2021 11:53:43 +0100 |
Hi Maxime,
The package ’openldap’ already exists and some of your comments ask
about the existing recipe. :-)
I think Jean-François just copy/pasted the current recipe and expand it
for their own requirements: having the service they need, IIUC.
On Fri, 17 Dec 2021 at 22:39, Maxime Devos <maximedevos@telenet.be> wrote:
>> + ,@(if (%current-target-system)
>> + '("--with-yielding_select=yes"
> "ac_cv_func_memcmp_working=yes")
>> + '()
>> + )
>
> is this speculation on what's necessary for cross-compilation, or has
> it been determined these flags are necessary?
This bits had been added by Mathieu 1c8b1870a60de12f6c19d809522f5d8362215131.
>>+ #:make-flags '("STRIP=")
>
> Why?
This one too.
>> + ,@(if (%current-target-system)
>> + '(
>> + (add-before 'make-depend 'fix-cross-gcc
>> + (lambda* (#:key target #:allow-other-keys)
>> + (setenv "CC" (string-append target "-gcc"))
>> + #t
>> + )
>> + )
>> + )
>> + '()
>
> You can use ,(cc-for-target) here. Also, CC can be set in #:make-flags.
Again this one.
>> + (synopsis "Implementation of the Lightweight Directory Access
> Protocol")
>> + (description "OpenLDAP is a free implementation of the
> Lightweight Directory Access Protocol.")
>
> That's a very terse description --- is it a server, a client
> application, programming APIs for communicating with a server, or all
> of these? Also, no need to mention it's free, everything in Guix is
> free.
This description is from 2013, 2a75d4e664e802d3a3e2ed6455c63f32541559c8. ;-)
Your comments about the package itself make sense but I am not convinced
that they are related to the first Jean-François submission. :-)
Cheers,
simon