Re: [PATCH] gnu: Add gloox.

[ng0]

Leo Famulari <address@hidden> writes:

> [ Unknown signature status ]
> On Thu, Sep 15, 2016 at 09:54:17PM +0000, ng0 wrote:
>> This patch adds gloox, a dependency for 0ad needs for their "lobby"
>> feature. Gentoo builds it with libressl without any patches applied,
>> they had a long discussion and are very strict in their testing, looking
>> at the tests comparing libressl and openssl builds I did there was no
>> failure and if it won't work I'll notice with 0ad. I'm positive that it
>> works.
>
>> Subject: [PATCH] gnu: Add gloox.
>> 
>> * gnu/packages/messaging.scm (gloox): New variable.
>
>> +    (inputs
>> +     `(("libidn" ,libidn)
>> +       ("gnutls" ,gnutls)
>> +       ("libressl" ,libressl)
>> +       ("zlib" ,zlib)))
>
> I don't think this package is using libressl:
>
> $ guix gc --references $(./pre-inst-env guix build gloox)
> /gnu/store/3fhnmvbdxlsh64pr3zg4y74x3hlx33qw-libtasn1-4.8
> /gnu/store/5992iq1v7arqa14ym3di58n4la0893nv-zlib-1.2.8
> /gnu/store/6004mqdv1y8wc6zvg9yxy4bh652sgzms-gloox-1.0.17
> /gnu/store/9nifwk709wajpyfwa0jzaa3p6mf10vxs-gcc-4.9.3-lib
> /gnu/store/cdkrfbl10kbyyjjw3yfk9hckfw8n1b7g-gmp-6.1.0
> /gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23
> /gnu/store/sbj1kgn8bs91bn7ba9qk4n3l2rr7dxbr-libidn-1.32
> /gnu/store/ykzwykkvr2c80rw4l1qh3mvfdkl7jibi-bash-4.3.42
> /gnu/store/yrl3c1mxqwcpppyh0sjlwn3sj2w5qj54-gnutls-3.5.2
>
> The Debian package uses GnuTLS only:
> https://packages.debian.org/source/sid/gloox

I always look at Gentoo first, and it is validated for
openssl+libressl without any additional patches: (link at gpo, which
mirrors them from packages.gentoo.org)
https://data.gpo.zugaina.org/gentoo/net-libs/gloox/gloox-1.0.17.ebuild

2016-09-24 17:06 <ng0> I think net-libs/gloox/gloox-* is wrong in using 
openssl/libressl. for version 0.17 the --with-ssl option is ignored. I packaged 
it for Guix and assumed openssl or libressl would work, but it is not even 
linked in the end. someone should fix that for Gentoo.
2016-09-24 17:09 <kensington> ng0: links ok here
2016-09-24 17:09 <kensington> ET_DYN 
libresolv.so.2,libssl.so.1.0.0,libcrypto.so.1.0.0,libz.so.1,libstdc++.so.6,libm.so.6,libc.so.6,libgcc_s.so.1
 /usr/lib64/libgloox.so

I'll see what I need to adjust, will send a new patch.
-- 
              ng0