![]() | |
![]() |
| | Thread Tools | Display Modes |
#1
| |||
| |||
|
#2
| |||
| |||
|
|
This patch to postgresql-7.4.6/config/perl.m4 fixes the problem: |
#3
| |||
| |||
|
|
David Walker <david (AT) cosmicfires (DOT) com> writes: This patch to postgresql-7.4.6/config/perl.m4 fixes the problem: |
|
Don't you think this is likely to break more cases than it fixes? You can't just arbitrarily say that no one else is going to need the ccdlflags. |
|
On the two platforms I checked it on, it seemed that the ccdlflags were a strict subset of what was in the ldopts result, so the proposed patch would make no difference, but I can't feel comfortable that it's true in general. (If it *is* true in general, then why does the patch fix your problem?) |
#4
| |||
| |||
|
|
On 2004-11-16, Tom Lane <tgl (AT) sss (DOT) pgh.pa.us> wrote: Don't you think this is likely to break more cases than it fixes? You're not correctly understanding what it does. |
|
As I originally said in IRC, I do not know why the configure script is trying to second-guess the ExtUtils::Embed output; however, what it is doing clearly produces the wrong results. |
#5
| |||
| |||
|
|
Andrew - Supernews <andrew+nonews (AT) supernews (DOT) com> writes: As I originally said in IRC, I do not know why the configure script is trying to second-guess the ExtUtils::Embed output; however, what it is doing clearly produces the wrong results. I'm not sure why it's doing that either; taking the Embed output as gospel would seem like a reasonable thing to do. But I'm hesitant to change code that's been the same since PG 7.3 and therefore has survived two port testing cycles without previous complaints. |
#6
| |||
| |||
|
|
The ldopts value contains compiler-specific shared library linking flags plus information about additional library to link in. The ccdlflags value contains only the compiler-specific shared library linking flags. If we subtract the second from the first, we get the information about additional library to link in, which is what we wanted to get out of it. |
|
The poster apparently needs the rpath information, but I think he'll have to get that some other way. |
#7
| |||
| |||
|
|
Peter, it was you that committed the current contents of config/perl.m4; do you recall why it wants to remove ccdlflags from the Embed results? |
#8
| |||
| |||
|
|
Peter Eisentraut <peter_e (AT) gmx (DOT) net> writes: The ldopts value contains compiler-specific shared library linking flags plus information about additional library to link in. The ccdlflags value contains only the compiler-specific shared library linking flags. If we subtract the second from the first, we get the information about additional library to link in, which is what we wanted to get out of it. |
|
[ thinks ... ] OK, so the problem case is where perl was built with a different compiler than we're building PG with, |
|
Where would he get it from, then? We did talk about providing macros for more flexible specification of rpath, so we can fix the problem if we can get the Perl library path, but I'm unsure where to learn that. |
#9
| |||
| |||
|
|
On 2004-11-16, Tom Lane <tgl (AT) sss (DOT) pgh.pa.us> wrote: [ thinks ... ] OK, so the problem case is where perl was built with a different compiler than we're building PG with, Which is a non-starter in any event - you can't reliably embed perl into a program that's compiled with a different compiler than perl was, |
|
Where would he get it from, then? We did talk about providing macros for more flexible specification of rpath, so we can fix the problem if we can get the Perl library path, but I'm unsure where to learn that. Oddly enough, that's what ccdlflags is for. So you'd end up removing that info from ldopts only to add it back... which seems somewhat pointless to me. |
![]() |
| Thread Tools | |
| Display Modes | |
| |