Re: [HACKERS] perl interface bug?

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Brook Milligan <brook(at)trillium(dot)NMSU(dot)Edu>
Cc: E(dot)Mergl(at)bawue(dot)de, hackers(at)postgreSQL(dot)org
Subject: Re: [HACKERS] perl interface bug?
Date: 1998-10-15 15:39:27
Message-ID: 9014.908465967@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Brook Milligan <brook(at)trillium(dot)NMSU(dot)Edu> writes:
> We cannot kill this idea because 'make test' doesn't run without the
> postmaster. After all, NONE of the regression tests do either!

Edmund is thinking of a different situation. The perl5 interface is
also a "Perl module", which means that it is supposed to build and
install under very rigid rules --- this is supposed to work:
perl Makefile.PL
make
make test
make install

If you were installing the perl5 module separately from Postgres proper,
and already had a running Postgres server, that should indeed work.
(With the caveat that you first have to set POSTGRES_HOME environment
variable to /usr/local/pgsql or local equivalent.)

It's not *entirely* clear to me that that's a very plausible scenario,
since we distribute the perl5 module along with Postgres. But it does
work and we probably shouldn't break it.

Come to think of it, Brook's proposed changes do break the perl5
directory as a standalone Perl module, because it no longer includes a
Makefile.PL, only a Makefile.PL.in. Can we avoid that by finding some
other way to get the install target directory name into the perl5
makefile? Maybe, instead of being rewritten by autoconf, Makefile.PL
could actively go look for the directory name, say by looking to see if
../../Makefile.global exists and contains a POSTGRESDIR= line. If not,
fall back to requiring POSTGRES_HOME to be set.

regards, tom lane

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 1998-10-15 15:51:47 Re: Wishes for PostgreSQL 6.4
Previous Message D'Arcy J.M. Cain 1998-10-15 15:10:23 Re: AW: [HACKERS] PostgreSQL v6.4 BETA2...