Re: 1.18.0 tarball uploaded - Mailing list pgadmin-hackers

From Dave Page
Subject Re: 1.18.0 tarball uploaded
Date
Msg-id CA+OCxoz0HhcO=JZx7NOu-qq6cD6uUwpC-GZXrAdY8hVywWT-DA@mail.gmail.com
Whole thread Raw
In response to Re: 1.18.0 tarball uploaded  (Richard PALO <richard.palo@free.fr>)
Responses Re: 1.18.0 tarball uploaded
Re: 1.18.0 tarball uploaded
List pgadmin-hackers
On Tue, Sep 3, 2013 at 10:31 PM, Richard PALO <richard.palo@free.fr> wrote:
> Le 30/08/13 13:45, Dave Page a écrit :
>
>> I've uploaded the 1.18.0 GA tarball to
>> ftp://ftp.pgadmin.org/release/v1.18.0/src/. Please take a look and
>> yelp if there are any showstopper issues!
>>
>> Thanks!
>>
> Hi, seems to build fine (and now without autoconf!) with pkgsrc.
>
> I happened to notice that we provide by default, because of wxGTK*,
> --with-libintl-prefix= and --with-libiconv-prefix= which are consequently
> ignored by configure... for info.

What do you mean by "provide"? pgAdmin doesn't use libiconv or libintl
directly, so configure makes no attempt to find them.

> Also, I would like confirmation concerning the two crypto options
> libgcrypt and openssl, that contrary to the comments in configure, libgcrypt
> is really intended to have precedence over openssl...

Hmm, yeah - it sure looks like libgcrypt would take precedence if both
are available to me, though I'm not entirely confident I'm reading
configure.ac.in correctly. Akshay, should the two "Look for xxx" tests
be swapped? I'm also thinking that we shouldn't be testing for "!= no"
either, but do an initial test to ensure both options aren't set to
"yes" (and thus conflicting), and if both aren't set to no, then force
OpenSSL to yes (so we have a default, but no/no can disable crypto).
Then, only test to see if one or the other option is set to yes.

Thoughts?

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgadmin-hackers by date:

Previous
From: Richard PALO
Date:
Subject: Re: 1.18.0 tarball uploaded
Next
From: Dieter Van de Walle
Date:
Subject: Issue 598