Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p
Date
Msg-id 200405192342.i4JNgqW14736@candle.pha.pa.us
Whole thread Raw
In response to Re: [COMMITTERS] pgsql-server/src backend/main/main.c b  (Claudio Natoli <claudio.natoli@memetrics.com>)
Responses Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p
List pgsql-patches
Claudio Natoli wrote:
>
> > Tom Lane wrote:
> > > Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > > > Tom Lane wrote:
> > > >> Also, in the non-exec case it seems unnecessary and possibly unsafe
> to
> > > >> repeat this computation in a backend.
> > >
> > > > Thanks.  It originally was as below and I forgot about that.  It now
> is:
> > > >     if (my_exec_path[0] == '\0' && find_my_exec(argv[0],
> my_exec_path) < 0)
> > > > The issue is that fork/exec will need to set it too.
> > >
> > > That fixes it for my_exec_path, but what of the pkglib_path stuff?
> >
> > OK, I will fix that one, though it is a cheap call.
>
> There's also the possibility of adding it to read/write_backend_variables,
> just as we add other variables for fork/exec that we want the backends to
> inherit.

In the exec case, the executable name is actually different from the
postmaster, so passing it doesn't make sense.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

pgsql-patches by date:

Previous
From: Claudio Natoli
Date:
Subject: Re: [COMMITTERS] pgsql-server/src backend/main/main.c b
Next
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p