Throw nice error if server is too old to support psql's \ef or \sf command.

Previously, you'd get "function pg_catalog.pg_get_functiondef(integer) does
not exist", which is at best rather unprofessional-looking.  Back-patch
to 8.4 where \ef was introduced.

Josh Kupershmidt

diff --git a/src/bin/psql/command.c b/src/bin/psql/command.c
index 7c44faeb2c62aaa470321c1e158fd17f242f37a3..98548d365510166137c8f62a62223c29ec4b970f 100644 (file)
--- a/src/bin/psql/command.c
+++ b/src/bin/psql/command.c
@@ -508,7 +508,13 @@ exec_command(const char *cmd,
         */
        else if (strcmp(cmd, "ef") == 0)
        {
-               if (!query_buf)
+               if (pset.sversion < 80400)
+               {
+                       psql_error("The server (version %d.%d) does not support editing function source.\n",
+                                          pset.sversion / 10000, (pset.sversion / 100) % 100);
+                       status = PSQL_CMD_ERROR;
+               }
+               else if (!query_buf)
                {
                        psql_error("no query buffer\n");
                        status = PSQL_CMD_ERROR;