Thread: pgsql: Backpatch BCC compile changes to 8.0.X for psql.

Log Message:
-----------
Backpatch BCC compile changes to 8.0.X for psql.

Modified Files:
--------------
    pgsql/src/bin/psql:
        bcc32.mak (r1.12 -> r1.13)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/bcc32.mak.diff?r1=1.12&r2=1.13)
        command.c (r1.142 -> r1.143)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/command.c.diff?r1=1.142&r2=1.143)
        startup.c (r1.113 -> r1.114)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/startup.c.diff?r1=1.113&r2=1.114)
        win32.mak (r1.26 -> r1.27)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/win32.mak.diff?r1=1.26&r2=1.27)
    pgsql/src/interfaces/libpq:
        Makefile (r1.131 -> r1.132)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/Makefile.diff?r1=1.131&r2=1.132)
        bcc32.mak (r1.15 -> r1.16)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/bcc32.mak.diff?r1=1.15&r2=1.16)
        fe-exec.c (r1.166 -> r1.167)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/fe-exec.c.diff?r1=1.166&r2=1.167)
        pthread-win32.c (r1.4 -> r1.5)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/pthread-win32.c.diff?r1=1.4&r2=1.5)
        win32.mak (r1.33 -> r1.34)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/win32.mak.diff?r1=1.33&r2=1.34)

momjian@svr1.postgresql.org (Bruce Momjian) writes:
>         startup.c (r1.113 -> r1.114)
>         (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/startup.c.diff?r1=1.113&r2=1.114)

You probably just broke at least one Unix platform.  My recollection is
that "int optreset", not "extern int optreset", is essential on the
platforms that don't HAVE_INT_OPTRESET.  Certainly we do that
everyplace else.

Please wrap this change in a suitable bcc #ifdef.

            regards, tom lane

Tom Lane wrote:
> momjian@svr1.postgresql.org (Bruce Momjian) writes:
> >         startup.c (r1.113 -> r1.114)
> >         (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/startup.c.diff?r1=1.113&r2=1.114)
>
> You probably just broke at least one Unix platform.  My recollection is
> that "int optreset", not "extern int optreset", is essential on the
> platforms that don't HAVE_INT_OPTRESET.  Certainly we do that
> everyplace else.
>
> Please wrap this change in a suitable bcc #ifdef.

OK, I just put it back to original because I couldn't get it to link
anyway for BCC.

--
  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