From: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org, Andres Freund <andres(at)anarazel(dot)de> |
Subject: | Re: Should contrib modules install .h files? |
Date: | 2018-07-02 14:10:42 |
Message-ID: | 87lgatohzp.fsf@news-spur.riddles.org.uk |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>>>>> "Tom" == Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> writes:
>> I'm thinking that $(includedir_server)/$(MODULEDIR) would be a
>> reasonable place? MODULEDIR defaults to either "contrib" or
>> "extension" depending on whether EXTENSION is set. Something like
>> the attached patch seem reasonable?
Tom> FWIW, I agree with Andres' thought that each contrib module should
Tom> have its own subdirectory under $(includedir_server). Otherwise
Tom> we're going to be faced with questions about whether .h files need
Tom> to be renamed because they're not globally unique enough. There
Tom> are already some that are pretty shaky from this standpoint:
I'm not suggesting that all modules should install a .h file or that all
of a module's .h files should be installed.
A slight snag in trying to use a subdir for each module is that there is
not in fact anywhere in the existing makefiles that uses or assigns such
a name. Indeed some contrib subdirs install multiple modules.
Tom> Not sure about whether the MODULEDIR part is useful. Seems like
Tom> making a distinction between extensions and other contrib is
Tom> likely to create more headaches than it avoids.
Sure, but that's just copied from DATA and DOCS which already do it that
way. For DATA there seems some justification based on CREATE EXTENSION,
but for docs?
--
Andrew (irc:RhodiumToad)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2018-07-02 14:16:22 | Re: Should contrib modules install .h files? |
Previous Message | Jesper Pedersen | 2018-07-02 13:54:40 | Re: [HACKERS] Fix performance degradation of contended LWLock on NUMA |