Remove bogus commutator marking --- the module doesn't actually supply

any commutator operator for =(chkpass,text), so this was creating a
shell operator that would fail on use.  Found by opr_sanity testing.

diff --git a/contrib/chkpass/chkpass.sql.in b/contrib/chkpass/chkpass.sql.in
index 68cd1e27659a3029d83e5e06e49961e837ef7253..40a0d722c908d8f841bfe02b175e72258ee39e61 100644 (file)
--- a/contrib/chkpass/chkpass.sql.in
+++ b/contrib/chkpass/chkpass.sql.in
@@ -4,7 +4,7 @@
 --  [email protected]
 --  http://www.druid.net/darcy/
 -- 
---  $PostgreSQL: pgsql/contrib/chkpass/chkpass.sql.in,v 1.6 2005/01/29 22:35:01 tgl Exp $
+--  $PostgreSQL: pgsql/contrib/chkpass/chkpass.sql.in,v 1.7 2007/09/29 23:32:42 tgl Exp $
 --
 --  best viewed with tabs set to 4
 --
@@ -53,17 +53,13 @@ CREATE FUNCTION ne(chkpass, text)
    LANGUAGE C STRICT;
 
 --
--- Now the operators.  Note how some of the parameters to some
--- of the 'create operator' commands are commented out.  This
--- is because they reference as yet undefined operators, and
--- will be implicitly defined when those are, further down.
+-- Now the operators.
 --
 
 CREATE OPERATOR = (
    leftarg = chkpass,
    rightarg = text,
-   commutator = =,
--- negator = <>,
+   negator = <>,
    procedure = eq
 );
 

diff --git a/contrib/chkpass/uninstall_chkpass.sql b/contrib/chkpass/uninstall_chkpass.sql
index 4a4737a1d559a8370030ee28f125a9d4d8c76d25..34ad3fde8fbf73f587ec36c72a8e524ab20e8fc2 100644 (file)
--- a/contrib/chkpass/uninstall_chkpass.sql
+++ b/contrib/chkpass/uninstall_chkpass.sql
@@ -4,8 +4,6 @@ DROP OPERATOR <>(chkpass, text);
 
 DROP OPERATOR =(chkpass, text);
 
-DROP OPERATOR =(text, chkpass);
-
 DROP FUNCTION ne(chkpass, text);
 
 DROP FUNCTION eq(chkpass, text);