Re: [patch] Please drop the dangerous libssl and

From: Raphaël Enrici <blacknoz(at)club-internet(dot)fr>
To: Andreas Pflug <pgadmin(at)pse-consulting(dot)de>
Cc: Dave Page <dpage(at)vale-housing(dot)co(dot)uk>, "Adam H(dot)Pendleton" <fmonkey(at)fmonkey(dot)net>, PgAdmin Hackers <pgadmin-hackers(at)postgresql(dot)org>, Loïc Minier <lool(at)dooz(dot)org>
Subject: Re: [patch] Please drop the dangerous libssl and
Date: 2005-12-27 10:17:53
Message-ID: 43B114D1.9040707@club-internet.fr
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Raphaël Enrici wrote:
> Raphaël Enrici wrote:
>
>>Andreas Pflug wrote:
>>
>>
>>>Raphaël Enrici wrote:
>>>
>>>
>>>
>>>>Dear friends,
>>>>
>>>>Loic Minier(CCed) provided a patch to prevent pgadmin3 1.2.2 from being
>>>>linked to a different libssl version than libpq when dynamically built
>>>>with an already ssl enabled libpq. The full bug report and original
>>>>patch by Loic can be found at
>>>>http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=341117
>>>>
>>>>Attached is an svn diff for 1.4.1 release that I'm using for the package
>>>>I'm about to upload to Debian (patch_libpqssl_1.4.1).
>>>>You'll also find a fully untested patch for trunk (patch_libpqssl_current).
>>>>
>>>>Please recheck and apply if eveything's ok with them.
>>>>
>>>>Cheers,
>>>>Raph
>>>>
>>>>
>>>>------------------------------------------------------------------------
>>>>
>>>>Index: acinclude.m4
>>>>===================================================================
>>>>--- acinclude.m4 (revision 4858)
>>>>+++ acinclude.m4 (working copy)
>>>>@@ -227,7 +227,10 @@
>>>> else
>>>> if test "$pgsql_ssl_libpq" = "yes"
>>>> then
>>>>- LIBS="$LIBS -lssl -lcrypto -lpq"
>>>>+ # no idea why -lssl and -lcrypto were included here, as this
>>>>+ # support is provided via libpq
>>>>+ #LIBS="$LIBS -lssl -lcrypto -lpq
>>>>+ LIBS="$LIBS -lpq"
>>>> else
>>>> LIBS="$LIBS -lcrypto -lpq"
>>>> fi
>>>
>>>
>>>We probably need this with static linking?
>>
>>
>>I think so, that's why the static part was not changed in the patch.
>
>
> mmmh sorry, replied too fast... I need to recheck.

So, that was it. The static part of the build should not be affected by
the patch. It only concerns the dynamic part of the build.

Cheers,
Raph

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Hiroshi Saito 2005-12-28 03:16:01 One problem of wxWidgets for MSW.
Previous Message Raphaël Enrici 2005-12-27 10:08:19 Re: [patch] Please drop the dangerous libssl and