| From: | Dave Page <dpage(at)pgadmin(dot)org> |
|---|---|
| To: | "Andreas 'ads' Scherbaum" <adsmail(at)wars-nicht(dot)de> |
| Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | Re: crypto library is not installed |
| Date: | 2014-03-14 12:40:22 |
| Message-ID: | CA+OCxowMhi=KdDBaWqTv1qUE26QzvD0xnHRJ-iBiWY+gvVnyQQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
On Fri, Mar 14, 2014 at 12:37 PM, Andreas 'ads' Scherbaum
<adsmail(at)wars-nicht(dot)de> wrote:
> On 03/14/2014 12:39 PM, Dave Page wrote:
>>
>>
>> I took a quick look at this (and then passed the buck to Ashesh for
>> additional review and commit :-p ) - I think it's fine except that the
>> test needs to be inside the OpenSSL block above. See attached.
>
>
> Not sure you want to move this into this check: as you pointed out, libpq is
> pulling in SSL anyway, therefore this check should be standalone. The
> optional --with-openssl should be removed instead, because it's no longer
> optional.
Libpq doesn't *always* pull in OpenSSL.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | benj dev | 2014-03-14 16:26:12 | Re: New functionality - Table tree view |
| Previous Message | Andreas 'ads' Scherbaum | 2014-03-14 12:37:55 | Re: crypto library is not installed |