OK, I have made Solaris gcc the same as Linux in Makefile.shlib, patch attached.
I am not backpatching this. We will get this tested for 8.3. --------------------------------------------------------------------------- Jignesh K. Shah wrote: > I dont think we solved this.. But I think the way to put -m64 should be same > as in Linux and Solaris > and not different. > > Thanks. > Regards, > Jignesh > > > Tom Lane wrote: > > Peter Eisentraut <[EMAIL PROTECTED]> writes: > >> Am Mittwoch, 17. Januar 2007 17:12 schrieb Tom Lane: > >>> "Jignesh K. Shah" <[EMAIL PROTECTED]> writes: > >>>> simple if I use -m64 for 64 bit then all end binaries are generated > >>>> 64-bit and the shared libraries are generated 32-bit and the compilation > >>>> fails (ONLY ON SOLARIS) since that particular line is only for the > >>>> condition Solaris AND gcc. > >>>> > >>>> If I use the COMPILER which is CC + CFLAGS it passes -m64 properly to it > >>>> and generates shared libraries 64-bit and the compile continues.. > >>> Hmm ... I see we're doing it that way already for some other platforms, > >>> but I can't help thinking it's a kluge. Wouldn't the correct answer be > >>> that -m64 needs to be in LDFLAGS? > > > >> The correct answer may be to put -m64 into CC. > > > > Did we conclude that that was a satisfactory solution, or is this still > > a live patch proposal? > > > > If -m64 in CC is the right solution, it should probably be mentioned in > > FAQ_Solaris. > > > > regards, tom lane > > ---------------------------(end of broadcast)--------------------------- > TIP 3: Have you checked our extensive FAQ? > > http://www.postgresql.org/docs/faq -- Bruce Momjian <[EMAIL PROTECTED]> http://momjian.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. +
Index: src/Makefile.shlib =================================================================== RCS file: /cvsroot/pgsql/src/Makefile.shlib,v retrieving revision 1.108 diff -c -c -r1.108 Makefile.shlib *** src/Makefile.shlib 11 Feb 2007 19:31:45 -0000 1.108 --- src/Makefile.shlib 20 Feb 2007 22:45:06 -0000 *************** *** 191,197 **** ifeq ($(PORTNAME), solaris) ifeq ($(GCC), yes) ! LINK.shared = $(CC) -shared else # CFLAGS added for X86_64 LINK.shared = $(CC) -G $(CFLAGS) --- 191,197 ---- ifeq ($(PORTNAME), solaris) ifeq ($(GCC), yes) ! LINK.shared = $(COMPILER) -shared # $(COMPILER) needed for -m64 else # CFLAGS added for X86_64 LINK.shared = $(CC) -G $(CFLAGS)
---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match