Changes 8.3.3:

* Make pg_get_ruledef() parenthesize negative constants (Tom)
  Before this fix, a negative constant in a view or rule might be
  dumped as, say, -42::integer, which is subtly incorrect: it should
  be (-42)::integer due to operator precedence rules. Usually this
  would make little difference, but it could interact with another
  recent patch to cause PostgreSQL to reject what had been a valid
  "SELECT DISTINCT" view query. Since this could result in pg_dump
  output failing to reload, it is being treated as a high-priority
  fix. The only released versions in which dump output is actually
  incorrect are 8.3.1 and 8.2.7.
* Make "ALTER AGGREGATE ... OWNER TO" update pg_shdepend (Tom)
  This oversight could lead to problems if the aggregate was later
  involved in a "DROP OWNED" or "REASSIGN OWNED" operation.

3 files changed, 9 insertions, 49 deletions

diff --git a/databases/postgresql83/Makefile.common b/databases/postgresql83/Makefile.common
index 55a72cd6840..3fcb8b83887 100644
--- a/databases/postgresql83/Makefile.common
+++ b/databases/postgresql83/Makefile.common
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile.common,v 1.2 2008/04/05 16:39:26 adam Exp $
+# $NetBSD: Makefile.common,v 1.3 2008/06/20 06:54:42 adam Exp $
 #
 # This Makefile fragment is included by all PostgreSQL packages built from
 # the main sources of the PostgreSQL distribution except jdbc-postgresql.
@@ -37,7 +37,7 @@ PATCHDIR?=		${.CURDIR}/../postgresql83/patches
 #
 # Note: Do not forget jdbc-postgresql83 when updating version
 #
-DIST_VERS?=		8.3.1
+DIST_VERS?=		8.3.3
 BASE_VERS?=		${DIST_VERS}
 
 BUILDLINK_API_DEPENDS.postgresql83-client+=	postgresql83-client>=${BASE_VERS}
diff --git a/databases/postgresql83/distinfo b/databases/postgresql83/distinfo
index 8eebe9915e5..71470230fd9 100644
--- a/databases/postgresql83/distinfo
+++ b/databases/postgresql83/distinfo
@@ -1,10 +1,10 @@
-$NetBSD: distinfo,v 1.2 2008/04/05 16:39:27 adam Exp $
+$NetBSD: distinfo,v 1.3 2008/06/20 06:54:42 adam Exp $
 
-SHA1 (postgresql-8.3.1.tar.bz2) = 7397e0d8fd3c420ee845d9ae4995ae347bfaea1d
-RMD160 (postgresql-8.3.1.tar.bz2) = 075efb67126f2b308d2a5fe4f9097fdbfbe94c89
-Size (postgresql-8.3.1.tar.bz2) = 13995572 bytes
+SHA1 (postgresql-8.3.3.tar.bz2) = b933f26a70bef9a5971c0ff2f392e293eed138b2
+RMD160 (postgresql-8.3.3.tar.bz2) = 8cc31cae862e328c8372245c8f953a1773dcc1f8
+Size (postgresql-8.3.3.tar.bz2) = 14004538 bytes
 SHA1 (patch-aa) = a7112ce2e9fc28f638afcf2e196d3cdc0f4c254d
-SHA1 (patch-ab) = 8bc81e1f95cf8933a4da2e557539764054b51f5f
+SHA1 (patch-ab) = 61ed51b483b84c3ca88f8955713dff27f9070a6c
 SHA1 (patch-ac) = 8e3a7021fdd01810d4ebcbd20002b28164b22279
 SHA1 (patch-ad) = 5612e447bbb76e81f72d39ad0153b9f77d47f653
 SHA1 (patch-ae) = 8b3e47320dfe05d94b769c9b079fd7ca6d26f5d6
diff --git a/databases/postgresql83/patches/patch-ab b/databases/postgresql83/patches/patch-ab
index c4ba986e814..2017473b549 100644
--- a/databases/postgresql83/patches/patch-ab
+++ b/databases/postgresql83/patches/patch-ab
@@ -1,6 +1,6 @@
-$NetBSD: patch-ab,v 1.2 2008/04/05 16:39:27 adam Exp $
+$NetBSD: patch-ab,v 1.3 2008/06/20 06:54:42 adam Exp $
 
---- configure.orig	2008-03-14 04:23:21.000000000 +0100
+--- configure.orig	2008-06-09 02:38:40.000000000 +0200
 +++ configure
 @@ -1510,6 +1510,7 @@ case $host_os in
    darwin*) template=darwin ;;
@@ -21,43 +21,3 @@ $NetBSD: patch-ab,v 1.2 2008/04/05 16:39:27 adam Exp $
  echo "$as_me:$LINENO: result: $perl_embed_ldflags" >&5
  echo "${ECHO_T}$perl_embed_ldflags" >&6
  fi
-@@ -24263,16 +24262,12 @@ cat >>conftest.$ac_ext <<_ACEOF
- /* end confdefs.h.  */
- 
- /* Override any gcc2 internal prototype to avoid an error.  */
--#ifdef __cplusplus
--extern "C"
--#endif
--/* We use char because int might match the return type of a gcc2
--   builtin and then its argument prototype would still apply.  */
--char gettext ();
-+#include <libintl.h>
-+
- int
- main ()
- {
--gettext ();
-+gettext ("");
-   ;
-   return 0;
- }
-@@ -24318,16 +24313,12 @@ cat >>conftest.$ac_ext <<_ACEOF
- /* end confdefs.h.  */
- 
- /* Override any gcc2 internal prototype to avoid an error.  */
--#ifdef __cplusplus
--extern "C"
--#endif
--/* We use char because int might match the return type of a gcc2
--   builtin and then its argument prototype would still apply.  */
--char gettext ();
-+#include <libintl.h>
-+
- int
- main ()
- {
--gettext ();
-+gettext ("");
-   ;
-   return 0;
- }