Re: [HACKERS] Linux/Alpha and pgsql....

Ryan Kirkpatrick <rkirkpat@nag.cs.colorado.edu>

From: Ryan Kirkpatrick <rkirkpat@nag.cs.colorado.edu>
To: The Hermit Hacker <scrappy@hub.org>
Cc: Bruce Momjian <maillist@candle.pha.pa.us>, pgsql-hackers@postgreSQL.org
Date: 1998-04-11T21:41:48Z
Lists: pgsql-hackers

Attachments

On Sat, 11 Apr 1998, The Hermit Hacker wrote:

> On Sat, 11 Apr 1998, Ryan Kirkpatrick wrote:
> 
> > 	These two patches should bring the current version of pgsql very
> > close to working cleaning on Linux/Alpha! :) As usual, if you have any
> > questions, feel free to email me.
> 
> 	Seeing as how close we are to a v6.3.2 release, I put these in,
> but I don't like them...
> 
> 	We've been moving *away* from using -D's in order to deal with
> operating system/hardware related issues...using tools/ccsym, are there no
> compiler defined variables you can use for this instead?

	Hold on a second here.... The first patches I sent you were done
by modifying the defined(linuxalpha) to (defined(linux) &&
defined(alpha)) (the tools/ccsym method). But I was told that there was a
template for linux/alpha, and I should modify that, and so I used the
-D's. Which way do you all want it???
	For what its worth, I agree, the output from tools/ccsym should be
used and not -D's.
	Ok, attached are a new set of patches. First of all, ignore all
previous patches I have sent on this thread. Now these three work on the
idea that 'alpha' and 'linux' are both defined. The former is defined by
CFLAGS in template/linuxalpha, and latter is defined by the gcc on my UDB
(and all other Linux/Alpha systems). This removes the need to add any
defines to the CFLAGS line. To get rid of the other define (-Dalpha), we
simply need to replace each 'defined(alpha)' with '(defined(alpha) ||
defined(__alpha__))', as gcc automatically defines '__alpha__' like it
does 'linux'. If you want me to do this, just ask.
	Ok, now are these patches ok by everyone? :) Thanks.

----------------------------------------------------------------------------
|   "For to me to live is Christ, and to die is gain."                     |
|                                            --- Philippians 1:21 (KJV)    |
----------------------------------------------------------------------------
|  Ryan Kirkpatrick  |  Boulder, Colorado  | rkirkpat@nag.cs.colorado.edu  |
----------------------------------------------------------------------------
|               http://www-ugrad.cs.colorado.edu/~rkirkpat/                |
----------------------------------------------------------------------------