Re: psql not showing triggers

[ There is text before PGP section. ]
>
-- Start of PGP signed section.
> While playing around with trying to add foreign keys to the
> \d table display in psql, I noticed that tableinfo.triggers
> is not used once it is set. I think it is meant to go here:
>
>  /* count triggers */
>         if (!error && tableinfo.hasrules)
>
> as:
>
>  /* count triggers */
>         if (!error && tableinfo.triggers)
>
>
> Does that seem right? Since the archive links on the web page,
> and cvs appear to be down, I cannot check if this has been
> already fixed. Also, after all the hullabaloo about cvs, is there
> a definitive page for using it? This one, maybe?:
>
> http://developer.postgresql.org/TODO/docs/cvs.html

Good catch.  Your guess was right, and it hadn not been fixed.  Patch
attached and applied.

As far as CVS, that page should be accurate.  It was updated a few days
ago with the correct information.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
Index: src/bin/psql/describe.c
===================================================================
RCS file: /cvsroot/pgsql/src/bin/psql/describe.c,v
retrieving revision 1.39
diff -c -r1.39 describe.c
*** src/bin/psql/describe.c    2001/08/21 16:36:05    1.39
--- src/bin/psql/describe.c    2001/10/06 14:39:07
***************
*** 793,799 ****
          }

          /* count triggers */
!         if (!error && tableinfo.hasrules)
          {
              sprintf(buf,
                      "SELECT t.tgname\n"
--- 793,799 ----
          }

          /* count triggers */
!         if (!error && tableinfo.triggers)
          {
              sprintf(buf,
                      "SELECT t.tgname\n"