Re: add server include files to default installation?

Поиск
Список
Период
Сортировка
От Fabien COELHO
Тема Re: add server include files to default installation?
Дата
Msg-id Pine.LNX.4.58.0405191336490.20575@sablons.cri.ensmp.fr
обсуждение исходный текст
Ответ на Re: add server include files to default installation?  (Bruce Momjian <pgman@candle.pha.pa.us>)
Ответы Re: add server include files to default installation?
Список pgsql-hackers
Dear Thomas,

> My point is, in order to maintain a good separation of concern, I
> should not use this knowledge.

I agree.

> IMHO, there's a need for a well documented "module extension build API"
> accessible from a well known place.

Sure.

I'm not really addressing that at the moment, I'm addressing having the
necessary files. Given gnu-make constraints about makefile inclusions,
that requires to have the same layout as the initial source tree, whatever
it is. I'm not sure that dropping all includes would be an easy move, nor
even desirable.


> This is more important than to include it in the default installation.

Well, the point of having an infrastructure if extensions cannot rely it
being there when needed is not clear to me.


> Your suggestion to rewrite the Makefile.global etc. is a good first step.

It is what I'm trying to avoid;-) It is not just this makefile, but the
whole include-based makefiles that are at stake. I think that an
infrastructure is possible without fixing these files.

So as I understand your point as "the layout constraints should not be
visible from the extensions API", I suggest that extension makefiles
should look like (adapted from contrib):
PGXSDIR := $(shell pg_config --pgxs-dir)include $(PGXSDIR)/pgxs_begin.mk[... set required macros as in contrib
...]include$(PGXSDIR)/pgxs_end.mk
 

It is independent from what my first patch addresses, that is
tryinbg to put all required files somewhere on install.

Thanks for your point,

-- 
Fabien Coelho - coelho@cri.ensmp.fr


В списке pgsql-hackers по дате отправления:

Предыдущее
От: "Thomas Hallgren"
Дата:
Сообщение: Re: add server include files to default installation?
Следующее
От: "Atesz"
Дата:
Сообщение: An Index Scanning Solution question