[Pkg-postgresql-public] slony1 1.2.15 available, memory leak fixes included

[Gerfried Fuchs]

* Peter Eisentraut <peter_e at gmx.net> [2008-09-18 14:55:10 CEST]:
> Gerfried Fuchs wrote:
> >  Just got the notification from DEHS about the new upstream version from
> > slony1 being available. Peter, did you already take a look at it? From
> > what I read on the website it's mainly a bugfix release.
> 
> Yeah, it should probably be uploaded to unstable and pushed into lenny. 
>   I don't know if I will find the time immediately.

 I've taken the liberty to give 1.2.15 a shot. The complete diffstat is:

$ diff -Nru slony1-1.2.14/ slony1-1.2.15/ | diffstat
 RELEASE                                  |   16 +
 config.h.in                              |    6 
 configure                                |   22 +-
 configure.ac                             |    2 
 debian/changelog                         |   14 +
 debian/compat                            |    2 
 debian/control                           |    6 
 debian/patches/05_includedir-order.patch |    8 
 debian/rules                             |    1 
 doc/adminguide/slonik_ref.sgml           |    8 
 src/backend/slony1_funcs.sql             |   14 +
 src/slon/conf-file.c                     |   43 ++--
 src/slon/remote_worker.c                 |   11 -
 src/slonik/parser.c                      |  282 +++++++++++++++----------------
 src/slonik/scan.c                        |  199 ++++++++++-----------
 src/slonik/slonik.c                      |   19 +-
 src/slonik/y.tab.h                       |  220 ++++++++++++++++++++++++
 tools/check_slon.sh                      |    4 
 tools/check_slony_cluster.sh             |    4 
 tools/launch_clusters.sh                 |   10 -
 tools/mkslonconf.sh                      |    4 
 tools/pull-gborg-mail.sh                 |    5 
 tools/release_checklist.sh               |    8 
 tools/search-logs.sh                     |   10 -
 24 files changed, 589 insertions(+), 329 deletions(-)


 When removing the ovious changes like version strings, documentation,
$Id$ header chunks, it's reduced to:

$ diffstat diff 
 src/backend/slony1_funcs.sql |   12 +
 src/slon/conf-file.c         |   43 +++---
 src/slon/remote_worker.c     |    9 -
 src/slonik/parser.c          |  282 +++++++++++++++++++++----------------------
 src/slonik/scan.c            |  199 +++++++++++++++---------------
 src/slonik/slonik.c          |   17 +-
 src/slonik/y.tab.h           |  220 +++++++++++++++++++++++++++++++++
 tools/check_slon.sh          |    2 
 tools/check_slony_cluster.sh |    2 
 tools/launch_clusters.sh     |    8 -
 tools/mkslonconf.sh          |    2 
 tools/pull-gborg-mail.sh     |    3 
 tools/release_checklist.sh   |    6 
 tools/search-logs.sh         |    8 -
 14 files changed, 523 insertions(+), 290 deletions(-)

 Most of the differents are quite similar.

 Peter, where do you usually want to store the .orig.tar.gz, I noticed
upstream offers .tar.bz2 only? Furthermore there were some packaging
changes done that I'm not sure how the release team might react to
(debhelper moved to 7), so I'm not sure if some of those should get
reverted.

 Do you want me to commit the changelog snippet? I also noticed that one
of the patches only applied with an offset difference
(05_includedir-order.patch), I suggest that a refresh on that one should
be done. I could do the upload if you don't object and suggest what
direction to go with respect to getting it approved into lenny.

 Thanks,
Rhonda