all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH 1/1] gnu: audit: Use GnuTLS instead of OpenSSL.
@ 2016-10-22  7:09 Leo Famulari
  2016-10-22 13:57 ` Christopher Allan Webber
  0 siblings, 1 reply; 4+ messages in thread
From: Leo Famulari @ 2016-10-22  7:09 UTC (permalink / raw)
  To: guix-devel

The audit build is failing because it can't find GnuTLS:

libtool: link: gcc -W -Wall -Wundef -D_GNU_SOURCE -fPIE -DPIE -g -O2 -pie -Wl,-z -Wl,relro -Wl,-z -Wl,now -o .libs/audispd-zos-remote audispd_zos_remote-zos-remote-plugin.o audispd_zos_remote-zos-remote-log.o audispd_zos_remote-zos-remote-ldap.o audispd_zos_remote-zos-remote-config.o audispd_zos_remote-zos-remote-queue.o  -lpthread /gnu/store/y9j75zc10h0za5wnbvrb349c8hxm2s6k-openldap-2.4.44/lib/libldap.so /gnu/store/xvsww5q2j9g4csi2i2mkpjm09mywnb6g-cyrus-sasl-2.1.26/lib/libsasl2.so -ldl -lgnutls /gnu/store/y9j75zc10h0za5wnbvrb349c8hxm2s6k-openldap-2.4.44/lib/liblber.so -lresolv -L../../../auparse /tmp/guix-build-audit-2.4.5.drv-0/audit-2.4.5/auparse/.libs/libauparse.so /tmp/guix-build-audit-2.4.5.drv-0/audit-2.4.5/lib/.libs/libaudit.so -Wl,-rpath -Wl,/gnu/store/y9j75zc10h0za5wnbvrb349c8hxm
 2s6k-openldap-2.4.44/lib -Wl,-rpath -Wl,/gnu/store/xvsww5q2j9g4csi2i2mkpjm09mywnb6g-cyrus-sasl-2.1.26/lib -Wl,-rpath -Wl,/gnu/store/ri7alh15j0bbaansb79y4f29ivqgvgws-audit-2.4.5/lib
ld: cannot find -lgnutls
collect2: error: ld returned 1 exit status

https://hydra.gnu.org/build/1550341/nixlog/2/tail-reload

It succeeds when building against GnuTLS instead of OpenSSL.

* gnu/packages/admin.scm (audit)[inputs]: Replace openssl with gnutls.
---
 gnu/packages/admin.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 9724c9b..7a83a8c 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -1556,7 +1556,7 @@ platform-specific methods.")
              #t)))))
     (inputs
      `(("openldap" ,openldap)
-       ("openssl" ,openssl)
+       ("gnutls" ,gnutls)
        ("sasl" ,cyrus-sasl)))
     (synopsis "User-space component to the Linux auditing system")
     (description
-- 
2.10.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] gnu: audit: Use GnuTLS instead of OpenSSL.
  2016-10-22  7:09 [PATCH 1/1] gnu: audit: Use GnuTLS instead of OpenSSL Leo Famulari
@ 2016-10-22 13:57 ` Christopher Allan Webber
  2016-10-22 14:11   ` Leo Famulari
  0 siblings, 1 reply; 4+ messages in thread
From: Christopher Allan Webber @ 2016-10-22 13:57 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari writes:

> The audit build is failing because it can't find GnuTLS:
>
> libtool: link: gcc -W -Wall -Wundef -D_GNU_SOURCE -fPIE -DPIE -g -O2 -pie -Wl,-z -Wl,relro -Wl,-z -Wl,now -o .libs/audispd-zos-remote audispd_zos_remote-zos-remote-plugin.o audispd_zos_remote-zos-remote-log.o audispd_zos_remote-zos-remote-ldap.o audispd_zos_remote-zos-remote-config.o audispd_zos_remote-zos-remote-queue.o  -lpthread /gnu/store/y9j75zc10h0za5wnbvrb349c8hxm2s6k-openldap-2.4.44/lib/libldap.so /gnu/store/xvsww5q2j9g4csi2i2mkpjm09mywnb6g-cyrus-sasl-2.1.26/lib/libsasl2.so -ldl -lgnutls /gnu/store/y9j75zc10h0za5wnbvrb349c8hxm2s6k-openldap-2.4.44/lib/liblber.so -lresolv -L../../../auparse /tmp/guix-build-audit-2.4.5.drv-0/audit-2.4.5/auparse/.libs/libauparse.so /tmp/guix-build-audit-2.4.5.drv-0/audit-2.4.5/lib/.libs/libaudit.so -Wl,-rpath -Wl,/gnu/store/y9j75zc10h0za5wnbvrb349c8h
 xm2s6k-openldap-2.4.44/lib -Wl,-rpath -Wl,/gnu/store/xvsww5q2j9g4csi2i2mkpjm09mywnb6g-cyrus-sasl-2.1.26/lib -Wl,-rpath -Wl,/gnu/store/ri7alh15j0bbaansb79y4f29ivqgvgws-audit-2.4.5/lib
> ld: cannot find -lgnutls
> collect2: error: ld returned 1 exit status
>
> https://hydra.gnu.org/build/1550341/nixlog/2/tail-reload
>
> It succeeds when building against GnuTLS instead of OpenSSL.

Seems like it built okay here without this patch?

I did do the "guix build audit" before doing a git pull, but after doing
the git pull re-running the command didn't build anything new?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] gnu: audit: Use GnuTLS instead of OpenSSL.
  2016-10-22 13:57 ` Christopher Allan Webber
@ 2016-10-22 14:11   ` Leo Famulari
  2016-10-22 15:06     ` Christopher Allan Webber
  0 siblings, 1 reply; 4+ messages in thread
From: Leo Famulari @ 2016-10-22 14:11 UTC (permalink / raw)
  To: Christopher Allan Webber; +Cc: guix-devel

On Sat, Oct 22, 2016 at 08:57:19AM -0500, Christopher Allan Webber wrote:
> Leo Famulari writes:
> > It succeeds when building against GnuTLS instead of OpenSSL.
> 
> Seems like it built okay here without this patch?

Does it work for you on core-updates?

It builds with OpenSSL on master for me, but not core-updates. Weird!

The other thing is that audit is distributed under the GPL2+ but,
AFAICT, there is no OpenSSL exception made to their GPL2+ license.

There are no results for `grep -ri openssl` in the audit source tree.

So we may have to switch to GnuTLS for licensing reasons.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] gnu: audit: Use GnuTLS instead of OpenSSL.
  2016-10-22 14:11   ` Leo Famulari
@ 2016-10-22 15:06     ` Christopher Allan Webber
  0 siblings, 0 replies; 4+ messages in thread
From: Christopher Allan Webber @ 2016-10-22 15:06 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari writes:

> On Sat, Oct 22, 2016 at 08:57:19AM -0500, Christopher Allan Webber wrote:
>> Leo Famulari writes:
>> > It succeeds when building against GnuTLS instead of OpenSSL.
>> 
>> Seems like it built okay here without this patch?
>
> Does it work for you on core-updates?
>
> It builds with OpenSSL on master for me, but not core-updates. Weird!
>
> The other thing is that audit is distributed under the GPL2+ but,
> AFAICT, there is no OpenSSL exception made to their GPL2+ license.
>
> There are no results for `grep -ri openssl` in the audit source tree.
>
> So we may have to switch to GnuTLS for licensing reasons.

Ah, I hadn't tried core-updates... I was testing against master.

Yes, makes sense.  Sounds like we ought to use GnuTLS anyway.  I'd say
push it!

 - Chris

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-10-22 15:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-22  7:09 [PATCH 1/1] gnu: audit: Use GnuTLS instead of OpenSSL Leo Famulari
2016-10-22 13:57 ` Christopher Allan Webber
2016-10-22 14:11   ` Leo Famulari
2016-10-22 15:06     ` Christopher Allan Webber

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.