Re: [PATCHES] contrib/spi makefile inconsistency

2007-03-23 Thread Magnus Hagander
On Thu, Mar 22, 2007 at 12:33:30PM -0400, Tom Lane wrote:
> Magnus Hagander <[EMAIL PROTECTED]> writes:
> > contrib/spi has a Makefile that uses $(addsuffix) makefile rules for
> > DATA_built and DOCS. It's the only Makefile in contrib that does it.
> > I would like to change that to actually listing the modules instead.
> 
> This seems like a definite regression in terms of writing the makefiles.
> I'm not very happy about it --- just how much complexity are you saving
> in the msvc support?  And have you looked at the other places where
> addsuffix is already used?  (It looks like many of the contrib modules
> rely on pgxs.mk's usages of addsuffix...)

A grep shows that only spi/Makefile uses addsuffix...

(The rules in the pgxs makefiles are "emulated" and does the same
functionality, but I don't actuallyi *parse* the pgxs makefile)

But. Given the comments from you and Peter, I'll hold back on doing this
and attempt to parse it. For the current testing ("just get the darn tests
running"), I'll hard-code the spi exception in the msvc build code instead
- pending a proper parser for it.

//Magnus

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


Re: [PATCHES] contrib/spi makefile inconsistency

2007-03-22 Thread Peter Eisentraut
Magnus Hagander wrote:
> I would like to change that to actually listing the modules instead.
> The reason for that is that parsing the Makefile for the msvc build
> will be a *lot* easier if I don't have to parse $addsuffix rules.

Let's not open that can of worms.  Even if you think it's only a small 
can.

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org


Re: [PATCHES] contrib/spi makefile inconsistency

2007-03-22 Thread Tom Lane
Magnus Hagander <[EMAIL PROTECTED]> writes:
> contrib/spi has a Makefile that uses $(addsuffix) makefile rules for
> DATA_built and DOCS. It's the only Makefile in contrib that does it.
> I would like to change that to actually listing the modules instead.

This seems like a definite regression in terms of writing the makefiles.
I'm not very happy about it --- just how much complexity are you saving
in the msvc support?  And have you looked at the other places where
addsuffix is already used?  (It looks like many of the contrib modules
rely on pgxs.mk's usages of addsuffix...)

regards, tom lane

---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

http://www.postgresql.org/about/donate