Re: [HACKERS] 4 pgcrypto regressions failures - 1 unsolved

From: Marko Kreen <marko(at)l-t(dot)ee>
To: Kris Jurka <books(at)ejurka(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: [HACKERS] 4 pgcrypto regressions failures - 1 unsolved
Date: 2005-07-16 10:07:41
Message-ID: 20050716100741.GA20959@l-t.ee
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Fri, Jul 15, 2005 at 08:06:15PM -0500, Kris Jurka wrote:
> On Fri, 15 Jul 2005, Marko Kreen wrote:
>
> > [buildfarm machine dragonfly]
> >
> > On Tue, Jul 12, 2005 at 01:06:46PM -0500, Kris Jurka wrote:
> > > Well the buildfarm machine kudu is actually the same machine just building
> > > with the Sun compiler and it works fine. It links all of libz.a into
> > > libpgcrypto.so while gcc refuses to.
> >
> > I googled a bit and found two suggestions:
> >
> > 1. http://curl.haxx.se/mail/lib-2002-01/0092.html
> > (Use -mimpure-text on linking line)
> >
> > The attached patch does #1. Could you try it and see if it fixes it?
> >
>
> This patch works, pgcrypto links and passes its installcheck test now.
>
> Kris Jurka

Thanks.

Here is the patch with a little comment.

It should not break anything as it just disables a extra
argument "-assert pure-text" to linker.

Linking static libraries into shared one is bad idea, as the
static parts wont be shared between processes, but erroring
out is worse, especially if another compiler for a platform
allows it.

This makes gcc act same way as Sun's cc.

--
marko

Attachment Content-Type Size
solgcc2.diff text/plain 799 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Mario Weilguni 2005-07-16 11:22:30 Re: pg_get_prepared?
Previous Message Kevin Brown 2005-07-16 06:38:01 Re: Checkpoint cost, looks like it is WAL/CRC

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-07-16 11:47:06 Re: backslashes in pgindent
Previous Message Luke Lonergan 2005-07-16 05:34:35 Re: backslashes in pgindent