Re: Regression - GNUmakefile - pg_usleep - Mailing list pgsql-patches

From Rocco Altier
Subject Re: Regression - GNUmakefile - pg_usleep
Date
Msg-id 6E0907A94904D94B99D7F387E08C4F5730ECB2@FALCON.INSIGHT
Whole thread Raw
In response to Regression - GNUmakefile - pg_usleep  ("Rocco Altier" <RoccoA@Routescape.com>)
Responses Re: Regression - GNUmakefile - pg_usleep
List pgsql-patches

> -----Original Message-----
> From: pgsql-patches-owner@postgresql.org
> [mailto:pgsql-patches-owner@postgresql.org] On Behalf Of Tom Lane
> Sent: Sunday, July 24, 2005 10:47 AM
> To: Rocco Altier
> Cc: Patches (PostgreSQL)
> Subject: Re: [PATCHES] Regression - GNUmakefile - pg_usleep
>
>
> "Rocco Altier" <RoccoA@Routescape.com> writes:
> > Attached patch fixes the SHLIB_LINK to add pgport now that
> pg_usleep is
> > added.
>
> It seems highly unlikely that this will accomplish anything,
> given that
> SHLIB_LINK is not used to construct regress.so.
>
I have another patch in queue for this.

> > This is needed for AIX to resolve symbols at compile time.
>
> I'm still wondering why that platform has such a hard time finding
> symbols that are in the backend.
>
Right now there are 2 different sets of linker rules.  One for a single
file -> .so (from Makefile.aix), and another for multiple files ->
lib<files>.so (from Makefile.shlib).

The patch I proposed to Makefile.shlib to pull all $LIBS in was rejected
since this would cause libpq to link against backend libraries, etc.
The suggested solution was to only pull in libraries as needed (which is
what is happening here, since pgport would be required).

I think the 2 sets of link rules should be more similar, since the
contrib moudules should get the same link option, instead of based on
how many files need to be linked together, thus Makefile.aix should be
changed to use SHLIB_LINK.

I am attaching all the patches for so for AIX to let it get past Contrib
on the buildfarm.

Makefile.aix.patch - will bring Makefile.aix in line with Makefile.shlib
by using SHLIB_LINK
contrib.libs.patch - will pull in the needed libraries for contrib
modules
regress.pgport.patch - same idea as contrib patch, pull in the needed
library

I hope this makes it a bit clearer what I am trying to achieve by having
all the pieces together in one place.

Thanks,
    -rocco

PS. I am starting to look at using the facilities in AIX to support run
time linked libraries instead of at compile time.  This does require AIX
4.2+, so I will try to leave the 4.1 code in place.

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] Patch to fix plpython on OS X
Next
From: "Rocco Altier"
Date:
Subject: Re: Regression - GNUmakefile - pg_usleep