Re: crypto library is not installed

From: Andreas 'ads' Scherbaum <adsmail(at)wars-nicht(dot)de>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: crypto library is not installed
Date: 2014-03-04 08:29:36
Message-ID: 53158EF0.6020807@wars-nicht.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On 03/03/2014 02:44 PM, Dave Page wrote:
> On Mon, Mar 3, 2014 at 1:05 PM, Andreas 'ads' Scherbaum
> <adsmail(at)wars-nicht(dot)de> wrote:
>> On 03/03/2014 11:32 AM, Dave Page wrote:
>>>
>>> On Mon, Mar 3, 2014 at 10:20 AM, Andreas 'ads' Scherbaum
>>> <adsmail(at)wars-nicht(dot)de> wrote:
>>>>
>>>>
>>>> Yes, I know how to fix this. My question is: why is "configure" not
>>>> checking
>>>> for this library, when it is required to build pgAdmin3?
>>>
>>>
>>> Noone wrote the code I guess. Patches welcome :-)
>>
>>
>> Just checking the code: why is it even requiring "crypto" when compiling
>> without --with-openssl?
>
> That's actually the likely reason why we don't do that test. It seems
> to be required because we're linking with libpq which is linked with
> OpenSSL, apps linking to it (on some platforms) also need to link with
> libcypto. It may also be that an element of bitrot has crept into that
> configure option - I very much doubt anyone has tested without SSL
> support in a long time.

Given what you said, attached is a patch which requires "openssl/ssl.h"
to be available. If not, configure bails out with an error message.

Regards,

--
Andreas 'ads' Scherbaum
German PostgreSQL User Group
European PostgreSQL User Group - Board of Directors
Volunteer Regional Contact, Germany - PostgreSQL Project

Attachment Content-Type Size
configure_2014-03-04.patch text/x-patch 534 bytes

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2014-03-04 09:45:49 Re: Documentation patch
Previous Message Andreas 'ads' Scherbaum 2014-03-03 23:00:52 Documentation patch