Re: Extensions, this time with a patch - Mailing list pgsql-hackers

From Dimitri Fontaine
Subject Re: Extensions, this time with a patch
Date
Msg-id m2aam7oa0t.fsf@2ndQuadrant.fr
Whole thread Raw
In response to Re: Extensions, this time with a patch  ("David E. Wheeler" <david@kineticode.com>)
Responses Re: Extensions, this time with a patch  ("David E. Wheeler" <david@kineticode.com>)
Re: Extensions, this time with a patch  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
"David E. Wheeler" <david@kineticode.com> writes:
> Sure. The reason to do it, though, is so that extension authors can create
> just one metadata file, instead of two (or three, if one must also put such
> data into the Makefile).

That's a good idea, but my guess is that the implementation cost of
supporting the control format in your perl infrastructure is at least an
order of magnitude lower than the cost for me to support your current
JSON file format, so I lean towards you having an automated way to fill
in the json file from the control one...

The Makefile supports $(VERSION) because chances are it's already there
(think packaging or tarball release targets). Having yet another place
where to manually maintain a version number ain't appealing.

In the latest patch, though, the only other thing you find in the
Makefile about the extension is its basename, which must be the one of
both the .control and the .sql files. And it's possible for $(EXTENSION)
to be a list of them, too, because of contrib/spi.

Regards,
-- 
Dimitri Fontaine
http://2ndQuadrant.fr     PostgreSQL : Expertise, Formation et Support


pgsql-hackers by date:

Previous
From: Mark Wong
Date:
Subject: Re: PostgreSQL and HugePage
Next
From: Mark Wong
Date:
Subject: Re: PostgreSQL and HugePage