Skip site navigation (1) Skip section navigation (2)

Re: [COMMITTERS] pgsql-server/src backend/main/main.c b

From: Claudio Natoli <claudio(dot)natoli(at)memetrics(dot)com>
To: 'Bruce Momjian' <pgman(at)candle(dot)pha(dot)pa(dot)us>,Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql-server/src backend/main/main.c b
Date: 2004-05-19 23:36:07
Message-ID: A02DEC4D1073D611BAE8525405FCCE2B55F46B@harris.memetrics.local (view raw or flat)
Thread:
Lists: pgsql-patches
> Tom Lane wrote:
> > Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)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.

Cheers,
Claudio

--- 
Certain disclaimers and policies apply to all email sent from Memetrics.
For the full text of these disclaimers and policies see 
<a
href="http://www.memetrics.com/emailpolicy.html">http://www.memetrics.com/em
ailpolicy.html</a>

Responses

pgsql-patches by date

Next:From: Bruce MomjianDate: 2004-05-19 23:42:52
Subject: Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p
Previous:From: Bruce MomjianDate: 2004-05-19 22:52:18
Subject: New native timezone library patch for all platforms

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group