Re: pg_ctl.c

Поиск
Список
Период
Сортировка
От Gaetano Mendola
Тема Re: pg_ctl.c
Дата
Msg-id 40B44DE2.4030209@mbigroup.it
обсуждение исходный текст
Ответ на pg_ctl.c  (Bruce Momjian <pgman@candle.pha.pa.us>)
Список pgsql-patches
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Bruce Momjian wrote:

| Gaetano Mendola wrote:
|
|>Bruce Momjian wrote:
|>
|>
|>>Here is the C version of pg_ctl.c written by Andrew Dunstan and updated
|>>by me.
|>>
|>>You can use it by creating a src/bin/pg_ctl_test directory and putting
|>>the C and Makefile into that directory.  You can then do a make install
|>>and use it for testing.
|>>
|>>Unless someone finds a problem, I will apply the change soon.  This
|>>removes our last shell script!
|>
|>It desn't compile on my platform:
|>
|>$ gcc -I /usr/include/pgsql/server pg_ctl.c
|>pg_ctl.c: In function `start_postmaster':
|>pg_ctl.c:219: error: `DEVNULL' undeclared (first use in this function)
|>pg_ctl.c:219: error: (Each undeclared identifier is reported only once
|>pg_ctl.c:219: error: for each function it appears in.)
|
|
| DEVNULL is in CVS port.h.  Are you running against CVS?

No. Against 7.4.1


|>2) xstrdup protected by duplicate NULL string
|
|
| You mean that you can't pass NULL to xstrdup?  Yea, but if we do, it
| will crash and we will hear about it right away.

Yea u'r right, it will crash without know why, for the same reason you can
avoid to use the xstrdup then.



Regards
Gaetano Mendola




-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (MingW32)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFAtE3h7UpzwH2SGd4RAjDhAKDV8lRn7XEAGqHLBJzQTvrBlyJsXgCgrw5c
CljcRIysGU3+IGoVbHU9vYg=
=9jrs
-----END PGP SIGNATURE-----


В списке pgsql-patches по дате отправления:

Предыдущее
От: "Magnus Hagander"
Дата:
Сообщение: Re: eventlog fix
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: [HACKERS] Configuration patch