extension installation faillure using VPATH and MODULEDIR

Started by Cédric Villemainover 13 years ago4 messagesbugs
Jump to latest
#1Cédric Villemain
cedric@2ndquadrant.com

the attached patch fix the extension installation using a VPATH when a
MODULEDIR is defined in the extension Makefile.

The current issue is that the sharedir/extension directory is not created if
MODULEDIR != 'extension' i.e. MODULEDIR is defined in the Makefile. Thus 'make
install' fail to copy the control file in an nonexistent directory.

Tested with debian extension packages.
--
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation

Attachments:

0001-Fix-extension-install.patchtext/x-patch; charset=UTF-8; name=0001-Fix-extension-install.patchDownload+3-1
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Cédric Villemain (#1)
Re: extension installation faillure using VPATH and MODULEDIR

=?iso-8859-1?q?C=E9dric_Villemain?= <cedric@2ndquadrant.com> writes:

the attached patch fix the extension installation using a VPATH when a
MODULEDIR is defined in the extension Makefile.

The current issue is that the sharedir/extension directory is not created if
MODULEDIR != 'extension' i.e. MODULEDIR is defined in the Makefile. Thus 'make
install' fail to copy the control file in an nonexistent directory.

[ scratches head ... ] What's this have to do with VPATH?

I see the point that if EXTENSION is set, and a nondefault MODULEDIR is
selected, the "install" target puts the control file into the extension
subdirectory which hasn't been made by installdirs. But that seems not
connected to VPATH builds.

regards, tom lane

--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs

#3Cédric Villemain
cedric@2ndquadrant.com
In reply to: Tom Lane (#2)
Re: extension installation faillure using VPATH and MODULEDIR

Le samedi 1 décembre 2012 18:44:12, Tom Lane a écrit :

=?iso-8859-1?q?C=E9dric_Villemain?= <cedric@2ndquadrant.com> writes:

the attached patch fix the extension installation using a VPATH when a
MODULEDIR is defined in the extension Makefile.

The current issue is that the sharedir/extension directory is not created
if MODULEDIR != 'extension' i.e. MODULEDIR is defined in the Makefile.
Thus 'make install' fail to copy the control file in an nonexistent
directory.

[ scratches head ... ] What's this have to do with VPATH?

I see the point that if EXTENSION is set, and a nondefault MODULEDIR is
selected, the "install" target puts the control file into the extension
subdirectory which hasn't been made by installdirs. But that seems not
connected to VPATH builds.

I just hit the bug and though it was relative to VPATH because it was the main
difference in my build. Obviously in my others scenariii the extension/
directory was pre-existent and I didn't notice the problem.

--
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation

#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Cédric Villemain (#3)
Re: extension installation faillure using VPATH and MODULEDIR

=?iso-8859-15?q?C=E9dric_Villemain?= <cedric@2ndquadrant.com> writes:

Le samedi 1 d�cembre 2012 18:44:12, Tom Lane a �crit :

[ scratches head ... ] What's this have to do with VPATH?

I see the point that if EXTENSION is set, and a nondefault MODULEDIR is
selected, the "install" target puts the control file into the extension
subdirectory which hasn't been made by installdirs. But that seems not
connected to VPATH builds.

I just hit the bug and though it was relative to VPATH because it was the main
difference in my build. Obviously in my others scenariii the extension/
directory was pre-existent and I didn't notice the problem.

No worries, I was just wondering if you were seeing something more than
the problem described. Committed to 9.1 and up.

regards, tom lane

--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs