Thread: VPATH problems in ECPG again
Hi, I just noticed a VPATH problem in the ecpg_config.h file that was just set up to be installed: $ pwd /pgsql/build/80_rel/src/interfaces/ecpg/include $ make install for i in ecpgerrno.h ecpglib.h ecpgtype.h sqlca.h sql3types.h ecpg_informix.h ecpg_config.h pgtypes_error.h pgtypes_numeric.hpgtypes_timestamp.h pgtypes_date.h pgtypes_interval.h; do /bin/sh /pgsql/source/80_rel/config/install-sh-c -m 644 /pgsql/source/80_rel/src/interfaces/ecpg/include/$i '/pgsql/install/80_rel/include'|| exit; done /pgsql/source/80_rel/config/install-sh: /pgsql/source/80_rel/src/interfaces/ecpg/include/ecpg_config.h does not exist. make: *** [install-headers] Error 1 This is natural, since ecpg_config.h lives in builddir, not in srcdir, unlike the rest of the headers. -- Alvaro Herrera http://www.CommandPrompt.com/ PostgreSQL Replication, Consulting, Custom Development, 24x7 support
Alvaro Herrera wrote: > Hi, > > I just noticed a VPATH problem in the ecpg_config.h file that was just > set up to be installed: > > $ pwd > /pgsql/build/80_rel/src/interfaces/ecpg/include > > $ make install > for i in ecpgerrno.h ecpglib.h ecpgtype.h sqlca.h sql3types.h ecpg_informix.h ecpg_config.h pgtypes_error.h pgtypes_numeric.hpgtypes_timestamp.h pgtypes_date.h pgtypes_interval.h; do /bin/sh /pgsql/source/80_rel/config/install-sh-c -m 644 /pgsql/source/80_rel/src/interfaces/ecpg/include/$i '/pgsql/install/80_rel/include'|| exit; done > /pgsql/source/80_rel/config/install-sh: /pgsql/source/80_rel/src/interfaces/ecpg/include/ecpg_config.h does not exist. > make: *** [install-headers] Error 1 > > This is natural, since ecpg_config.h lives in builddir, not in srcdir, > unlike the rest of the headers. I think this patch fixes it. -- Alvaro Herrera http://www.CommandPrompt.com/ PostgreSQL Replication, Consulting, Custom Development, 24x7 support
Attachment
Alvaro Herrera <alvherre@commandprompt.com> writes: >> I just noticed a VPATH problem in the ecpg_config.h file that was just >> set up to be installed: > I think this patch fixes it. Looks reasonable to me, please apply. regards, tom lane