From: | Richard PALO <richard(dot)palo(at)baou(dot)fr> |
---|---|
To: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Fwd: patch proposal for acinclude.m4 to better support gcc on illumos/smartos |
Date: | 2013-07-19 06:51:24 |
Message-ID: | ksanl6$5he$1@ger.gmane.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
As this seemed to fall upon deaf ears, I'll reiterate my request for
integration of the following patch which fixes the fact that the linker
directive '-as-needed' is not supported by Solaris ld.
-------- Message original --------
Sujet: patch proposal for acinclude.m4 to better support gcc on
illumos/smartos
Date : Fri, 19 Oct 2012 08:36:23 +0200
De : Richard PALO <richard(dot)palo(at)baou(dot)fr>
Pour : pgadmin-hackers(at)postgresql(dot)org
Groupes de discussion: gmane.comp.db.postgresql.pgadmin.devel
As /usr/bin/ld on illumos doesn't (yet) support --as-needed, this patch
to acinclude.m4 will suppress the LDFLAGS -Wl,-as-needed passed when gcc
is used as the compiler in the "bash bootstrap" command.
This patch assumes that all gcc variants running on illumos (gcc3 and
gcc4) are configured with --without-gnu-ld --with-ld=/usr/bin/ld,
which after brief research, seems typically the case.
This has been tested and seems to work fine on oi_151a7 with gcc47 under
pkgsrc http://pkgsrc.smartos.org/packages/illumos/2012Q3/All
~/src/pgadmin3$ git diff
diff --git a/acinclude.m4 b/acinclude.m4
index e9c5a5d..c964e93 100644
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -684,7 +684,9 @@ AC_DEFUN([SETUP_POSTGRESQL],
case "${host}" in
*-apple-darwin*)
;;
-
+ *solaris*)
+ LDFLAGS="$LDFLAGS"
+ ;;
*)
LDFLAGS="$LDFLAGS -Wl,-as-needed"
;;
Attachment | Content-Type | Size |
---|---|---|
gcc-illumos.patch | text/x-patch | 323 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2013-07-19 08:56:20 | pgAdmin III commit: Don't use --as-needed on Solaris. |
Previous Message | Neel Patel | 2013-07-19 06:21:29 | Re: Materialized View Patch File |