Tom Lane wrote:
> Bruce Momjian <br...@momjian.us> writes:
> > Tom Lane wrote:
> >> Also, the recommendation to specify prefix here is redundant and
> >> error-prone.  It can get the correct prefix from pg_config.
> 
> > Again, see my email just posted about using pg_migrator in a
> > multi-pg_config-binary environment.
> 
> What's your point?  If the wrong pg_config is first in the path,
> PGXS will do the wrong things anyway.  Specifying install prefix
> manually is not enough to prevent that.

I already tell them to make sure the new one is first in the path:

        A third install method is to use PGXS (assuming the new 'pg_config'
        is the first 'pg_config' in your $PATH):

> If the pg_migrator makefiles conformed to the accepted syntax for
> PGXS invocation (click click ... which they do now) then the preferred

Good, thanks.

> solution for this would be to explicitly specify the pg_config
> location, *and nothing else*, on the make command line.
> 
>       gmake USE_PGXS=1 PG_CONFIG=/path/to/pg_config install

OK, I have updated the INSTALL file to use that and not tell them to
modify their path.

> I'm inclined to agree with Peter that the current documentation
> is just going to confuse people by giving them too many options.
> I would agree with removing the "ifdef USE_PGXS" from the makefiles
> so that the USE_PGXS=1 bit can go away, and then saying
> 
>       If the new installation's pg_config is first in your path,
>       you can just say
>               gmake install
>       Otherwise, or to be extra sure, say
>               gmake PG_CONFIG=/path/to/pg_config install
> 
> (Although actually, if that pg_config isn't in your path, the
> installed pg_migrator won't be either.  It might be better to just
> say "fix things so that the new installation's executables are
> first in your PATH", and be done with it.)

I am betting many people will do src/pg_migrator to run it.  However,
there will be no pg_migrator binary in the old cluster (hopefully) so it
would still find it even it the new cluster is not first in the path.

Here is the new text I am using:

        The simplest build option is to use PGXS:
        
                gmake USE_PGXS=1 PG_CONFIG=/usr/local/pgsql/bin/pg_config
        install
        
        
        Another option is to point to the top of the new PostgreSQL source tree
        by running something like:
        
                gmake top_builddir=/usr/src/pgsql install
        
        Replace '/usr/src/pgsql' with your new source directory.  pg_migrator
        also understands the 'prefix=' specification if you installed Postgres
        in a custom location.
        
I am happy to remove the USE_PGXS from the Makefile, but it seems all
the other extensions require that so I want to be consistent.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

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

Reply via email to