Fix psql's \dd version check for operator families.

Report and patch by Josh Kupershmidt; comment revisions by me.

diff --git a/src/bin/psql/describe.c b/src/bin/psql/describe.c
index b5c14c57672e4438f24ae00d9263e3402f545a13..2ff6d7dc997da21141dd689f091295a48be7e630 100644 (file)
--- a/src/bin/psql/describe.c
+++ b/src/bin/psql/describe.c
@@ -876,8 +876,7 @@ objectDescription(const char *pattern, bool showSystem)
                                                  "pg_catalog.pg_table_is_visible(c.oid)");
 
        /*
-        * pg_opclass.opcmethod only available in 8.3+, and comment on operator
-        * family only available in 8.3+
+        * pg_opclass.opcmethod only available in 8.3+
         */
        if (pset.sversion >= 80300)
        {
@@ -902,7 +901,14 @@ objectDescription(const char *pattern, bool showSystem)
                processSQLNamePattern(pset.db, &buf, pattern, true, false,
                                                          "n.nspname", "o.opcname", NULL,
                                                          "pg_catalog.pg_opclass_is_visible(o.oid)");
+       }
 
+       /*
+        * although operator family comments have been around since 8.3,
+        * pg_opfamily_is_visible is only available in 9.2+
+        */
+       if (pset.sversion >= 90200)
+       {
                /* Operator family descriptions */
                appendPQExpBuffer(&buf,
                                                  "UNION ALL\n"