From: | Jerry Sievert <jerry(at)legitimatesounding(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-bugs(at)lists(dot)postgresql(dot)org |
Subject: | Re: BUG #15414: extension compilation fails when c++ files are named with .cc |
Date: | 2018-10-01 20:50:19 |
Message-ID: | 0C026788-9773-4AEF-B5DD-436EDFCD489A@legitimatesounding.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
Hi, see comments inline:
> On Oct 1, 2018, at 1:25 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
>
> I'm a bit worried that supporting extensions that PG itself doesn't use
Pg chose the extension .cpp, and chose it fairly recently. It seems to have been an arbitrary choice for the three files added as part of the JIT addition.
> I guess we have two choices:
> 1) Add rules for ".cpp", ".cc", ".C", ".cxx", and ".c++" and whatever
> else we can think of. Add additional variants whenever somebody
> presents one (and backpatch it).
This is a rather disingenuous choice. Perhaps a better approach would be to do a quick survey of the existing c++ extensions (there aren’t very many), and make a determination of what the community has chosen to support. As an aside, .C will be problematic on filesystems that are case-insensitive.
> 2) Insist that pg extensions follow PGXS rules, which atm are to use
> .cpp for c++ source files.
>
> I don't really have an opinion here.
You have an obvious opinion here, and had a very strong opinion on irc.
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2018-10-01 21:07:26 | Re: BUG #15414: extension compilation fails when c++ files are named with .cc |
Previous Message | Andres Freund | 2018-10-01 20:31:28 | Re: BUG #15414: extension compilation fails when c++ files are named with .cc |