patch proposal for acinclude.m4 to better support gcc on illumos/smartos

From: Richard PALO <richard(dot)palo(at)baou(dot)fr>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: patch proposal for acinclude.m4 to better support gcc on illumos/smartos
Date: 2012-10-19 06:36:23
Message-ID: k5qsd5$8q2$1@ger.gmane.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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 322 bytes

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dhiraj Chawla 2012-10-20 08:38:15 pgAdmin3 crash when creating new FTS Configuration
Previous Message Dave Page 2012-10-17 14:12:31 Re: pgAdmin with SSHTunneling support