Bug#698341: pu: package spatialite/3.0.0~beta20110817-3+deb7u1

2013-03-11 Thread David Paleino
On Sat, 09 Mar 2013 18:17:05 +, Adam D. Barratt wrote:

 On Sun, 2013-03-03 at 10:24 +0100, David Paleino wrote:
  On Thu, 28 Feb 2013 20:31:40 +, Adam D. Barratt wrote:
   Cool; thanks. Did you have chance to look at whether fixing the testing
   package just requires the source file split or if there were any other
   issues?
  
  The package I prepared and built for testing also required 2 other
  patches, to fix #683075. I guess that hasn't changed since then :)
 
 The debdiff included in MID:20130119194518.0c49613d@local only
 appeared to include the srsinit changes. Apologies if I simply missed
 it, but was a debdiff with the other changes also posted somewhere?

You did see it already, but I didn't include it in the patch I posted. This is
because I only made a patch for the source to be split; the actual package for
testing carries two more patches, attaching them to this mail.

The bigger patch is needed because an API change in freexl happened
unnoticed between testing and unstable.
See also your message
MID:1358594334.12995.35.ca...@jacala.jungle.funky-badger.org (Sat, 19 Jan
2013 11:18:54 +).

Sorry for the confusion, and thank you!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
From: Evgeni Golov evg...@debian.org
Subject: fix linking with libgeos
Origin: vendor, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683075#39
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683075

---
 libspatialite/configure   |4 ++--
 libspatialite/configure.ac|4 ++--
 spatialite-tools/configure|2 +-
 spatialite-tools/configure.ac |2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

--- spatialite.orig/libspatialite/configure
+++ spatialite/libspatialite/configure
@@ -16593,7 +16593,7 @@ for ac_lib in '' geos_c; do
 ac_res=none required
   else
 ac_res=-l$ac_lib
-LIBS=-l$ac_lib -lm -lgeos $ac_func_search_save_LIBS
+LIBS=-l$ac_lib $ac_func_search_save_LIBS
   fi
   if ac_fn_c_try_link $LINENO; then :
   ac_cv_search_GEOSTopologyPreserveSimplify=$ac_res
@@ -16663,7 +16663,7 @@ for ac_lib in '' geos_c; do
 ac_res=none required
   else
 ac_res=-l$ac_lib
-LIBS=-l$ac_lib -lm -lgeos $ac_func_search_save_LIBS
+LIBS=-l$ac_lib $ac_func_search_save_LIBS
   fi
   if ac_fn_c_try_link $LINENO; then :
   ac_cv_search_GEOSCoveredBy=$ac_res
--- spatialite.orig/libspatialite/configure.ac
+++ spatialite/libspatialite/configure.ac
@@ -120,7 +120,7 @@ AC_ARG_ENABLE(geos, [AS_HELP_STRING(
 if test x$enable_geos != xno; then
   OMIT_GEOS_FLAGS=
   AC_CHECK_HEADERS(geos_c.h,, [AC_MSG_ERROR([cannot find geos_c.h, bailing out])])
-  AC_SEARCH_LIBS(GEOSTopologyPreserveSimplify,geos_c,,AC_MSG_ERROR(['libgeos_c' is required but it doesn't seems to be installed on this system.]),-lm -lgeos)
+  AC_SEARCH_LIBS(GEOSTopologyPreserveSimplify,geos_c,,AC_MSG_ERROR(['libgeos_c' is required but it doesn't seems to be installed on this system.]),)
   #---
   #   --enable-geosadvanced
   #
@@ -129,7 +129,7 @@ if test x$enable_geos != xno; then
 	  [], [geosadvanced=yes])
   if test x$enable_geosadvanced != xno; then
 	  GEOSADVANCED_FLAGS=-DGEOS_ADVANCED
-	  AC_SEARCH_LIBS(GEOSCoveredBy,geos_c,,AC_MSG_ERROR([obsolete 'libgeos_c' ( v.3.3.0). please retry specifying: --disable-geosadvanced.]),-lm -lgeos)
+	  AC_SEARCH_LIBS(GEOSCoveredBy,geos_c,,AC_MSG_ERROR([obsolete 'libgeos_c' ( v.3.3.0). please retry specifying: --disable-geosadvanced.]),)
   else
 	  GEOSADVANCED_FLAGS=
   fi
--- spatialite.orig/spatialite-tools/configure
+++ spatialite/spatialite-tools/configure
@@ -16547,7 +16547,7 @@ if test ${ac_cv_lib_geos_c_GEOSTopology
   $as_echo_n (cached)  6
 else
   ac_check_lib_save_LIBS=$LIBS
-LIBS=-lgeos_c -lm -lgeos $LIBS
+LIBS=-lgeos_c $LIBS
 cat confdefs.h - _ACEOF conftest.$ac_ext
 /* end confdefs.h.  */
 
--- spatialite.orig/spatialite-tools/configure.ac
+++ spatialite/spatialite-tools/configure.ac
@@ -72,7 +72,7 @@ AC_SUBST(READLINE_LIBS)
 
 AC_CHECK_LIB(expat,XML_ParserCreate,,AC_MSG_ERROR(['expat' is required but it doesn't seems to be installed on this system.]))
 AC_CHECK_LIB(proj,pj_init_plus,,AC_MSG_ERROR(['libproj' is required but it doesn't seems to be installed on this system.]),-lm)
-AC_CHECK_LIB(geos_c,GEOSTopologyPreserveSimplify,,AC_MSG_ERROR(['libgeos_c' is required but it doesn't seems to be installed on this system.]),-lm -lgeos)
+AC_CHECK_LIB(geos_c,GEOSTopologyPreserveSimplify,,AC_MSG_ERROR(['libgeos_c' is required but it doesn't seems to be installed on this system.]),)
 
 PKG_CHECK_MODULES([LIBFREEXL], [freexl], , AC_MSG_ERROR(['libfreexl' is required but it doesn't seems to be installed on this system.]))
 LIBSPATIALITE_CFLAGS=
From: Felix Geyer fge...@debian.org
Subject: fix

Bug#698341: pu: package spatialite/3.0.0~beta20110817-3+deb7u1

2013-03-03 Thread David Paleino
On Thu, 28 Feb 2013 20:31:40 +, Adam D. Barratt wrote:

 On Fri, 2013-02-15 at 21:13 +0100, David Paleino wrote:
  On Fri, 15 Feb 2013 18:21:29 +, Adam D. Barratt wrote:
  
   On Sun, 2013-01-27 at 20:13 +0100, David Paleino wrote:
On Sun, 27 Jan 2013 14:16:08 +, Adam D. Barratt wrote:
 Please go ahead with the upload to unstable, as we need the issues
 fixing there in any case, however we proceed; thanks.

Done, 3.1.0~rc2-2 just uploaded to sid.
   
   That was a little over two weeks ago now. Have there been any reported
   issues with the package in sid?
  
  None up to now.
 
 Cool; thanks. Did you have chance to look at whether fixing the testing
 package just requires the source file split or if there were any other
 issues?

The package I prepared and built for testing also required 2 other patches, to
fix #683075. I guess that hasn't changed since then :)

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#698341: pu: package spatialite/3.0.0~beta20110817-3+deb7u1

2013-02-15 Thread David Paleino
On Fri, 15 Feb 2013 18:21:29 +, Adam D. Barratt wrote:

 On Sun, 2013-01-27 at 20:13 +0100, David Paleino wrote:
  On Sun, 27 Jan 2013 14:16:08 +, Adam D. Barratt wrote:
   Please go ahead with the upload to unstable, as we need the issues
   fixing there in any case, however we proceed; thanks.
  
  Done, 3.1.0~rc2-2 just uploaded to sid.
 
 That was a little over two weeks ago now. Have there been any reported
 issues with the package in sid?

None up to now.

Thanks for your heads up! :)
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#698341: pu: package spatialite/3.0.0~beta20110817-3+deb7u1

2013-01-27 Thread David Paleino
On Sun, 27 Jan 2013 14:16:08 +, Adam D. Barratt wrote:

 On Sat, 2013-01-19 at 19:45 +0100, David Paleino wrote:
 [..]
 
  I made a patch both for the version in sid, and the one in testing.
  I'd really prefer to upload it to sid, and then let it flow to testing, but
  the current upstream version in sid is greater than the one in testing.
  That upload is pre-freeze though, maybe an unblock could be granted?
 
 Given the size of the diff, I strongly suspect that we'd still end up
 looking at tpu.

ACK.

  Both patches work the same way: spatial_ref_sys data is being split out
  of a single file (srs_init.c in one case, epsg_inlined.c in the other) into
  multiple smaller files. This makes gcc not die on ppc. Besides this, the
  patches also contain changes to Makefile.{am.in}, to make it compile
  correctly.
  
  I'm not attaching both patches, since they're big -- 9.8MB (testing) and 12M
  (unstable). The unstable one is bigger because the code contained more code
  definitions than the testing one. You can find them here:
  
http://alioth.debian.org/~dapal/spatialite_testing.patch.bz2
http://alioth.debian.org/~dapal/spatialite_unstable.patch.bz2
 
 Unfortunately the patches /were/ attached, so the mail never made it to
 the debian-release list. :(

Sorry for the mess :/

  If I got an ACK for the sid version, I'll investigate whether it compiles
  both with libfreexl-dev from testing and sid. In case not, I'll patch it to
  compile with the version in testing (causing a sid-only RC bug), wait for
  it to enter testing, and then de-patch it to compile with freexl from sid.
  The API change in freexl must've slipped through, sorry for that.
 
 Please go ahead with the upload to unstable, as we need the issues
 fixing there in any case, however we proceed; thanks.

Done, 3.1.0~rc2-2 just uploaded to sid.

Thanks for your interest and work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#698341: Bug#661018: Bug#698341: RM: spatialite-tools/3.0.0~beta20110817-3

2013-01-24 Thread David Paleino
Hello Andreas, hello everybody,

On Wed, 23 Jan 2013 08:55:54 +0100, Andreas Tille wrote:

 [..]

before continuing, please check #698341.
There have been developments, and I asked for comments about two possible
uploads. I'm waiting for comments by the RT on what to do.

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#698341: pu: package spatialite/3.0.0~beta20110817-3+deb7u1

2013-01-19 Thread David Paleino
retitle 698341 pu: package spatialite/3.0.0~beta20110817-3+deb7u1
user 698341 release.debian@packages.debian.org
usertags 698341 = pu
thanks

Hello RT,

let's try to fix this instead of going for removal.

I've applied the debdiff prepared by Felix Geyer in #683075 [0], and checked
that it compiles fine in wheezy.

In any case, I'm attaching the final diff (modulo
s/UNRELEASED/testing-proposed-updates/) to this email.

The other bug afflicting spatialite is #664547. To fix this, we need to split
some huge file (I guess it's spatialite.c, 17k+ LOC) into smaller files, so
that gcc doesn't die on some arches. Would you be ok with such a huge patch?

Kindly,
David

[0]: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683075#44

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index be5e24a..35c1a73 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
+spatialite (3.0.0~beta20110817-3+deb7u1) UNRELEASED; urgency=low
+
+  [ Felix Geyer ]
+  * Fix FTBFS in wheezy. (Closes: #683075)
+- Add 05-fix_libgeos_search.patch from Evgeni Golov to fix configure checks.
+- Add 06-freexl_compat.patch to restore compatibility with freexl in wheezy.
+
+ -- David Paleino da...@debian.org  Sat, 19 Jan 2013 08:51:49 +0100
+
 spatialite (3.0.0~beta20110817-3) unstable; urgency=medium
 
   [ Francesco Paolo Lovergine ]
diff --git a/debian/patches/05-fix_libgeos_search.patch b/debian/patches/05-fix_libgeos_search.patch
new file mode 100644
index 000..a048771
--- /dev/null
+++ b/debian/patches/05-fix_libgeos_search.patch
@@ -0,0 +1,74 @@
+From: Evgeni Golov evg...@debian.org
+Subject: fix linking with libgeos
+Origin: vendor, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683075#39
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=683075
+
+---
+ libspatialite/configure   |4 ++--
+ libspatialite/configure.ac|4 ++--
+ spatialite-tools/configure|2 +-
+ spatialite-tools/configure.ac |2 +-
+ 4 files changed, 6 insertions(+), 6 deletions(-)
+
+--- spatialite.orig/libspatialite/configure
 spatialite/libspatialite/configure
+@@ -16593,7 +16593,7 @@ for ac_lib in '' geos_c; do
+ ac_res=none required
+   else
+ ac_res=-l$ac_lib
+-LIBS=-l$ac_lib -lm -lgeos $ac_func_search_save_LIBS
++LIBS=-l$ac_lib $ac_func_search_save_LIBS
+   fi
+   if ac_fn_c_try_link $LINENO; then :
+   ac_cv_search_GEOSTopologyPreserveSimplify=$ac_res
+@@ -16663,7 +16663,7 @@ for ac_lib in '' geos_c; do
+ ac_res=none required
+   else
+ ac_res=-l$ac_lib
+-LIBS=-l$ac_lib -lm -lgeos $ac_func_search_save_LIBS
++LIBS=-l$ac_lib $ac_func_search_save_LIBS
+   fi
+   if ac_fn_c_try_link $LINENO; then :
+   ac_cv_search_GEOSCoveredBy=$ac_res
+--- spatialite.orig/libspatialite/configure.ac
 spatialite/libspatialite/configure.ac
+@@ -120,7 +120,7 @@ AC_ARG_ENABLE(geos, [AS_HELP_STRING(
+ if test x$enable_geos != xno; then
+   OMIT_GEOS_FLAGS=
+   AC_CHECK_HEADERS(geos_c.h,, [AC_MSG_ERROR([cannot find geos_c.h, bailing out])])
+-  AC_SEARCH_LIBS(GEOSTopologyPreserveSimplify,geos_c,,AC_MSG_ERROR(['libgeos_c' is required but it doesn't seems to be installed on this system.]),-lm -lgeos)
++  AC_SEARCH_LIBS(GEOSTopologyPreserveSimplify,geos_c,,AC_MSG_ERROR(['libgeos_c' is required but it doesn't seems to be installed on this system.]),)
+   #---
+   #   --enable-geosadvanced
+   #
+@@ -129,7 +129,7 @@ if test x$enable_geos != xno; then
+ 	  [], [geosadvanced=yes])
+   if test x$enable_geosadvanced != xno; then
+ 	  GEOSADVANCED_FLAGS=-DGEOS_ADVANCED
+-	  AC_SEARCH_LIBS(GEOSCoveredBy,geos_c,,AC_MSG_ERROR([obsolete 'libgeos_c' ( v.3.3.0). please retry specifying: --disable-geosadvanced.]),-lm -lgeos)
++	  AC_SEARCH_LIBS(GEOSCoveredBy,geos_c,,AC_MSG_ERROR([obsolete 'libgeos_c' ( v.3.3.0). please retry specifying: --disable-geosadvanced.]),)
+   else
+ 	  GEOSADVANCED_FLAGS=
+   fi
+--- spatialite.orig/spatialite-tools/configure
 spatialite/spatialite-tools/configure
+@@ -16547,7 +16547,7 @@ if test ${ac_cv_lib_geos_c_GEOSTopology
+   $as_echo_n (cached)  6
+ else
+   ac_check_lib_save_LIBS=$LIBS
+-LIBS=-lgeos_c -lm -lgeos $LIBS
++LIBS=-lgeos_c $LIBS
+ cat confdefs.h - _ACEOF conftest.$ac_ext
+ /* end confdefs.h.  */
+ 
+--- spatialite.orig/spatialite-tools/configure.ac
 spatialite/spatialite-tools/configure.ac
+@@ -72,7 +72,7 @@ AC_SUBST(READLINE_LIBS)
+ 
+ AC_CHECK_LIB(expat,XML_ParserCreate,,AC_MSG_ERROR(['expat' is required but it doesn't seems to be installed on this system.]))
+ AC_CHECK_LIB(proj,pj_init_plus,,AC_MSG_ERROR(['libproj' is required but it doesn't seems to be installed on this system.]),-lm)
+-AC_CHECK_LIB(geos_c,GEOSTopologyPreserveSimplify

Bug#696652: unblock: fuseiso/20070708-3

2012-12-25 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package fuseiso.
The new version in sid fixes an RC bug, and a typo in the sourcecode.

  unblock fuseiso/20070708-3

Attaching the git diff.

Thanks for your work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 6205145..21d4dbd 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+fuseiso (20070708-3) unstable; urgency=low
+
+  * Renamed dependency from fuse-utils to fuse (Closes: #689009)
+  * Fixed typo in sourcecode (Closes: #598021)
+  * Updated years in debian/copyright
+
+ -- David Paleino da...@debian.org  Tue, 25 Dec 2012 08:47:20 +0100
+
 fuseiso (20070708-2) unstable; urgency=low
 
   * Maintenance release
diff --git a/debian/control b/debian/control
index d37c9ef..6b96d30 100644
--- a/debian/control
+++ b/debian/control
@@ -18,7 +18,7 @@ Architecture: any
 Depends:
  ${shlibs:Depends}
  , ${misc:Depends}
- , fuse-utils
+ , fuse
 Description: FUSE module to mount ISO filesystem images
  This package provides a module to mount ISO filesystem images
  using FUSE.
diff --git a/debian/copyright b/debian/copyright
index 0184527..016abbb 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -1,7 +1,7 @@
 Format-Specification: http://dep.debian.net/deps/dep5
 
 Files: debian/*
-Copyright: © 2007-2010, David Paleino da...@debian.org
+Copyright: © 2007-2012, David Paleino da...@debian.org
 License: GPL-2+
 
 Files: *
diff --git a/debian/patches/01-fix_typo.patch b/debian/patches/01-fix_typo.patch
new file mode 100644
index 000..eb80d15
--- /dev/null
+++ b/debian/patches/01-fix_typo.patch
@@ -0,0 +1,20 @@
+From: Jakub Wilk jw...@debian.org
+Subject: fix typo in sourcecode
+Origin: vendor, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=598021
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=598021
+
+---
+ src/fuseiso.c |2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- fuseiso.orig/src/fuseiso.c
 fuseiso/src/fuseiso.c
+@@ -295,7 +295,7 @@ void usage(const char* prog) {
+ -f -- run in foreground, do not daemonize\n
+ -d -- run in foreground and print debug information\n
+ -s -- run single-threaded\n
+-\nPlease consult with FUSE ducumentation for more information\n,
++\nPlease consult with FUSE documentation for more information\n,
+ VERSION, 
+ prog);
+ };
diff --git a/debian/patches/series b/debian/patches/series
index ed05433..d5f1cf6 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 00-support_large_iso.patch
+01-fix_typo.patch


signature.asc
Description: PGP signature


Bug#695471: unblock: bootchart2/0.14.4-2

2012-12-12 Thread David Paleino
Control: tags -1 - moreinfo

On Mon, 10 Dec 2012 20:10:43 +, Adam D. Barratt wrote:

 On Sat, 2012-12-08 at 19:47 +0100, David Paleino wrote:
  I just uploaded bootchart2/0.14.4-2 to unstable. This upload fixes RC bug
  #694403 -- the package shipped a systemd unit, but no classic initscript.
 
 +# Define LSB log_* functions.
 +# Depend on lsb-base (= 3.0-6) to ensure that this file is present.
 +. /lib/lsb/init-functions
 
 As far as I can see, bootchart2 doesn't depend on lsb-base at all? (I
 realise it's basically guaranteed to be present.)

Uops.

Uploaded -3 to sid. Thanks for noticing this! :)

David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#695471: unblock: bootchart2/0.14.4-2

2012-12-08 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Hello RT,
I just uploaded bootchart2/0.14.4-2 to unstable. This upload fixes RC bug
#694403 -- the package shipped a systemd unit, but no classic initscript.

I'm attaching a git diff -- please note that this is a particular initscript,
since starting it effectively *stops* the daemon -- and that's why it's
started after all the other services (i.e. stop collecting boot info as soon as
all other initscripts did their job).

  unblock bootchart2/0.14.4-2

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/bootchart2.bootchart-done.init b/debian/bootchart2.bootchart-done.init
new file mode 100644
index 000..da3f590
--- /dev/null
+++ b/debian/bootchart2.bootchart-done.init
@@ -0,0 +1,58 @@
+#! /bin/sh
+
+### BEGIN INIT INFO
+# Provides:  bootchart2
+# Required-Start:$remote_fs $all
+# Required-Stop:
+# Default-Start: 1 2 3 4 5
+# Default-Stop:
+# Short-Description: Stop bootchartd
+# Description:   This script stops the bootchartd daemon after the
+#system booted.
+### END INIT INFO
+
+# Author: David Paleino da...@debian.org
+
+# Do NOT set -e
+
+# PATH should only include /usr/* if it runs after the mountnfs.sh script
+PATH=/sbin:/usr/sbin:/bin:/usr/bin
+DESC=boot process analyser
+NAME=bootchartd
+DAEMON=/sbin/$NAME
+DAEMON_ARGS=
+PIDFILE=/var/run/$NAME.pid
+SCRIPTNAME=/etc/init.d/bootchart2
+
+# Exit if the package is not installed
+[ -x $DAEMON ] || exit 0
+
+# Read configuration variable file if it is present
+[ -r /etc/bootchartd.conf ]  . /etc/bootchartd.conf
+
+# Load the VERBOSE setting and other rcS variables
+. /lib/init/vars.sh
+
+# Define LSB log_* functions.
+# Depend on lsb-base (= 3.0-6) to ensure that this file is present.
+. /lib/lsb/init-functions
+
+case $1 in
+  start)
+	[ $VERBOSE != no ]  log_daemon_msg Stopping $DESC $NAME
+	$DAEMON stop
+	case $? in
+		0|1) [ $VERBOSE != no ]  log_end_msg 0 ;;
+		2) [ $VERBOSE != no ]  log_end_msg 1 ;;
+	esac
+	;;
+  stop|restart|force-reload)
+	# do nothing
+	;;
+  *)
+	echo Usage: $SCRIPTNAME start 2
+	exit 3
+	;;
+esac
+
+:
diff --git a/debian/changelog b/debian/changelog
index 9876687..a769a35 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+bootchart2 (0.14.4-2) unstable; urgency=low
+
+  * Acknowledge NMU.
+  * Provide also an initscript (Closes: #694403)
+
+ -- David Paleino da...@debian.org  Sat, 08 Dec 2012 19:35:08 +0100
+
 bootchart2 (0.14.4-1.1) unstable; urgency=low
 
   * Non-maintainer upload.
diff --git a/debian/rules b/debian/rules
index 0b97af1..158f0dc 100755
--- a/debian/rules
+++ b/debian/rules
@@ -13,3 +13,8 @@ override_dh_installchangelogs:
 
 override_dh_auto_test:
 
+override_dh_installinit:
+	dh_installinit \
+		--name=bootchart-done \
+		--no-start \
+		--update-rcd-params=start 99 1 2 3 4 5 6 . stop 99 0 .


signature.asc
Description: PGP signature


Bug#693867: unblock: pycountry/0.14.1+ds1-3

2012-11-21 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear RT,

please unblock package pycountry from sid. Latest upload fixes RC bug #693739.
Attaching the diff.

  unblock pycountry/0.14.1+ds1-3

Thanks for your hard work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 9bb8816..4608183 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+pycountry (0.14.1+ds1-3) unstable; urgency=low
+
+  * Support common_name attribute for languages, thanks to
+Jerome Flesch (Closes: #693739).
+
+ -- David Paleino da...@debian.org  Tue, 20 Nov 2012 19:14:08 +0100
+
 pycountry (0.14.1+ds1-2) unstable; urgency=low
 
   * Oops, install README.txt in the appropriate place (Closes: #644348)
diff --git a/debian/patches/02-support_language_common-name.patch b/debian/patches/02-support_language_common-name.patch
new file mode 100644
index 000..3933c9a
--- /dev/null
+++ b/debian/patches/02-support_language_common-name.patch
@@ -0,0 +1,20 @@
+From: Jerome Flesch jfle...@gmail.com
+Subject: support common_name attribute for languages
+Origin: vendor, http://bugs.debian.org/693739
+Bug-Debian: http://bugs.debian.org/693739
+Forwarded: no
+
+---
+ src/pycountry/__init__.py |1 +
+ 1 file changed, 1 insertion(+)
+
+--- pycountry.orig/src/pycountry/__init__.py
 pycountry/src/pycountry/__init__.py
+@@ -52,6 +52,7 @@ class Languages(pycountry.db.Database):
+ field_map = dict(iso_639_2B_code='bibliographic',
+  iso_639_2T_code='terminology',
+  iso_639_1_code='alpha2',
++ common_name='common_name',
+  name='name')
+ data_class_name = 'Language'
+ xml_tag = 'iso_639_entry'
diff --git a/debian/patches/series b/debian/patches/series
index 625a4ca..c9ffadd 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 00-use_system_iso-codes.patch
 01-prevent_logger_warning.patch
+02-support_language_common-name.patch


signature.asc
Description: PGP signature


Bug#693239: RM: ninja-ide/2.0~b-2

2012-11-14 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: rm
X-Debbugs-CC: Diego Sarmentero diego.sarment...@ninja-ide.org

Dear RT,

please remove ninja-ide 2.0~b-2 from testing. This version is largely outdated,
and has bugs (the large majority of which not reported in Debian) which were
already fixed by upstream in latest versions (I just uploaded 2.1.1 to sid).

I don't think the current version in testing is suitable for a Debian stable
release, so better nothing than a buggy version -- which would undermine both
the project's and Debian's reputation.

Please note: this removal is ONLY from testing.

Upstream CCed in this mail.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#692858: Bug #692858: Re: please accept josm-plugins_0.0.svn28420+ds2-1 into testing-proposed-updates

2012-11-12 Thread David Paleino
On Mon, 12 Nov 2012 13:13:50 +, Adam D. Barratt wrote:

 Hi,
 
 This mail appears never to have made it to debian-release@, most likely
 due to the size of the debdiff.
 
 On Fri, 2012-11-09 at 21:37 +0100, David Paleino wrote:
  please accept josm-plugins version 0.0.svn28420+ds2-1 into
  testing-proposed-updates. It is a new upstream version because I removed
  the livegps plugin from the previous (already repackaged) tarball.
 
 There doesn't appear to be a josm-plugins package in t-p-u currently?

Not yet, I was waiting for an (official) ACK.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#692919: unblock: wicd/1.7.2.4-4

2012-11-10 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package wicd. Latest upload in sid fixes RC bug #692916.

Attaching diff to -3 (which was already unblocked recently).
I moved the code adding users from debconf's .config to postinst; adduser was
already a dependency of wicd-daemon.
Also, I changed the call to getent to be a bit clearer (thanks to Vagrant on
IRC).

unblock wicd/1.7.2.4-4

Thanks!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index fdfb349..1e0c95c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+wicd (1.7.2.4-4) unstable; urgency=low
+
+  * Move user-adding code from debconf-config to wicd-daemon postinst
+(Closes: #692916)
+
+ -- David Paleino da...@debian.org  Sat, 10 Nov 2012 21:41:50 +0100
+
 wicd (1.7.2.4-3) unstable; urgency=low
 
   * Fix debian/watch
diff --git a/debian/wicd-daemon.config b/debian/wicd-daemon.config
index 1622e51..d4c4e16 100755
--- a/debian/wicd-daemon.config
+++ b/debian/wicd-daemon.config
@@ -25,14 +25,3 @@ db_subst wicd/users users $users
 
 db_input high wicd/users || true
 db_go
-
-db_get wicd/users
-users=$(echo $RET | sed -e 's@,@@g')
-
-# Add selected users
-for u in $users; do
-	hasuser=$((getent passwd | grep -w $u) || true)
-	if [ -n $hasuser ]; then
-		adduser --quiet $u netdev
-	fi
-done
diff --git a/debian/wicd-daemon.postinst b/debian/wicd-daemon.postinst
index 17451fd..2aa4eec 100644
--- a/debian/wicd-daemon.postinst
+++ b/debian/wicd-daemon.postinst
@@ -37,6 +37,16 @@ case $1 in
 fi
 fi
 fi
+
+# Add users previously selected in debconf
+db_get wicd/users
+users=$(echo $RET | sed -e 's@,@@g')
+for u in $users; do
+hasuser=$(getent passwd $u || true)
+if [ -n $hasuser ]; then
+adduser --quiet $u netdev
+fi
+done
 ;;
 
 abort-upgrade|abort-remove|abort-deconfigure)


signature.asc
Description: PGP signature


Bug#691990: unblock: wicd/1.7.2.4-3 (pre-approval)

2012-10-31 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Hello RT,

I'm requesting pre-approval for an upload of wicd to unstable (targeted to
testing). Please see the attached patch.

It fixes a RC bug, and a couple other easy bugs (which don't affect the
functionality of the package in a significant way).

If the diff is OK, I'll upload the package, and please unblock it for migration
to testing.

Thanks for your hard work,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 3ceb288..e4c976a 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
+wicd (1.7.2.4-3) UNRELEASED; urgency=low
+
+  * Fix debian/watch
+  * Recommend rfkill in wicd-daemon (Closes: #683559)
+  * Substitute dependency on dhcp3-client with isc-dhcp-client
+(Closes: #680976)
+  * Fix handling of /etc/resolv.conf when it's a symlink (Closes: #691973)
+
+ -- David Paleino da...@debian.org  Thu, 01 Nov 2012 00:30:25 +0100
+
 wicd (1.7.2.4-2) unstable; urgency=high
 
   * Fix comments inside /etc/default/wicd (Closes: #668327)
diff --git a/debian/control b/debian/control
index b0639f0..6bc40a8 100644
--- a/debian/control
+++ b/debian/control
@@ -47,7 +47,7 @@ Depends:
  , dbus
  , wpasupplicant
  , wireless-tools
- , dhcpcd | dhcp3-client | pump | udhcpc
+ , dhcpcd | isc-dhcp-client | pump | udhcpc
  , net-tools | ethtool
  , net-tools | iproute
  , adduser
@@ -57,6 +57,7 @@ Depends:
  , python-wicd (= ${source:Version})
 Pre-Depends: debconf (= 1.5.34)
 Recommends: wicd-gtk (= ${source:Version}) | wicd-curses (= ${source:Version}) | wicd-cli (= ${source:Version}) | wicd-client
+ , rfkill
 Suggests: pm-utils
 Breaks: wicd (= 1.6.2.2-4)
 Replaces: wicd (= 1.6.2.2-4)
diff --git a/debian/patches/01-remove_unused_icons.patch b/debian/patches/01-remove_unused_icons.patch
index 7361f66..022a095 100644
--- a/debian/patches/01-remove_unused_icons.patch
+++ b/debian/patches/01-remove_unused_icons.patch
@@ -5,7 +5,7 @@ Don't install rarely-used icons, save disk space
 
 --- wicd.orig/setup.py
 +++ wicd/setup.py
-@@ -535,15 +535,11 @@ try:
+@@ -538,15 +538,11 @@ try:
  if not wpath.no_install_man:
  data.append((wpath.mandir + 'man1/', [ 'man/wicd-client.1' ]))
  data.append((wpath.icons + 'scalable/apps/', ['icons/scalable/wicd-gtk.svg']))
diff --git a/debian/patches/02-workaround_dhclient_bug.patch b/debian/patches/02-workaround_dhclient_bug.patch
index 8f46a1e..d4119fa 100644
--- a/debian/patches/02-workaround_dhclient_bug.patch
+++ b/debian/patches/02-workaround_dhclient_bug.patch
@@ -10,7 +10,7 @@ Forwarded: no
 
 --- wicd.orig/wicd/wnettools.py
 +++ wicd/wicd/wnettools.py
-@@ -284,7 +284,7 @@ class BaseInterface(object):
+@@ -285,7 +285,7 @@ class BaseInterface(object):
  
  client_dict = {
  dhclient : 
diff --git a/debian/patches/04-fix_resolv.conf_backup-restore.patch b/debian/patches/04-fix_resolv.conf_backup-restore.patch
new file mode 100644
index 000..25903d6
--- /dev/null
+++ b/debian/patches/04-fix_resolv.conf_backup-restore.patch
@@ -0,0 +1,40 @@
+From: David Paleino da...@debian.org
+Subject: fix handling of resolv.conf when it's a symlink
+Forwarded: not-needed
+Origin: upstream
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=691973
+
+---
+ wicd/wicd-daemon.py |   14 --
+ 1 file changed, 12 insertions(+), 2 deletions(-)
+
+--- wicd.orig/wicd/wicd-daemon.py
 wicd/wicd/wicd-daemon.py
+@@ -1703,7 +1703,11 @@ def main(argv):
+ # don't back up if .orig exists, probably there cause
+ # wicd exploded
+ if not os.path.exists(backup_location):
+-shutil.copy2('/etc/resolv.conf', backup_location)
++if os.path.islink('/etc/resolv.conf'):
++dest = os.readlink('/etc/resolv.conf')
++os.symlink(dest, backup_location)
++else:
++shutil.copy2('/etc/resolv.conf', backup_location)
+ os.chmod(backup_location, 0644)
+ except IOError:
+ print 'error backing up resolv.conf'
+@@ -1750,7 +1754,13 @@ def main(argv):
+ 
+ # restore resolv.conf on quit
+ try:
+-shutil.move(wpath.varlib + 'resolv.conf.orig', '/etc/resolv.conf')
++backup_location = wpath.varlib + 'resolv.conf.orig'
++if os.path.islink(backup_location):
++dest = os.readlink(backup_location)
++os.remove('/etc/resolv.conf')
++os.symlink(dest, '/etc/resolv.conf')
++else:
++shutil.move(backup_location, '/etc/resolv.conf')
+ os.chmod('/etc/resolv.conf', 0644)
+ except IOError:
+ print 'error restoring

Bug#691260: unblock: greenwich/0.8.2-6

2012-10-23 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package greenwich. Latest upload to sid fixes two bugs, one of
which RC.

  unblock greenwich/0.8.2-6

Attaching diff between -5 and -6.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 64b0da0..3a42f79 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+greenwich (0.8.2-6) unstable; urgency=low
+
+  * Add dependency on liblocale-gettext-perl (Closes: #690864)
+  * Fix Polish translation encoding, thanks to Jakub Wilk
+(Closes: #690867)
+
+ -- David Paleino da...@debian.org  Tue, 23 Oct 2012 17:59:38 +0200
+
 greenwich (0.8.2-5) unstable; urgency=low
 
   * debian/control:
diff --git a/debian/control b/debian/control
index 6c1329d..8a57e2b 100644
--- a/debian/control
+++ b/debian/control
@@ -18,6 +18,7 @@ Depends: whois
  , libgtk2-perl
  , libgnome2-gconf-perl
  , libgtk2-gladexml-perl
+ , liblocale-gettext-perl
  , ${misc:Depends}
 Description: graphical whois client for GNOME
  A GNOME application which allows you to perform whois lookups. You enter the
diff --git a/debian/patches/04-fix_polish_translation.patch b/debian/patches/04-fix_polish_translation.patch
new file mode 100644
index 000..a803504
--- /dev/null
+++ b/debian/patches/04-fix_polish_translation.patch
@@ -0,0 +1,40 @@
+From: Jakub Wilk jw...@debian.org
+Subject: fix Polish translation encoding
+Origin: vendor, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=690867
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=690867
+Forwarded: no
+
+---
+ src/po/pl.po |8 
+ 1 file changed, 4 insertions(+), 4 deletions(-)
+
+--- greenwich.orig/src/po/pl.po
 greenwich/src/po/pl.po
+@@ -38,7 +38,7 @@ msgid 
+ Do you want to replace it?
+ msgstr Plik  '%s' istnieje.\n
+ \n
+-Czy chcesz nadpisać zmiany?
++Czy chcesz nadpisać zmiany?
+ 
+ msgid A whois client for GNOME.
+ msgstr Whois klient pod GNOME.
+@@ -65,10 +65,10 @@ msgid Greenwich Preferences
+ msgstr Greenwich Preferencje
+ 
+ msgid span weight=\bold\Network:/span
+-msgstr span weight=\bold\Sieć:/span
++msgstr span weight=\bold\Sieć:/span
+ 
+ msgid Timeout for network connections:
+-msgstr Upłynął żądany limit czasu:
++msgstr Upłynął żądany limit czasu:
+ 
+ msgid span weight=\bold\Recursive:/span
+ msgstr span weight=\bold\Rekursywny:/span
+@@ -80,4 +80,4 @@ msgid Question
+ msgstr Pytanie
+ 
+ msgid _Replace
+-msgstr _Zamień
++msgstr _Zamień
diff --git a/debian/patches/series b/debian/patches/series
index 72d1ab2..822b7f6 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,4 @@
 01-fix_manpage.patch
 02-fix_desktop-file.patch
 03-add_italian_translation.patch
+04-fix_polish_translation.patch


signature.asc
Description: PGP signature


Bug#688495: unblock: osm2pgsql/0.80.0+r27899-3

2012-09-23 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package osm2pgsql.

Latest version (which should've been -2, but I've messed up with the changelog,
and didn't notice it was -3 only after the dput) fixes RC bug #687965.

I'm attaching the git diff with the version currently in testing.

  unblock osm2pgsql/0.80.0+r27899-3

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 13a19fd..def62db 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+osm2pgsql (0.80.0+r27899-3) unstable; urgency=low
+
+  * Update Recommends of postgresql-8.4-postgis to -9.1
+  * Backport support for 64-bit IDs (Closes: #687965)
+
+ -- David Paleino da...@debian.org  Sun, 23 Sep 2012 09:14:15 +0200
+
 osm2pgsql (0.80.0+r27899-1) unstable; urgency=low
 
   * New SVN snapshot
diff --git a/debian/control b/debian/control
index e616531..21e9d9f 100644
--- a/debian/control
+++ b/debian/control
@@ -31,7 +31,7 @@ Depends:
  , ${misc:Depends}
 Recommends:
  postgis
- , postgresql-8.4-postgis
+ , postgresql-9.1-postgis
 Suggests:
  josm
  , gosmore 
diff --git a/debian/patches/02-backport_64bit_ids_support.patch b/debian/patches/02-backport_64bit_ids_support.patch
new file mode 100644
index 000..e135d06
--- /dev/null
+++ b/debian/patches/02-backport_64bit_ids_support.patch
@@ -0,0 +1,43 @@
+From: David Paleino da...@debian.org
+Subject: backport support for 64-bit IDs
+Forwarded: not-needed
+Origin: vendor
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=687965
+
+---
+ osmtypes.h |3 +--
+ output-gazetteer.c |4 ++--
+ 2 files changed, 3 insertions(+), 4 deletions(-)
+
+--- osm2pgsql.orig/osmtypes.h
 osm2pgsql/osmtypes.h
+@@ -6,8 +6,7 @@
+ #include inttypes.h
+ #include time.h
+ 
+-// uncomment the following to build a version that supports 64bit IDs.
+-// #define OSMID64
++#define OSMID64
+ 
+ #ifdef OSMID64
+ typedef int64_t osmid_t;
+--- osm2pgsql.orig/output-gazetteer.c
 osm2pgsql/output-gazetteer.c
+@@ -32,7 +32,7 @@
+CREATE TABLE place (   \
+  place_id BIGINT, \
+  osm_type CHAR(1) NOT NULL,   \
+- osm_id BIGINT NOT NULL,  \
++ osm_id  POSTGRES_OSMID_TYPE  NOT NULL, \
+  class TEXT NOT NULL, \
+  type TEXT NOT NULL,  \
+  name keyvalue[], \
+@@ -51,7 +51,7 @@
+ #define V2_CREATE_PLACE_TABLE   \
+CREATE TABLE place (   \
+  osm_type CHAR(1) NOT NULL,   \
+- osm_id BIGINT NOT NULL,  \
++ osm_id  POSTGRES_OSMID_TYPE  NOT NULL, \
+  class TEXT NOT NULL, \
+  type TEXT NOT NULL,  \
+  name HSTORE, \
diff --git a/debian/patches/series b/debian/patches/series
index dde124e..be78265 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 00-fix_build.patch
 01-disable_gazetteer.patch
+02-backport_64bit_ids_support.patch


signature.asc
Description: PGP signature


Bug#688495: unblock: osm2pgsql/0.80.0+r27899-3

2012-09-23 Thread David Paleino
On Sun, 23 Sep 2012 17:11:08 +0100, Adam D. Barratt wrote:

 Control: tags -1 + moreinfo
 
 On Sun, 2012-09-23 at 09:24 +0200, David Paleino wrote:
 
  Please unblock package osm2pgsql.
  
  Latest version (which should've been -2, but I've messed up with the
  changelog, and didn't notice it was -3 only after the dput) fixes RC bug
  #687965.
 
 The bug report indicates that database tables would need to be rebuilt
 in line with the 32- to 64-bit change.  Would that happen automatically,
 or is it something that users are expected to do after upgrading?

They're expected to do that on their own.

However, I noticed that osm2pgsql/testing already used BIGINT columns for osm_id
(BIGINT == int_8), maybe for some other bug of the software (which created 64
bit columns even though one asked for 32 bit). I haven't checked the version in
stable.

Now the change has been made more explicit in the code; end users (of
testing) shouldn't notice much of a difference. Should I add a short notice to
NEWS.Debian, or the like, for stable users?

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#688495: unblock: osm2pgsql/0.80.0+r27899-3

2012-09-23 Thread David Paleino
On Sun, 23 Sep 2012 17:31:38 +0100, Adam D. Barratt wrote:

 On Sun, 2012-09-23 at 18:19 +0200, David Paleino wrote:
  On Sun, 23 Sep 2012 17:11:08 +0100, Adam D. Barratt wrote:
   On Sun, 2012-09-23 at 09:24 +0200, David Paleino wrote:
Please unblock package osm2pgsql.

Latest version (which should've been -2, but I've messed up with the
changelog, and didn't notice it was -3 only after the dput) fixes RC bug
#687965.
   
   The bug report indicates that database tables would need to be rebuilt
   in line with the 32- to 64-bit change.  Would that happen automatically,
   or is it something that users are expected to do after upgrading?
  
  They're expected to do that on their own.
  
  However, I noticed that osm2pgsql/testing already used BIGINT columns for
  osm_id (BIGINT == int_8), maybe for some other bug of the software (which
  created 64 bit columns even though one asked for 32 bit). I haven't checked
  the version in stable.
 
 0.69+r20104-2 (squeeze) has:
 
 #define CREATE_PLACE_TABLE  \
CREATE TABLE place (   \
  place_id BIGINT, \
  osm_type CHAR(1) NOT NULL,   \
  osm_id BIGINT NOT NULL,  \
 
 That would imply that the field is already 64-bit even in stable?

Uhm. I missed one tiny bit.

My patch fixes also output-gazetteer.c -- which already used BIGINT, but
output-pgsql.c was using POSTGRES_OSMID_TYPE. That means 32 bit in
stable/testing, and now 64 bit in unstable.

Should I upload a new version with a notice about the database migration?

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#688495: unblock: osm2pgsql/0.80.0+r27899-3

2012-09-23 Thread David Paleino
On Sun, 23 Sep 2012 18:07:58 +0100, Adam D. Barratt wrote:

 On Sun, 2012-09-23 at 18:43 +0200, David Paleino wrote:
  Should I upload a new version with a notice about the database migration?
 
 If there's not a sane way to avoid users having to fix things up, yes
 please.

There you go, -4 uploaded to sid, diff attached.

Thanks!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/NEWS.Debian b/debian/NEWS.Debian
new file mode 100644
index 000..9d87413
--- /dev/null
+++ b/debian/NEWS.Debian
@@ -0,0 +1,20 @@
+osm2pgsql (0.80.0+r27899-4) unstable; urgency=low
+
+  Since version 0.80.0+r27899-3, osm2pgsql supports 64-bit OSM IDs. This
+  means that the column osm_id in created tables changed its type from
+  INTEGER (int4) to BIGINT (int8). This change will allow storing new
+  OSM objects, as the OSM database gets larger.
+
+  Databases created before this version should be migrated. Remember to do
+  a BACKUP before attempting anything.
+
+  You can either recreate the database from scratch using osm2pgsql, or
+  issue the following SQL statement on each table containing a osm_id
+  column:
+
+ALTER TABLE table COLUMN osm_id TYPE bigint;
+
+  Remember to make PROPER BACKUPS before attempting any such operation on
+  your database.
+
+ -- David Paleino da...@debian.org  Sun, 23 Sep 2012 21:52:38 +0200
diff --git a/debian/changelog b/debian/changelog
index 13a19fd..87b5e30 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,17 @@
+osm2pgsql (0.80.0+r27899-4) unstable; urgency=low
+
+  * Added debian/NEWS.Debian explaining what changes from the 32-bit
+version.
+
+ -- David Paleino da...@debian.org  Sun, 23 Sep 2012 21:57:34 +0200
+
+osm2pgsql (0.80.0+r27899-3) unstable; urgency=low
+
+  * Update Recommends of postgresql-8.4-postgis to -9.1
+  * Backport support for 64-bit IDs (Closes: #687965)
+
+ -- David Paleino da...@debian.org  Sun, 23 Sep 2012 09:14:15 +0200
+
 osm2pgsql (0.80.0+r27899-1) unstable; urgency=low
 
   * New SVN snapshot
diff --git a/debian/control b/debian/control
index e616531..21e9d9f 100644
--- a/debian/control
+++ b/debian/control
@@ -31,7 +31,7 @@ Depends:
  , ${misc:Depends}
 Recommends:
  postgis
- , postgresql-8.4-postgis
+ , postgresql-9.1-postgis
 Suggests:
  josm
  , gosmore 
diff --git a/debian/patches/02-backport_64bit_ids_support.patch b/debian/patches/02-backport_64bit_ids_support.patch
new file mode 100644
index 000..e135d06
--- /dev/null
+++ b/debian/patches/02-backport_64bit_ids_support.patch
@@ -0,0 +1,43 @@
+From: David Paleino da...@debian.org
+Subject: backport support for 64-bit IDs
+Forwarded: not-needed
+Origin: vendor
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=687965
+
+---
+ osmtypes.h |3 +--
+ output-gazetteer.c |4 ++--
+ 2 files changed, 3 insertions(+), 4 deletions(-)
+
+--- osm2pgsql.orig/osmtypes.h
 osm2pgsql/osmtypes.h
+@@ -6,8 +6,7 @@
+ #include inttypes.h
+ #include time.h
+ 
+-// uncomment the following to build a version that supports 64bit IDs.
+-// #define OSMID64
++#define OSMID64
+ 
+ #ifdef OSMID64
+ typedef int64_t osmid_t;
+--- osm2pgsql.orig/output-gazetteer.c
 osm2pgsql/output-gazetteer.c
+@@ -32,7 +32,7 @@
+CREATE TABLE place (   \
+  place_id BIGINT, \
+  osm_type CHAR(1) NOT NULL,   \
+- osm_id BIGINT NOT NULL,  \
++ osm_id  POSTGRES_OSMID_TYPE  NOT NULL, \
+  class TEXT NOT NULL, \
+  type TEXT NOT NULL,  \
+  name keyvalue[], \
+@@ -51,7 +51,7 @@
+ #define V2_CREATE_PLACE_TABLE   \
+CREATE TABLE place (   \
+  osm_type CHAR(1) NOT NULL,   \
+- osm_id BIGINT NOT NULL,  \
++ osm_id  POSTGRES_OSMID_TYPE  NOT NULL, \
+  class TEXT NOT NULL, \
+  type TEXT NOT NULL,  \
+  name HSTORE, \
diff --git a/debian/patches/series b/debian/patches/series
index dde124e..be78265 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 00-fix_build.patch
 01-disable_gazetteer.patch
+02-backport_64bit_ids_support.patch


signature.asc
Description: PGP signature


Bug#682478: unblock: josm/0.0.svn5267+dfsg1-2

2012-07-23 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package josm 0.0.svn5267+dfsg1-2.

It fixes #682315 which, even if not RC, would make the package almost unusable
in wheezy.

OpenStreetMap has undergone a license change in its data, so some data was
removed/hidden, and the current JOSM in wheezy is unable to deal with that. The
reporter proposed to package a newer upstream version, but I managed to
backport the needed fixes to the version in wheezy.

Please find the diff attached (there are also some patch renames there, sorry
for the noise).

unblock josm/0.0.svn5267+dfsg1-2

Thanks for your work,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index e879535..f548210 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
+josm (0.0.svn5267+dfsg1-2) unstable; urgency=low
+
+  * Backported patches from upstream repository, to handle recent
+OSM license change (Closes: #682315)
+  - 06-handle_redacted_objects.patch
+  - 07-handle_deleted_nodes_without_coordinates.patch
+
+ -- David Paleino da...@debian.org  Mon, 23 Jul 2012 10:16:59 +0200
+
 josm (0.0.svn5267+dfsg1-1) unstable; urgency=low
 
   * New tested snapshot
diff --git a/debian/patches/10-build.patch b/debian/patches/00-build.patch
similarity index 100%
rename from debian/patches/10-build.patch
rename to debian/patches/00-build.patch
diff --git a/debian/patches/20-bts.patch b/debian/patches/01-bts.patch
similarity index 100%
rename from debian/patches/20-bts.patch
rename to debian/patches/01-bts.patch
diff --git a/debian/patches/40-elemstyles.patch b/debian/patches/02-elemstyles.patch
similarity index 100%
rename from debian/patches/40-elemstyles.patch
rename to debian/patches/02-elemstyles.patch
diff --git a/debian/patches/70-default_look_and_feel.patch b/debian/patches/03-default_look_and_feel.patch
similarity index 100%
rename from debian/patches/70-default_look_and_feel.patch
rename to debian/patches/03-default_look_and_feel.patch
diff --git a/debian/patches/80-fix_images.patch b/debian/patches/04-fix_images.patch
similarity index 100%
rename from debian/patches/80-fix_images.patch
rename to debian/patches/04-fix_images.patch
diff --git a/debian/patches/90-fix_version.patch b/debian/patches/05-fix_version.patch
similarity index 100%
rename from debian/patches/90-fix_version.patch
rename to debian/patches/05-fix_version.patch
diff --git a/debian/patches/06-handle_redacted_objects.patch b/debian/patches/06-handle_redacted_objects.patch
new file mode 100644
index 000..3ce0066
--- /dev/null
+++ b/debian/patches/06-handle_redacted_objects.patch
@@ -0,0 +1,46 @@
+From: Don-vip
+Subject: handle redacted objects resulting from OSM License change process
+Origin: upstream, http://josm.openstreetmap.de/changeset/5339/josm
+ upstream, http://josm.openstreetmap.de/changeset/5340/josm
+Bug-Debian: http://bugs.debian.org/682315
+
+---
+ src/org/openstreetmap/josm/data/osm/history/History.java|   14 ++
+ src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java |5 +++
+ 2 files changed, 19 insertions(+)
+
+--- josm.orig/src/org/openstreetmap/josm/data/osm/history/History.java
 josm/src/org/openstreetmap/josm/data/osm/history/History.java
+@@ -230,4 +230,18 @@ public class History{
+ public OsmPrimitiveType getType() {
+ return type;
+ }
++
++@Override
++public String toString() {
++String result = History [
+++ (type != null ? type= + type + ,  : ) + id= + id;
++if (versions != null) {
++result += , versions=\n;
++for (HistoryOsmPrimitive v : versions) {
++result += \t + v + ,\n;
++}
++}
++result += ];
++return result;
++}
+ }
+--- josm.orig/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java
 josm/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java
+@@ -161,6 +161,11 @@ public class HistoryBrowserModel extends
+ if (history.getByVersion(primitive.getVersion()) != null)
+ return primitive.isModified();
+ 
++// if latest version from history is higher than a non existing primitive version,
++// that means this version has been redacted and the primitive cannot be used.
++if (history.getLatest().getVersion()  primitive.getVersion())
++return false;
++
+ // latest has a higher version than one of the primitives
+ // in the history (probably because the history got out of sync
+ // with uploaded data) - show the primitive as latest
diff --git a/debian/patches/07-handle_deleted_nodes_without_coordinates.patch b/debian/patches/07

Bug#681038: unblock: osm-gps-map/0.7.3-3

2012-07-10 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: freeze-exception

Hello RT,

I just uploaded osm-gps-map 0.7.3-3 to unstable, fixing a RC bug (#680815). It's
a tiny diff (attached), please unblock it.

unblock osm-gps-map/0.7.3-3

Thanks for your work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index e5c8e75..98b3cc7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+osm-gps-map (0.7.3-3) unstable; urgency=low
+
+  * Add missing dependency on python-gtk2 (Closes: #680815)
+
+ -- David Paleino da...@debian.org  Tue, 10 Jul 2012 09:01:42 +0200
+
 osm-gps-map (0.7.3-2) unstable; urgency=low
 
   * Don't install *.la files anymore (Closes: #621487)
diff --git a/debian/control b/debian/control
index 76a7d22..5fd3767 100644
--- a/debian/control
+++ b/debian/control
@@ -73,6 +73,7 @@ Architecture: any
 Depends: ${misc:Depends}
  , ${shlibs:Depends}
  , ${python:Depends}
+ , python-gtk2
 Provides: ${python:Provides}
 Suggests: libosmgpsmap2-dbg
 Description: GTK+ library to embed OpenStreetMap maps - Python bindings


signature.asc
Description: PGP signature


Bug#681039: unblock: osmosis/0.40.1+ds1-7

2012-07-10 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Hello RT,

I just uploaded to unstable osmosis 0.40.1+ds1-7, fixing a FTBFS (#680820).
Tiny diff attached.

unblock osmosis/0.40.1+ds1-7

Thanks for your work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 6cbdc77..b342366 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+osmosis (0.40.1+ds1-7) unstable; urgency=low
+
+  * Added missing dependency on libxz-java (Closes: #680820)
+
+ -- David Paleino da...@debian.org  Tue, 10 Jul 2012 09:10:35 +0200
+
 osmosis (0.40.1+ds1-6) unstable; urgency=low
 
   * Use a dedicate ivy cache, to don't interfere with user cache.
diff --git a/debian/control b/debian/control
index 47b5e67..0ec9cee 100644
--- a/debian/control
+++ b/debian/control
@@ -26,6 +26,7 @@ Build-Depends:
  , libosmpbf-java
  , libplexus-classworlds-java
  , libxerces2-java
+ , libxz-java
  , checkstyle (= 5.4)
  , ivy
  , junit4 (= 4.10)
@@ -58,6 +59,7 @@ Depends:
  , libbatik-java
  , libcommons-pool-java
  , libxerces2-java
+ , libxz-java
  , ${misc:Depends}
 Description: Command line OpenStreetMap data processor
  Osmosis consists of a series of components that can be chained together


signature.asc
Description: PGP signature


Re: Bug#674587: transition: mapnik

2012-06-01 Thread David Paleino
On Fri, 25 May 2012 19:34:15 +0200, David Paleino wrote:

 Package: release.debian.org
 Severity: normal
 User: release.debian@packages.debian.org
 Usertags: transition
 
 Hello RT,
 
 I'm requesting to add Mapnik to the transition tracker.
 
 [..]

*gentle ping*

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Bug#674587: transition: mapnik

2012-06-01 Thread David Paleino
On Fri, 1 Jun 2012 11:06:11 +0200, Cyril Brulebois wrote:

 David Paleino da...@debian.org (01/06/2012):
  On Fri, 25 May 2012 19:34:15 +0200, David Paleino wrote:
   ^^^
 
   I'm requesting to add Mapnik to the transition tracker.
  
  *gentle ping*
 
 Your mail is still marked as unread (= needs processing) in my maildir,
 but 25  20, and:
   https://lists.debian.org/debian-devel-announce/2012/05/msg4.html

Oops.

However, I'd still like the transition to be put in the tracker, in the
planned section.

Sorry for the noise,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#674587: transition: mapnik

2012-05-25 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: transition

Hello RT,

I'm requesting to add Mapnik to the transition tracker.

Upstream released version 2.0.1 a while ago (Debian has 2.0.0 currently), which
changed SONAME back to the old scheme. Before 2.0.0, the library was
libmapnik.so, afterwards it became libmapnik2.so -- with all the
consequences it had (package renames, binNMUs, ...)
However, this SONAME change was meant to be kept private by upstream (to ensure
co-installability on devs machines), so I need to re-transition it, and all
r-deps.

Fortunately, I had previously kept the libmapnik-dev package, so hopefully it
won't need too many sourceful uplods.

I can't tell whether it will be ready for wheezy (I'm still ironing out some
bugs of 2.0.1, and 2.0.2 will come out in a few weeks), but it would be nice to
have the transition tracked, at least.

  is_affected = .build-depends ~ /libmapnik.*-dev/;
  is_good = .depends ~ /libmapnik \(= 2\.0\.1)//;
  is_bad = .depends ~ /libmapnik2|libmapnik,|libmapnik$/;

(hope I got the regexes fine)

Thank you for your work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174



-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20120525193415.76be1891@local



Bug#668456: pu: package wicd/1.7.0+ds1-5+squeeze2

2012-04-30 Thread David Paleino
retitle 668456 pu: package wicd/1.7.0+ds1-5+squeeze2
tags 668456 - confirmed pending
thanks

Hello people,

sorry to bother you again, but... my 1.7.2.3 release of wicd didn't really fix
anything. Now I _really_ fixed it with 1.7.2.4, which I just uploaded to sid.

Unfortunately, I pushed the wrong patch to p-u already; so here is an updated
patch, with an updated version (don't know if it's needed).

Please evaluate the patch (it's a bit different from the one in sid since
squeeze still had py2.5 available), and advice on how to proceed (re the
version number)

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 8a20bc0..c245a54 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,19 @@
+wicd (1.7.0+ds1-5+squeeze2) UNRELEASED; urgency=low
+
+  * debian/patches/:
+- 31-fix_local_privilege_escalation.patch, CVE-2012-2095,
+  improved. Really fixes the bug. (Closes: #668397)
+
+ -- David Paleino da...@debian.org  Mon, 30 Apr 2012 21:53:47 +0200
+
+wicd (1.7.0+ds1-5+squeeze1) proposed-updates; urgency=low
+
+  * debian/patches/:
+- fix local privilege escalation, CVE-2012-2095
+  (31-fix_local_privilege_escalation.patch) (Closes: #668397)
+
+ -- David Paleino da...@debian.org  Tue, 24 Apr 2012 22:05:28 +0200
+
 wicd (1.7.0+ds1-5) unstable; urgency=medium
 
   * debian/patches/series:
diff --git a/debian/patches/31-fix_local_privilege_escalation.patch b/debian/patches/31-fix_local_privilege_escalation.patch
new file mode 100644
index 000..6201cfe
--- /dev/null
+++ b/debian/patches/31-fix_local_privilege_escalation.patch
@@ -0,0 +1,65 @@
+From: David Paleino d.pale...@gmail.com
+Subject: fix local privilege escalation, CVE-2012-2095
+Origin: upstream, http://bazaar.launchpad.net/~wicd-devel/wicd/experimental/revision/767
+
+---
+ wicd/misc.py|8 
+ wicd/wicd-daemon.py |9 ++---
+ 2 files changed, 14 insertions(+), 3 deletions(-)
+
+--- wicd.orig/wicd/wicd-daemon.py
 wicd/wicd/wicd-daemon.py
+@@ -1088,7 +1088,8 @@ class WirelessDaemon(dbus.service.Object
+ def SetWirelessProperty(self, netid, prop, value):
+  Sets property to value in network specified. 
+ # We don't write script settings here.
+-if (prop.strip()).endswith(script):
++prop = misc.sanitize_config(prop)
++if prop.endswith('script'):
+ print Setting script properties through the daemon is not \
+   +  permitted.
+ return False
+@@ -1280,7 +1281,8 @@ class WirelessDaemon(dbus.service.Object
+ @dbus.service.method('org.wicd.daemon.wireless')
+ def SaveWirelessNetworkProperty(self, id, option):
+  Writes a particular wireless property to disk. 
+-if (option.strip()).endswith(script):
++option = misc.sanitize_config(option)
++if option.endswith(script):
+ print 'You cannot save script information to disk through ' + \
+   'the daemon.'
+ return
+@@ -1410,7 +1412,8 @@ class WiredDaemon(dbus.service.Object):
+ def SetWiredProperty(self, property, value):
+  Sets the given property to the given value. 
+ if self.WiredNetwork:
+-if (property.strip()).endswith(script):
++property = misc.sanitize_config(property)
++if property.endswith('script'):
+ print Setting script properties through the daemon \
+   +  is not permitted.
+ return False
+--- wicd.orig/wicd/misc.py
 wicd/wicd/misc.py
+@@ -26,6 +26,7 @@ import os
+ import locale
+ import sys
+ import re
++import string
+ import gobject
+ from threading import Thread
+ from subprocess import Popen, STDOUT, PIPE, call
+@@ -378,6 +379,13 @@ def noneToString(text):
+ else:
+ return str(text)
+ 
++def sanitize_config(s):
++ Sanitize property names to be used in config-files. 
++allowed = string.ascii_letters + '_' + string.digits
++table = string.maketrans(allowed, ' ' * len(allowed))
++blank = string.maketrans('', '')
++return s.translate(blank, table)
++
+ def to_unicode(x):
+  Attempts to convert a string to utf-8. 
+ # If this is a unicode string, encode it and return
diff --git a/debian/patches/series b/debian/patches/series
index f813495..d46fe16 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -21,3 +21,4 @@
 28-announce_dhclient.conf.template.patch
 29-document_variables_passed_to_scripts.patch
 30-make_connection_info_selectable.patch
+31-fix_local_privilege_escalation.patch


signature.asc
Description: PGP signature


Bug#656829: Transition status

2012-04-27 Thread David Paleino
Hello people,

what's the status of the exiv2 transition? It would really be nice to have
exiv2 0.22 in wheezy.

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#656829: Transition status

2012-04-27 Thread David Paleino
On Fri, 27 Apr 2012 21:25:47 +0200, Cyril Brulebois wrote:

 David Paleino da...@debian.org (27/04/2012):
  what's the status of the exiv2 transition? It would really be nice to have
  exiv2 0.22 in wheezy.
 
 http://release.debian.org/transitions/html/exiv2.html says not started.

I meant: what are the plans to start it?

Sorry if I wasn't clearer before.

David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#668456: pu: package wicd/1.7.0+ds1-5+squeeze1

2012-04-24 Thread David Paleino
Hello Adam,

On Tue, 17 Apr 2012 20:51:37 +0100, Adam D. Barratt wrote:

 On Thu, 2012-04-12 at 00:31 +0200, David Paleino wrote:
  I'm hereby requesting permission to upload a fix for wicd to p-u, bug
  #668397 (CCed), CVE-2012-2095. git diff attached.
  
  The patch for stable is slightly different from the one just pushed in
  unstable: namely, it needed an additional has_profile, which was used in
  pre-1.7.1 versions.
 
 +wicd (1.7.0+ds1-5+squeeze1) UNRELEASED; urgency=low
 +
 +  * debian/patches/:
 +- fix local privilege escalation, CVE-2012-2095
 +  (31-fix_local_privilege_escalation.patch) (Closes: #668397)
 +
 
 With the distribution fixed, please go ahead; thanks.

I changed the patch quite a bit, since it was buggy and made wicd not work at
all (see #669388).

Please find it attached: that's the one I'd like to push into p-u.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
From: David Paleino d.pale...@gmail.com
Subject: fix local privilege escalation, CVE-2012-2095
Origin: upstream, http://bazaar.launchpad.net/~wicd-devel/wicd/experimental/revision/758

---
 wicd/misc.py|8 
 wicd/wicd-daemon.py |   16 
 2 files changed, 16 insertions(+), 8 deletions(-)

--- wicd.orig/wicd/wicd-daemon.py
+++ wicd/wicd/wicd-daemon.py
@@ -1088,9 +1088,9 @@ class WirelessDaemon(dbus.service.Object
 def SetWirelessProperty(self, netid, prop, value):
  Sets property to value in network specified. 
 # We don't write script settings here.
-if (prop.strip()).endswith(script):
-print Setting script properties through the daemon is not \
-  +  permitted.
+if misc.sanitize_config(prop).endswith('script'):
+print 'Setting script properties through the daemon' \
+  + ' is not permitted.'
 return False
 self.LastScan[netid][prop] = misc.to_unicode(misc.Noneify(value))
 
@@ -1407,14 +1407,14 @@ class WiredDaemon(dbus.service.Object):
 return str(iface)
 
 @dbus.service.method('org.wicd.daemon.wired')
-def SetWiredProperty(self, property, value):
+def SetWiredProperty(self, prop, value):
  Sets the given property to the given value. 
 if self.WiredNetwork:
-if (property.strip()).endswith(script):
-print Setting script properties through the daemon \
-  +  is not permitted.
+if misc.sanitize_config(prop).endswith('script'):
+print 'Setting script properties through the daemon' \
+  + ' is not permitted.'
 return False
-self.WiredNetwork[property] = misc.to_unicode(misc.Noneify(value))
+self.WiredNetwork[prop] = misc.to_unicode(misc.Noneify(value))
 return True
 else:
 print 'SetWiredProperty: WiredNetwork does not exist'
--- wicd.orig/wicd/misc.py
+++ wicd/wicd/misc.py
@@ -378,6 +378,14 @@ def noneToString(text):
 else:
 return str(text)
 
+def sanitize_config(s):
+ Sanitize property names to be used in config-files. 
+s = s.strip()
+s = s.replace('=', '')
+s = s.replace(' ', '')
+s = s.replace('\n', '')
+return s
+
 def to_unicode(x):
  Attempts to convert a string to utf-8. 
 # If this is a unicode string, encode it and return


signature.asc
Description: PGP signature


Bug#668456: pu: package wicd/1.7.0+ds1-5+squeeze1

2012-04-11 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: pu

Hello RT,

I'm hereby requesting permission to upload a fix for wicd to p-u, bug #668397
(CCed), CVE-2012-2095. git diff attached.

The patch for stable is slightly different from the one just pushed in
unstable: namely, it needed an additional has_profile, which was used in
pre-1.7.1 versions.

Thanks for your work,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 8a20bc0..21cd923 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+wicd (1.7.0+ds1-5+squeeze1) UNRELEASED; urgency=low
+
+  * debian/patches/:
+- fix local privilege escalation, CVE-2012-2095
+  (31-fix_local_privilege_escalation.patch) (Closes: #668397)
+
+ -- David Paleino da...@debian.org  Thu, 12 Apr 2012 00:19:08 +0200
+
 wicd (1.7.0+ds1-5) unstable; urgency=medium
 
   * debian/patches/series:
diff --git a/debian/patches/31-fix_local_privilege_escalation.patch b/debian/patches/31-fix_local_privilege_escalation.patch
new file mode 100644
index 000..819eb20
--- /dev/null
+++ b/debian/patches/31-fix_local_privilege_escalation.patch
@@ -0,0 +1,99 @@
+From: David Paleino d.pale...@gmail.com
+Subject: fix local privilege escalation, CVE-2012-2095
+Origin: upstream, http://bazaar.launchpad.net/~wicd-devel/wicd/experimental/revision/751
+
+---
+ wicd/wicd-daemon.py |   58 
+ 1 file changed, 50 insertions(+), 8 deletions(-)
+
+--- wicd.orig/wicd/wicd-daemon.py
 wicd/wicd/wicd-daemon.py
+@@ -980,6 +980,28 @@ class WirelessDaemon(dbus.service.Object
+  wireless-settings.conf),
+ debug=debug)
+ 
++self._validProperties = (
++'bssid',
++'essid',
++'hidden',
++'channel',
++'mode',
++'enctype',
++'encryption_method',
++'key',
++'automatic',
++'ip',
++'netmask',
++'broadcast',
++'gateway',
++'use_static_dns',
++'use_global_dns',
++'dns1',
++'dns2',
++'dns3',
++'use_settings_globally',
++)
++
+ def get_debug_mode(self):
+ return self._debug_mode
+ def set_debug_mode(self, mode):
+@@ -1088,9 +1110,9 @@ class WirelessDaemon(dbus.service.Object
+ def SetWirelessProperty(self, netid, prop, value):
+  Sets property to value in network specified. 
+ # We don't write script settings here.
+-if (prop.strip()).endswith(script):
+-print Setting script properties through the daemon is not \
+-  +  permitted.
++if prop.strip() not in self._validProperties:
++print Trying to set invalid property (or property not  \
++  permitted): + prop.strip() + .
+ return False
+ self.LastScan[netid][prop] = misc.to_unicode(misc.Noneify(value))
+ 
+@@ -1367,6 +1389,26 @@ class WiredDaemon(dbus.service.Object):
+  wired-settings.conf), 
+ debug=debug)
+ 
++self._validProperties = (
++'ip',
++'broadcast',
++'netmask',
++'gateway',
++'search_domain',
++'dns_domain',
++'dns1',
++'dns2',
++'dns3',
++'encryption_enabled',
++'default',
++'dhcphostname',
++'lastused',
++'profilename',
++'use_global_dns',
++'use_static_dns',
++'has_profile',
++)
++
+ def get_debug_mode(self):
+ return self._debug_mode
+ def set_debug_mode(self, mode):
+@@ -1407,14 +1449,14 @@ class WiredDaemon(dbus.service.Object):
+ return str(iface)
+ 
+ @dbus.service.method('org.wicd.daemon.wired')
+-def SetWiredProperty(self, property, value):
++def SetWiredProperty(self, prop, value):
+  Sets the given property to the given value. 
+ if self.WiredNetwork:
+-if (property.strip()).endswith(script):
+-print Setting script properties through the daemon \
+-  +  is not permitted.
++if prop.strip() not in self._validProperties:
++print Trying to set invalid property (or property not  \
++  permitted): + prop.strip() + .
+ return False
+-self.WiredNetwork[property] = misc.to_unicode(misc.Noneify(value))
++self.WiredNetwork[prop] = misc.to_unicode(misc.Noneify

Bug#663632: transition: geos

2012-03-12 Thread David Paleino
On Mon, 12 Mar 2012 21:19:34 +0100, Julien Cristau wrote:

 [..]
 - osmium FTBFS (no bug report found)

Fixed in git, a package will soon be uploaded.

http://anonscm.debian.org/gitweb/?p=pkg-osm/osmium.git;a=commitdiff;h=b79f5511de3c6a22dfa1e9cb1cf4f85886244dad

David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


RM: josm-plugins/0.0.svn26626+ds1-2

2011-12-07 Thread David Paleino
user 651372 release.debian@packages.debian.org
usertags 651372 rm
reassign 651372 release.debian.org
retitle 651372 RM: josm-plugins/0.0.svn26626+ds1-2
thanks


Hello Antoine,

On Thu, 08 Dec 2011 01:33:24 -0500, Antoine Beaupré wrote:

 I am not sure I understand why, but it seems that 4550 is stuck in
 unstable and doesn't migrate down to testing:
 
 http://packages.qa.debian.org/j/josm.html
 
 Indeed, it looks like it's blocked by some dependency, although I am
 not sure the blame is on the dependency as much as josm...
 
 http://release.debian.org/migration/testing.pl?package=josm
 
 Any ideas?

It's because it would make josm-plugins uninstallable in testing.

Release team: please remove josm-plugins/0.0.svn26626+ds1-2 from testing, since
I haven't yet had time to update it to latest josm, and I'd prefer having this
one in testing.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#651197: RM: desktopcouch/1.0.8-1

2011-12-06 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: rm

Hello,
please remove desktopcouch from testing, since it's currently unusable at all
(see bug #651142, X-Debbugs-CCed). I just raised that bug's severity to grave.

However, it has some rdepends, all coming from the same source package: dmedia.
I don't know if I'm required to contact dmedia's maintainers before asking
removal of desktopcouch -- in any case, they are CCed.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#622973: pu: package josm/0.0.svn3376-1+squeeze1

2011-04-16 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: pu

Dear Release Team,
I'm writing to you to ask for a possible p-u upload for JOSM.

OpenStreetMap is pursuing a process to change the license of its data.
Tomorrow it will enter Phase III, i.e. a contributor has to explicitely accept
or decline the new license [0], otherwise he'll get a 403 error [1].

The suggested fix [2] was to improve the error message, including the response
body. Upstream did this [3], and released a new snapshot (4021, currently in
sid).

I'm attaching the backported patch.

Thanks,
David

[0]: http://lists.openstreetmap.org/pipermail/talk/2011-April/057649.html
[1]: http://lists.openstreetmap.org/pipermail/josm-dev/2011-April/005504.html
[2]: http://lists.openstreetmap.org/pipermail/josm-dev/2011-April/005508.html
[3]:
https://josm.openstreetmap.de/changeset?reponame=josmnew=4020%40%2Fold=4019%40%2F

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
From: David Paleino da...@debian.org
Subject: backport fixed failed authorisation dialog from 4021
Origin: vendor
Forwarded: not-needed

---
 src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java |3 +-
 src/org/openstreetmap/josm/tools/ExceptionUtil.java |   24 
 2 files changed, 26 insertions(+), 1 deletion(-)

--- josm.orig/src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java
+++ josm/src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java
@@ -271,9 +271,10 @@ public class ExceptionDialogUtil {
  * @param e the exception
  */
 public static void explainAuthorizationFailed(OsmApiException e) {
+// Fixme: add special handling that calls ExceptionUtil.explainFailedOAuthAuthorisation(e)
 HelpAwareOptionPane.showOptionDialog(
 Main.parent,
-ExceptionUtil.explainFailedOAuthAuthorisation(e),
+ExceptionUtil.explainFailedAuthorisation(e),
 tr(Authorisation Failed),
 JOptionPane.ERROR_MESSAGE,
 ht(/ErrorMessages#AuthenticationFailed)
--- josm.orig/src/org/openstreetmap/josm/tools/ExceptionUtil.java
+++ josm/src/org/openstreetmap/josm/tools/ExceptionUtil.java
@@ -128,6 +128,30 @@ public class ExceptionUtil {
 );
 }
 
+   public static String explainFailedAuthorisation(OsmApiException e) {
+e.printStackTrace();
+String header = e.getErrorHeader();
+String body = e.getErrorBody();
+String msg = null;
+if (header != null) {
+if (body != null  !header.equals(body)) {
+msg = header +  ( + body + );
+} else {
+msg = header;
+}
+} else {
+msg = body;
+}
+
+return tr(html
++ Authorisation at the OSM server failed.br
++ The server reported the following error:br
++ ''{0}''
++ /html,
+msg
+);
+}
+
 public static String explainFailedOAuthAuthorisation(OsmApiException e) {
 e.printStackTrace();
 return tr(html


signature.asc
Description: PGP signature


Bug#622973: pu: package josm/0.0.svn3376-1+squeeze1

2011-04-16 Thread David Paleino
On Sat, 16 Apr 2011 11:32:42 +0100, Adam D. Barratt wrote:

 On Sat, 2011-04-16 at 11:55 +0200, David Paleino wrote:
  I'm writing to you to ask for a possible p-u upload for JOSM.
  
  OpenStreetMap is pursuing a process to change the license of its data.
  Tomorrow it will enter Phase III, i.e. a contributor has to explicitely
  accept or decline the new license [0], otherwise he'll get a 403 error [1].
 
 If this is phase 3, presumably this has been known about for a while?

The date wasn't known up to two days ago; when it was announced (see link [0]
in my mail).

 If so, it would have been nicer if the patch could have been included in
 the version which we shipped in Squeeze.  (I realise this was only just
 fixed upstream, looking at the links you provided; thanks for those).

Heh :)
It's really an easy patch, but the problem really is that it was announced
suddenly, and with little pre-notice.

  The suggested fix [2] was to improve the error message, including the
  response body. Upstream did this [3], and released a new snapshot (4021,
  currently in sid).
 
 Assuming the resulting package has been tested in a Squeeze environment,
 please feel free to upload including (just) that patch.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#617828: nmu: mapnik_0.7.1-3

2011-03-11 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: binnmu

Dear RT,

  nmu mapnik_0.7.1-3 . ALL . -m Bump libgdal dependency

Currently libmapnik0.7 depends on libgdal1-1.6.0, while libgdal1-1.7.0 is
available in sid.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Bug#609520: unblock: python-xattr/0.4-5

2011-01-13 Thread David Paleino
On Tue, 11 Jan 2011 00:46:46 +0100, Julien Cristau wrote:

 On Sun, Jan  9, 2011 at 23:03:56 +, Federico Ceratto wrote:
 
  The diff between version 0.4.5 and 0.4-5+squeeze1 is below.
 
 Ack, please upload.

I sponsored this yesterday, and has been ACCEPTED into t-p-u.

David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#606797: unblock: desktopcouch/0.6.6-2

2010-12-11 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear Release Team,
please unblock package desktopcouch.

Version 0.6.6-2 fixes a bug with Futon authentication caused by broken
(intended) HTTP behaviour by couchdb. It would be nice if this was included in
Squeeze and, since the patch was tiny (2 lines), I already uploaded it to sid.


unblock desktopcouch/0.6.6-2


Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Pre-upload approval for gThumb

2010-11-28 Thread David Paleino
On Sun, 28 Nov 2010 20:14:23 +0100, Mehdi Dogguy wrote:

 On 10/24/2010 03:17 PM, David Paleino wrote:
  
  I could just check the other two too. However, I assume that at
  least image width and height are set by any camera -- that's why I
  didn't add the check to those as well, in first place. However, it's
  just an assumption, and I haven't googled for it. Also, for those
  attributes there's no reasonable default that could be set.
 
 Ok, I'd like to have this bug fixed in Squeeze as well. Could you please
 upload this fix in sid?

Done.
Would you please unblock it?

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#604108: unblock: mapnik/0.7.1-3

2010-11-20 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear Release Team,
please unblock package mapnik.
This version is compiled using the embedded AGG source, instead of the
system-wide libagg-dev, which is broken (at least for Mapnik's usage) and
mostly dead upstream.
I already received an ACK from the security team for this.


unblock mapnik/0.7.1-3


Thank you for your work!
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#603667: unblock: bash-completion/1:1.2-3

2010-11-16 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear Release Team,
please unblock package bash-completion.

This version fixes an upgrade-path issue from older versions when bash
shipped /etc/bash_completion , i.e. bug #564254 .


unblock bash-completion/1:1.2-3


Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Bug#603470: libmapnik0.7: package linked against broken external AGG

2010-11-15 Thread David Paleino
Dear Security and Release Teams,

On Mon, 15 Nov 2010 12:07:56 +0100, Sven Geggus wrote:

 Francesco P. Lovergine schrieb am Montag, den 15. November um 11:19 Uhr:
 
  My best guessing is moving to the embedded copy if resulting issues are
  grave enough to compromise its use.
 
 There are occasional hangs of the rendering library resulting in 100% CPU
 usage.

I'm writing to you because of #603470. The obvious solution is to link against
the internal patched libagg, and this is what mapnik upstream is expecting us to
do.

The AGG+Mapnik case is unfortunate; the problem is twofold: upstream relicensed
the code from MIT to GPL-2 from versions 2.4 → 2.5 (and Mapnik is LGPL, so
we're basically restricting its usage when linking to the GPL library), and
development of AGG has now stopped.
It seems like there are some forks in the wild of the 2.4 branch (because of
license concerns). Mapnik embeds a patched 2.3 version of AGG -- I'd like to
know if:

- security team: would it be acceptable to use the embedded copy?
- release team: would such a change have a freeze exception granted? (attaching
  diff -- the only change is the drop of INTERNAL_LIBAGG=no, but I split it to
  make it easier to read in future)

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 119090b..019f990 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+mapnik (0.7.1-3) UNRELEASED; urgency=low
+
+  * Compile using internal copy of libagg. (Closes: #603470)
+
+ -- David Paleino da...@debian.org  Mon, 15 Nov 2010 16:05:05 +0100
+
 mapnik (0.7.1-2) unstable; urgency=low
 
   * debian/patches/03-fix_ImportError_mips.patch added, fixes
diff --git a/debian/control b/debian/control
index 678646c..dcd8651 100644
--- a/debian/control
+++ b/debian/control
@@ -26,7 +26,6 @@ Build-Depends:
  libfribidi-dev,
  libgdal1-dev,
  libxml2-dev,
- libagg-dev,
  libicu-dev,
  libcairo2-dev,
  libcairomm-1.0-dev,
diff --git a/debian/rules b/debian/rules
index a19bd94..067d3c2 100755
--- a/debian/rules
+++ b/debian/rules
@@ -5,7 +5,12 @@
 #export DH_VERBOSE=1
 
 # scons flags
-SCONS_FLAGS=INPUT_PLUGINS=raster,sqlite,postgis,ogr,shape,osm,gdal,kismet PROJ_INCLUDES=/usr/include PROJ_LIBS=/usr/lib INTERNAL_LIBAGG=no SYSTEM_FONTS=/usr/share/fonts/truetype/ttf-dejavu XMLPARSER=libxml2 DESTDIR=$(CURDIR)/debian/tmp PREFIX=/usr LIB_DIR_NAME=/mapnik/0.7
+SCONS_FLAGS := INPUT_PLUGINS=raster,sqlite,postgis,ogr,shape,osm,gdal,kismet
+SCONS_FLAGS += PROJ_INCLUDES=/usr/include PROJ_LIBS=/usr/lib
+SCONS_FLAGS += SYSTEM_FONTS=/usr/share/fonts/truetype/ttf-dejavu
+SCONS_FLAGS += XMLPARSER=libxml2
+SCONS_FLAGS += DESTDIR=$(CURDIR)/debian/tmp
+SCONS_FLAGS += PREFIX=/usr LIB_DIR_NAME=/mapnik/0.7
 
 CFLAGS = -Wall -g
 


signature.asc
Description: PGP signature


Bug#603486: Upload permission and future unblock: gphoto2/2.4.5-3

2010-11-14 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear RT,
am I allowed to upload a package with the following debdiff? It fixes #512883
(which is RC).

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 9c76be8..79b1f32 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+gphoto2 (2.4.5-3) unstable; urgency=low
+
+  * Warn and exit with an error if files couldn't be written to
+disk (Closes: #512883)
+  * Added myself to Uploaders
+
+ -- David Paleino da...@debian.org  Sun, 14 Nov 2010 17:14:36 +0100
+
 gphoto2 (2.4.5-2) unstable; urgency=low
 
   * debian/control:
diff --git a/debian/control b/debian/control
index 9043ec4..f7e69ad 100644
--- a/debian/control
+++ b/debian/control
@@ -3,6 +3,7 @@ Section: utils
 Priority: extra
 Maintainer: Debian PhotoTools Maintainers pkg-phototools-de...@lists.alioth.debian.org
 Uploaders: Frederic Peters fpet...@debian.org, Cyril Brulebois k...@debian.org
+ , David Paleino da...@debian.org
 Build-Depends: debhelper (= 7),
quilt,
type-handling (= 0.2.1),
diff --git a/debian/patches/01-handle_EACCES.patch b/debian/patches/01-handle_EACCES.patch
new file mode 100644
index 000..c5e5492
--- /dev/null
+++ b/debian/patches/01-handle_EACCES.patch
@@ -0,0 +1,31 @@
+From: David Paleino da...@debian.org
+Subject: exit with an error if files couldn't be written to disk
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=512883
+Origin: vendor
+Forwarded: yes, https://sourceforge.net/tracker/index.php?func=detailaid=3109145group_id=8874atid=308874
+
+---
+ gphoto2/main.c |5 +
+ 1 file changed, 5 insertions(+)
+
+--- gphoto2-2.4.5.orig/gphoto2/main.c
 gphoto2-2.4.5/gphoto2/main.c
+@@ -43,6 +43,7 @@
+ #include fcntl.h
+ #include utime.h
+ #include limits.h
++#include errno.h
+ 
+ #ifdef HAVE_POPT
+ #  include popt.h
+@@ -501,6 +502,10 @@ save_file_to_file (Camera *camera, GPCon
+ 	strcpy (tmpname, tmpfileXX);
+ 	fd = mkstemp(tmpname);
+ 	if (fd == -1) {
++	if (errno == EACCES) {
++	gp_context_error (context, _(Permission denied));
++	return GP_ERROR;
++	}
+ 	CR (gp_file_new (file));
+ 		tmpfilename = NULL;
+ 	} else {
diff --git a/debian/patches/series b/debian/patches/series
index 1d77bb1..e4ef4ce 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
+01-handle_EACCES.patch
 20_german_translation


signature.asc
Description: PGP signature


Bug#603486: Upload permission and future unblock: gphoto2/2.4.5-3

2010-11-14 Thread David Paleino
On Sun, 14 Nov 2010 17:40:26 +, Adam D. Barratt wrote:

 On Sun, 2010-11-14 at 17:33 +0100, David Paleino wrote:
  am I allowed to upload a package with the following debdiff? It fixes
  #512883 (which is RC).
 
 Please go ahead.

Uploaded :)

Please,

unblock gphoto2/2.4.5-3

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#603514: unblock: tilecache/2.03-2

2010-11-14 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package tilecache, fixing maintainer e-mail addresses, as
discussed before on-list.

unblock tilecache/2.03-2

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Upload to unstable and freeze exception for tilecache

2010-11-13 Thread David Paleino
Dear Release Team,
I'm writing to you because of #602932 .
The e-mail address for the maintainer in debian/control doesn't work (he now
works for another company, AFAICT). I already adopted the package with a new
version in experimental.

Would it make sense to make an upload only changing debian/control Maintainer
and Uploaders fields?

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#602893: unblock: libgphoto2/2.4.6-2

2010-11-10 Thread David Paleino
On Wed, 10 Nov 2010 00:02:54 +0100, Michael Biebl wrote:

 There still seems to be something missing:
 
 On my (constantly upgraded) Debian sid system I have:
 
 /etc/udev/rules.d/025_libgphoto2.rules - ../libgphoto2.rules
 /etc/udev/libgphoto2.rules
 
 but also
 
 /lib/udev/rules.d/60-libgphoto2-2.rules
 
 
 I guess the files in /etc should be cleaned up on upgrades.

Right.

/me reads dpkg-maintscript-helper(1) :-)

 See also [1]
 This would also fix [2].
 
 Would be nice to have those old conffileS cleaned up on upgrades (maybe you
 want to look into [3] while at it)

I'll take a look at those -- I was going to upload a -3 nevertheless (I forgot
to apply one more patch in -2). Will reping as soon as it lands in sid.

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#602893: unblock: libgphoto2/2.4.6-2

2010-11-10 Thread David Paleino
Hello,

On Wed, 10 Nov 2010 09:07:38 +0100, David Paleino wrote:

 [..]
 Will reping as soon as it lands in sid.

Maybe it's better if I attach the debdiff for review before uploading :-)

I'm attaching 2.4.6-2 → 2.4.6-3. If it is ok, I'll upload it to sid.

For conffile removal, I couldn't use dpkg-maintscript-helper -- until
2.4.6-1, they were created at postinst-time (thus they weren't marked
as conffiles), and in 2.4.6-2 the rules file was already in /lib/. So I just
rm them in postinst and postrm (just to be sure). Any better idea?

Other than that, I also merged the patch from #531979 by Martin Pitt.

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff -u libgphoto2-2.4.6/debian/rules libgphoto2-2.4.6/debian/rules
--- libgphoto2-2.4.6/debian/rules
+++ libgphoto2-2.4.6/debian/rules
@@ -97,6 +97,11 @@
 		mkdir -p debian/libgphoto2-port0/lib/udev  \
 		mv debian/tmp/usr/lib/udev/check-mtp-device \
 			debian/libgphoto2-port0/lib/udev/check-mtp-device
+
+	# create hal FDI
+	mkdir -p debian/libgphoto2-$(major)/usr/share/hal/fdi/preprobe/10osvendor/
+	CAMLIBS=`ls -d debian/libgphoto2-$(major)/usr/lib/libgphoto2/2*` ./packaging/generic/print-camera-list hal-fdi  \
+			debian/libgphoto2-$(major)/usr/share/hal/fdi/preprobe/10osvendor/20-libgphoto2.fdi
 endif
 
 	# remove *.la files before trying to remove debian/tmp
diff -u libgphoto2-2.4.6/debian/libgphoto2-2.postinst libgphoto2-2.4.6/debian/libgphoto2-2.postinst
--- libgphoto2-2.4.6/debian/libgphoto2-2.postinst
+++ libgphoto2-2.4.6/debian/libgphoto2-2.postinst
@@ -7,15 +7,18 @@
 
 case $1 in
 configure)
-	# create FDI file
-	mkdir -p /usr/share/hal/fdi/preprobe/10osvendor/ || true
-	/usr/lib/$PACKAGE/print-camera-list hal-fdi  \
-		/usr/share/hal/fdi/preprobe/10osvendor/20-$PACKAGE.fdi
-	
 	# remove old fdi files
 	rm -f /usr/share/hal/fdi/information/10freedesktop/10-camera-$PACKAGE.fdi
 	rm -f /usr/share/hal/fdi/information/20thirdparty/$PACKAGE.fdi
 	rm -f /usr/share/hal/fdi/information/10freedesktop/10-camera-$PACKAGE-device.fdi
+
+	# remove old config files
+	rm -f /etc/udev/$PACKAGE.rules
+	rm -f /etc/udev/rules.d/025_libgphoto2.rules
+	rm -f /etc/hotplug/usb/$PACKAGE.usermap
+	rm -f /etc/udev/libgphoto2_generic-ptp_support.rules
+	rm -f /etc/udev/rules.d/025_$PACKAGE.rules
+	rm -f /etc/udev/rules.d/020_libgphoto2_generic-ptp_support.rules
 	;;
 
 abort-upgrade|abort-remove|abort-deconfigure)
diff -u libgphoto2-2.4.6/debian/libgphoto2-2.postrm libgphoto2-2.4.6/debian/libgphoto2-2.postrm
--- libgphoto2-2.4.6/debian/libgphoto2-2.postrm
+++ libgphoto2-2.4.6/debian/libgphoto2-2.postrm
@@ -14,7 +14,6 @@
 	rm -f /usr/share/hal/fdi/information/20thirdparty/$PACKAGE.fdi || /bin/true
 	rm -f /usr/share/hal/fdi/information/10freedesktop/10-camera-$PACKAGE.fdi || /bin/true
 	rm -f /usr/share/hal/fdi/information/10freedesktop/10-camera-$PACKAGE-device.fdi || /bin/true
-	rm -f /usr/share/hal/fdi/preprobe/10osvendor/20-$PACKAGE.fdi || /bin/true
 	;;
 
 disappear|upgrade|failed-upgrade|abort-install|abort-upgrade)
diff -u libgphoto2-2.4.6/debian/changelog libgphoto2-2.4.6/debian/changelog
--- libgphoto2-2.4.6/debian/changelog
+++ libgphoto2-2.4.6/debian/changelog
@@ -1,3 +1,17 @@
+libgphoto2 (2.4.6-3) unstable; urgency=low
+
+  * Merge one more patch by Martin Pitt (thanks!): move HAL FDI
+creation from debian/libgphoto2-2.postinst to debian/rules.
+This avoids problems with half-configured libraries during
+upgrade/installation (LP: #352271, Closes: #531979)
+  * Remove old conffiles. We can't use dpkg-maintscript-helper,
+because they were created at install-time and thus not marked
+as conffiles. (Closes: #461397, #511129)
+  * Move from /etc/udev/rules.d/ to /lib/udev/rules.d should now
+be complete (Closes: #586668)
+
+ -- David Paleino da...@debian.org  Wed, 10 Nov 2010 10:26:49 +0100
+
 libgphoto2 (2.4.6-2) unstable; urgency=low
 
   * Merge Ubuntu patches, by Martin Pitt (thanks!):


signature.asc
Description: PGP signature


Bug#602893: unblock: libgphoto2/2.4.6-2

2010-11-09 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear RT,
please unblock package libgphoto2 2.4.6-2. It fixes an RC bug, and fixes a
wrong Build-Dependency.
I'm attaching the diff from 2.4.6-1, which is currently in testing.


unblock libgphoto2/2.4.6-2


Thank you,
David
-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index d707f60..f6d9e47 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,17 @@
+libgphoto2 (2.4.6-2) unstable; urgency=low
+
+  * Merge Ubuntu patches, by Martin Pitt (thanks!):
+- Update Build-Depends: Move from libltdl3-dev to libltdl-dev
+  (Closes: #531974).
+- Create modern udev rules at build-time instead of at install-time
+  thanks to a patch sent (and now merged) upstream. Drop a lot of
+  code from libgphoto2-2.postinst accordingly, while adding the
+  generation code to debian/rules. Also bump the versioned dependency
+  on udev from 0.113-1 to 0.136 (Closes: #531978).
+  * Added myself to Uploaders
+
+ -- David Paleino da...@debian.org  Mon, 08 Nov 2010 11:19:51 +0100
+
 libgphoto2 (2.4.6-1) unstable; urgency=low
 
   * New upstream release.
diff --git a/debian/control b/debian/control
index 6256992..2bef34b 100644
--- a/debian/control
+++ b/debian/control
@@ -3,11 +3,12 @@ Section: libs
 Priority: optional
 Maintainer: Debian PhotoTools Maintainers pkg-phototools-de...@lists.alioth.debian.org
 Uploaders: Frederic Peters fpet...@debian.org, Cyril Brulebois k...@debian.org
+ , David Paleino da...@debian.org
 Build-Depends: debhelper ( 5.0.0),
quilt,
type-handling,
libtool,
-   libltdl3-dev,
+   libltdl-dev,
zlib1g-dev,
libusb-dev (= 1:0.1.5) [!hurd-i386],
libgpmg1-dev | not+linux-gnu,
diff --git a/debian/libgphoto2-2.dirs b/debian/libgphoto2-2.dirs
index 86610e6..e69de29 100644
--- a/debian/libgphoto2-2.dirs
+++ b/debian/libgphoto2-2.dirs
@@ -1 +0,0 @@
-etc/udev/rules.d
diff --git a/debian/libgphoto2-2.postinst b/debian/libgphoto2-2.postinst
index 6975dd9..3fe44c2 100644
--- a/debian/libgphoto2-2.postinst
+++ b/debian/libgphoto2-2.postinst
@@ -16,25 +16,6 @@ case $1 in
 	rm -f /usr/share/hal/fdi/information/10freedesktop/10-camera-$PACKAGE.fdi
 	rm -f /usr/share/hal/fdi/information/20thirdparty/$PACKAGE.fdi
 	rm -f /usr/share/hal/fdi/information/10freedesktop/10-camera-$PACKAGE-device.fdi
-
-	# create udev rules file
-	if [ -d /etc/udev/ ]; then
-	/usr/lib/$PACKAGE/print-camera-list udev-rules version 0.98 mode 0664 group plugdev  /etc/udev/$PACKAGE.rules
-	# install the udev file only once:
-	#- the first time the package is installed
-	# OR - the first time the package is upgraded from a version earlier than 2.1.6-5.1
-	if [ -z $2 ] || dpkg --compare-versions $2 lt 2.1.6-5.1 ; then
-		ln -sf ../$PACKAGE.rules /etc/udev/rules.d/025_$PACKAGE.rules
-	fi
-
-	if [ -L /etc/udev/rules.d/020_libgphoto2_generic-ptp_support.rules ]; then
-		rm -f /etc/udev/rules.d/020_libgphoto2_generic-ptp_support.rules
-	fi
-
-	if [ -f /etc/udev/libgphoto2_generic-ptp_support.rules ]; then
-		rm -f /etc/udev/libgphoto2_generic-ptp_support.rules
-	fi
-	fi
 	;;
 
 abort-upgrade|abort-remove|abort-deconfigure)
diff --git a/debian/patches/01_udev_rules_136 b/debian/patches/01_udev_rules_136
new file mode 100644
index 000..053a830
--- /dev/null
+++ b/debian/patches/01_udev_rules_136
@@ -0,0 +1,108 @@
+# Description: Add a new udev mode 136 to create contemporary udev rules which will avoid using the check_ptp_camera script, and provide udev DB tags for the hal deprecation. See http://lists.freedesktop.org/archives/devkit-devel/2009-June/000191.html for discussion.
+# Upstream: http://sourceforge.net/tracker/?func=detailaid=2801117group_id=8874atid=308874 (in svn head)
+Index: libgphoto2-2.4.6/packaging/generic/print-camera-list.c
+===
+--- libgphoto2-2.4.6.orig/packaging/generic/print-camera-list.c	2009-06-05 13:34:52.0 +0200
 libgphoto2-2.4.6/packaging/generic/print-camera-list.c	2009-06-05 13:35:17.0 +0200
+@@ -366,12 +366,14 @@
+ 
+ typedef enum {
+ 		UDEV_PRE_0_98 = 0,
+-		UDEV_0_98 = 1
++		UDEV_0_98 = 1,
++		UDEV_136 = 2
+ } udev_version_t;
+ 
+ static const StringFlagItem udev_version_t_map[] = {
+ 	{ pre-0.98, UDEV_PRE_0_98 },
+ 	{ 0.98, UDEV_0_98 },
++	{ 136, UDEV_136 },
+ 	{ NULL, 0 }
+ };
+ 
+@@ -399,13 +401,20 @@
+ 		ACTION!=\add\, GOTO=\libgphoto2_rules_end\\n\n,
+ 		/* UDEV_0_98 */
+ 		SUBSYSTEM!=\usb|usb_device\, GOTO=\libgphoto2_rules_end\\n
++		ACTION!=\add\, GOTO=\libgphoto2_rules_end\\n\n

Bug#602893: unblock: libgphoto2/2.4.6-2

2010-11-09 Thread David Paleino
On Tue, 09 Nov 2010 21:34:00 +, Adam D. Barratt wrote:

 On Tue, 2010-11-09 at 09:09 +0100, David Paleino wrote:
  please unblock package libgphoto2 2.4.6-2. It fixes an RC bug, and fixes a
  wrong Build-Dependency.
  I'm attaching the diff from 2.4.6-1, which is currently in testing.
 
 The removal
 of /etc/udev/rules.d/020_libgphoto2_generic-ptp_support.rules
 and /etc/udev/libgphoto2_generic-ptp_support.rules in the postinst seems
 to have been dropped without a replacement.  Was that intentional?

Yes.

Those lines have been there since before 2.4.0-8 (which is the first version
available in current's git repository), and Lenny released with 2.4.1-3. So it
should be safe to remove them.

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Pre-upload approval for gThumb

2010-10-24 Thread David Paleino
On Sun, 24 Oct 2010 13:28:29 +0200, Philipp Kern wrote:

 On Sat, Oct 23, 2010 at 10:36:08PM +0200, David Paleino wrote:
  today I received #601137 -- i.e. gThumb segfaults because of an unhandled
  situation of a missing Exif tag (the orientation).
  I could track it down, and made a patch [1] for it.
 [...]
  [1]:
  http://git.debian.org/?p=collab-maint/gthumb.git;a=blob;f=debian/patches/14-safe_exif_read.patch;hb=refs/heads/exp/master
 
 Does this mean that it still crashes if any of the other attributes are not
 present?  In the context diff it looks like there could be more.

Only the two that you can see in the diff.

 That would seem like bad design to me...

I could just check the other two too.
However, I assume that at least image width and height are set by any camera --
that's why I didn't add the check to those as well, in first place. However,
it's just an assumption, and I haven't googled for it.
Also, for those attributes there's no reasonable default that could be set.

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Pre-upload approval for gThumb

2010-10-23 Thread David Paleino
Dear RT,
today I received #601137 -- i.e. gThumb segfaults because of an unhandled
situation of a missing Exif tag (the orientation).
I could track it down, and made a patch [1] for it.
I tested it locally with the image provided by the user, and it works. I already
uploaded a package to experimental (which has a newer upstream version).

Would it be ok to upload a package to sid with the above patch applied?

Thank you,
David

[1]:
http://git.debian.org/?p=collab-maint/gthumb.git;a=blob;f=debian/patches/14-safe_exif_read.patch;hb=refs/heads/exp/master

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#600842: unblock: colorname/0.4+dfsg.1-2

2010-10-20 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear RT,
please unblock package colorname.
I just uploaded a new version to sid, fixes an RC bug (easy to fix, patch
attached).

unblock colorname/0.4+dfsg.1-2

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/debian/changelog b/debian/changelog
index 8dc9715..ab77659 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+colorname (0.4+dfsg.1-2) unstable; urgency=low
+
+  * debian/colorname.desktop: remove Terminal=true (Closes: #600838)
+  * debian/control: Standards-Version bumped to 3.9.1, no changes
+needed
+
+ -- David Paleino da...@debian.org  Wed, 20 Oct 2010 15:57:48 +0200
+
 colorname (0.4+dfsg.1-1) unstable; urgency=low
 
   * Initial release (Closes: #565623)
diff --git a/debian/colorname.desktop b/debian/colorname.desktop
index be817f2..bd700e1 100644
--- a/debian/colorname.desktop
+++ b/debian/colorname.desktop
@@ -4,5 +4,4 @@ Type=Application
 Name=colorname
 Categories=Graphics;Utility;GTK
 Exec=/usr/bin/colorname
-Terminal=true
 Icon=/usr/share/pixmaps/colorname.xpm
diff --git a/debian/control b/debian/control
index 8d6e73d..fb0cbd4 100644
--- a/debian/control
+++ b/debian/control
@@ -7,7 +7,7 @@ Build-Depends: quilt (= 0.46-7~),
  python (= 2.4),
  python-gtk2 (= 2.10),
  python-gnome2 (= 2.18)
-Standards-Version: 3.8.3
+Standards-Version: 3.9.1
 Homepage: http://code.foosel.org/colorname
 Vcs-Git: git://git.debian.org/git/collab-maint/colorname.git
 Vcs-Browser: http://git.debian.org/?p=collab-maint/colorname.git


signature.asc
Description: PGP signature


Bug#600018: Pre-upload approval for python-bottle/0.8.4-1

2010-10-13 Thread David Paleino
retitle 600018 unblock: python-bottle/0.8.4-1
user release.debian@packages.debian.org
usertags 600018 freeze-exception
thanks

Hello,

On Wed, 13 Oct 2010 09:38:34 +0200, Mehdi Dogguy wrote:

 On 10/12/2010 10:46 PM, David Paleino wrote:
  Package: release.debian.org
  Severity: normal
  
  Hello RT,
  I have a new upstream version of python-bottle (0.8.4, currently 0.8.2-1 in
  sid and 0.8.3-1 in experimental), which fixes a couple of bugs. Not much
  important, but would be nice to have them pushed in Squeeze, since the
  fixes are tiny and cause no harm, even if they don't meet the freeze
  criteria.
 
 Looks ok. Please go ahead with the upload.

Package accepted, requesting unblock.


unblock python-bottle/0.8.4-1


Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#600018: Pre-upload approval for python-bottle/0.8.4-1

2010-10-12 Thread David Paleino
Package: release.debian.org
Severity: normal

Hello RT,
I have a new upstream version of python-bottle (0.8.4, currently 0.8.2-1 in sid
and 0.8.3-1 in experimental), which fixes a couple of bugs. Not much important,
but would be nice to have them pushed in Squeeze, since the fixes are tiny and
cause no harm, even if they don't meet the freeze criteria.

I'm attaching a git diff from the tagged debian/0.8.2-1. Would such an upload
be unblocked?

Thank you for your work,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff --git a/apidoc/changelog.rst b/apidoc/changelog.rst
index db36a54..776e03e 100755
--- a/apidoc/changelog.rst
+++ b/apidoc/changelog.rst
@@ -5,6 +5,15 @@
 Release Notes and Changelog
 ===
 
+Bugfix Release 0.8.4
+=
+
+* Fixed Using DictMixins with the @view decorator (Issue #102 and #103)
+
+Bugfix Release 0.8.3
+=
+
+* Fixed Reloading server dies on slow hardware. (Issue #90)
 
 Bugfix Release 0.8.2
 =
diff --git a/bottle.py b/bottle.py
index 7b7ae07..8a7cae4 100755
--- a/bottle.py
+++ b/bottle.py
@@ -62,7 +62,7 @@ This is an example::
 from __future__ import with_statement
 
 __author__ = 'Marcel Hellkamp'
-__version__ = '0.8.2'
+__version__ = '0.8.4'
 __license__ = 'MIT'
 
 import base64
@@ -1432,7 +1432,7 @@ class FileCheckerThread(threading.Thread):
 self.status = 3
 if not exists(self.lockfile):
 self.status = 2
-elif mtime(self.lockfile)  time.time() - self.interval * 2:
+elif mtime(self.lockfile)  time.time() - self.interval - 5:
 self.status = 1
 if not self.status:
 time.sleep(self.interval)
@@ -1806,7 +1806,7 @@ def view(tpl_name, **defaults):
 @functools.wraps(func)
 def wrapper(*args, **kwargs):
 result = func(*args, **kwargs)
-if isinstance(result, dict):
+if isinstance(result, (dict, DictMixin)):
 tplvars = defaults.copy()
 tplvars.update(result)
 return template(tpl_name, **tplvars)
diff --git a/debian/changelog b/debian/changelog
index 09cfb5e..713036a 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,16 @@
+python-bottle (0.8.4-1) UNRELEASED; urgency=low
+
+  * New upstream version
+
+ -- David Paleino da...@debian.org  Tue, 12 Oct 2010 22:09:37 +0200
+
+python-bottle (0.8.3-1) experimental; urgency=low
+
+  * New upstream version
+  * Upload to experimental, since we're in hard Squeeze freeze now.
+
+ -- David Paleino da...@debian.org  Sun, 05 Sep 2010 07:52:39 +0200
+
 python-bottle (0.8.2-1) unstable; urgency=low
 
   * New upstream version


signature.asc
Description: PGP signature


Bug#599613: unblock: mapnik/0.7.1-2

2010-10-09 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package mapnik

This version fixes #599330 , which renders the package unusable on mips{,-el}

unblock mapnik/0.7.1-2


Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Upload/unblock request for syslog-summary (was: Re: python transition incomplete)

2010-09-27 Thread David Paleino
Dear RT,

On Sun, 26 Sep 2010 23:53:23 +0200, Jakub Wilk wrote:
 syslog-summary 1.14-1
 #588524

I'm attaching the debdiff 1.14-1 → 1.14-2.
Is it ok to upload?

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff -u syslog-summary-1.14/debian/control syslog-summary-1.14/debian/control
--- syslog-summary-1.14/debian/control
+++ syslog-summary-1.14/debian/control
@@ -1,6 +1,5 @@
 Source: syslog-summary
-Maintainer: David Paleino d.pale...@gmail.com
-DM-Upload-Allowed: yes
+Maintainer: David Paleino da...@debian.org
 Section: admin
 Priority: optional
 Standards-Version: 3.8.3
diff -u syslog-summary-1.14/debian/changelog syslog-summary-1.14/debian/changelog
--- syslog-summary-1.14/debian/changelog
+++ syslog-summary-1.14/debian/changelog
@@ -1,3 +1,13 @@
+syslog-summary (1.14-2) unstable; urgency=low
+
+  * debian/control:
+- updated my email address
+- DMUA removed
+  * debian/patches/01-fix_shebang.patch added: use /usr/bin/python
+instead of hardcoding python2.5 (Closes: #588524)
+
+ -- David Paleino da...@debian.org  Mon, 27 Sep 2010 09:28:06 +0200
+
 syslog-summary (1.14-1) unstable; urgency=low
 
   * New upstream release:
diff -u syslog-summary-1.14/debian/patches/series syslog-summary-1.14/debian/patches/series
--- syslog-summary-1.14/debian/patches/series
+++ syslog-summary-1.14/debian/patches/series
@@ -1,0 +2 @@
+01-fix_shebang.patch
only in patch2:
unchanged:
--- syslog-summary-1.14.orig/debian/patches/01-fix_shebang.patch
+++ syslog-summary-1.14/debian/patches/01-fix_shebang.patch
@@ -0,0 +1,16 @@
+From: David Paleino da...@debian.org
+Subject: don't hardcode python2.5
+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=588524
+
+---
+ syslog-summary |2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- syslog-summary.orig/syslog-summary
 syslog-summary/syslog-summary
+@@ -1,4 +1,4 @@
+-#!/usr/bin/env python2.5
++#!/usr/bin/python
+ # -*- coding: utf-8 -*-
+ 
+ # Copyright © 2008-2009, David Paleino d.pale...@gmail.com


signature.asc
Description: PGP signature


Re: Upload/unblock request for syslog-summary (was: Re: python transition incomplete)

2010-09-27 Thread David Paleino
On Mon, 27 Sep 2010 20:01:15 +0100, Adam D. Barratt wrote:

 On Mon, 2010-09-27 at 09:36 +0200, David Paleino wrote:
  On Sun, 26 Sep 2010 23:53:23 +0200, Jakub Wilk wrote:
   syslog-summary 1.14-1
   #588524
  
  I'm attaching the debdiff 1.14-1 → 1.14-2.
  Is it ok to upload?
 
 debian/pyversions in the source package contains 2.5.

Oops, that slipped through. I'm going to use 2.6- in debian/pyversions.

May I upload with this additional change to the previous debdiff?

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Upload/unblock request for syslog-summary (was: Re: python transition incomplete)

2010-09-27 Thread David Paleino
On Mon, 27 Sep 2010 20:33:15 +0100, Adam D. Barratt wrote:

 On Mon, 2010-09-27 at 21:09 +0200, David Paleino wrote:
  On Mon, 27 Sep 2010 20:01:15 +0100, Adam D. Barratt wrote:
   debian/pyversions in the source package contains 2.5.
  
  Oops, that slipped through. I'm going to use 2.6- in debian/pyversions.
 
 If the package supports python 2.5 (which the current dependencies
 rather suggest it does) would 2.5- not be better?

Yes.

 Apologies if this is a silly question and I'm missing a reason not to do so.

Sorry for rushing a reply before thinking :)

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Pre-approval of python-liblas/1.2.1-2 upload

2010-09-12 Thread David Paleino
Hello RT,
I prepared an upload for python-liblas, version 1.2.1-2, targetted at Squeeze.
This upload fixes an RC bug (#595603), and another one which is
minor (#595608, but the fix is really easy, and they're really almost the
same bug).

The patch I want to upload is:

http://svn.debian.org/viewsvn/pkg-grass/packages/python-liblas/trunk/debian/patches/00-fix_library_search.patch?view=markup

I'll wait a reply before uploading to sid.

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Pre-approval of python-liblas/1.2.1-2 upload

2010-09-12 Thread David Paleino
On Sun, 12 Sep 2010 12:26:14 +0100, Adam D. Barratt wrote:

 On Sun, 2010-09-12 at 10:56 +0200, David Paleino wrote:
  I prepared an upload for python-liblas, version 1.2.1-2, targetted at
  Squeeze. This upload fixes an RC bug (#595603), and another one which is
  minor (#595608, but the fix is really easy, and they're really almost the
  same bug).
 
 Please go ahead, and let us know once the package has been accepted.

Accepted.

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#595669: unblock: osm2pgsql/0.69+r20104-2

2010-09-05 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Dear release team,
I uploaded osm2pgsql 0.69+r20104-2 to unstable today. This upload was ready
months ago already but, unfortunately, never uploaded.

It fixes RC bug #559604 . I've been pinged by Adam (CCed) a month ago but, due
to VAC (mine and of frankie), I haven't been able to make an upload for it,
until today.

Please unblock package osm2pgsql.

unblock osm2pgsql/0.69+r20104-2



Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#595669: unblock: osm2pgsql/0.69+r20104-2

2010-09-05 Thread David Paleino
On Sun, 5 Sep 2010 18:44:09 +0200, David Paleino wrote:

 Please unblock package osm2pgsql.

 unblock osm2pgsql/0.69+r20104-2

Sorry, forgot to attach the debdiff. Please find it attached.

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
diff -u osm2pgsql-0.69+r20104/debian/changelog osm2pgsql-0.69+r20104/debian/changelog
--- osm2pgsql-0.69+r20104/debian/changelog
+++ osm2pgsql-0.69+r20104/debian/changelog
@@ -1,3 +1,18 @@
+osm2pgsql (0.69+r20104-2) unstable; urgency=low
+
+  [ Francesco Paolo Lovergine ]
+  * Changed debian/rules to autodetect latest svn revision and upstream version. 
+  * Now recommends both postgis and last available postgresql-postgis revision.
+(closes: #559604)
+  * Fixed previous changelog for repeated logs.
+
+  [ David Paleino ]
+  * debian/control:
+- fix Vcs-* fields to point to trunk only.
+- added myself to Uploaders
+
+ -- David Paleino da...@debian.org  Sun, 05 Sep 2010 08:38:52 +0200
+
 osm2pgsql (0.69+r20104-1) unstable; urgency=low
 
   * Added 900913.sql, containing EPSG:900913 for postgis, as example. (Andreas)
@@ -7,10 +22,6 @@
   * Added Vcs-* fields to debian/control.
   * Policy bumped to 3.8.4 without changes.
   * Added a new set of examples (gazetteer and other scripts).
-  * New upstream svn snapshot as taken from upstream svn. Now using svn release
-number instead of date to identify better the relevant snapshot.
-  * Added Svn-* fields to debian/control.
-  * Policy bumped to 3.8.4 without changes.
 
  -- Francesco Paolo Lovergine fran...@debian.org  Mon, 22 Feb 2010 10:43:12 +0100
 
diff -u osm2pgsql-0.69+r20104/debian/control osm2pgsql-0.69+r20104/debian/control
--- osm2pgsql-0.69+r20104/debian/control
+++ osm2pgsql-0.69+r20104/debian/control
@@ -3,17 +3,18 @@
 Priority: optional
 Maintainer: Debian GIS Project pkg-grass-de...@lists.alioth.debian.org
 Uploaders: Andreas Putzo andr...@putzo.net, Francesco Paolo Lovergine fran...@debian.org
+ , David Paleino da...@debian.org
 DM-Upload-Allowed: yes
 Build-Depends: debhelper (= 7), dpatch, libbz2-dev, libgeos-dev, libpq-dev, libxml2-dev, libproj-dev
 Standards-Version: 3.8.4
 Homepage: http://wiki.openstreetmap.org/wiki/Osm2pgsql
-Vcs-Browser: http://svn.debian.org/wsvn/pkg-grass/packages/osm2pgsql/
-Vcs-Svn: svn://svn.debian.org/pkg-grass/packages/osm2pgsql/
+Vcs-Browser: http://svn.debian.org/wsvn/pkg-grass/packages/osm2pgsql/trunk/
+Vcs-Svn: svn://svn.debian.org/pkg-grass/packages/osm2pgsql/trunk/
 
 Package: osm2pgsql
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends}
-Recommends: postgresql-8.3-postgis
+Recommends: postgis, postgresql-8.4-postgis
 Suggests: josm, gosmore 
 Description: Openstreetmap data to PostgreSQL converter
  Convert OSM planet snapshot data to SQL suitable for loading
diff -u osm2pgsql-0.69+r20104/debian/rules osm2pgsql-0.69+r20104/debian/rules
--- osm2pgsql-0.69+r20104/debian/rules
+++ osm2pgsql-0.69+r20104/debian/rules
@@ -7,9 +7,9 @@
 include /usr/share/dpatch/dpatch.make
 
 SVNREPO := http://svn.openstreetmap.org/applications/utils/export/osm2pgsql
-SVNREV  := 20104
-SUFFIX  := +r20104
-VERSION := 0.69
+SVNREV  := $(shell svn info $(SVNREPO)|grep Revision:|cut -d' ' -f2)
+SUFFIX  := +r$(SVNREV)
+VERSION := $(shell dpkg-parsechangelog|grep Version|cut -d' ' -f2|cut -d+ -f1)
 
 CFLAGS = -Wall -g
 



signature.asc
Description: PGP signature


Bug#593843: unblock: gthumb/3:2.11.5-3

2010-08-21 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: freeze-exception

I uploaded gthumb 2.11.5-3 in sid, which fixes RC bug #593813 . The bug is
already fixed upstream (the version in experimental, 2.11.90), and I backported
the fix from there.

The patch is tiny, attaching it to this message.

It would be nice to have this in Squeeze. Thank you :)

unblock gthumb/3:2.11.5-3

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
Author: Paolo Bacchilega pao...@src.gnome.org
Description: fix photo comments loss when upgrading from 2.10
Bug: http://bugzilla.gnome.org/show_bug.cgi?id=626893
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=593813
Comment: Patch backported by David Paleino da...@debian.org, using the
 following git commits:
 - http://git.gnome.org/browse/gthumb/commit/?id=b80580d3341d2b050bf59fdc42973ed28e93bc54
 - http://git.gnome.org/browse/gthumb/commit/?id=08fcb249ec7e6f2daaf99f50ece6612cfa5a272c
 - http://git.gnome.org/browse/gthumb/commit/?id=66bc3971668f06970594c55c777b5f628882c4e9

---
 extensions/exiv2_tools/exiv2-utils.cpp   |   99 +--
 extensions/exiv2_tools/gth-metadata-provider-exiv2.c |   23 +++-
 extensions/exiv2_tools/main.c|   14 ++
 po/POTFILES.skip |2 
 4 files changed, 107 insertions(+), 31 deletions(-)

--- gthumb.orig/extensions/exiv2_tools/exiv2-utils.cpp
+++ gthumb/extensions/exiv2_tools/exiv2-utils.cpp
@@ -57,6 +57,14 @@ const char *_DATE_TAG_NAMES[] = {
 	NULL
 };
 
+const char *_LAST_DATE_TAG_NAMES[] = {
+	Exif::Image::DateTime,
+	Xmp::exif::DateTime,
+	Xmp::xmp::ModifyDate,
+	Xmp::xmp::MetadataDate,
+	NULL
+};
+
 const char *_ORIGINAL_DATE_TAG_NAMES[] = {
 	Exif::Photo::DateTimeOriginal,
 	Xmp::exif::DateTimeOriginal,
@@ -132,11 +140,11 @@ const char *_ORIENTATION_TAG_NAMES[] = {
 };
 
 const char *_DESCRIPTION_TAG_NAMES[] = {
+	Iptc::Application2::Caption,
+	Xmp::dc::description,
 	Exif::Photo::UserComment,
 	Exif::Image::ImageDescription,
-	Xmp::dc::description,
 	Xmp::tiff::ImageDescription,
-	Iptc::Application2::Caption,
 	Iptc::Application2::Headline,
 	NULL
 };
@@ -263,24 +271,15 @@ set_file_info (GFileInfo  *info,
 
 
 static void
-set_attribute_from_tagset (GFileInfo  *info,
-			   const char *attribute,
-			   const char *tagset[])
-{
-	GObject *metadata;
-	int  i;
-	char*key;
-	char*description;
-	char*formatted_value;
-	char*raw_value;
-	char*type_name;
-
-	metadata = NULL;
-	for (i = 0; tagset[i] != NULL; i++) {
-		metadata = g_file_info_get_attribute_object (info, tagset[i]);
-		if (metadata != NULL)
-			break;
-	}
+set_attribute_from_metadata (GFileInfo  *info,
+			 const char *attribute,
+			 GObject*metadata)
+{
+	char *key;
+	char *description;
+	char *formatted_value;
+	char *raw_value;
+	char *type_name;
 
 	if (metadata == NULL)
 		return;
@@ -292,6 +291,7 @@ set_attribute_from_tagset (GFileInfo  *i
 		  raw, raw_value,
 		  value-type, type_name,
 		  NULL);
+
 	set_file_info (info,
 		   attribute,
 		   description,
@@ -303,6 +303,26 @@ set_attribute_from_tagset (GFileInfo  *i
 
 
 static void
+set_attribute_from_tagset (GFileInfo  *info,
+			   const char *attribute,
+			   const char *tagset[])
+{
+	GObject *metadata;
+	int  i;
+
+	metadata = NULL;
+	for (i = 0; tagset[i] != NULL; i++) {
+		metadata = g_file_info_get_attribute_object (info, tagset[i]);
+		if (metadata != NULL)
+			break;
+	}
+
+	if (metadata != NULL)
+		set_attribute_from_metadata (info, attribute, metadata);
+}
+
+
+static void
 set_string_list_attribute_from_tagset (GFileInfo  *info,
    const char *attribute,
    const char *tagset[])
@@ -336,9 +356,32 @@ set_string_list_attribute_from_tagset (G
 static void
 set_attributes_from_tagsets (GFileInfo *info)
 {
-	set_attribute_from_tagset (info, general::datetime, _DATE_TAG_NAMES);
+	set_attribute_from_tagset (info, general::datetime, _LAST_DATE_TAG_NAMES);
+	if (g_file_info_get_attribute_object (info, general::datetime) == NULL)
+		set_attribute_from_tagset (info, general::datetime, _ORIGINAL_DATE_TAG_NAMES);
+
 	set_attribute_from_tagset (info, general::description, _DESCRIPTION_TAG_NAMES);
 	set_attribute_from_tagset (info, general::title, _TITLE_TAG_NAMES);
+
+	/* if iptc::caption and iptc::headline are different use iptc::headline
+	 * to set general::title, if not already set. */
+
+	if (g_file_info_get_attribute_object (info, general::title) == NULL) {
+		GObject *iptc_caption;
+		GObject *iptc_headline;
+
+		iptc_caption = g_file_info_get_attribute_object (info, Iptc::Application2::Caption);
+		iptc_headline = g_file_info_get_attribute_object (info, Iptc::Application2::Headline);
+
+		if ((iptc_caption != NULL

Bug#593843: unblock: gthumb/3:2.11.5-3

2010-08-21 Thread David Paleino
On Sat, 21 Aug 2010 17:00:39 +0100, Adam D. Barratt wrote:

 On Sat, 2010-08-21 at 16:29 +0200, David Paleino wrote:
  I uploaded gthumb 2.11.5-3 in sid, which fixes RC bug #593813 . The bug is
  already fixed upstream (the version in experimental, 2.11.90), and I
  backported the fix from there.
  
  The patch is tiny, attaching it to this message.
 
  const char *_DESCRIPTION_TAG_NAMES[] = {
 +   Iptc::Application2::Caption,
 +   Xmp::dc::description,
 Exif::Photo::UserComment,
 Exif::Image::ImageDescription,
 -   Xmp::dc::description,
 Xmp::tiff::ImageDescription,
 -   Iptc::Application2::Caption,
 
 I'm hoping those aren't exported anywhere near gthumb-dev :-)

Ehm, no :)

Thank you for looking,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#584430: transition: osm-gps-map

2010-06-10 Thread David Paleino
On Thu, 3 Jun 2010 21:31:41 +0200, David Paleino wrote:

 Hello Adam,
 
 On Thu, 03 Jun 2010 18:46:24 +0100, Adam D. Barratt wrote:
 
  The package does actually have one reverse dependency - gpxviewer, which
  uses the Python bindings.
 
 Oops, I missed to check those. Yes, you're right. I'm CCing the gpxviewer
 maintainer.
 
  Are there any changes which are likely to affect the bindings?
 
 Yes, there are, since those are direct bindings to the .so library. Upstream
 provided new functions, and old ones are deprecated, but they're using new
 ones for compatibility.
 
 The only thing happening is some more output on the console, with the
 deprecation warnings. Otherwise, gpxviewer works fine.
 
 [..]

Any news for this transition?

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#584430: transition: osm-gps-map

2010-06-03 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: transition

Hello RT,
osm-gps-map upstream just released 0.7.0, and bumped LT_VERSION_INFO
from 1:0:1 to 2:0:0. This caused a change in SONAME, from libosmgpsmap.so.0 to
libosmgpsmap.so.2.

There are no reverse dependencies, am I allowed to upload libosmgpsmap2?

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#584430: transition: osm-gps-map

2010-06-03 Thread David Paleino
Hello Adam,

On Thu, 03 Jun 2010 18:46:24 +0100, Adam D. Barratt wrote:

 On Thu, 2010-06-03 at 16:43 +0200, David Paleino wrote:
  osm-gps-map upstream just released 0.7.0, and bumped LT_VERSION_INFO
  from 1:0:1 to 2:0:0. This caused a change in SONAME, from libosmgpsmap.so.0
  to libosmgpsmap.so.2.
  
  There are no reverse dependencies, am I allowed to upload libosmgpsmap2?
 
 The package does actually have one reverse dependency - gpxviewer, which
 uses the Python bindings.

Oops, I missed to check those. Yes, you're right. I'm CCing the gpxviewer
maintainer.

 Are there any changes which are likely to affect the bindings?

Yes, there are, since those are direct bindings to the .so library. Upstream
provided new functions, and old ones are deprecated, but they're using new ones
for compatibility.

The only thing happening is some more output on the console, with the
deprecation warnings. Otherwise, gpxviewer works fine.

$ gpxviewer 

(gpxviewer:3960): OsmGpsMap-WARNING **: osm_gps_map_set_mapcenter is deprecated

(gpxviewer:3960): OsmGpsMap-WARNING **: osm_gps_map_clear_tracks is deprecated

(gpxviewer:3960): OsmGpsMap-WARNING **: osm_gps_map_add_track is deprecated

(gpxviewer:3960): OsmGpsMap-WARNING **: osm_gps_map_add_track is deprecated

(gpxviewer:3960): OsmGpsMap-WARNING **: osm_gps_map_add_track is deprecated


Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: library tranistion proposal exiv2(0.20): libexiv2-6 - libexiv2-9

2010-06-01 Thread David Paleino
On Tue, 1 Jun 2010 07:53:23 +1000, Mark Purcell wrote:

 debian-release,
 
 The following transition is proposed:
 
 exiv2(0.20): libexiv2-6 - libexiv2-9
 
 Important rdepends:
 
 [..]
 +  gthumb

Unfortunately, I won't have a decent connection to download all its
Build-Depends until friday night, so I can't really test if it builds right
now.

David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: library tranistion proposal exiv2(0.20): libexiv2-6 - libexiv2-9

2010-06-01 Thread David Paleino
On Tue, 1 Jun 2010 08:02:29 +0200, David Paleino wrote:

 On Tue, 1 Jun 2010 07:53:23 +1000, Mark Purcell wrote:
 
  debian-release,
  
  The following transition is proposed:
  
  exiv2(0.20): libexiv2-6 - libexiv2-9
  
  Important rdepends:
  
  [..]
  +  gthumb
 
 Unfortunately, I won't have a decent connection to download all its
 Build-Depends until friday night, so I can't really test if it builds right
 now.

Ok, I got lucky and could download everything and build gthumb. It built fine,
and seems to work fine as well.

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Bug#573974: RM: josm-plugins -- ROM; prevents josm from migrating; doesn't work with sid's josm

2010-03-17 Thread David Paleino
reassign 573974 release.debian.org
thanks

Hello,
moving the bug to release.debian.org, sorry :)

(the RM is from testing)

On Monday 15 March 2010 13:38:33, David Paleino wrote:
 Hello,
 as can be seen in #571140, josm-plugins doesn't work anymore with the
 version of JOSM in Sid. This is because the two packages are closely
 tied, and josm-plugins works only for a specific version of josm. This is
 why we usually make coordinate uploads of both.
 This time, with JOSM 2561, it happened that we couldn't upload josm-plugins
 in  a timely manner, because of licensing issues to be solved upstream.
 This is also preventing josm from migrating to testing
 
 We'll keep the RC-buggy package in sid for the moment, in the hope we can
 make  a new upload ASAP.
 
 
 Thank you,
 David
 (I'm not in Uploaders because the last upload of josm-plugins happened when
  I  wasn't listed there yet)

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 |
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: This is a digitally signed message part.


Bug#570763: transition: mapnik

2010-02-21 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: transition

Hello,
I prepared a new upstream release of Mapnik, which has a SONAME bump. 
Therefore, I had to rename the package from libmapnik0.6 to libmapnik0.7.

The only package (Build-)Depending on it is gpsdrive, and it's maintained by 
the same team libmapnik is under (DebianGIS, CCed). It builds and runs 
correctly with libmapnik0.7.

Am I allowed to upload it to NEW?

Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 |
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: This is a digitally signed message part.


Bug#570763: nmu: gpsdrive_2.10~pre4-6.dfsg-4

2010-02-21 Thread David Paleino
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: binnmu

Hello,

nmu gpsdrive_2.10~pre4-6.dfsg-4 . ALL . -m rebuild against libmapnik0.7

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 |
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: This is a digitally signed message part.


Bug#570859: nmu: gpsdrive_2.10~pre4-6.dfsg-4

2010-02-21 Thread David Paleino
On Sunday 21 February 2010 23:03:29, David Paleino wrote:
 Package: release.debian.org
 Severity: normal
 User: release.debian@packages.debian.org
 Usertags: binnmu
 
 Hello,
 
 nmu gpsdrive_2.10~pre4-6.dfsg-4 . ALL . -m rebuild against libmapnik0.7

Please, also:

dw gpsdrive_2.10~pre4-6.dfsg-4 . ALL . -m libmapnik0.7

since mapnik currently FTBFS for various reasons on different arches.

Thank you,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 |
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: This is a digitally signed message part.


nmu: gthumb 3:2.10.11-3 (was: Re: Bug#565350: image that crashes gthumb with a segfault)

2010-01-15 Thread David Paleino
tags 565350 =
retitle 565350 nmu: gthumb 3:2.10.11-3
reassign 565350 release.debian.org

user release.debian@packages.debian.org
usertag 565350 + binnmu

Hello release team,

On Friday 15 January 2010 10:11:57, David Paleino wrote:
 On Friday 15 January 2010 01:04:27, Joey Hess wrote:
  Package: gthumb
  Version: 3:2.10.11-3
  Severity: normal
 
  j...@gnu:~/lib/raw/fonefoto/latestgthumb CIMG0024.jpg
  [..]
  gth_minimal_exif_tag_read: TAG value does not fit, Can't read full value
  zsh: segmentation fault  gthumb CIMG0024.jpg
 
  The picture is attached. It was taken with a Palm Pre, and
  my other similar pictures do not cause this.
 
 I can confirm this bug happens. However, just rebuilding the package seems
  to be fixing it, i.e. I was able to see the image without it segfaulting.
  I'll do some more tests to be sure a binNMU would suffice.

I confirmed this behaviour, tested in a sid chroot. The version in the 
repository segfaults with the image Joey attached to the original report, 
while a recompiled one doesn't. I suppose this could be related to libexif's 
#557137 , but I'm not really sure.

Please schedule a binNMU for gthumb on all architectures.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 |
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: This is a digitally signed message part.


s-p-u upload permission for gthumb

2009-06-20 Thread David Paleino
Hello,
yesterday a user reported #533641 (gthumb) in Lenny. It's not a major bug, but
it's still a bug, and the fix is really a one-line patch:

  
http://git.debian.org/?p=collab-maint/gthumb.git;a=blob;f=debian/patches/10-dont_follow_symlinks_on_dups_search.patch;hb=lenny

Do I have permission to upload it to s-p-u? It would be nice to have it in
5.0.2 :)

I'll eventually patch the version in Sid as well.

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: RFC: gThumb upload for Lenny

2009-03-07 Thread David Paleino
On Sat, 7 Mar 2009 11:18:50 +0100, Adeodato Simó wrote:

 * David Paleino [Thu, 26 Feb 2009 14:52:02 +0100]:
 
  On Thu, 26 Feb 2009 09:31:07 +0100, David Paleino wrote:
 
   On Thu, 26 Feb 2009 00:35:08 +0100, Bart Martens wrote:
 
Hello Luk,
 
I suggest that David follows this approach for getting the critical bug
fixed in stable :
 
1.  file a temporary serious bug against gthumb 3:2.10.10-2
to prevent it from entering testing,
 
   There's no need for such a bug: why would we want to prevent it from
   entering Squeeze? Once in testing, I can upload to s-p-u without any
   problem -- the upload to t-p-u would just happen to makes things faster.
 
  Obviously, if RMs are ok with an upload to t-p-u, I'll do it that way, no
  need to pointlessly delay a fix.
 
 Hello,

Hello dato,

 not sure if this still needs reply from us. In case it wasn't
 mentioned, I think an upload to p-u will work now, since the checks have
 been relaxed by ftpmaster at our request.

The previous upload I made was moved by luk into the pending
queue:

http://release.debian.org/proposed-updates/stable.html
http://release.debian.org/proposed-updates/stable_diffs/gthumb_2.10.8-1+lenny1.debdiff

In the meanwhile, gthumb 3:2.10.10-2 (containing the fix backported to the
Lenny package) migrated to testing, so the t-p-u upload isn't needed anymore.

 The only caveat is that the fixed/new package will have to be in testing

It already is, see above.

 by the time we want to release 5.0.1 (i.e., the check is delayed until
 point release time).

I believe we're done then. The fix is in testing already (mixed with a new
upstream version), and a package containing the important fixes only is already
in p-u pending (see link above too)

Thank you for your interest,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: library transition proposal exiv2/ libexiv2-5

2009-03-07 Thread David Paleino
On Sat, 7 Mar 2009 13:35:29 +1100, Mark Purcell wrote:

 Hi,

Hello Mark,

 I propose a transition with libexiv2-5 (already in experimental) source 
 package exiv2. Which is a new upstream release from unstable.
 
 Could be large rdepends;
 
 libexiv2-4
 Reverse Depends:
   [..]
   gthumb

This is only in gthumb/experimental (sid's one depends on libexif12).

I'll test-build it with libexiv2-5/experimental, and report back to you.

Thank you,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: library transition proposal exiv2/ libexiv2-5

2009-03-07 Thread David Paleino
On Sat, 7 Mar 2009 12:30:22 +0100, David Paleino wrote:

 On Sat, 7 Mar 2009 13:35:29 +1100, Mark Purcell wrote:
 
  Hi,
 
 Hello Mark,
 
  I propose a transition with libexiv2-5 (already in experimental) source 
  package exiv2. Which is a new upstream release from unstable.
  
  Could be large rdepends;
  
  libexiv2-4
  Reverse Depends:
[..]
gthumb
 
 This is only in gthumb/experimental (sid's one depends on libexif12).
 
 I'll test-build it with libexiv2-5/experimental, and report back to you.

It miserably failed :/

gth-exiv2-utils.cpp: In function 'void write_metadata(const char*, const char*,
GList*)':
gth-exiv2-utils.cpp:620: error: 'class Exiv2::ExifData' has no member named
'eraseThumbnail'

The line in question is:

image1-exifData().eraseThumbnail();

Where image1 is:

Exiv2::Image::AutoPtr image1 = Exiv2::ImageFactory::open (from_file);

I'm going to read the docs on how to patch that -- but I was already thinking
at RMing gthumb from experimental nevertheless... if it's a simple patch, I
might do it and update the package, otherwise I'll ask for its removal.

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Please give back gthumb

2009-03-01 Thread David Paleino
Hello,
would you please give back gthumb on amd64, hppa and ia64? The build failed for
(seems) temporary reasons.

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: RFC: gThumb upload for Lenny

2009-02-26 Thread David Paleino
On Thu, 26 Feb 2009 00:35:08 +0100, Bart Martens wrote:

 Hello Luk,
 
 I suggest that David follows this approach for getting the critical bug
 fixed in stable :
 
 1.  file a temporary serious bug against gthumb 3:2.10.10-2
 to prevent it from entering testing,

There's no need for such a bug: why would we want to prevent it from entering
Squeeze? Once in testing, I can upload to s-p-u without any problem -- the
upload to t-p-u would just happen to makes things faster.

Ciao,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: RFC: gThumb upload for Lenny

2009-02-26 Thread David Paleino
On Thu, 26 Feb 2009 09:31:07 +0100, David Paleino wrote:

 On Thu, 26 Feb 2009 00:35:08 +0100, Bart Martens wrote:
 
  Hello Luk,
  
  I suggest that David follows this approach for getting the critical bug
  fixed in stable :
  
  1.  file a temporary serious bug against gthumb 3:2.10.10-2
  to prevent it from entering testing,
 
 There's no need for such a bug: why would we want to prevent it from entering
 Squeeze? Once in testing, I can upload to s-p-u without any problem -- the
 upload to t-p-u would just happen to makes things faster.

Obviously, if RMs are ok with an upload to t-p-u, I'll do it that way, no need
to pointlessly delay a fix.

David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: RFC: gThumb upload for Lenny

2009-02-22 Thread David Paleino
On Sun, 22 Feb 2009 16:03:22 +0100, Luk Claes wrote:

 David Paleino wrote:
  On Mon, 16 Feb 2009 20:02:48 +0100, David Paleino wrote:
  
  Hello,
  bug #420195, gthumb: Drag and Drop from Folders Pane Causes Copying Loop,
  was reported more than a year ago. Unfortunately, I had a look at it only
  very recently, on Feb, 12. I underestimated it, leaving it at priority
  normal, while it's a critical bug -- sorry for having missed Lenny in
  a so clamorous way :/.
 
  [..]
 
  I managed to make a patch (attached), [..]
 
  After an ACK from RM, I'll prepare an upload of gthumb 3:2.10.8-1~lenny1
  for proposed-updates, and report the debdiff here before effectively
  uploading.
 
 I guess you should use a version number higher than the one currently in
 stable,

Oops, yes, that ~ should have been a +:

$ dpkg --compare-versions 3:2.10.8-1 lt 3:2.10.8-1~lenny1  echo OK
$ dpkg --compare-versions 3:2.10.8-1 lt 3:2.10.8-1+lenny1  echo OK
OK

Thank you for making me notice!

 but please do upload and add an entry on [1].

Ok, will do ASAP.

  By the way, should I upload a -2 to sid before going proposed-updates?
 
 Done in the meantime

Yes, but it's a different upstream version too. :-)

 and it's indeed required atm.

Ok, I've also read your other reply ;-)

 Cheers
 
 Luk

Thank you!

Ciao,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: RFC: gThumb upload for Lenny

2009-02-22 Thread David Paleino
Hello Luk,

On Sun, 22 Feb 2009 16:03:22 +0100, Luk Claes wrote:

 David Paleino wrote:
 [..]

 I guess you should use a version number higher than the one currently in
 stable, but please do upload and add an entry on [1].

I did, but I got a REJECT:

Mapping stable-proposed-updates to proposed-updates.
Rejected: gthumb-data_2.10.8-1+lenny1_all.deb: old version (3:2.10.8-1) in 
testing = new version (3:2.10.8-1+lenny1) targeted at proposed-updates.
Rejected: gthumb_2.10.8-1+lenny1_i386.deb: old version (3:2.10.8-1) in testing 
= new version (3:2.10.8-1+lenny1) targeted at proposed-updates.
Rejected: gthumb_2.10.8-1+lenny1.dsc: old version (3:2.10.8-1) in testing = 
new version (3:2.10.8-1+lenny1) targeted at proposed-updates.

  By the way, should I upload a -2 to sid before going proposed-updates?
 
 Done in the meantime and it's indeed required atm.

Obviously, this means I should wait for gthumb 2.10.10-1 to migrate to
Squeeze? Isn't this a bit broken? :)

Ciao,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: RFC: gThumb upload for Lenny

2009-02-19 Thread David Paleino
On Mon, 16 Feb 2009 20:02:48 +0100, David Paleino wrote:

 Hello,
 bug #420195, gthumb: Drag and Drop from Folders Pane Causes Copying Loop,
 was reported more than a year ago. Unfortunately, I had a look at it only very
 recently, on Feb, 12. I underestimated it, leaving it at priority normal,
 while it's a critical bug -- sorry for having missed Lenny in a so clamorous
 way :/.
 
 [..]
 
 I managed to make a patch (attached), [..]
 
 After an ACK from RM, I'll prepare an upload of gthumb 3:2.10.8-1~lenny1 for
 proposed-updates, and report the debdiff here before effectively uploading.

Ping.

By the way, should I upload a -2 to sid before going proposed-updates?

Ciao,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: very late freeze exception for texlive-extra 2007.dfsg.16-1

2009-02-17 Thread David Paleino
Hello Norbert,

On Tue, 17 Feb 2009 16:43:46 +0100, Norbert Preining wrote:

 On Fr, 13 Feb 2009, Luk Claes wrote:
   Ok ... here is another one but I will not be able to upload a fix for it
   before Monday (16.) morning. (Bug #515086)
   
   Anything we can do here?
  
  Will need to go to proposed-updates (next to unstable) after the release
  (so Monday would be fine).
 
 Anything I have to make sure while uploading? Should the changelog be
 changed in a certain way, specific version numbering?

If the version in stable is 2007.dfsg.16-1, 2007.dfsg.16-1+lenny1 should be
fine. Everything _lower_ than any version in squeeze/sid (to ease upgrades) but
_higher_ than the current version in lenny.

You may want to check --compare-versions:

$ dpkg --compare-versions 2007.dfsg.16-1 lt 2007.dfsg.16-1+lenny1
$ echo $?
0
$

Ciao,
David :)

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: very late freeze exception for texlive-extra 2007.dfsg.16-1

2009-02-17 Thread David Paleino
On Tue, 17 Feb 2009 17:23:10 +0100, Norbert Preining wrote:

 Hi David,

Ciao Norbert,

 On Di, 17 Feb 2009, David Paleino wrote:
  fine. Everything _lower_ than any version in squeeze/sid (to ease upgrades)
  but _higher_ than the current version in lenny.
 
 But in sid = lenny atm, and we will probably only do lenny stuff on the
 2007 packages anyway ...

You should do separate uploads nevertheless, if I correctly understood the
proposed-updates mechanism.

And this also applies if you want to make the same changes in sid and in lenny
-- you should do two uploads (version-1+lenny1 and version-2)

 It was more about the distribution line, whether it should be unstable
 or proposed-updated or something else.

If it's meant to be uploaded in lenny, stable-proposed-updates (or just
proposed-updates) is the way to go. And if you want to make the same changes
in sid, you should upload a -2 there, separately. :)

Then again, I might be wrong, and RMs will tell us.
I hope I got everything right though, someone (*cough*NM*cough*) might be
watching me. ;)

Ciao,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Draft for lenny release announcement

2009-02-09 Thread David Paleino
On Mon, 9 Feb 2009 13:46:55 +0200, Riku Voipio wrote:

 On Mon, Feb 09, 2009 at 11:15:23AM +0100, Alexander Reichle-Schmehl wrote:
  pDebian GNU/Linux runs on computers ranging from palmtops and handheld
  systems to supercomputers, and on nearly everything in between.  A total
  of eleven architectures are supported including:  Sun SPARC (sparc), HP
  Alpha (alpha), Motorola/IBM PowerPC (powerpc), Intel IA-32 (i386) and
  IA-64 (ia64), HP PA-RISC (hppa), MIPS (mips, mipsel), ARM (arm), IBM
  ^^^ armel
  S/390 (s390) and AMD64 and Intel EM64T (amd64)./p
 
 p Debian GNU/Linux 5.0 includes the ARM EABI port, armel. The
 new port is provides more efficient usage of modern and future ARM
 processors. The old ARM port (arm) is now depreceated./p
^^^
s/depreceated/deprecated/

But... has it really been deprecated? Didn't know that :P

David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Draft for lenny release announcement

2009-02-09 Thread David Paleino
On Mon, 09 Feb 2009 14:41:12 +0100, Daniel Baumann wrote:

 W. Martin Borgert wrote:
  Maybe I overlooked it, but I suggest to mention:
 
  - Debian Live 
 
 jup. therefore, please include the following paragraph below the
 pDebian GNU/Linux can be installed [...] paragraph:
 
 pIn addition to the regular installation media, Debian GNU/Linux can
 now also be directly used without prior installation. These special
 images are also known as live images and are availabe for CDs, USB
 sticks and different forms of network setups. Initially, these are
 provided for the amd64 and i386 architecture only./p
 ^^
Typo: architectures, it's plural :)

David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Freeze exception for packeth

2009-01-31 Thread David Paleino
On Sat, 31 Jan 2009 16:48:28 +0100, Adeodato Simó wrote:

 * David Paleino [Fri, 30 Jan 2009 11:05:13 +0100]:
 
  packeth (1.6-2) unstable; urgency=low
 [..]
  Please unblock it for Lenny.
 
 Okay, unblocked.

Thank you,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Freeze exception for macchanger

2009-01-31 Thread David Paleino
On Sat, 31 Jan 2009 17:15:40 +0100, Adeodato Simó wrote:

 * David Paleino [Fri, 30 Jan 2009 11:09:06 +0100]:
 
  Hello,
  I've uploaded macchanger 1.5.0-3 to sid, while Lenny has 1.5.0-2. Here's the
  changelog, debdiff attached:
 
 The diff contains this:
 
 --- macchanger-1.5.0/debian/copyright
 +++ macchanger-1.5.0/debian/copyright
 @@ -1,8 +1,4 @@
 -Format-Specification: http://wiki.debian.org/Proposals/CopyrightFormat
 -Upstream-Author: Alvaro Lopez Ortega alv...@alobbs.com
 -Packaged-By: Domenico Andreoli ca...@debian.org
 -Packaged-Date: Sun, 24 Aug 2003 17:18:37 +0200
 -Original-Source-Location: http://www.alobbs.com/macchanger/
 +Source: http://www.alobbs.com/macchanger
  
 I think dropping Upstream-Author is a loss.

Uhm, /me too, and can't really tell why that happened. I usually leave at least
Format-Specification too. :/

 [..]
 The diff also contains this:
 
 --- macchanger-1.5.0/debian/changelog
 +++ macchanger-1.5.0/debian/changelog
 @@ -16,7 +27,7 @@
warning
  - 02-fix_usage_message.patch added, fixes usage message (Closes: #391400)
  
 - -- David Paleino d.pale...@gmail.com  Fri, 06 Jun 2008 18:55:58 +0200
 + -- David Paleino d.pale...@gmail.com  Fri, 06 Jun 2008 17:08:41 +0200
 
 I'm sensing this is going to be due to some VCS vs. uploaded version
 war: please try that they don't happen as a norm.

ACK.

 In any case, unblocked.

Thank you,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Freeze exception for packeth

2009-01-30 Thread David Paleino
Hello,
I've uploaded packeth 1.6-2 to sid, while Lenny has 1.6-1. Here's the
changelog, debdiff attached:

packeth (1.6-2) unstable; urgency=low

  * [67fe44f] Packaging converted to git: fixed Vcs-* urls accordingly
  * [dc4ed8d] Adding .desktop file
  * [a2b61db] Added menu file (Closes: #501384)
  * [dd20a8f] debian/rules updated
  * [e999125] debian/changelog updated
  * [bc31532] Added descriptions to the patches
  * [3a7c40c] Standards-Version bump to 3.8.0, debian/README.source
added.
  * [2a748ea] Fixed menu section

 -- David Paleino d.pale...@gmail.com  Wed, 31 Dec 2008 16:03:20 +0100

Please unblock it for Lenny.

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


packeth_1.6-1_1.6-2.diff.bz2
Description: application/bzip


signature.asc
Description: PGP signature


Freeze exception for macchanger

2009-01-30 Thread David Paleino
Hello,
I've uploaded macchanger 1.5.0-3 to sid, while Lenny has 1.5.0-2. Here's the
changelog, debdiff attached:

macchanger (1.5.0-3) unstable; urgency=low

  * debian/patches/:
- 01-fix_manpage.patch updated, added -s|--show option.
  Thanks to Geoff Simmons (Closes: #506922)
  * debian/control:
- DM-Upload-Allowed set.
  * debian/watch added

 -- David Paleino d.pale...@gmail.com  Wed, 31 Dec 2008 11:30:48 +0100


Please unblock it for Lenny.

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


macchanger_1.5.0-2_1.5.0-3.diff.bz2
Description: application/bzip


signature.asc
Description: PGP signature


Please unblock libnet

2008-12-23 Thread David Paleino
Hello,
please unblock libnet:

$ grep-excuses libnet
libnet (1.1.2.1-2 to 1.1.2.1-4)
Maintainer: David Paleino 
94 days old (needed 10 days)
Not touching package, as requested by freeze (contact debian-release if
update is needed)
Not considered
$

There are many cosmetic changes since -2 (in -3 I adopted it), but most
importantly, in -3 I updated debian/copyright which was missing lots of
information, and fixed some bugs present in -2.

(in -4 I just reverted a patch which caused bigger issues)

Thank you,
David Paleino

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Please unblock cdebconf

2008-12-19 Thread David Paleino
Hello,
I'm not the maintainer, but I'm asking unblocking of cdebconf nevertheless:

http://qa.debian.org/excuses.php?package=cdebconf

Excuse for cdebconf

* 18 days old (needed 10 days)
* Not touching package, as requested by freeze (contact debian-release if
  update is needed)
* Updating cdebconf fixes old bugs: #502244
* Not considered 

Kindly,
David Paleino

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: Bug#504875: Unblock suggestion: libggi

2008-12-14 Thread David Paleino
On Thu, 11 Dec 2008 10:38:30 -0800 (PST), Asheesh Laroia wrote:

 Howdy Debian Releasers,
 
 I was examining the remaining Lenny RC bugs and found 
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=504875 via 
 http://bts.turmzimmer.net/details.php?bydist=bothsortby=packagesfullcomment=on
  
 .  This is an RC bug against libggi2-dev (in src:libggi).
 
 This issue is fixed in unstable, and the fix is a tiny patch, and the -3 
 release that is in sid corrects only that RC bug and the maintainer's 
 email address.

The fix is in unstable since Nov 08. Is there any plan unblocking libggi?

 I believe that it should get unblocked and go into Lenny.  I'm probably 
 not authorized to formally request it, since I'm neither the maintainer 
 nor the Release Team.

The same applies to me, but I'd like to see another RC bug gone.

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


gthumb/experimental needs rebuild

2008-11-22 Thread David Paleino
Hello,
would you please schedule a rebuild of gthumb (the version in experimental)?
It currently depends on libexiv2-2, while now the archive has libexiv2-4 and
libexiv2-5 (thus, gthumb is not installable)

Kindly,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


binNMUs for pinot

2008-04-17 Thread David Paleino
Hi,
please schedule binNMUs for pinot 0.84-1 on alpha, amd64, arm, armel, hppa,
i386, ia64, mips, mipsel, powerpc, s390, sparc [1], because of the libgmime ABI
change without SONAME bump.

Thanks,
David

[1] this is the output of rmadison pinot

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


Re: pending libgmime transition

2008-04-16 Thread David Paleino
2008/4/16, Stephen Gran [EMAIL PROTECTED]:
 This one time, at band camp, Stephen Gran said:
  libgmime had an ABI change (it changed sizeof(off_t)) without a soname
  bump.  As a short term work around, I've NMUed it with a package rename.
  This means it will hit NEW, but when it comes out and is autobuilt,
  we'll need a small binNMU campaign for the following packages:
 
  tracker
  pinot
  ...

 This one time, at band camp, David Paleino said:
  I currently maintain pinot. Please feel free to write me privately when the 
  new
  libgmime completes NEW. [1]

 David - it's time.

I'm sorry I can't help right now. I'll probably have an Internet
connection on Saturday (I'm writing from an University terminal).
Please feel free to binNMU pinot, and sorry for the noise (I believed
libgmime would have taken some more time).

Thanks,
David

-- 
 . ''`.  Debian packager! | http://snipurl.com/gofoxygo/
 : :'  :   User #334216   |  http://www.hanskalabs.net/
 `. `'`   GPG: 1392B174   | http://www.debianizzati.org/
  `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: pending libgmime transition

2008-04-06 Thread David Paleino
On Sun, 6 Apr 2008 13:29:22 +0100, Stephen Gran wrote:

 Hi all,

Hi,

 ...
 pinot
 ...

I currently maintain pinot. Please feel free to write me privately when the new
libgmime completes NEW. [1]

Kindly,
David

[1] or, at least, give a = 7d delay for the NMU :)

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature


  1   2   >