Fabien COELHO wrote:
> Dear patchers,
>
> Please find attached version number 3 for a patch to enable
> extensions such as contribs or external add-ons to be installed
> simply with an already installed postgresql.

OK, I think we're getting somewhere.

I am still opposed to adding more targets of the form "light-install", 
"client-only install", etc.  Please discuss this on -hackers.  It can 
be done as a separate patch later on if need be.

While I now understand what you are doing in contrib, I ask who is going 
to want to maintain two parallel sets of makefiles, one for PGXS and 
one for in-tree builds?  One who is going to want to use the PGXS ones 
anyway?  I realize they're good examples, but examples should be put 
into the documentation, so people can find them.

Paste your documentation (pgxs.sgml) somewhere into xfunc.sgml, where it 
discusses writing user-defined functions.  This material isn't long 
enough to warrant a chapter of its own.

+ ifdef PGXS
+ LDFLAGS += -L$(pkglibdir)
+ endif

needs to disappear.  There is nothing to link with in there.  (If there 
is, that's a bug.)

libpgport should be installed in the normal libdir.

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


---------------------------(end of broadcast)---------------------------
TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]

Reply via email to