On Fri, Jun 05, 2020 at 02:41:42PM +0100, Stuart Henderson wrote:
> On 2020/06/05 14:54, Theo Buehler wrote:
> > It's not the first time that it would have been very useful to me to
> > have debug symbols for the libraries in openssl available.
> > 
> > This seems to work for me. Not sure if the revision bump is necessary.
> 
> OK. It adds symbols to the static library archives so the bump is
> warranted.

Oh yes. Thanks!

> 
> > I don't expect to need symbols for 1.0.2, so I didn't add a debug
> > package there, but as there are a few consumers it might be useful,
> > too...
> 
> 
> Seems a good time to have a look over those.
> 
> sslscan-openssl depends on the :patch target so wouldn't be affected
> by a debug change in openssl/1.0.2 (and does still want 1.0.2).

I don't know what this does but I believe you :)

> waagent uses the openssl binary for cms so probably that dep can be
> removed. Diff below, I have no way to test though I don't think there
> should be a problem.

I scanned through the diff -w bewtween our cms.c and the one of
OpenSSL 1.0.2-stable and couldn't spot a change that makes me expect
any real incompatibility.

ok tb for the diff below.

> nsca-ng requires PSK so has to stay using OpenSSL. It could move to
> 1.1 though (there's an update, https://github.com/weiss/nsca-ng/releases)
>
> IIRC nrpe 2.x needs 1.0.2. newer versions upstream work with newer OpenSSL
> and *maybe* also LibreSSL but aren't compatible with old versions so
> handling an update means we need overlapping versions. (it's also
> deprecated upstream now in favour of NCPA).

In summary, I see no compelling reason to add debug packages to 1.0.2
unless someone really wants them (not me).

> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/sysutils/waagent/Makefile,v
> retrieving revision 1.4
> diff -u -p -r1.4 Makefile
> --- Makefile  12 Jul 2019 20:49:54 -0000      1.4
> +++ Makefile  5 Jun 2020 13:28:32 -0000
> @@ -19,9 +19,6 @@ MODPY_SETUPTOOLS=   Yes
>  
>  MODPY_ADJ_FILES=     bin/waagent
>  
> -# LibreSSL has removed CMS support
> -RUN_DEPENDS=         security/openssl/1.0.2
> -
>  NO_TEST=             Yes
>  
>  pre-configure:
> Index: patches/patch-config_openbsd_waagent_conf
> ===================================================================
> RCS file: 
> /cvs/ports/sysutils/waagent/patches/patch-config_openbsd_waagent_conf,v
> retrieving revision 1.1.1.1
> diff -u -p -r1.1.1.1 patch-config_openbsd_waagent_conf
> --- patches/patch-config_openbsd_waagent_conf 23 Jun 2017 12:37:09 -0000      
> 1.1.1.1
> +++ patches/patch-config_openbsd_waagent_conf 5 Jun 2020 13:28:32 -0000
> @@ -5,6 +5,15 @@ disable extensions and autoupdate
>  Index: config/openbsd/waagent.conf
>  --- config/openbsd/waagent.conf.orig
>  +++ config/openbsd/waagent.conf
> +@@ -61,7 +61,7 @@ OS.EnableFIPS=n
> + OS.RootDeviceScsiTimeout=300
> + 
> + # If "None", the system default version is used.
> +-OS.OpensslPath=/usr/local/bin/eopenssl
> ++OS.OpensslPath=/usr/bin/openssl
> + 
> + # Set the path to SSH keys and configuration files
> + OS.SshDir=/etc/ssh
>  @@ -84,6 +84,9 @@ OS.PasswordPath=/etc/master.passwd
>   #
>   # Pid.File=/var/run/waagent.pid
> 
> 
> 
> 
> 
> 
> > Index: Makefile
> > ===================================================================
> > RCS file: /var/cvs/ports/security/openssl/1.1/Makefile,v
> > retrieving revision 1.15
> > diff -u -p -r1.15 Makefile
> > --- Makefile        22 Apr 2020 04:31:53 -0000      1.15
> > +++ Makefile        5 Jun 2020 12:48:58 -0000
> > @@ -2,12 +2,15 @@
> >  
> >  PORTROACH= limit:^1\.1\.[0-9][a-z] skipb:0 skipv:1.1.1
> >  V=         1.1.1g
> > +REVISION=  0
> >  
> >  SHLIBVER=  11.5
> >  SHARED_LIBS=       crypto ${SHLIBVER} \
> >             ssl ${SHLIBVER}
> >  
> >  WANTLIB=   c
> > +
> > +DEBUG_PACKAGES = ${BUILD_PACKAGES}
> >  
> >  CONFIGURE_STYLE=   simple
> >  CONFIGURE_SCRIPT=  config
> > 
> 

Reply via email to