Re: parallel 3.0.0 released

[Orion Poplawski]

On 07/14/2015 01:53 PM, Olaf Till wrote:
> On Tue, Jul 14, 2015 at 10:11:58AM -0600, Orion Poplawski wrote:
> > On 07/14/2015 09:50 AM, Olaf Till wrote:
> > > Thanks for checking. I've added a configure link test for this
> > > symbol. Hope it works, did not check the negative case since I'd have
> > > to recompile gnutls fot this ...
> >
> > I'm afraid the check doesn't work when compiled with -O2 because the
> > optimization throws away the variable.  Would need -O0 or to do something with
> > the values, like:
> >
> >        [AC_LANG_PROGRAM([[#include <gnutls/gnutls.h>]
> >                          [#include <stdio.h>]],
> >                         [[printf("%x",&gnutls_srp_2048_group_prime);]])],
>
> It's corrected now, thanks.
>
> On Tue, Jul 14, 2015 at 10:28:26AM -0600, Orion Poplawski wrote:
> > I Fedora we have to patch the makefile to remove the "-s" (strip) option so
> > that proper debuginfo information would be present.  I don't know of many
> > other octave packages that do this and it would be nice to not have to
> > maintain this patch.
>
> Removed the -s option for now. But maybe there will be some policy in
> OF for this later.
>
> If nothing else happens, I'll request a bug fix release tomorrow.
>
> Olaf

Just getting back around to this - looks like lack of 
gnutls_srp_2048_group_prime doesn't properly disable TLS.  I still get:

#define HAVE_LIBGNUTLS 1
/* #undef HAVE_LIBGNUTLS_EXTRA */

which results in:

/tmp/rpm/octave-parallel-3.0.2-1.fc24.x86_64/install/parallel-3.0.2/x86_64-redhat-linux-gnu-api-v49+/parallel_interface.oct: 
failed to load: 
/tmp/rpm/octave-parallel-3.0.2-1.fc24.x86_64/install/parallel-3.0.2/x86_64-redhat-linux-gnu-api-v49+/parallel_interface.oct: 
undefined symbol: octave_parallel_server_credentials_callback

because parallel-3.0.2/src/p-streams.h:

#ifdef HAVE_LIBGNUTLS
extern "C" int octave_parallel_server_credentials_callback
(gnutls_session_t session, const char *username, gnutls_datum_t *salt,
 gnutls_datum_t *verifier, gnutls_datum_t *g, gnutls_datum_t *n);
#endif

and parallel-3.0.2/src/gnutls-callbacks.cc:

uses #ifdef HAVE_LIBGNUTLS_EXTRA to define the callback.

So we're inconsistent here it seems.


--
Orion Poplawski
Technical Manager                     303-415-9701 x222
NWRA/CoRA Division                    FAX: 303-415-9702
3380 Mitchell Lane                  address@hidden
Boulder, CO 80301              http://www.cora.nwra.com