Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p
От | Bruce Momjian |
---|---|
Тема | Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p |
Дата | |
Msg-id | 200405192342.i4JNgqW14736@candle.pha.pa.us обсуждение исходный текст |
Ответ на | Re: [COMMITTERS] pgsql-server/src backend/main/main.c b (Claudio Natoli <claudio.natoli@memetrics.com>) |
Ответы |
Re: [COMMITTERS] pgsql-server/src backend/main/main.c b ackend/p
|
Список | 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 по дате отправления: