Bug#733645: morse2ascii: Please move $(LIBS) to the end of the $(CC) call

2013-12-30 Thread Michael Bienia
Package: morse2ascii
Version: 0.2+dfsg-1
Severity: wishlist

Hello,

could you please move the $(LIBS) placement in the $(CC) call to the
end? This allows morse2ascii to build with ld --as-needed (which is
default on Ubuntu) as the order matters there (the used libaries must
appear after the object files using them).

Simply modify debian/patches/makefile.patch from
+   $(CC) $(CPPFLAGS) $(CFLAGS) $(LDFLAGS) -o $(EXE) $(LIBS) morse2ascii.o 
resample2.o
to
+   $(CC) $(CPPFLAGS) $(CFLAGS) $(LDFLAGS) -o $(EXE) morse2ascii.o 
resample2.o $(LIBS)


Thanks,
Michael


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#733650: readseq: Buffer overflow in ureadseq.c

2013-12-30 Thread Michael Bienia
Package: readseq
Version: 1-10
Severity: normal
Tags: patch

Hello,

looking at why readseq failed to build in Ubuntu trusty, gcc found a
buffer overflow in ureadseq.c:

In function 'strcpy',
inlined from 'writeSeq' at ureadseq.c:1925:13:
/usr/include/x86_64-linux-gnu/bits/string3.h:104:3: warning: call to 
__builtin___memcpy_chk will always overflow destination buffer [enabled by 
default]

which then got triggered by the test suite:

Test of NCBI ASN.1 conversions:
./readseq -p -f=asn test.m-ig  -otest.m-asn
*** buffer overflow detected ***: ./readseq terminated

Looking at ureadseq.c:writeSeq() it's pretty obvious that
  char  idword[31], endstr[10];  (line 1771)
and
  strcpy(endstr,\\n  } } ,); (line 1925)
won't match (the string copied is 14 bytes long).

Regards,
Michael
Index: readseq-1/ureadseq.c
===
--- readseq-1.orig/ureadseq.c	2013-12-30 18:06:59.0 +0100
+++ readseq-1/ureadseq.c	2013-12-30 18:13:37.0 +0100
@@ -1768,7 +1768,7 @@
 
   short linesout = 0, seqtype = kNucleic;
   long  i, j, l, l1, ibase;
-  char  idword[31], endstr[10];
+  char  idword[31], endstr[14];
   char  seqnamestore[128], *seqname = seqnamestore;
   char  s[kMaxseqwidth], *cp;
   char  nameform[10], numform[10], nocountsymbols[10];


Bug#693372: ubuntu-dev-tools: requestsync fails ValueError: IV must be 16 bytes long

2013-01-02 Thread Michael Bienia
On 2012-12-30 18:40:23 -0800, Vincent Cheng wrote:
Hi,

 Michael: the reason why python-keyring can't migrate to testing right
 now is because Debian is in freeze, and updates such as new upstream
 releases don't comply with the freeze policy [1]. Is there a way to
 fix this bug with the current version of python-keyring in testing
 instead?

There is no other way than to fix (by either backporting the fix or
allowing python-keyring to migrate) python-keyring in testing[1]. The
current python-keyring from testing doesn't (partly) work with
python-crypto from testing as python-keyring from testing uses an empty
initialisation vector for the cypher to encrypt the keyring. Older
version of python-crypto wrongly allowed this but it got fixed in
python-crypto 2.6 which migrated to testing while a fixed python-keyring
didn't.

So someone needs to talk to the release team and security team how to
resolve the current situation regarding python-keyring by either
backporting the fix from python-keyring 0.9.1 to 0.7.1 or letting
python-keyring migrate:
CryptedFileKeyring now uses PBKDF2 to derive the key from the user's
password and a random hash. The IV is chosen randomly as well. All the
stored passwords are encrypted at once. [...] (this is CVE-2012-4571
and unfixed in testing).
Ideally python-crypto would have a Breaks: python-keyring (= 0.7.1-1)
to enforce an upgrade of python-keyring when python-crypto get upgraded.

Unfortunately python-keyring 0.9.x also introduced a new keyring format
with this change. python-keyring 0.9 contains migration code from the
old format to the new format. This code got already removed again in
python-keyring 1.0 and any unconverted keyrings have to get converted
manually. While 1.0 won't be in wheezy, it will certainly be in jessie
and the Debian package would need to carry the migration code or
instruct the user how to upgrade its keyring files. (See also
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=675379%3b#38)


1: There would be an other option: to undo the change in python-crypto
which enforces an non-empty IV but it's not a sane option security-wise.

Michael


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#693372: ubuntu-dev-tools: requestsync fails ValueError: IV must be 16 bytes long

2013-01-02 Thread Michael Bienia
On 2013-01-02 14:34:58 +0100, Sebastian Ramacher wrote:
 On 2013-01-02 12:35:36, Michael Bienia wrote:
  1: There would be an other option: to undo the change in python-crypto
  which enforces an non-empty IV but it's not a sane option security-wise.
 
 NACK with my python-crypto maintainer hat on. I'm not opening this can
 of worms again. One CVE because of that is already one to much.

I didn't expect that this solution would be acceptable and would be
surprised if it would have become the chosen solution (I just listed it
for completeness) as it would re-open a bug and mask an other bug in
python-keyring instead of fixing it.

Michael


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#693372: ubuntu-dev-tools: requestsync fails ValueError: IV must be 16 bytes long

2012-11-25 Thread Michael Bienia
On 2012-11-15 20:51:59 +, Jean-Michel Vourgère wrote:
 ValueError: IV must be 16 bytes long

This comes from a bad combination of python-crypto and python-keyring
(see http://bugs.debian.org/675379; python-keyring: CryptedFileKeyring
is insecure). python-keyring got fixed already in unstable but didn't
migrate to testing yet.
Someone familiar with the Debian process should check if it's still
possible to let python-keyring migrate to testing.

Michael


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#689629: Please use python-argparse instead of python2.6-argparse as builddep

2012-10-07 Thread Michael Bienia
On 2012-10-04 20:15:44 +0200, Daniel Holbach wrote:
 On 04.10.2012 18:47, Jakub Wilk wrote:
  * Daniel Holbach daniel.holb...@ubuntu.com, 2012-10-04, 17:54:
  -   python2.6-argparse,
  +   python-argparse,
  
  No, this is wrong. python2.6-argparse is fully intentional here. Quoting 
  the changelog:
  
  | * Change D-B to python2.6-argparse to force installation during build
  |   (this would otherwise be satisfied by python2.7 which provides
  |   python-argparse).
  
  (I filed #689633, BTW.)
  
  Maybe I could propose a better patch, but I don't know what you're 
  trying to achieve. :)

The problem is that Ubuntu doesn't support Python 2.6 anymore (only
2.7), therefore argparse got removed and sorl-thumbnail was sitting
in dependency wait on python2.6-argparse indefinitely.

If replacing python2.6-argparse with python-argparse is the wrong
solution, do you have a better solution on how to make it build in
Ubuntu? Or should the package get removed completely due to the indirect
dependency on python2.6?

Thanks for your help.

Michael Bienia


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#673286: gem: FTBFS with gcc-4.7

2012-05-17 Thread Michael Bienia
Package: gem
Version: 1:0.93.3-3
Severity: normal
Tags: patch
User: debian-...@lists.debian.org
Usertags: ftbfs-gcc-4.7

Hi,

while trying to build gem with gcc-4.7 (tried in Ubuntu), it FTBFS with:
,
| make[3]: Entering directory `/build/buildd/gem-0.93.3/plugins/recordV4L2'
| /bin/bash ../../libtool  --tag=CXX   --mode=compile g++ -DHAVE_CONFIG_H -I. 
-I../../src  -I../../src  -DPD -I/usr/include/pd  -freg-struct-return -O3 
-falign-loops -falign-functions -falign-jumps -funroll-loops -ffast-math -mmmx 
-MT gem_recordV4L2_la-recordV4L2.lo -MD -MP -MF 
.deps/gem_recordV4L2_la-recordV4L2.Tpo -c -o gem_recordV4L2_la-recordV4L2.lo 
`test -f 'recordV4L2.cpp' || echo './'`recordV4L2.cpp
| libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I../../src -I../../src -DPD 
-I/usr/include/pd -freg-struct-return -O3 -falign-loops -falign-functions 
-falign-jumps -funroll-loops -ffast-math -mmmx -MT 
gem_recordV4L2_la-recordV4L2.lo -MD -MP -MF 
.deps/gem_recordV4L2_la-recordV4L2.Tpo -c recordV4L2.cpp  -fPIC -DPIC -o 
.libs/gem_recordV4L2_la-recordV4L2.o
| recordV4L2.cpp: In member function 'virtual void 
gem::plugins::recordV4L2::close()':
| recordV4L2.cpp:81:5: error: '::close' has not been declared
| recordV4L2.cpp: In member function 'virtual bool 
gem::plugins::recordV4L2::open(std::string)':
| recordV4L2.cpp:101:65: warning: format '%s' expects a matching 'char*' 
argument [-Wformat]
| recordV4L2.cpp: In member function 'virtual bool 
gem::plugins::recordV4L2::init(const imageStruct*, int)':
| recordV4L2.cpp:159:3: error: '::write' has not been declared
| recordV4L2.cpp: In member function 'virtual bool 
gem::plugins::recordV4L2::putFrame(imageStruct*)':
| recordV4L2.cpp:184:3: error: '::write' has not been declared
| make[3]: *** [gem_recordV4L2_la-recordV4L2.lo] Error 1
`
Adding the missing #include unistd.h in that file fixes it.

Regards,
Michael
Index: gem-0.93.3/plugins/recordV4L2/recordV4L2.cpp
===
--- gem-0.93.3.orig/plugins/recordV4L2/recordV4L2.cpp	2012-05-17 15:43:18.0 +0200
+++ gem-0.93.3/plugins/recordV4L2/recordV4L2.cpp	2012-05-17 15:45:16.0 +0200
@@ -31,7 +31,7 @@
 #include fcntl.h
 #include sys/ioctl.h
 #include stdio.h
-
+#include unistd.h
 #include stdlib.h
 
 


Bug#671765: xpdf: FTBFS with ld --as-needed

2012-05-06 Thread Michael Bienia
Package: xpdf
Version: 3.03-9
Severity: normal
Tags: patch
User: debian-...@lists.debian.org
Usertags: ld-as-needed

Hello,

xpdf 3.03-9 will FTBFS when trying to link with --as-needed.
The problem is in debian/rules passing the libs to link with in LDFLAGS
instead of LIBS (the correct variable for this).
Attached is a patch which fixes it.

This isn't a problem for Debian (yet), but as Ubuntu uses ld --as-needed
already by default, xpdf FTBFS there:
,[ 
https://launchpadlibrarian.net/104399108/buildlog_ubuntu-quantal-i386.xpdf_3.03-9_FAILEDTOBUILD.txt.gz
 ]-
| g++ -Wl,-Bsymbolic-functions -fPIE -pie -Wl,-z,relro -Wl,-z,now -lXm 
-lpoppler -lXt -lX11   -o build/xpdf.real build/*.o
| build/xpdf.o: In function `main':
| /build/buildd/xpdf-3.03/build/xpdf.cc:342: undefined reference to 
`Object::memCheck(_IO_FILE*)'
| /build/buildd/xpdf-3.03/build/xpdf.cc:251: undefined reference to 
`GooString::GooString(char const*)'
| /build/buildd/xpdf-3.03/build/xpdf.cc:302: undefined reference to 
`GooString::GooString(char const*)'
| /build/buildd/xpdf-3.03/build/xpdf.cc:325: undefined reference to 
`GooString::~GooString()'
`

Regards,
Michael

diff -Nru xpdf-3.03/debian/rules xpdf-3.03/debian/rules
--- xpdf-3.03/debian/rules	2012-02-25 02:05:38.0 +0100
+++ xpdf-3.03/debian/rules	2012-05-06 19:55:56.0 +0200
@@ -4,7 +4,7 @@
 export DEB_BUILD_MAINT_OPTIONS=hardening=+all
 
 includes=$(strip $(shell pkg-config --cflags poppler))
-LDFLAGS+=-lXm $(shell pkg-config --libs poppler --libs xt --libs x11)
+LIBS+=-lXm $(shell pkg-config --libs poppler --libs xt --libs x11)
 CPPFLAGS+=$(includes) $(includes)/goo $(includes)/splash -Wno-write-strings -DHAVE_DIRENT_H
 CPPFLAGS+=-Wno-format-extra-args -DSYSTEM_XPDFRC=\/etc/xpdf/xpdfrc\
 
@@ -28,7 +28,7 @@
 	mv build/parseargs.c build/parseargs.cc
 
 override_dh_auto_build: $(objects)
-	$(CXX) $(LDFLAGS) -o build/xpdf.real build/*.o
+	$(CXX) $(LDFLAGS) -o build/xpdf.real build/*.o $(LIBS)
 
 %:
 	dh ${@}


Bug#671016: ax25-tools: FTBFS with ld --as-needed

2012-05-01 Thread Michael Bienia
Package: ax25-tools
Version: 0.0.10-rc2+cvs20120204-3
Severity: normal
User: debian-...@lists.debian.org
Usertags: ld-as-needed
Tags: patch

Hello,

trying to build your package in Ubuntu failed as the linking doesn't
respect the needed argument order for linking with ld --as-needed:

,[ 
https://launchpadlibrarian.net/103676001/buildlog_ubuntu-quantal-i386.ax25-tools_0.0.10-rc2%2Bcvs20120204-3_FAILEDTOBUILD.txt.gz
 ]-
| g++  -g -O2  -lX11 -lfltk -lGL  -lSM -lICE -lXi -lXext -lX11  ../hdrvcomm.o  
-o xfsmmixer xfsmmixer.o xfsmmixer_main.o  
| xfsmmixer_main.o: In function `main':
| 
/build/buildd/ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/xfsmmixer_main.cxx:321:
 undefined reference to `Fl::arg(int, char**, int)'
| 
/build/buildd/ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/xfsmmixer_main.cxx:416:
 undefined reference to `Fl_Slider::bounds(double, double)'
| 
/build/buildd/ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/xfsmmixer_main.cxx:418:
 undefined reference to `Fl_Valuator::value(double)'
| 
/build/buildd/ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/xfsmmixer_main.cxx:420:
 undefined reference to `Fl_Slider::bounds(double, double)'
| 
/build/buildd/ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/xfsmmixer_main.cxx:422:
 undefined reference to `Fl_Valuator::value(double)'
`

Attached is a patch to fix it but it would be nice if someone more
experienced with autotools double checked it (I also patched Makefile.in
instead of regenerating it to keep the changes small).


Regards,
Michael
diff -u ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/Makefile.in ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/Makefile.in
--- ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/Makefile.in
+++ ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/Makefile.in
@@ -50,16 +50,16 @@
 am_xfhdlcchpar_OBJECTS = xfhdlcchpar.$(OBJEXT) \
 	xfhdlcchpar_main.$(OBJEXT)
 xfhdlcchpar_OBJECTS = $(am_xfhdlcchpar_OBJECTS)
-xfhdlcchpar_LDADD = $(LDADD)
+xfhdlcchpar_LDADD = $(LDADD) ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
 am_xfhdlcst_OBJECTS = xfhdlcst.$(OBJEXT) xfhdlcst_main.$(OBJEXT)
 xfhdlcst_OBJECTS = $(am_xfhdlcst_OBJECTS)
-xfhdlcst_LDADD = $(LDADD)
+xfhdlcst_LDADD = $(LDADD) ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
 am_xfsmdiag_OBJECTS = xfsmdiag.$(OBJEXT) xfsmdiag_main.$(OBJEXT)
 xfsmdiag_OBJECTS = $(am_xfsmdiag_OBJECTS)
-xfsmdiag_LDADD = $(LDADD)
+xfsmdiag_LDADD = $(LDADD) ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
 am_xfsmmixer_OBJECTS = xfsmmixer.$(OBJEXT) xfsmmixer_main.$(OBJEXT)
 xfsmmixer_OBJECTS = $(am_xfsmmixer_OBJECTS)
-xfsmmixer_LDADD = $(LDADD)
+xfsmmixer_LDADD = $(LDADD) ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
 DEFAULT_INCLUDES = -I.@am__isrc@ -I$(top_builddir)
 depcomp = $(SHELL) $(top_srcdir)/depcomp
 am__depfiles_maybe = depfiles
@@ -184,8 +184,6 @@
 top_builddir = @top_builddir@
 top_srcdir = @top_srcdir@
 AM_CPPFLAGS = $(X_CFLAGS) -I$(srcdir)/..
-LDDADD = ../hdrvcomm.o
-AM_LDFLAGS = $(X_LIBS) -lX11 $(FLTK_LIB) $(LDDADD)
 xfsmmixer_SOURCES = xfsmmixer.cxx xfsmmixer_main.cxx xfsmmixer.h
 xfhdlcchpar_SOURCES = xfhdlcchpar.cxx xfhdlcchpar_main.cxx xfhdlcchpar.h
 xfhdlcst_SOURCES = xfhdlcst.cxx xfhdlcst_main.cxx xfhdlcst.h
--- ax25-tools-0.0.10-rc2+cvs20120204.orig/hdlcutil/fl/Makefile.am
+++ ax25-tools-0.0.10-rc2+cvs20120204/hdlcutil/fl/Makefile.am
@@ -4,12 +4,15 @@
 endif
 
 AM_CPPFLAGS = $(X_CFLAGS) -I$(srcdir)/..
-LDDADD = ../hdrvcomm.o
-AM_LDFLAGS = $(X_LIBS) -lX11 $(FLTK_LIB) $(LDDADD)
 
 xfsmmixer_SOURCES = xfsmmixer.cxx xfsmmixer_main.cxx xfsmmixer.h
 xfhdlcchpar_SOURCES = xfhdlcchpar.cxx xfhdlcchpar_main.cxx xfhdlcchpar.h
 xfhdlcst_SOURCES = xfhdlcst.cxx xfhdlcst_main.cxx xfhdlcst.h
 xfsmdiag_SOURCES = xfsmdiag.cxx xfsmdiag_main.cxx xfsmdiag.h xfsmdiag_main.h
 
+xfsmmixer_LDADD = ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
+xfhdlcchpar_LDADD = ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
+xfhdlcst_LDADD = ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
+xfsmdiag_LDADD = ../hdrvcomm.o $(X_LIBS) -lX11 $(FLTK_LIB)
+
 hdrvcomm.o: ../hdrvcomm.o


Bug#671052: gtklp: FTBFS with ld --as-needed

2012-05-01 Thread Michael Bienia
Package: gtklp
Version: 1.2.7-2.3
Severity: normal
Tags: patch
User: debian-...@lists.debian.org
Usertags: ld-as-needed

Hello,

trying to build gtklp in Ubuntu (which uses ld --as-needed by default)
resulted in the following linking error:

,[ 
https://launchpadlibrarian.net/103699915/buildlog_ubuntu-quantal-i386.gtklp_1.2.7-2.3_FAILEDTOBUILD.txt.gz
 ]-
| libtool: link: gcc -g -O2 -fstack-protector --param=ssp-buffer-size=4 
-Wformat -Wformat-security -Werror=format-security -Wall -o gtklp file.o 
general.o gtklp.o gtklp_functions.o gtklptab.o hpgl2.o image.o output.o ppd.o 
special.o text.o -Wl,-Bsymbolic-functions -Wl,-z -Wl,relro  
-L/usr/lib/i386-linux-gnu -lX11 ../libgtklp/libgtklp.a -lgtk-x11-2.0 
-lgdk-x11-2.0 -latk-1.0 -lgio-2.0 -lpangoft2-1.0 -lpangocairo-1.0 
-lgdk_pixbuf-2.0 /usr/lib/i386-linux-gnu/libcairo.so -lpango-1.0 
/usr/lib/i386-linux-gnu/libfreetype.so -lfontconfig -lgobject-2.0 -lglib-2.0 
-lcups -lgssapi_krb5 -lkrb5 -lk5crypto -lcom_err -lgnutls -L/lib/i386-linux-gnu 
-lgcrypt -lz -lpthread -lcrypt -lm
| /usr/bin/ld: ../libgtklp/libgtklp.a(libgtklp.o): undefined reference to 
symbol 'XOpenDisplay'
| /usr/bin/ld: note: 'XOpenDisplay' is defined in DSO 
/usr/lib/i386-linux-gnu/libX11.so so try adding it to the linker command line
`

Although -lX11 is in the linker call the problem is in the argument
order (it's listed before libgtklp.a).
The patch from bug #554752 fixed only linking with --no-add-needed but
not with --as-needed.

Attached is a fixed patch which adds -lX11 to gtklp_LDADD (in
gtklp/Makefile.am) instead of LDFLAGS, fixing linking both with
--no-add-needed and --as-needed.

Regards,
Michael
Index: gtklp-1.2.7/gtklp/Makefile.am
===
--- gtklp-1.2.7.orig/gtklp/Makefile.am	2009-06-21 14:25:37.0 +0200
+++ gtklp-1.2.7/gtklp/Makefile.am	2012-05-01 16:04:00.0 +0200
@@ -9,5 +9,5 @@
 	duplex_short.xpm logo.xpm logo2.xpm logo2_k.xpm logo_k.xpm rarrow.xpm \
 	white.xpm
 
-gtklp_LDADD = $(top_builddir)/libgtklp/libgtklp.a $(GTK2_LIBS) $(GTK_LIBS) $(CUPS_LIBS) $(FORTE_LDFLAGS) 
+gtklp_LDADD = $(top_builddir)/libgtklp/libgtklp.a -lX11 $(GTK2_LIBS) $(GTK_LIBS) $(CUPS_LIBS) $(FORTE_LDFLAGS) 
 gtklp_LDFLAGS = $(CUPS_LDFLAGS)


Bug#646033: Patch for FTBFS with ld --as-needed

2011-10-31 Thread Michael Bienia
Hello,

here is a minimal patch (patches both src/Makefile.am and
src/Makefile.in) to fix the FTBFS.
It replaces the wrong usage of $binary_LDFLAGS with the correct
$binary_LDADD to pass additional libraries to link with.

I'm not sure if -lX11 should be kept in LDADD (from #554058) as I could
build the package without the -lX11 in LDADD in Ubuntu precise.

Michael
diff -u brightside-1.4.0/src/Makefile.in brightside-1.4.0/src/Makefile.in
--- brightside-1.4.0/src/Makefile.in
+++ brightside-1.4.0/src/Makefile.in
@@ -206,7 +206,7 @@
 	brightside.c brightside.h
 
 
-brightside_LDFLAGS = \
+brightside_LDADD = \
 	$(EXTRA_GNOME_LIBS)\
 	$(ALSA_LIBS)	\
 	$(INTLLIBS)
@@ -225,7 +225,7 @@
 
 EGGFILES2 = eggtrayicon.h eggtrayicon.c
 
-brightside_properties_LDFLAGS = \
+brightside_properties_LDADD = \
 	$(EXTRA_GNOME_LIBS)			\
 	$(INTLLIBS)
 
@@ -287,14 +287,12 @@
 	$(am__objects_2) $(am__objects_3) $(am__objects_4) \
 	brightside-util.$(OBJEXT) brightside.$(OBJEXT)
 brightside_OBJECTS = $(am_brightside_OBJECTS)
-brightside_LDADD = $(LDADD) -lX11
 brightside_DEPENDENCIES =
 am_brightside_properties_OBJECTS = brightside-properties.$(OBJEXT) \
 	gconf-property-editor.$(OBJEXT) \
 	gconf-property-editor-marshal.$(OBJEXT) \
 	preview-file-selection.$(OBJEXT) custom-action.$(OBJEXT)
 brightside_properties_OBJECTS = $(am_brightside_properties_OBJECTS)
-brightside_properties_LDADD = $(LDADD) -lX11
 brightside_properties_DEPENDENCIES =
 
 DEFAULT_INCLUDES =  -I. -I$(srcdir) -I$(top_builddir)
only in patch2:
unchanged:
--- brightside-1.4.0.orig/src/Makefile.am
+++ brightside-1.4.0/src/Makefile.am
@@ -36,7 +36,7 @@
 	brightside-util.c brightside-util.h			\
 	brightside.c brightside.h
 
-brightside_LDFLAGS =		\
+brightside_LDADD =		\
 	$(EXTRA_GNOME_LIBS)\
 	$(ALSA_LIBS)	\
 	$(INTLLIBS)
@@ -53,7 +53,7 @@
 
 EGGFILES2 = eggtrayicon.h eggtrayicon.c
 
-brightside_properties_LDFLAGS = 			\
+brightside_properties_LDADD = 			\
 	$(EXTRA_GNOME_LIBS)			\
 	$(INTLLIBS)
 


Bug#637169: /usr/bin/requestsync: default mail configuration fails (with a traceback)

2011-08-09 Thread Michael Bienia
On 2011-08-09 13:07:30 +1000, Karl Goetz wrote:
 Package: ubuntu-dev-tools
 Version: 0.101
 Severity: normal
 File: /usr/bin/requestsync
 
 I've stepped all the way through requestsync, and the final step bombs out on 
 me:
 Press [Enter] to continue or [Ctrl-C] to abort. 
 Connecting to fiordland.ubuntu.com:25 ...
 Traceback (most recent call last):
   File /usr/bin/requestsync, line 240, in module
 mailBug(srcpkg, subscribe, status, title, report, options.keyid)
   File /usr/lib/pymodules/python2.6/ubuntutools/requestsync/mail.py, line 
 226, in mailBug
 s.sendmail(myemailaddr, to, mail.encode('utf-8'))
   File /usr/lib/python2.6/smtplib.py, line 709, in sendmail
 raise SMTPRecipientsRefused(senderrs)
 smtplib.SMTPRecipientsRefused: {'n...@bugs.launchpad.net': (554, '5.7.1 
 n...@bugs.launchpad.net: Relay access denied')}
 
 I invoked the command as `requestsync -d sid -s -e freeciv oneiric`

That ubuntu-dev-tools version still has a SMTP server hardcoded which
isn't a MX for launchpad.net anymore. This got fixed in ubuntu-dev-tools
0.115 but didn't made it yet as a SRU into stable.

The relevant commit is
http://bazaar.launchpad.net/~ubuntu-dev/ubuntu-dev-tools/trunk/revision/996

Till then you can either use requestsync --lp to directly file the bug
through the LP API (if you have allowed the requestsync script to access
LP on your behalf). Alternatively you can try to set the UBUSMTP (or
DEBSMTP) environment variable to a current MX for launchpad.net (or any
other SMTP server which will accept your mail).

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#625984: bluez-tools: FTBFS with ld --as-needed

2011-05-07 Thread Michael Bienia
Package: bluez-tools
Version: 0.1.38+git662e-2
Severity: normal
Tags: patch

Hello,

bluez-tools fails to build in Ubuntu as it uses ld --as-needed for
linking by default and blueztools misuses AM_LDFLAGS to specify the
libraries to link with. As configure uses AM_LDFLAGS before the object
files in the linker call the linking fails. The proper variable for them
is LDADD, program_LDADD or even LIBS (I'm not very familiar with
automake which one is prefered when).

The attached patch renames AM_LDFLAGS in src/Makefile.{am,in} to LDADD
(instead of adding program_LDADD for each binary).

Regards,
Michael


https://launchpadlibrarian.net/71108320/buildlog_ubuntu-oneiric-i386.bluez-tools_0.1.38%2Bgit662e-2_FAILEDTOBUILD.txt.gz
gcc -std=gnu99  -g -O2 -L/usr/lib/i386-linux-gnu -ldbus-1 -lpthread -lrt   
-L/usr/lib/i386-linux-gnu -lglib-2.0   -pthread -L/usr/lib/i386-linux-gnu 
-ldbus-glib-1 -ldbus-1 -lpthread -lgobject-2.0 -lgthread-2.0 -lrt -lglib-2.0   
-Wl,-Bsymbolic-functions -o bt-monitor lib/marshallers.o lib/dbus-common.o 
lib/helpers.o lib/sdp.o  lib/bluez/adapter.o lib/bluez/agent.o 
lib/bluez/audio.o lib/bluez/device.o lib/bluez/input.o lib/bluez/manager.o 
lib/bluez/network.o lib/bluez/network_server.o lib/bluez/serial.o 
lib/obexd/obexagent.o lib/obexd/obexclient.o 
lib/obexd/obexclient_file_transfer.o lib/obexd/obexclient_session.o 
lib/obexd/obexclient_transfer.o lib/obexd/obexmanager.o lib/obexd/obexsession.o 
lib/obexd/obextransfer.o bt-monitor.o  
bt-monitor.o: In function `adapter_device_disappeared':
/build/buildd/bluez-tools-0.1.38+git662e/src/bt-monitor.c:123: undefined 
reference to `g_print'
bt-monitor.o: In function `device_disconnect_requested':
/build/buildd/bluez-tools-0.1.38+git662e/src/bt-monitor.c:182: undefined 
reference to `g_print'
bt-monitor.o: In function `reload_device_services':
/build/buildd/bluez-tools-0.1.38+git662e/src/bt-monitor.c:371: undefined 
reference to `g_hash_table_lookup'
/build/buildd/bluez-tools-0.1.38+git662e/src/bt-monitor.c:378: undefined 
reference to `g_slist_nth_data'
/build/buildd/bluez-tools-0.1.38+git662e/src/bt-monitor.c:379: undefined 
reference to `g_slist_remove'
/build/buildd/bluez-tools-0.1.38+git662e/src/bt-monitor.c:380: undefined 
reference to `g_object_unref'
[...]
Index: bluez-tools-0.1.38+git662e/src/Makefile.am
===
--- bluez-tools-0.1.38+git662e.orig/src/Makefile.am	2011-05-07 17:06:38.0 +0200
+++ bluez-tools-0.1.38+git662e/src/Makefile.am	2011-05-07 17:07:10.0 +0200
@@ -1,5 +1,5 @@
 AM_CPPFLAGS = $(DBUS_CFLAGS) $(GLIB_CFLAGS) $(DBUS_GLIB_CFLAGS)
-AM_LDFLAGS = $(DBUS_LIBS) $(GLIB_LIBS) $(DBUS_GLIB_LIBS)
+LDADD = $(DBUS_LIBS) $(GLIB_LIBS) $(DBUS_GLIB_LIBS)
 
 # Marshallers generation
 #BUILT_SOURCES = lib/marshallers.c lib/marshallers.h
@@ -57,7 +57,7 @@
 if OBEX
 bin_PROGRAMS += bt-obex
 bt_obex_SOURCES = $(lib_sources) $(bluez_sources) $(obexd_sources) bt-obex.c
-bt_obex_LDADD = $(LIBREADLINE)
+bt_obex_LDADD = $(LDADD) $(LIBREADLINE)
 endif
 
 dist_man_MANS = bt-monitor.1 bt-adapter.1 bt-agent.1 bt-device.1 bt-input.1 bt-audio.1 bt-network.1 bt-serial.1
Index: bluez-tools-0.1.38+git662e/src/Makefile.in
===
--- bluez-tools-0.1.38+git662e.orig/src/Makefile.in	2011-05-07 17:07:16.0 +0200
+++ bluez-tools-0.1.38+git662e/src/Makefile.in	2011-05-07 17:07:58.0 +0200
@@ -419,7 +419,7 @@
 top_builddir = @top_builddir@
 top_srcdir = @top_srcdir@
 AM_CPPFLAGS = $(DBUS_CFLAGS) $(GLIB_CFLAGS) $(DBUS_GLIB_CFLAGS)
-AM_LDFLAGS = $(DBUS_LIBS) $(GLIB_LIBS) $(DBUS_GLIB_LIBS)
+LDADD = $(DBUS_LIBS) $(GLIB_LIBS) $(DBUS_GLIB_LIBS)
 
 # Marshallers generation
 #BUILT_SOURCES = lib/marshallers.c lib/marshallers.h
@@ -462,7 +462,7 @@
 bt_network_SOURCES = $(lib_sources) $(bluez_sources) $(obexd_sources) bt-network.c
 bt_serial_SOURCES = $(lib_sources) $(bluez_sources) $(obexd_sources) bt-serial.c
 @OBEX_TRUE@bt_obex_SOURCES = $(lib_sources) $(bluez_sources) $(obexd_sources) bt-obex.c
-@OBEX_TRUE@bt_obex_LDADD = $(LIBREADLINE)
+@OBEX_TRUE@bt_obex_LDADD = $(LDADD) $(LIBREADLINE)
 dist_man_MANS = bt-monitor.1 bt-adapter.1 bt-agent.1 bt-device.1 \
 	bt-input.1 bt-audio.1 bt-network.1 bt-serial.1 $(am__append_3)
 all: all-am


Bug#624686: vim-runtime: Update of the supported Ubuntu codenames

2011-04-30 Thread Michael Bienia
Package: vim-runtime
Version: 2:7.3.154+hg~74503f6ee649-2
Severity: normal
Tags: patch

Hello,

attached is a patch to update the supported Ubuntu codenames in
runtime/syntax/debchangelog.vim and runtime/syntax/debsources.vim:
- Drop jaunty and karmic
- Add oneiric

Michael
Index: vim-7.3.154+hg~74503f6ee649/runtime/syntax/debchangelog.vim
===
--- vim-7.3.154+hg~74503f6ee649.orig/runtime/syntax/debchangelog.vim	2011-04-30 17:17:21.0 +0200
+++ vim-7.3.154+hg~74503f6ee649/runtime/syntax/debchangelog.vim	2011-04-30 17:17:48.0 +0200
@@ -19,7 +19,7 @@
  Define some common expressions we can use later on
 syn match debchangelogName	contained ^[[:alnum:]][[:alnum:].+-]\+ 
 syn match debchangelogUrgency	contained ; urgency=\(low\|medium\|high\|critical\|emergency\)\( \S.*\)\=
-syn match debchangelogTarget	contained \v %(frozen|unstable|%(testing|%(old)=stable)%(-proposed-updates|-security)=|experimental|%(lenny|squeeze)-%(backports%(-sloppy)=|volatile)|%(dapper|hardy|jaunty|karmic|lucid|maverick|natty)%(-%(security|proposed|updates|backports|commercial|partner))=)+
+syn match debchangelogTarget	contained \v %(frozen|unstable|%(testing|%(old)=stable)%(-proposed-updates|-security)=|experimental|%(lenny|squeeze)-%(backports%(-sloppy)=|volatile)|%(dapper|hardy|lucid|maverick|natty|oneiric)%(-%(security|proposed|updates|backports|commercial|partner))=)+
 syn match debchangelogVersion	contained (.\{-})
 syn match debchangelogCloses	contained closes:\_s*\(bug\)\=#\=\_s\=\d\+\(,\_s*\(bug\)\=#\=\_s\=\d\+\)*
 syn match debchangelogLP	contained \clp:\s\+#\d\+\(,\s*#\d\+\)*
Index: vim-7.3.154+hg~74503f6ee649/runtime/syntax/debsources.vim
===
--- vim-7.3.154+hg~74503f6ee649.orig/runtime/syntax/debsources.vim	2011-04-30 17:18:42.0 +0200
+++ vim-7.3.154+hg~74503f6ee649/runtime/syntax/debsources.vim	2011-04-30 17:19:16.0 +0200
@@ -23,7 +23,7 @@
 
  Match uri's
 syn match debsourcesUri+\(http://\|ftp://\|[rs]sh://\|debtorrent://\|\(cdrom\|copy\|file\):\)[^' 	]\++
-syn match debsourcesDistrKeyword   +\([[:alnum:]_./]*\)\(lenny\|squeeze\|\(old\)\=stable\|testing\|unstable\|sid\|rc-buggy\|experimental\|dapper\|hardy\|jaunty\|karmic\|lucid\|maverick\|natty\)\([-[:alnum:]_./]*\)+
+syn match debsourcesDistrKeyword   +\([[:alnum:]_./]*\)\(lenny\|squeeze\|\(old\)\=stable\|testing\|unstable\|sid\|rc-buggy\|experimental\|dapper\|hardy\|lucid\|maverick\|natty\|oneiric\)\([-[:alnum:]_./]*\)+
 
  Associate our matches and regions with pretty colours
 hi def link debsourcesLineError


Bug#624688: vim-gui-common: Drop path and file extension for Icon in gvim.desktop

2011-04-30 Thread Michael Bienia
Package: vim-gui-common
Version: 2:7.3.154+hg~74503f6ee649-2
Severity: normal
Tags: patch

Hi,

this was originally filed in Ubuntu as bug #740842 [1].
To allow theming of the icon, the desktop file shouldn't contain an
absolute path name (and also no file extension).

Michael

1: https://launchpad.net/bugs/740842

diff -Nru vim-7.3.154+hg~74503f6ee649/debian/gvim.desktop 
vim-7.3.154+hg~74503f6ee649/debian/gvim.desktop
--- vim-7.3.154+hg~74503f6ee649/debian/gvim.desktop 2011-03-18 
02:33:29.0 +0100
+++ vim-7.3.154+hg~74503f6ee649/debian/gvim.desktop 2011-04-30 
17:34:57.0 +0200
@@ -72,7 +72,7 @@
 Exec=gvim -f %F
 Terminal=false
 Type=Application
-Icon=/usr/share/pixmaps/vim.svg
+Icon=vim
 Categories=Utility;TextEditor;
 StartupNotify=true
 MimeType=text/plain;



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#624707: FTBFS caused by error: format not a string literal and no format arguments

2011-04-30 Thread Michael Bienia
Package: dctl-tools
Version: 2.18
Severity: normal
Tags: patch

Hello,

trying to build dctrl-tools 2.18 in Ubuntu oneiric leads to a FTBFS
because of error: format not a string literal and no format arguments
[1]. The error comes from a wrong argument order for message() and some
missing format specifiers. Attached is a patch to fix it.

Regards,
Michael


1: 
https://launchpadlibrarian.net/70771161/buildlog_ubuntu-oneiric-i386.dctrl-tools_2.18_FAILEDTOBUILD.txt.gz
gcc  -g -Wall -Werror -std=gnu99 -Ilib -DENABLE_L_DEBUG -D_GNU_SOURCE 
-DSYSCONF=\/etc\ -DHAVE_GETTEXT -DPACKAGE=\dctrl-tools\ 
-DLOCALEDIR=\/usr/share/locale\  -DVERSION=\2.18\ 
-DMAINTAINER='dctrl-tools developers 
dctrl-tools-de...@lists.alioth.debian.org' -c -o grep-dctrl/grep-dctrl.o 
grep-dctrl/grep-dctrl.c
In file included from grep-dctrl/grep-dctrl.c:35:0:
lib/msg.h: In function 'debug_message':
lib/msg.h:115:3: error: format not a string literal and no format arguments 
[-Werror=format-security]
lib/msg.h: In function 'errno_msg':
lib/msg.h:140:2: error: format not a string literal and no format arguments 
[-Werror=format-security]
lib/msg.h: In function 'enomem':
lib/msg.h:148:3: error: format not a string literal and no format arguments 
[-Werror=format-security]
lib/msg.h: In function 'fatal_enomem':
lib/msg.h:154:3: error: format not a string literal and no format arguments 
[-Werror=format-security]
cc1: all warnings being treated as errors
diff -Nru dctrl-tools-2.18/grep-dctrl/rc.c dctrl-tools-2.18ubuntu1/grep-dctrl/rc.c
--- dctrl-tools-2.18/grep-dctrl/rc.c	2010-12-04 11:31:28.0 +0100
+++ dctrl-tools-2.18ubuntu1/grep-dctrl/rc.c	2011-04-30 21:16:39.0 +0200
@@ -64,14 +64,14 @@
 		return false;
 	}
 	if (stat.st_uid != 0  stat.st_uid != getuid()) {
-		message(L_IMPORTANT, _(not owned by you or root, ignoring),
-			fname);
+		message(L_IMPORTANT, fname,
+			_(not owned by you or root, ignoring));
 		return false;
 	}
 	if ((stat.st_mode  (S_IWGRP | S_IWOTH)) != 0) {
-		message(L_IMPORTANT, _(write permissions for 
-   group or others, ignoring),
-			fname);
+		message(L_IMPORTANT, fname,
+			_(write permissions for 
+			  group or others, ignoring));
 		return false;
 	}
 	return true;
@@ -109,11 +109,11 @@
 		return (struct ifile){ .mode = m_error, .s = 0 };
 	}
 
-	message(L_INFORMATIONAL, _(reading config file), fname);
+	message(L_INFORMATIONAL, fname, _(reading config file));
 
 	f = fopen(fname, r);
 	if (f == 0) {
-		message(L_INFORMATIONAL, strerror(errno), fname);
+		message(L_INFORMATIONAL, fname, %s, strerror(errno));
 		return (struct ifile){ .mode = m_error, .s = 0 };
 	}
 
@@ -136,8 +136,8 @@
 
 		line = getaline (f);
 		if (line == 0) {
-			message(L_FATAL, _(read failure or out of memory),
-fname);
+			message(L_FATAL, fname,
+_(read failure or out of memory));
 			fail();
 		}
 
diff -Nru dctrl-tools-2.18/join-dctrl/join-dctrl.c dctrl-tools-2.18ubuntu1/join-dctrl/join-dctrl.c
--- dctrl-tools-2.18/join-dctrl/join-dctrl.c	2010-12-04 11:31:28.0 +0100
+++ dctrl-tools-2.18ubuntu1/join-dctrl/join-dctrl.c	2011-04-30 21:03:15.0 +0200
@@ -91,7 +91,7 @@
 for (size_t i = 0; i  2; i++) {
 if (key == the_other_key[i]) continue;
 if (args-join_field[0] != NULL) {
-message(L_FATAL, 0, gettext(errmsg[i]));
+message(L_FATAL, 0, %s, gettext(errmsg[i]));
 fail();
 }
 }
diff -Nru dctrl-tools-2.18/lib/msg.h dctrl-tools-2.18ubuntu1/lib/msg.h
--- dctrl-tools-2.18/lib/msg.h	2010-12-04 11:31:29.0 +0100
+++ dctrl-tools-2.18ubuntu1/lib/msg.h	2011-04-30 21:12:32.0 +0200
@@ -112,7 +112,7 @@
 debug_message (const char * s, const char * fname)
 {
 #ifdef INCLUDE_DEBUG_MSGS
-  message (L_DEBUG, s, fname);
+  message (L_DEBUG, fname, %s, s);
 #endif
 }
 
@@ -137,7 +137,7 @@
 inline static void
 errno_msg(int severity, char const * fname)
 {
-	message(severity, strerror(errno), fname);
+	message(severity, fname, %s, strerror(errno));
 }
 
 #define enomem_msg _(cannot find enough memory)
@@ -145,13 +145,13 @@
 inline static void
 enomem (const char * fname)
 {
-  message (L_IMPORTANT, enomem_msg, fname);
+  message (L_IMPORTANT, fname, enomem_msg);
 }
 
 inline static void
 fatal_enomem (const char * fname)
 {
-  message(L_FATAL, enomem_msg, fname);
+  message(L_FATAL, fname, enomem_msg);
   fail();
 }
 


Bug#611652: libexplain: Missing include and two code errors

2011-01-31 Thread Michael Bienia
Package: libexplain
Version: 0.37.D001-1
Severity: normal
Tags: patch

Hello,

attached is the patch which I needed to get libexplain build in Ubuntu
natty:
* libexplain/iocontrol/siocshwtstamp.c:
  + Add missing include of libexplain/ac/linux/net_tstamp.h.
  + Fix two code errors.

Here the part of the Ubuntu build log:
,[ 
http://launchpadlibrarian.net/62570091/buildlog_ubuntu-natty-i386.libexplain_0.37.D001-1_FAILEDTOBUILD.txt.gz
 ]-
| libtool: compile:  gcc -Wall -g -O2 -Wall -Wextra -Wl,--as-needed -I. -c 
libexplain/iocontrol/siocshwtstamp.c  -fPIC -DPIC -o 
libexplain/iocontrol/.libs/siocshwtstamp.o
| libexplain/iocontrol/siocshwtstamp.c: In function 'print_explanation':
| libexplain/iocontrol/siocshwtstamp.c:74:21: warning: assignment from 
incompatible pointer type
| libexplain/iocontrol/siocshwtstamp.c:77:60: error: dereferencing pointer to 
incomplete type
| libexplain/iocontrol/siocshwtstamp.c:79:24: error: dereferencing pointer to 
incomplete type
| libexplain/iocontrol/siocshwtstamp.c:90:10: error: expected ':' or '...' 
before '{' token
| libexplain/iocontrol/siocshwtstamp.c:98:21: warning: assignment from 
incompatible pointer type
| libexplain/iocontrol/siocshwtstamp.c:99:60: error: dereferencing pointer to 
incomplete type
| libexplain/iocontrol/siocshwtstamp.c:103:32: error: dereferencing pointer to 
incomplete type
| libexplain/iocontrol/siocshwtstamp.c:105:26: error: 'HWTSTAMP_TX_OFF' 
undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:105:26: note: each undeclared identifier 
is reported only once for each function it appears in
| libexplain/iocontrol/siocshwtstamp.c:106:26: error: 'HWTSTAMP_TX_ON' 
undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:125:29: error: 'cf' undeclared (first 
use in this function)
| libexplain/iocontrol/siocshwtstamp.c:127:26: error: 'HWTSTAMP_FILTER_NONE' 
undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:128:26: error: 'HWTSTAMP_FILTER_ALL' 
undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:129:26: error: 'HWTSTAMP_FILTER_SOME' 
undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:130:26: error: 
'HWTSTAMP_FILTER_PTP_V1_L4_EVENT' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:131:26: error: 
'HWTSTAMP_FILTER_PTP_V1_L4_SYNC' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:132:26: error: 
'HWTSTAMP_FILTER_PTP_V1_L4_DELAY_REQ' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:133:26: error: 
'HWTSTAMP_FILTER_PTP_V2_L4_EVENT' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:134:26: error: 
'HWTSTAMP_FILTER_PTP_V2_L4_SYNC' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:135:26: error: 
'HWTSTAMP_FILTER_PTP_V2_L4_DELAY_REQ' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:136:26: error: 
'HWTSTAMP_FILTER_PTP_V2_L2_EVENT' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:137:26: error: 
'HWTSTAMP_FILTER_PTP_V2_L2_SYNC' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:138:26: error: 
'HWTSTAMP_FILTER_PTP_V2_L2_DELAY_REQ' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:139:26: error: 
'HWTSTAMP_FILTER_PTP_V2_EVENT' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:140:26: error: 
'HWTSTAMP_FILTER_PTP_V2_SYNC' undeclared (first use in this function)
| libexplain/iocontrol/siocshwtstamp.c:141:26: error: 
'HWTSTAMP_FILTER_PTP_V2_DELAY_REQ' undeclared (first use in this function)
| make[1]: *** [libexplain/iocontrol/siocshwtstamp.lo] Error 1
`

Regards,
Michael
--- libexplain-0.37.D001.orig/libexplain/iocontrol/siocshwtstamp.c
+++ libexplain-0.37.D001/libexplain/iocontrol/siocshwtstamp.c
@@ -21,6 +21,7 @@
 #include libexplain/ac/linux/sockios.h
 #include libexplain/ac/net/if.h
 #include libexplain/ac/sys/ioctl.h
+#include libexplain/ac/linux/net_tstamp.h
 
 #include libexplain/buffer/dac.h
 #include libexplain/buffer/einval.h
@@ -86,7 +87,7 @@
  }
  goto generic;
 
-case ERANGE
+case ERANGE:
  {
 const struct ifreq *rq;
 
@@ -122,7 +123,7 @@
 
 }
 
-switch (cf-rx_filter)
+switch (cfg-rx_filter)
 {
 case HWTSTAMP_FILTER_NONE:
 case HWTSTAMP_FILTER_ALL:


Bug#608878: lemonldap-ng: libhtml-template-perl and libxml-simple-perl missing in Build-Depends-Indep

2011-01-04 Thread Michael Bienia
Package: lemonldap-ng
Version: 1.0.1-1
Severity: normal

Hello,

lemonldap-ng is missing libhtml-template-perl and libxml-simple-perl in
Build-Depends-Indep. Both packages are needed by the test suite. Without
it the build on Ubuntu fails:

,[ 
http://launchpadlibrarian.net/60877365/buildlog_ubuntu-natty-i386.lemonldap-ng_1.0.1-1_FAILEDTOBUILD.txt.gz
 ]-
| #   Failed test 'use Lemonldap::NG::Manager::_Struct;'
| #   at t/30-Manager-Struct.t line 9.
| # Tried to use 'Lemonldap::NG::Manager::_Struct'.
| # Error:  Can't locate HTML/Template.pm in @INC (…)
`

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#606486: xmltooling: FTBFS with ld --as-needed

2010-12-09 Thread Michael Bienia
Package: xmltooling
Version: 1.3.3-2
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch
User: debian-...@lists.debian.org
Usertags: ld-as-needed

Hello,

here is a patch to fix the FTBFS in Ubuntu with ld --as-needed before it
gets lost in the archives of debian-devel
(http://lists.debian.org/debian-devel/2010/12/msg00090.html).

Michael
--- xmltooling-1.3.3.orig/configure
+++ xmltooling-1.3.3/configure
@@ -23030,7 +23030,7 @@
 
 fi;
 if test -f ${LOG4SHIB_CONFIG}; then
-LDFLAGS=`${LOG4SHIB_CONFIG} --libs` $LDFLAGS
+LIBS=`${LOG4SHIB_CONFIG} --libs` $LIBS
 CPPFLAGS=`${LOG4SHIB_CONFIG} --cflags` $CPPFLAGS
 
 if test ${ac_cv_header_log4shib_CategoryStream_hh+set} = set; then
@@ -23292,7 +23292,7 @@
 	if test -f ${LOG4CPP_CONFIG}; then
 		{ echo $as_me:$LINENO: WARNING: will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe 5
 echo $as_me: WARNING: will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe 2;}
-	LDFLAGS=`${LOG4CPP_CONFIG} --libs` $LDFLAGS
+	LIBS=`${LOG4CPP_CONFIG} --libs` $LIBS
 	CPPFLAGS=`${LOG4CPP_CONFIG} --cflags` $CPPFLAGS
 		if test ${ac_cv_header_log4cpp_CategoryStream_hh+set} = set; then
   echo $as_me:$LINENO: checking for log4cpp/CategoryStream.hh 5
--- xmltooling-1.3.3.orig/configure.ac
+++ xmltooling-1.3.3/configure.ac
@@ -105,7 +105,7 @@
 fi
 ])
 if test -f ${LOG4SHIB_CONFIG}; then
-LDFLAGS=`${LOG4SHIB_CONFIG} --libs` $LDFLAGS
+LIBS=`${LOG4SHIB_CONFIG} --libs` $LIBS
 CPPFLAGS=`${LOG4SHIB_CONFIG} --cflags` $CPPFLAGS
 	AC_CHECK_HEADER([log4shib/CategoryStream.hh],,AC_MSG_ERROR([unable to find log4shib header files]))
 	AC_TRY_LINK(
@@ -130,7 +130,7 @@
 	])
 	if test -f ${LOG4CPP_CONFIG}; then
 		AC_MSG_WARN([will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe])
-	LDFLAGS=`${LOG4CPP_CONFIG} --libs` $LDFLAGS
+	LIBS=`${LOG4CPP_CONFIG} --libs` $LIBS
 	CPPFLAGS=`${LOG4CPP_CONFIG} --cflags` $CPPFLAGS
 		AC_CHECK_HEADER([log4cpp/CategoryStream.hh],,AC_MSG_ERROR([unable to find log4cpp header files]))
 		AC_TRY_LINK(


Bug#606488: opensaml2: FTBFS with ld --as-needed

2010-12-09 Thread Michael Bienia
Package: opensaml2
Version: 2.3-2
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch
User: debian-...@lists.debian.org
Usertags: ld-as-needed

Hello,

here is a patch to fix the FTBFS in Ubuntu with ld --as-needed before it
gets lost in the archives of debian-devel
(http://lists.debian.org/debian-devel/2010/12/msg00090.html).

Michael
--- opensaml2-2.3.orig/configure
+++ opensaml2-2.3/configure
@@ -22997,7 +22997,7 @@
 
 fi;
 if test -f ${LOG4SHIB_CONFIG}; then
-LDFLAGS=`${LOG4SHIB_CONFIG} --libs` $LDFLAGS
+LIBS=`${LOG4SHIB_CONFIG} --libs` $LIBS
 CPPFLAGS=`${LOG4SHIB_CONFIG} --cflags` $CPPFLAGS
 
 if test ${ac_cv_header_log4shib_CategoryStream_hh+set} = set; then
@@ -23259,7 +23259,7 @@
 	if test -f ${LOG4CPP_CONFIG}; then
 		{ echo $as_me:$LINENO: WARNING: will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe 5
 echo $as_me: WARNING: will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe 2;}
-	LDFLAGS=`${LOG4CPP_CONFIG} --libs` $LDFLAGS
+	LIBS=`${LOG4CPP_CONFIG} --libs` $LIBS
 	CPPFLAGS=`${LOG4CPP_CONFIG} --cflags` $CPPFLAGS
 		if test ${ac_cv_header_log4cpp_CategoryStream_hh+set} = set; then
   echo $as_me:$LINENO: checking for log4cpp/CategoryStream.hh 5
--- opensaml2-2.3.orig/configure.ac
+++ opensaml2-2.3/configure.ac
@@ -132,7 +132,7 @@
 fi
 ])
 if test -f ${LOG4SHIB_CONFIG}; then
-LDFLAGS=`${LOG4SHIB_CONFIG} --libs` $LDFLAGS
+LIBS=`${LOG4SHIB_CONFIG} --libs` $LIBS
 CPPFLAGS=`${LOG4SHIB_CONFIG} --cflags` $CPPFLAGS
 	AC_CHECK_HEADER([log4shib/CategoryStream.hh],,AC_MSG_ERROR([unable to find log4shib header files]))
 	AC_TRY_LINK(
@@ -157,7 +157,7 @@
 	])
 	if test -f ${LOG4CPP_CONFIG}; then
 		AC_MSG_WARN([will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe])
-	LDFLAGS=`${LOG4CPP_CONFIG} --libs` $LDFLAGS
+	LIBS=`${LOG4CPP_CONFIG} --libs` $LIBS
 	CPPFLAGS=`${LOG4CPP_CONFIG} --cflags` $CPPFLAGS
 		AC_CHECK_HEADER([log4cpp/CategoryStream.hh],,AC_MSG_ERROR([unable to find log4cpp header files]))
 		AC_TRY_LINK(


Bug#606489: shibboleth-sp2: FTBFS with ld --as-needed

2010-12-09 Thread Michael Bienia
Package: shibboleth-sp2
Version: 2.3.1+dfsg-4
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch
User: debian-...@lists.debian.org
Usertags: ld-as-needed

Hello,

here is a patch to fix the FTBFS in Ubuntu with ld --as-needed before it
gets lost in the archives of debian-devel
(http://lists.debian.org/debian-devel/2010/12/msg00090.html).

Michael
--- shibboleth-sp2-2.3.1+dfsg.orig/configure.ac
+++ shibboleth-sp2-2.3.1+dfsg/configure.ac
@@ -157,7 +157,7 @@
 fi
 ])
 if test -f ${LOG4SHIB_CONFIG}; then
-LDFLAGS=`${LOG4SHIB_CONFIG} --libs` $LDFLAGS
+LIBS=`${LOG4SHIB_CONFIG} --libs` $LIBS
 CPPFLAGS=`${LOG4SHIB_CONFIG} --cflags` $CPPFLAGS
 	AC_CHECK_HEADER([log4shib/CategoryStream.hh],,AC_MSG_ERROR([unable to find log4shib header files]))
 	AC_TRY_LINK(
@@ -182,7 +182,7 @@
 	])
 	if test -f ${LOG4CPP_CONFIG}; then
 		AC_MSG_WARN([will try to use log4cpp, note that most non-Internet2 supplied versions are not thread-safe])
-	LDFLAGS=`${LOG4CPP_CONFIG} --libs` $LDFLAGS
+	LIBS=`${LOG4CPP_CONFIG} --libs` $LIBS
 	CPPFLAGS=`${LOG4CPP_CONFIG} --cflags` $CPPFLAGS
 		AC_CHECK_HEADER([log4cpp/CategoryStream.hh],,AC_MSG_ERROR([unable to find log4cpp header files]))
 		AC_TRY_LINK(


Bug#603281: publican: Missing build-dependency on libxml-simple-perl

2010-11-12 Thread Michael Bienia
Package: publican
Version: 2.3-1
Severity: normal

Hello,

building publican in Ubuntu natty fails because libxml-simple-perl is
missing in Build-Depends:

,[ 
http://launchpadlibrarian.net/58474459/buildlog_ubuntu-natty-i386.publican_2.3-1_FAILEDTOBUILD.txt.gz
 ]-
| cd /build/buildd/publican-2.3/Users_Guide  \
|   perl -I /build/buildd/publican-2.3/blib/lib 
/build/buildd/publican-2.3/blib/script/publican build \
|  --formats=html-desktop --publish --langs=all \
|  --common_config=/build/buildd/publican-2.3/blib/datadir \
|  
--common_content=/build/buildd/publican-2.3/blib/datadir/Common_Content
| Can't locate XML/Simple.pm in @INC (@INC contains: 
/build/buildd/publican-2.3/blib/lib /etc/perl /usr/local/lib/perl/5.10.1 
/usr/local/share/perl/5.10.1 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.10 
/usr/share/perl/5.10 /usr/local/lib/site_perl .) at 
/build/buildd/publican-2.3/blib/lib/Publican/WebSite.pm line 19.
| BEGIN failed--compilation aborted at 
/build/buildd/publican-2.3/blib/lib/Publican/WebSite.pm line 19.
| Compilation failed in require at 
/build/buildd/publican-2.3/blib/script/publican line 15.
| BEGIN failed--compilation aborted at 
/build/buildd/publican-2.3/blib/script/publican line 15.
| make[1]: *** [override_dh_auto_build] Error 2
`

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#601013: sqlalchemy: FTBFS with python3.2 (in Ubuntu natty)

2010-10-22 Thread Michael Bienia
Package: sqlalchemy
Version: 0.6.3-3
Severity: normal
Tags: patch

Hi,

python3.2-dev got added to python3-all-dev in Ubuntu natty leading to a
FTBFS of sqlalchemy because it has only a special install target in
debian/rules for python3.1. Copying that target for python3.2 fixes it.

Regards,
Michael

,[ 
http://launchpadlibrarian.net/58038464/buildlog_ubuntu-natty-i386.sqlalchemy_0.6.3-3_FAILEDTOBUILD.txt.gz
 ]-
| python3.2 ./setup.py install --skip-build \
|   --root /build/buildd/sqlalchemy-0.6.3/debian/python3-sqlalchemy 
--install-layout=deb
| running install
| running install_lib
| warning: install_lib: 'build/lib.linux-i686-3.2' does not exist -- no Python 
modules to install
| […]
| # move extensions to -ext binary package
| mkdir -p 
/build/buildd/sqlalchemy-0.6.3/debian/python3-sqlalchemy-ext/usr/lib/python3/dist-packages/sqlalchemy
| mv 
/build/buildd/sqlalchemy-0.6.3/debian/python3-sqlalchemy/usr/lib/python3/dist-packages/sqlalchemy/*so
 \
|  
/build/buildd/sqlalchemy-0.6.3/debian/python3-sqlalchemy-ext/usr/lib/python3/dist-packages/sqlalchemy/
| mv: cannot stat 
`/build/buildd/sqlalchemy-0.6.3/debian/python3-sqlalchemy/usr/lib/python3/dist-packages/sqlalchemy/*so':
 No such file or directory
| make: *** [install-python3.2] Error 1
`


--- sqlalchemy-0.6.3/debian/rules
+++ sqlalchemy-0.6.3/debian/rules
@@ -72,6 +72,11 @@
python3.1 ./setup.py install $(py_setup_install_args) \
--root $(call pkgdir,3.1)
 
+install-python3.2:
+   # no extension for 3.2 yet...
+   python3.2 ./setup.py install $(py_setup_install_args) \
+   --root $(call pkgdir,3.2)
+
 install-python2.5: build-python2.5
# WTF? Why setuptools doesn't work with --skip-build in Python 2.5?
python2.5 ./setup.py install \



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#600560: vim: Add natty to recognized Ubuntu codenames

2010-10-18 Thread Michael Bienia
Package: vim
Version: 2:7.3.000+hg~ee53a39d5896-1
Severity: wishlist
Tags: patch

Hi,

attached is a patch to add the next Ubuntu codename to the syntax
highlighting.

Michael
Index: vim-7.3.000+hg~ee53a39d5896/runtime/syntax/debchangelog.vim
===
--- vim-7.3.000+hg~ee53a39d5896.orig/runtime/syntax/debchangelog.vim	2010-10-17 12:42:46.0 +0200
+++ vim-7.3.000+hg~ee53a39d5896/runtime/syntax/debchangelog.vim	2010-10-17 12:43:32.0 +0200
@@ -19,7 +19,7 @@
  Define some common expressions we can use later on
 syn match debchangelogName	contained ^[[:alnum:]][[:alnum:].+-]\+ 
 syn match debchangelogUrgency	contained ; urgency=\(low\|medium\|high\|critical\|emergency\)\( \S.*\)\=
-syn match debchangelogTarget	contained \v %(frozen|unstable|%(testing|%(old)=stable)%(-proposed-updates|-security)=|experimental|%(etch|lenny)-%(backports|volatile)|%(dapper|hardy|jaunty|karmic|lucid|maverick)%(-%(security|proposed|updates|backports|commercial|partner))=)+
+syn match debchangelogTarget	contained \v %(frozen|unstable|%(testing|%(old)=stable)%(-proposed-updates|-security)=|experimental|%(etch|lenny)-%(backports|volatile)|%(dapper|hardy|jaunty|karmic|lucid|maverick|natty)%(-%(security|proposed|updates|backports|commercial|partner))=)+
 syn match debchangelogVersion	contained (.\{-})
 syn match debchangelogCloses	contained closes:\_s*\(bug\)\=#\=\_s\=\d\+\(,\_s*\(bug\)\=#\=\_s\=\d\+\)*
 syn match debchangelogLP	contained \clp:\s\+#\d\+\(,\s*#\d\+\)*
Index: vim-7.3.000+hg~ee53a39d5896/runtime/syntax/debsources.vim
===
--- vim-7.3.000+hg~ee53a39d5896.orig/runtime/syntax/debsources.vim	2010-10-17 12:43:54.0 +0200
+++ vim-7.3.000+hg~ee53a39d5896/runtime/syntax/debsources.vim	2010-10-17 12:45:04.0 +0200
@@ -23,7 +23,7 @@
 
  Match uri's
 syn match debsourcesUri+\(http://\|ftp://\|[rs]sh://\|debtorrent://\|\(cdrom\|copy\|file\):\)[^' 	]\++
-syn match debsourcesDistrKeyword   +\([[:alnum:]_./]*\)\(etch\|lenny\|squeeze\|\(old\)\=stable\|testing\|unstable\|sid\|rc-buggy\|experimental\|dapper\|hardy\|jaunty\|karmic\|lucid\|maverick\)\([-[:alnum:]_./]*\)+
+syn match debsourcesDistrKeyword   +\([[:alnum:]_./]*\)\(etch\|lenny\|squeeze\|\(old\)\=stable\|testing\|unstable\|sid\|rc-buggy\|experimental\|dapper\|hardy\|jaunty\|karmic\|lucid\|maverick\|natty\)\([-[:alnum:]_./]*\)+
 
  Associate our matches and regions with pretty colours
 hi def link debsourcesLineError


Bug#600562: vim: Add Ubuntu's metapackages section to syntax highlighting

2010-10-18 Thread Michael Bienia
Package: vim
Version: 2:7.3.000+hg~ee53a39d5896-1
Severity: wishlist
Tags: patch

Hi,

Ubuntu caries a patch to add their metapackages section to syntax
hightlighting. It would be nice if that patch could get included in the
Debian package.

Michael
Index: vim-7.3.000+hg~ee53a39d5896/runtime/syntax/debcontrol.vim
===
--- vim-7.3.000+hg~ee53a39d5896.orig/runtime/syntax/debcontrol.vim	2010-10-17 12:49:06.0 +0200
+++ vim-7.3.000+hg~ee53a39d5896/runtime/syntax/debcontrol.vim	2010-10-17 12:50:28.0 +0200
@@ -27,7 +27,7 @@
 syn match debcontrolArchitecture contained \%(all\|any\|alpha\|amd64\|arm\%(e[bl]\)\=\|avr32\|hppa\|i386\|ia64\|lpia\|m32r\|m68k\|mips\%(el\)\=\|powerpc\|ppc64\|s390x\=\|sh[34]\(eb\)\=\|sh\|sparc\%(64\)\=\|hurd-i386\|kfreebsd-\%(i386\|amd64\|gnu\)\|knetbsd-i386\|kopensolaris-i386\|netbsd-\%(alpha\|i386\)\)
 syn match debcontrolName contained [a-z0-9][a-z0-9+.-]\+
 syn match debcontrolPriority contained \(extra\|important\|optional\|required\|standard\)
-syn match debcontrolSection contained \v((contrib|non-free|non-US/main|non-US/contrib|non-US/non-free|restricted|universe|multiverse)/)?(admin|cli-mono|comm|database|debian-installer|debug|devel|doc|editors|electronics|embedded|fonts|games|gnome|gnustep|gnu-r|graphics|hamradio|haskell|httpd|interpreters|java|kde|kernel|libs|libdevel|lisp|localization|mail|math|misc|net|news|ocaml|oldlibs|otherosfs|perl|php|python|ruby|science|shells|sound|text|tex|utils|vcs|video|web|x11|xfce|zope)
+syn match debcontrolSection contained \v((contrib|non-free|non-US/main|non-US/contrib|non-US/non-free|restricted|universe|multiverse)/)?(admin|cli-mono|comm|database|debian-installer|debug|devel|doc|editors|electronics|embedded|fonts|games|gnome|gnustep|gnu-r|graphics|hamradio|haskell|httpd|interpreters|java|kde|kernel|libs|libdevel|lisp|localization|mail|math|metapackages|misc|net|news|ocaml|oldlibs|otherosfs|perl|php|python|ruby|science|shells|sound|text|tex|utils|vcs|video|web|x11|xfce|zope)
 syn match debcontrolPackageType contained u\?deb
 syn match debcontrolVariable contained \${.\{-}}
 syn match debcontrolDmUpload contained \cyes


Bug#598600: ascii2binary: Upstream's endian.h conflicts with system's endian.h causing a FTBFS in Ubuntu

2010-09-30 Thread Michael Bienia
Package: ascii2binary
Version: 2.13-1
Severity: normal
Tags: patch

Hello,

ascii2binary FTBFS in Ubuntu maverick with the following error:
,[ 
http://people.ubuntuwire.org/~lucas/ubuntu-nbs/32/ascii2binary_2.13-1_lubuntu32.buildlog
 ]-
| In file included from //usr/include/stdlib.h:43,
|  from binary2ascii.c:27:
| //usr/include/bits/waitstatus.h:80: error: duplicate member '__w_retcode'
| //usr/include/bits/waitstatus.h:80: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:80: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '__w_coredump'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '__w_termsig'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:94: error: duplicate member '__w_stopsig'
| //usr/include/bits/waitstatus.h:94: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:94: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '__w_stopval'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '({anonymous})'
`
The problem is that upstream code contains an endian.h and the source
code is compiled with -I. This combination causes that including
stdlib.h pulls the wrong endian.h (the local one and not the system one)
leading to the above error as the guarding #if constants aren't defined
anymore.

A solution is to rename the local endian.h so the correct header is
included again. The attached patch renames endian.h to a2b_endian.h (the
rename is not that good visible in the debdiff).

(This is similar to bug #594826 in uni2ascii)

Michael
--- ascii2binary-2.13.orig/Makefile.am
+++ ascii2binary-2.13/Makefile.am
@@ -2,7 +2,7 @@
 dist_man_MANS = binary2ascii.1 ascii2binary.1
 binary2ascii_SOURCES = binary2ascii.c binfmt.c MachineInfo.c endian.c exitcodes.c
 ascii2binary_SOURCES = ascii2binary.c GetWord.c MachineInfo.c endian.c exitcodes.c
-noinst_HEADERS = exitcodes.h endian.h
+noinst_HEADERS = exitcodes.h a2b_endian.h
 LOCALEDIR = /usr/local/share/ascii2binary/locale
 AM_CFLAGS=-std=gnu99
 ACLOCAL_AMFLAGS = -I m4
only in patch2:
unchanged:
--- ascii2binary-2.13.orig/MachineInfo.c
+++ ascii2binary-2.13/MachineInfo.c
@@ -8,7 +8,7 @@
 #else
 #define gettext(x) (x)
 #endif
-#include endian.h
+#include a2b_endian.h
 
 void
 Print_Byte_Order(void)
only in patch2:
unchanged:
--- ascii2binary-2.13.orig/a2b_endian.h
+++ ascii2binary-2.13/a2b_endian.h
@@ -0,0 +1,6 @@
+/* Constants defining endian-ness values */
+
+#define E_LITTLE_ENDIAN 0
+#define E_BIG_ENDIAN 1
+#define E_PDP_ENDIAN 2
+#define E_UNKNOWN_ENDIAN 3
only in patch2:
unchanged:
--- ascii2binary-2.13.orig/endian.c
+++ ascii2binary-2.13/endian.c
@@ -19,7 +19,7 @@
  *
  */
 
-#include endian.h
+#include a2b_endian.h
 
 /* Returns a value indicating the endianness of the machine */
 
only in patch2:
unchanged:
--- ascii2binary-2.13.orig/Makefile.in
+++ ascii2binary-2.13/Makefile.in
@@ -163,7 +163,7 @@
 dist_man_MANS = binary2ascii.1 ascii2binary.1
 binary2ascii_SOURCES = binary2ascii.c binfmt.c MachineInfo.c endian.c exitcodes.c
 ascii2binary_SOURCES = ascii2binary.c GetWord.c MachineInfo.c endian.c exitcodes.c
-noinst_HEADERS = exitcodes.h endian.h
+noinst_HEADERS = exitcodes.h a2b_endian.h
 LOCALEDIR = /usr/local/share/ascii2binary/locale
 AM_CFLAGS = -std=gnu99
 ACLOCAL_AMFLAGS = -I m4


Bug#594765: apt-cacher-ng: FTBFS in Ubuntu maverick

2010-08-29 Thread Michael Bienia
Package: apt-cacher-ng
Version: 0.5.3-1
Severity: normal
Tags: experimental

Hi,

I tried building apt-cacher-ng 0.5.3-1 from experimental in my Ubuntu
maverick pbuilder and it FTBFS:
,
| g++ -c -Wall -g -O2 -D_FILE_OFFSET_BITS=64 
-I/tmp/buildd/apt-cacher-ng-0.5.3/include  source/job.cc -o job.o
| source/job.cc: In member function 'void job::HandleLocalDownload(const 
std::string, const std::string, const std::string)':
| source/job.cc:288: error: no matching function for call to 'stat::stat(const 
char*, stat*)'
| //usr/include/bits/stat.h:47: note: candidates are: stat::stat()
| //usr/include/bits/stat.h:47: note: stat::stat(const stat)
| source/job.cc:336: error: 'S_ISDIR' was not declared in this scope
| source/job.cc:387: error: no matching function for call to 'stat::stat(const 
char*, stat*)'
| //usr/include/bits/stat.h:47: note: candidates are: stat::stat()
| //usr/include/bits/stat.h:47: note: stat::stat(const stat)
| source/job.cc:421: error: 'S_ISREG' was not declared in this scope
| make[1]: *** [job.o] Error 1
| make[1]: Leaving directory `/tmp/buildd/apt-cacher-ng-0.5.3'
`

Michael

PS: Thanks for writing apt-cacher-ng. I use it for sharing debs between
my host system and my pbuilders and it works really great.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#594826: uni2ascii: Upstream's endian.h conflicts with system's endian.h causing a FTBFS in Ubuntu

2010-08-29 Thread Michael Bienia
Package: uni2ascii
Version: 4.14-3
Severity: normal
Tags: patch

Hello,

uni2ascii FTBFS in Ubuntu maverick with the following error:
,[ 
http://launchpadlibrarian.net/49745563/buildlog_ubuntu-maverick-i386.uni2ascii_4.14-3_FAILEDTOBUILD.txt.gz
 ]-
| if i686-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I. -I.-DNEWSUMMARY  -g -O2 -MT 
enttbl.o -MD -MP -MF .deps/enttbl.Tpo -c -o enttbl.o enttbl.c; \
|   then mv -f .deps/enttbl.Tpo .deps/enttbl.Po; else rm -f 
.deps/enttbl.Tpo; exit 1; fi
| In file included from //usr/include/stdlib.h:43,
|  from enttbl.c:22:
| //usr/include/bits/waitstatus.h:80: error: duplicate member '__w_retcode'
| //usr/include/bits/waitstatus.h:80: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:80: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '__w_coredump'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:81: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '__w_termsig'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:82: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:94: error: duplicate member '__w_stopsig'
| //usr/include/bits/waitstatus.h:94: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:94: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '__w_stopval'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '({anonymous})'
| //usr/include/bits/waitstatus.h:95: error: duplicate member '({anonymous})'
| make[2]: *** [enttbl.o] Error 1
`

The problem is that upstream code contains an endian.h and the source
code is compiled with -I. This combination causes that including
stdlib.h pulls the wrong endian.h (the local one and not the system one)
leading to the above error as the guarding #if constants aren't defined
anymore.

A solution is to rename the local endian.h so the correct header is
included again. The attached patch renames endian.h to myendian.h.

Michael
Subject: rename endian.h to myendian.h to not conflict anymore with the system endian.h
From: Michael Bienia ge...@ubuntu.com

Index: uni2ascii-4.14/Makefile.am
===
--- uni2ascii-4.14.orig/Makefile.am	2010-08-29 23:39:46.0 +0200
+++ uni2ascii-4.14/Makefile.am	2010-08-29 23:39:53.0 +0200
@@ -3,7 +3,7 @@
 dist_man_MANS = uni2ascii.1 ascii2uni.1
 uni2ascii_SOURCES = endian.c enttbl.c SetFormat.c uni2ascii.c UTF8in.c putu8.c
 ascii2uni_SOURCES = ascii2uni.c enttbl.c GetWord.c putu8.c SetFormat.c
-noinst_HEADERS = endian.h enttbl.h exitcode.h formats.h unicode.h utf8error.h
+noinst_HEADERS = myendian.h enttbl.h exitcode.h formats.h unicode.h utf8error.h
 AM_CFLAGS =
 if NEWSUMMARY
 AM_CFLAGS += -DNEWSUMMARY
Index: uni2ascii-4.14/Makefile.in
===
--- uni2ascii-4.14.orig/Makefile.in	2010-08-29 23:39:55.0 +0200
+++ uni2ascii-4.14/Makefile.in	2010-08-29 23:40:03.0 +0200
@@ -174,7 +174,7 @@
 dist_man_MANS = uni2ascii.1 ascii2uni.1
 uni2ascii_SOURCES = endian.c enttbl.c SetFormat.c uni2ascii.c UTF8in.c putu8.c
 ascii2uni_SOURCES = ascii2uni.c enttbl.c GetWord.c putu8.c SetFormat.c
-noinst_HEADERS = endian.h enttbl.h exitcode.h formats.h unicode.h utf8error.h
+noinst_HEADERS = myendian.h enttbl.h exitcode.h formats.h unicode.h utf8error.h
 AM_CFLAGS = $(am__append_1) $(am__append_2)
 AUTOMAKE_OPTIONS = dist-zip dist-bzip2
 EXTRA_DIST = TestSuiteAscii2Uni CREDITS uni2html.py ascii2uni.py u2a.tcl uni2ascii.info uni2ascii-${VERSION}.lsm
Index: uni2ascii-4.14/endian.c
===
--- uni2ascii-4.14.orig/endian.c	2010-08-29 23:39:04.0 +0200
+++ uni2ascii-4.14/endian.c	2010-08-29 23:39:18.0 +0200
@@ -17,7 +17,7 @@
  * or go to the web page:  http://www.gnu.org/licenses/gpl.txt.
  */
 
-#include endian.h
+#include myendian.h
 
 /* Returns a value indicating the endianness of the machine */
 
Index: uni2ascii-4.14/endian.h
===
--- uni2ascii-4.14.orig/endian.h	2010-08-29 23:38:28.0 +0200
+++ /dev/null	1970-01-01 00:00:00.0 +
@@ -1,11 +0,0 @@
-#ifndef GET_ENDIAN
-extern short Get_Endianness(void);
-#endif
-#define GET_ENDIAN
-
-/* Constants defining endian-ness values */
-
-#define E_LITTLE_ENDIAN 0
-#define E_BIG_ENDIAN 1
-#define

Bug#590291: apt-cacher-ng 0.5.1 still missing the needed include in source/acfg.cc for umask() and mkdir()

2010-08-01 Thread Michael Bienia
found 590291 0.5.1-1
thanks

Hi,

I tried compiling apt-cacher-ng 0.5.1-1 in my Ubuntu maverick pbuilder
and it failed:

,
| g++ -c -Wall -g -O2 -D_FILE_OFFSET_BITS=64 
-I/tmp/buildd/apt-cacher-ng-0.5.1/include  source/acfg.cc -o acfg.o
| source/acfg.cc: In function 'void acfg::PostProcConfig()':
| source/acfg.cc:633: error: 'umask' was not declared in this scope
| source/acfg.cc: In function 'void mkbasedir(const std::string)':
| source/acfg.cc:923: error: 'mkdir' was not declared in this scope
| make[1]: *** [acfg.o] Error 1
| make[1]: Leaving directory `/tmp/buildd/apt-cacher-ng-0.5.1'
| make: *** [build-stamp] Error 2
`

Looks like source/acfg.cc should also include fileio.h.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#589141: drivel: FTBFS on Ubuntu with error: format not a string literal and no format arguments

2010-07-15 Thread Michael Bienia
Package: drivel
Version: 3.0.1-1
Severity: normal
Tags: patch

Hi,

as the Ubuntu gcc has some additional security checks enabled, drivel fails to
build with:
,
| blog_lj.c: In function 'parse_login_request':
| blog_lj.c:255: error: format not a string literal and no format arguments
`

Here is also a small patch to fix it.

Regards,
Michael

--- drivel-3.0.1.orig/src/blog_lj.c
+++ drivel-3.0.1/src/blog_lj.c
@@ -252,6 +252,7 @@
GTK_DIALOG_DESTROY_WITH_PARENT,
GTK_MESSAGE_INFO,
GTK_BUTTONS_OK,
+   %s,
message);

gtk_dialog_run (GTK_DIALOG (dialog));
--- drivel-3.0.1.orig/src/dialogs.c
+++ drivel-3.0.1/src/dialogs.c
@@ -1327,6 +1327,7 @@
GTK_DIALOG_DESTROY_WITH_PARENT,
GTK_MESSAGE_WARNING,
GTK_BUTTONS_NONE,
+   %s,
msg);
gtk_dialog_add_buttons (GTK_DIALOG (dialog), GTK_STOCK_NO,
GTK_RESPONSE_REJECT,
@@ -1363,6 +1364,7 @@
GTK_DIALOG_DESTROY_WITH_PARENT,
GTK_MESSAGE_WARNING,
GTK_BUTTONS_NONE,
+   %s,
msg);
gtk_dialog_add_buttons (GTK_DIALOG (dialog),
GTK_STOCK_NO, GTK_RESPONSE_REJECT,
--- drivel-3.0.1.orig/src/main.c
+++ drivel-3.0.1/src/main.c
@@ -73,6 +73,7 @@
GTK_DIALOG_DESTROY_WITH_PARENT,
GTK_MESSAGE_ERROR,
GTK_BUTTONS_CLOSE,
+   %s,
error_string);
 
gtk_dialog_run (GTK_DIALOG (dialog));



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#588778: Function `sapwood_socket_path_get_default' implicitly converted to pointer

2010-07-12 Thread Michael Bienia
Package: sapwood
Version: 3.1.1.debian.10+0m5-1
Severity: normal
Tags: patch

Hi,

the Ubuntu buildds for amd64 and ia64 check for functions implicitly
converted to pointers and fails if it finds them.

,[ 
http://launchpadlibrarian.net/48786369/buildlog_ubuntu-maverick-amd64.sapwood_3.1.1.debian.10%2B0m5-1_FAILEDTOBUILD.txt.gz
 ]-
| sapwood-client.c: In function 'sapwood_client_get_socket':
| sapwood-client.c:60: warning: implicit declaration of function 
'sapwood_socket_path_get_default'
| sapwood-client.c:60: warning: assignment makes pointer from integer without a 
cast
| Function `sapwood_socket_path_get_default' implicitly converted to pointer at 
sapwood-client.c:60
`
More information can be found at:
http://wiki.debian.org/ImplicitPointerConversions

Attached is a patch to add the missing include for the declarations.

Regards,
Michael
Index: sapwood-3.1.1.debian.10+0m5/engine/sapwood-client.c
===
--- sapwood-3.1.1.debian.10+0m5.orig/engine/sapwood-client.c	2010-07-11 19:33:21.0 +0200
+++ sapwood-3.1.1.debian.10+0m5/engine/sapwood-client.c	2010-07-11 19:33:23.0 +0200
@@ -32,6 +32,7 @@
 #include unistd.h
 
 #include sapwood-client.h
+#include ../protocol/sapwood-proto.h
 
 GQuark
 sapwood_client_get_error_quark (void)


Bug#588440: tex4ht: Versioned build dependency on default-jdk-builddep which is now provided by gcj-native-helper

2010-07-08 Thread Michael Bienia
Package: tex4ht
Version: 20090611-1
Severity: normal

Hi,

tex4ht has a versioned build dependency on default-jdk-builddep which
got renamed by a recent java-common upload to gcj-native-helper.
gcj-native-helper provides default-jdk-builddep but provides only work
on unversioned dependencies.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#571008: still exists

2010-06-19 Thread Michael Bienia
On 2010-04-07 10:11:27 -0600, dann frazier wrote:
 Looks like this issue still exists in 0.8.1-1.2.

The patch got added to -1.2 but forgotten to also add it to
debian/patches/00list. Therefore it doesn't get applied during build.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#584099: ipgrab: Function `my_malloc' implicitly converted to pointer at parse_cl.c:206

2010-06-01 Thread Michael Bienia
Package: ipgrab
Version: 0.9.10-1
Severity: normal
Tags: patch

Hi,

as the Ubuntu buildds check for implicit pointer conversion during the
build, ipgrab failed to build on 64 bit architectures (AMD64 and IA64)
in Ubuntu:

,[ 
http://launchpadlibrarian.net/48792489/buildlog_ubuntu-maverick-amd64.ipgrab_0.9.10-1_FAILEDTOBUILD.txt.gz
 ]-
| parse_cl.c: In function 'free_args':
| parse_cl.c:163: warning: implicit declaration of function 'my_free'
| parse_cl.c: In function 'Cmdline':
| parse_cl.c:206: warning: implicit declaration of function 'my_malloc'
| parse_cl.c:206: warning: cast to pointer from integer of different size
| Function `my_malloc' implicitly converted to pointer at parse_cl.c:206
`
And at the end of the build log:
,
| Function `my_malloc' implicitly converted to pointer at parse_cl.c:206
| 
| Our automated build log filter detected the problem(s) above that will
| likely cause your package to segfault on architectures where the size of
| a pointer is greater than the size of an integer, such as ia64 and amd64.
| 
| This is often due to a missing function prototype definition.
| 
| Since use of implicitly converted pointers is always fatal to the application
| on ia64, they are errors.  Please correct them for your next upload.
| 
| More information can be found at:
| http://wiki.debian.org/ImplicitPointerConversions
`

Attached is a patch to add the missing include and fix it.

Michael

Index: ipgrab-0.9.10/src/parse_cl.c
===
--- ipgrab-0.9.10.orig/src/parse_cl.c   2010-06-01 11:13:40.0 +0200
+++ ipgrab-0.9.10/src/parse_cl.c2010-06-01 11:13:56.0 +0200
@@ -20,6 +20,7 @@
 #include stdlib.h
 #include getopt.h
 #include parse_cl.h
+#include utilities.h
 
 /*
 **



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#583594: php-doc: FTBFS in Ubuntu

2010-05-28 Thread Michael Bienia
Package: php-doc
Version: 20100521-1
Severity: normal
Tags: patch

Hi,

php-doc 20100521-1 fails to build in Ubuntu maverick:

,[ 
http://launchpadlibrarian.net/49075473/buildlog_ubuntu-maverick-i386.php-doc_20100521-1_FAILEDTOBUILD.txt.gz
 ]-
| rm doc-base/.manual.xml doc-base/version.xml doc-base/manual.xml \
|   doc-base/scripts/file-entities.php \
|   doc-base/entities/version.ent doc-base/entities/file-entities.ent
| rm: cannot remove `doc-base/.manual.xml': No such file or directory
| rm: cannot remove `doc-base/version.xml': No such file or directory
| rm: cannot remove `doc-base/manual.xml': No such file or directory
| rm: cannot remove `doc-base/scripts/file-entities.php': No such file or 
directory
| rm: cannot remove `doc-base/entities/version.ent': No such file or directory
| rm: cannot remove `doc-base/entities/file-entities.ent': No such file or 
directory
| make: *** [clean] Error 1
`

Replacing the rm call with rm -f in debian/rules fixes it.

Michael

diff -u php-doc-20100521/debian/rules php-doc-20100521/debian/rules
--- php-doc-20100521/debian/rules
+++ php-doc-20100521/debian/rules
@@ -30,7 +30,7 @@
#-$(MAKE) distclean
find en -name entities.\*.xml | xargs rm -f
find doc-base/ -name \*.ser | xargs rm -f
-   rm doc-base/.manual.xml doc-base/version.xml doc-base/manual.xml \
+   rm -f doc-base/.manual.xml doc-base/version.xml doc-base/manual.xml \
doc-base/scripts/file-entities.php \
doc-base/entities/version.ent doc-base/entities/file-entities.ent
rm -rf html



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#581819: gimp-plugin-registry: Function `lcms_open_profile' implicitly converted to pointer at separate-core.c:142

2010-05-16 Thread Michael Bienia
Package: gimp-plugin-registry
Version: 3.1-1
Severity: normal
Tags: patch

Hello,

gimp-plugin-registry 3.1-1 failed to build in Ubuntu Maverick on AMD64
(and IA64) as the buildd checks for implicitly converted pointers on
64bit architectures:

,[ 
http://launchpadlibrarian.net/48305399/buildlog_ubuntu-maverick-amd64.gimp-plugin-registry_3.1-1_FAILEDTOBUILD.txt.gz
 ]-
| make[2]: Entering directory 
`/build/buildd/gimp-plugin-registry-3.1/separate+/separate+-0.5.6'
| gcc -g -Wall -g -O2 -fno-tree-pre `pkg-config gimp-2.0 --cflags` `pkg-config 
gimpui-2.0 --cflags` `pkg-config gtk+-2.0 --cflags` `pkg-config lcms --cflags` 
-DENABLE_COLOR_MANAGEMENT -DENABLE_NLS -DGETTEXT_PACKAGE=\gimp20-separate\ 
-DSEPARATE_SEPARATE -DUSE_ICC_BUTTON -c separate-core.c
| separate-core.c: In function 'setup_transform':
| separate-core.c:142: warning: implicit declaration of function 
'lcms_open_profile'
| separate-core.c:142: warning: assignment makes pointer from integer without a 
cast
| Function `lcms_open_profile' implicitly converted to pointer at 
separate-core.c:142
`

Attached is a patch to #include the missing header file.

Regards,
Michael
Index: gimp-plugin-registry-3.1/separate+/separate+-0.5.6/separate-core.c
===
--- gimp-plugin-registry-3.1.orig/separate+/separate+-0.5.6/separate-core.c 
2010-05-16 11:47:24.0 +0200
+++ gimp-plugin-registry-3.1/separate+/separate+-0.5.6/separate-core.c  
2010-05-16 11:49:21.0 +0200
@@ -35,6 +35,8 @@
 #include separate-core.h
 #include util.h
 
+#include iccbutton.h
+
 
 
 #ifdef ENABLE_COLOR_MANAGEMENT


Bug#569368: squishdot: FTBFS: /bin/sh: dos2unix: not found

2010-03-12 Thread Michael Bienia
On 2010-02-11 20:08:04 +0100, Lucas Nussbaum wrote:
  /bin/sh: dos2unix: not found

diff -u squishdot-1.5.0/debian/rules squishdot-1.5.0/debian/rules
--- squishdot-1.5.0/debian/rules
+++ squishdot-1.5.0/debian/rules
@@ -27,7 +27,7 @@
dh_clean -k
dh_installdirs
dh_installzope Squishdot/
-   dos2unix `find debian/squishdot/${INSTDIR} -name *.py`
+   fromdos `find debian/squishdot/${INSTDIR} -name *.py`
touch install-stamp
 
 # We have nothing to do by default.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#569446: xtalk: FTBFS: /bin/sh: dos2unix: not found

2010-03-12 Thread Michael Bienia
On 2010-02-11 20:08:11 +0100, Lucas Nussbaum wrote:
  /bin/sh: dos2unix: not found

diff -u xtalk-1.3/debian/rules xtalk-1.3/debian/rules
--- xtalk-1.3/debian/rules
+++ xtalk-1.3/debian/rules
@@ -33,7 +33,7 @@
dh_installdirs
 
$(MAKE) DESTDIR=$(CURDIR)/debian/xtalk install
-   dos2unix debian/xtalk/usr/share/xtalk/*.py \
+   fromdos debian/xtalk/usr/share/xtalk/*.py \
 debian/xtalk/usr/bin/xtalk
 
 binary-indep: build install



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#571008: `stringToQualifiedNameList' implicitly converted to pointer

2010-03-12 Thread Michael Bienia
On 2010-02-22 13:10:00 -0700, dann frazier wrote:
   Function `stringToQualifiedNameList' implicitly converted to pointer at 
 plluaspi.c:541

This is because of missing includes. The attached dpatch fixes it.

Michael
#! /bin/sh /usr/share/dpatch/dpatch-run
## 21_missing_includes.dpatch by Michael Bienia ge...@ubuntu.com
##
## All lines beginning with `## DP:' are a description of the patch.
## DP: Add missing PostgreSQL includes.

@DPATCH@
diff -urNad postgresql-pllua-0.8.1~/plluaapi.c postgresql-pllua-0.8.1/plluaapi.c
--- postgresql-pllua-0.8.1~/plluaapi.c  2010-03-12 17:28:30.0 +0100
+++ postgresql-pllua-0.8.1/plluaapi.c   2010-03-12 17:29:25.0 +0100
@@ -10,6 +10,7 @@
 
 #include utils/builtins.h
 #include utils/guc.h
+#include utils/datum.h
 
 /* extended function info */
 typedef struct luaP_Info {
diff -urNad postgresql-pllua-0.8.1~/plluaspi.c postgresql-pllua-0.8.1/plluaspi.c
--- postgresql-pllua-0.8.1~/plluaspi.c  2008-04-01 01:53:07.0 +0200
+++ postgresql-pllua-0.8.1/plluaspi.c   2010-03-12 17:30:03.0 +0100
@@ -6,6 +6,7 @@
  */
 
 #include pllua.h
+#include utils/builtins.h
 
 #ifndef SPI_prepare_cursor
 #define SPI_prepare_cursor(cmd, nargs, argtypes, copts) \


Bug#572634: gtk-led-askpass: FTBFS with error: format not a string literal and no format arguments in Ubuntu

2010-03-05 Thread Michael Bienia
Package: gtk-led-askpass
Version: 0.10-2
Severity: normal
Tags: patch

Hello,

as the Ubuntu gcc has some additional checks enabled by default,
gtk-led-askpass FTBFS with the following error message:
,
| make[1]: Entering directory `/tmp/buildd/gtk-led-askpass-0.10'
| cc -Wall -Werror `pkg-config --cflags gtk+-2.0` -g -O2 -c gtk-led-askpass.c 
-o gtk-led-askpass.o
| cc1: warnings being treated as errors
| gtk-led-askpass.c: In function 'fail':
| gtk-led-askpass.c:85: error: format not a string literal and no format 
arguments
| gtk-led-askpass.c: In function 'main':
| gtk-led-askpass.c:245: error: format not a string literal and no format 
arguments
| make[1]: *** [gtk-led-askpass.o] Error 1
| make[1]: Leaving directory `/tmp/buildd/gtk-led-askpass-0.10'
| make: *** [build-stamp] Error 2
`

Michael

--- gtk-led-askpass-0.10.orig/gtk-led-askpass.c
+++ gtk-led-askpass-0.10/gtk-led-askpass.c
@@ -82,7 +82,7 @@
 void fail(gchar *message)
 {
GtkWidget *dialog = gtk_message_dialog_new(NULL, 0, GTK_MESSAGE_ERROR,
-   GTK_BUTTONS_CLOSE, message);
+   GTK_BUTTONS_CLOSE, %s, message);
 
gtk_window_set_position(GTK_WINDOW(dialog), GTK_WIN_POS_CENTER);
gtk_dialog_run(GTK_DIALOG(dialog));
@@ -242,7 +242,7 @@
 
/* Question dialog with no parent; OK and Cancel buttons. */
dialog = gtk_message_dialog_new(NULL, 0, GTK_MESSAGE_QUESTION,
-   GTK_BUTTONS_OK_CANCEL, message);
+   GTK_BUTTONS_OK_CANCEL, %s, message);
gtk_window_set_title(GTK_WINDOW(dialog), TITLE);
/* Place the dialog in the middle of the screen. */
gtk_window_set_position(GTK_WINDOW(dialog), GTK_WIN_POS_CENTER);



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#569118: php-imlib: FTBFS with php 5.3

2010-03-05 Thread Michael Bienia
On 2010-02-10 00:39:21 -0600, Raphael Geissert wrote:
  /tmp/buildd/php-imlib-0.7/imlib.c:46: error: duplicate 'static'
  /tmp/buildd/php-imlib-0.7/imlib.c:54: error: duplicate 'static'
  /tmp/buildd/php-imlib-0.7/imlib.c:77: error: 'second_arg_force_ref'
  undeclared here (not in a function)
  /tmp/buildd/php-imlib-0.7/imlib.c:114:
  error: 'third_arg_force_ref' undeclared here (not in a function)

After googling a little bit I found the commit which removed those
declarations from zend_modules.h:
http://github.com/php/php-src/commit/d09789e53096b5afe625240172ee397e60198289

Copying the needed declarations into imlib.c lets it build again.
Attached is a patch to fix the FTBFS (the deprecation warnings are still
there). It would be good if an user of it could confirm that is still
works as expected (didn't had time to test it yet).

Michael

--- php-imlib-0.7/imlib.c
+++ php-imlib-0.7/imlib.c
@@ -42,7 +42,17 @@
 
 ZEND_DECLARE_MODULE_GLOBALS(imlib);
 
-static
+   ZEND_BEGIN_ARG_INFO(second_arg_force_ref, 0)
+   ZEND_ARG_PASS_INFO(0)
+   ZEND_ARG_PASS_INFO(1)
+   ZEND_END_ARG_INFO();
+
+   ZEND_BEGIN_ARG_INFO(third_arg_force_ref, 0)
+   ZEND_ARG_PASS_INFO(0)
+   ZEND_ARG_PASS_INFO(0)
+   ZEND_ARG_PASS_INFO(1)
+   ZEND_END_ARG_INFO();
+
ZEND_BEGIN_ARG_INFO(third_and_fourth_arg_force_ref, 0)
ZEND_ARG_PASS_INFO(0)
ZEND_ARG_PASS_INFO(0)
@@ -50,7 +60,6 @@
ZEND_ARG_PASS_INFO(1)
ZEND_END_ARG_INFO();
 
-static
ZEND_BEGIN_ARG_INFO(second_through_fifth_arg_force_ref, 0)
ZEND_ARG_PASS_INFO(0)
ZEND_ARG_PASS_INFO(1)



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#562364: transfig build failure

2010-03-05 Thread Michael Bienia
On 2010-02-16 15:47:36 -0800, Daniel Schepler wrote:
 I'm seeing this build failure still (or maybe again?) with texlive-extra 
 version 2009-7, which is supposed to have #562497 fixed:

The problem now seems to be that epstopdf ignores the --outfile in
--filter mode. I don't know if epstopdf itself is buggy or only its
documentation as epstopdf -h still lists -f in combination with -o in
the examples.

Anyways, with the following patch and texlive-font-utils instead of
texlive-extra-utils in Build-Depends the package builds again.

Michael

--- transfig-3.2.5.a/debian/patches/14_fig2mpdf.dpatch
+++ transfig-3.2.5.a/debian/patches/14_fig2mpdf.dpatch
@@ -318,7 +318,7 @@
 +  if (iPdfOutputs)
 +  {
 +  fprintf(ptCreateFile,   %s -L pstex -D +%d:%d %s, prog, 
depth, iStartDepth, from, szFileName);
-+  fprintf(ptCreateFile,  | epstopdf -f --outfile %s.pdf\n, 
szFileName, szFileName);
++  fprintf(ptCreateFile,  | epstopdf -f  %s.pdf\n, szFileName, 
szFileName);
 +  }
 +  else
 +  fprintf(ptCreateFile, %s -L pstex -D +%d:%d %s %s.eps\n, 
prog, depth, iStartDepth, from, szFileName);



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#572700: gsl-doc: Missing build-dependency on texlive-font-utils

2010-03-05 Thread Michael Bienia
Package: gsl-doc
Version: 1.13-1
Severity: normal

Hello,

during an archive rebuild test in Ubuntu lucid gsl-doc FTBFS.
,[ 
http://people.ubuntuwire.org/~lucas/ubuntu-nbs/32/gsl-doc_1.13-1_llucid32.buildlog
 ]-
| Creating output file random-walk.eps...ready.
| /bin/sh: Converting 12-cities.eps to pdf...
| epstopdf: not found
| Converting bspline.eps to pdf...
| /bin/sh: epstopdf: not found
| Converting cheb.eps to pdf...
| /bin/sh: epstopdf: not found
| Converting dwt-orig.eps to pdf...
| /bin/sh: epstopdf: not found
| Converting dwt-samp.eps to pdf...
| /bin/sh: epstopdf: not found
`
Replacing texlive-extra-utils with texlive-font-utils in Build-Depends fixes
it.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#571637: python-reportlab-doc is missing the documentation

2010-02-26 Thread Michael Bienia
Package: python-reportlab-doc
Version: 2.3-1
Severity: normal

After a local backport of python-reportlab from unstable to lenny, I
noticed that my -doc package is missing the documention. I first assumed
that it got lost during my backporting, but after checking the package
contents as listed on packages.debian.org I noticed that they don't list
it either.

According to the package description the -doc package should contain the
documentation as PDF but it only contains examples [1,2]

From a look at the deb sizes mentioned in the -changes mail I assume
that the PDFs got already lost in the 2.2-1 upload [3,4].

Michael

1: http://packages.debian.org/squeeze/all/python-reportlab-doc/filelist
2: http://packages.debian.org/sid/all/python-reportlab-doc/filelist
3: http://packages.qa.debian.org/p/python-reportlab/news/20080427T161708Z.html
4: http://packages.qa.debian.org/p/python-reportlab/news/20081012T192717Z.html



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#569695: dia: Don't use LOCALMODLIBS from python for linking

2010-02-13 Thread Michael Bienia
Package: dia
Version: 0.97.1-1
Severity: normal
Tags: patch

Hello,

dia uses LOCALMODLIBS from python for linking. But as LOCALMODLIBS is
only intended to be used by the python intrepreter itself and not for
application linking against libpython2.6 this will cause a FTBFS as it
doesn't find -lssl (libssl-dev is not python2.6-dev's dependencies).
This is the case in Ubuntu lucid (dia FTBFS). The attached patch fixes
this by removing the use of LOCALMODLIBS.

Michael


diff -u dia-0.97.1/configure dia-0.97.1/configure
--- dia-0.97.1/configure
+++ dia-0.97.1/configure
@@ -18400,12 +18400,11 @@
 
 py_makefile=${py_config_dir}/Makefile
 if test -f $py_makefile; then
-  py_localmodlibs=`sed -n -e 's/^LOCALMODLIBS=\(.*\)/\1/p' $py_makefile`
   py_basemodlibs=`sed -n -e 's/^BASEMODLIBS=\(.*\)/\1/p' $py_makefile`
   py_other_libs=`sed -n -e 's/^LIBS=\(.*\)/\1/p' $py_makefile`
 
   if test -e ${py_config_dir}/libpython${PYTHON_VERSION}.a; then
-PYTHON_LIBS=-L${py_config_dir} -lpython${PYTHON_VERSION} $py_localmodlibs 
$py_basemodlibs $py_other_libs
+PYTHON_LIBS=-L${py_config_dir} -lpython${PYTHON_VERSION} $py_basemodlibs 
$py_other_libs
 { $as_echo $as_me:${as_lineno-$LINENO}: result: found 5
 $as_echo found 6; }
   else
--- dia-0.97.1.orig/acinclude.m4
+++ dia-0.97.1/acinclude.m4
@@ -230,13 +230,12 @@
 py_makefile=${py_config_dir}/Makefile
 if test -f $py_makefile; then
 dnl extra required libs
-  py_localmodlibs=`sed -n -e 's/^LOCALMODLIBS=\(.*\)/\1/p' $py_makefile`
   py_basemodlibs=`sed -n -e 's/^BASEMODLIBS=\(.*\)/\1/p' $py_makefile`
   py_other_libs=`sed -n -e 's/^LIBS=\(.*\)/\1/p' $py_makefile`
 
 dnl now the actual libpython
   if test -e ${py_config_dir}/libpython${PYTHON_VERSION}.a; then
-PYTHON_LIBS=-L${py_config_dir} -lpython${PYTHON_VERSION} $py_localmodlibs 
$py_basemodlibs $py_other_libs
+PYTHON_LIBS=-L${py_config_dir} -lpython${PYTHON_VERSION} $py_basemodlibs 
$py_other_libs
 AC_MSG_RESULT(found)
   else
 AC_MSG_RESULT(not found)



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#569300: python-geoclue: Missing build dependency on python-dbus and python-gobject

2010-02-11 Thread Michael Bienia
Package: python-geoclue
Version: 0.1.0-1
Severity: normal

Hello,

trying to build python-geoclue in Ubuntu lucid fails because of missing
build dependency on python-dbus and python-gobject.

,[ 
http://launchpadlibrarian.net/39018948/buildlog_ubuntu-lucid-i386.python-geoclue_0.1.0-1_FAILEDTOBUILD.txt.gz
 ]-
|  /usr/bin/fakeroot debian/rules clean
| dh  clean
|dh_testdir
|dh_auto_clean
| Traceback (most recent call last):
|   File setup.py, line 5, in module
| import Geoclue
|   File /build/buildd/python-geoclue-0.1.0/Geoclue/__init__.py, line 17, in 
module
| from Base import *
|   File /build/buildd/python-geoclue-0.1.0/Geoclue/Base.py, line 21, in 
module
| import dbus
| ImportError: No module named dbus
| dh_auto_clean: python setup.py clean -a returned exit code 1
| make: *** [clean] Error 1
`

After installing python-dbus it failed with:
,
|  debian/rules clean
| dh  clean
|dh_testdir
|dh_auto_clean
| Traceback (most recent call last):
|   File setup.py, line 5, in module
| import Geoclue
|   File /tmp/buildd/python-geoclue-0.1.0/Geoclue/__init__.py, line 17, in 
module
| from Base import *
|   File /tmp/buildd/python-geoclue-0.1.0/Geoclue/Base.py, line 23, in 
module
| import gobject
| ImportError: No module named gobject
| dh_auto_clean: python setup.py clean -a returned exit code 1
| make: *** [clean] Error 1
`

After install python-gobject too it finally build.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#568633: albatross: Missing build dependency on texlive-latex-recommended

2010-02-06 Thread Michael Bienia
Package: albatross
Version: 1.36-5.2
Severity: normal

Hello,

building albatross in Ubuntu lucid failed because of a missing build
dependency on texlive-latex-recommended:
,
| ! LaTeX Error: File `fancybox.sty' not found.
`

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#567930: ilisp: Missing build-dependency on texlive-latex-base

2010-02-01 Thread Michael Bienia
Package: ilisp
Version: 5.12.0+cvs.2004.12.26-7
Severity: normal

Hello,

trying to build ilisp in Ubuntu lucid failed with the following error:
,[ 
http://launchpadlibrarian.net/38029869/buildlog_ubuntu-lucid-i386.ilisp_5.12.0%2Bcvs.2004.12.26-7_FAILEDTOBUILD.txt.gz
 ]-
| texi2dvi  ilisp.texi
| This is pdfTeXk, Version 3.141592-1.40.3 (Web2C 7.5.6)
|  %-line parsing enabled.
| entering extended mode
| (./txiversion.tex (/usr/share/texmf/tex/texinfo/texinfo.tex
| Loading texinfo [version 2008-04-18.10]: pdf, fonts, page headings, tables,
| conditionals, indexing, sectioning, toc, environments,
| ! Font \circle=lcircle10 not loadable: Metric (TFM) file not found.
| to be read again 
|\allocationnumber 
| \alloc@ ...ifnum \count 1#1  #4\allocationnumber 
|   =\count 1#1 \global 
\advan...
| l.5802 \newdimen\circthick
|   
| ? 
| ! Emergency stop.
| to be read again 
|\allocationnumber 
| \alloc@ ...ifnum \count 1#1  #4\allocationnumber 
|   =\count 1#1 \global 
\advan...
| l.5802 \newdimen\circthick
|   
| No pages of output.
| Transcript written on txiversion.log.
| kpathsea: Running mktextfm lcircle10
| mktextfm: Running mf-nowin -progname=mf \mode:=ljfour; mag:=1; nonstopmode; 
input lcircle10
| This is METAFONT, Version 2.71828 (Web2C 7.5.6)
| kpathsea: Running mktexmf lcircle10
| 
| ! I can't find file `lcircle10'.
| * ...jfour; mag:=1; nonstopmode; input lcircle10
|   
| Please type another input file name
| ! Emergency stop.
| * ...jfour; mag:=1; nonstopmode; input lcircle10
|   
| Transcript written on mfput.log.
| grep: lcircle10.log: No such file or directory
| mktextfm: `mf-nowin -progname=mf \mode:=ljfour; mag:=1; nonstopmode; input 
lcircle10' failed to make lcircle10.tfm.
| kpathsea: Appending font creation commands to missfont.log.
| /usr/bin/texi2dvi: texinfo.tex appears to be broken, quitting.
`
The fix is to replace texlive-base with texlive-latex-base in
Build-Depends-Indep.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#551250: Fix for libmatio FTBFS

2010-02-01 Thread Michael Bienia
Hello,

as the package also FTBFS in Ubuntu lucid I looked if I can find the
cause and a possible fix for it. The problem is
,[ excerpt from doxygen/latex/refman.log ]-
| ! LaTeX Error: File `a4.sty' not found.
`
Thus the fix is simple: add texlive-latex-recommended to Build-Depends.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#568007: openbmap-logger: FTBFS with python2.6

2010-02-01 Thread Michael Bienia
Package: openbmap-logger
Version: 0.4.0-3
Severity: important
Tags: patch
User: debian-pyt...@lists.debian.org
Usertags: python2.6

Hello,

openbmap-logger fails to build with Python 2.6 (like on Ubuntu) because
debian/install contains usr/lib/python2.5.  Replacing that line with
usr/lib/python2.* fixes it.

,[ 
http://launchpadlibrarian.net/36573865/buildlog_ubuntu-lucid-i386.openbmap-logger_0.4.0-3_FAILEDTOBUILD.txt.gz
 ]-
| Writing 
/build/buildd/openbmap-logger-0.4.0/debian/tmp/usr/lib/python2.6/site-packages/openbmap_logger-0.4.0-py2.6.egg-info
| [...]
| cp: cannot stat `debian/tmp/usr/lib/python2.5': No such file or directory
| dh_install: cp returned exit code 1
`

Michael

--- openbmap-logger-0.4.0/debian/install
+++ openbmap-logger-0.4.0/debian/install
@@ -1,4 +1,4 @@
-usr/lib/python2.5
+usr/lib/python2.*
 usr/share/applications
 usr/share/pixmaps
 usr/share/openBmap/*.png



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#567639: make-doc-non-dfsg: Missing build dependency on texlive-latex-base

2010-01-30 Thread Michael Bienia
Package: make-doc-non-dfsg
Version: 3.81-5
Severity: normal

Hello,

when trying to build make-doc-non-dfsg on Ubuntu it failed to build
because of a missing build dependency on texlive-latex-base:

,[ 
http://launchpadlibrarian.net/36498388/buildlog_ubuntu-lucid-i386.make-doc-non-dfsg_3.81-5_FAILEDTOBUILD.txt.gz
 ]-
| make[2]: Entering directory `/build/buildd/make-doc-non-dfsg-3.81/doc'
| texi2html -split_chapter ./make.texi
| TEXINPUTS=.:$TEXINPUTS \
|   MAKEINFO='/bin/bash /build/buildd/make-doc-non-dfsg-3.81/missing --run 
makeinfo   -I .' \
|   texi2dvi make.texi
| You don't have a working TeX binary installed, but the texi2dvi script
| can't proceed without it. If you want to use this script, you have to
| install some kind of TeX, for example TeX Live Debian packages. You can do
| that with this command:
|apt-get install texlive
| make[2]: *** [make.dvi] Error 1
`

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#552917: Patch to fix stow FTBFS

2010-01-30 Thread Michael Bienia
Hello,

here is a patch to fix the FTBFS.

Michael

diff -u stow-1.3.3/debian/rules stow-1.3.3/debian/rules
--- stow-1.3.3/debian/rules
+++ stow-1.3.3/debian/rules
@@ -7,8 +7,7 @@
 build/stow::
[ -d texi ] || ( mkdir texi ; cd texi \
; ln -s ../stow.texi ../version.texi . \
-   ; texi2html -split_node -menu stow.texi \
-   ; mv stow html )
+   ; texi2html -split_node -menu --subdir=html stow.texi )
 
 clean::
rm -Rf texi



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#551243: php-htmlpurifier: FTBFS: Console_Getopt: unrecognized option -- /

2010-01-29 Thread Michael Bienia
On 2009-10-21 21:52:11 +0200, Christian BAYLE wrote:
 Hi, I think i found why
 when used in a builder the
 
 /usr/share/dh-make-php/phppkginfo . package
 and
 /usr/share/dh-make-php/phppkginfo . version
 
 commands don't return anything
[...] 
 I couldn't figure out yet why this happens

I looked at it and might have found the reason for this.  After
commenting in the print_r() in line 47 of
/usr/share/dh-make-php/phppkginfo I can see the error which happens
during parsing of the package.xml file:
'Parsing of package.xml from file 
/tmp/buildd/php-htmlpurifier-4.0.0/package.xml failed'
and later 'Unknown channel htmlpurifier.org'.

After adding the channel like described on
http://htmlpurifier.org/download#PEAR
(pear channel-discover htmlpurifier.org) phppkginfo works again.

As I'm not familiar with PHP packaging, I don't know how to properly fix
this but hope this helps you.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566841: ftplugin/debchangelog.vim: Use python-launchpadlib (and the LP API) for bug completion for Ubuntu bugs

2010-01-25 Thread Michael Bienia
Package: vim-runtime
Version: 2:7.2.330-1
Severity: normal
Tags: patch

Hello,

I noticed that ftplugin/debchangelog.vim still uses
python-launchpad-bugs to fetch the bug listings for an Ubuntu package.
The attached patch changes it to use the LP API (through
python-launchpadlib) instead. It uses the new anonymous access to LP
which will be rolled-out to the 'production' servers on 2010-01-27 (but
you can already use it on the 'edge' servers[0]).
As you might notice, I don't have much experience in writing vim
extensions, so feel free to improve the patch where necessary.

Michael

0: 
http://blog.launchpad.net/general/anonymous-access-to-the-launchpad-web-service-api

--- vim-7.2.330.orig/runtime/ftplugin/debchangelog.vim
+++ vim-7.2.330/runtime/ftplugin/debchangelog.vim
@@ -8,7 +8,7 @@
  URL: 
http://git.debian.org/?p=pkg-vim/vim.git;a=blob_plain;f=runtime/ftplugin/debchangelog.vim;hb=debian
 
  Bug completion requires apt-listbugs installed for Debian packages or
- python-launchpad-bugs installed for Ubuntu packages
+ python-launchpadlib = 1.5.4 installed for Ubuntu packages
 
 if exists(b:did_ftplugin)
   finish
@@ -329,17 +329,23 @@
   python  EOF
 import vim
 try:
-from launchpadbugs import connector
-buglist = connector.ConnectBugList()
-bl = list(buglist('https://bugs.launchpad.net/ubuntu/+source/%s' % 
vim.eval('pkgsrc')))
-bl.sort(None, int)
-liststr = '['
-for bug in bl:
-liststr += '#%d - %s', % (int(bug), bug.summary.replace('\'', 
'\'\''))
-liststr += ']'
-vim.command('silent let bug_lines = %s' % liststr)
+from launchpadlib.launchpad import Launchpad
+from lazr.restfulclient.errors import HTTPError
+# login anonymously
+lp = Launchpad.login_anonymously('debchangelog.vim', 'production')
+try:
+dsp = lp.load('%s/ubuntu/+source/%s' % (lp._root_uri, 
vim.eval('pkgsrc')))
+tasklist = dsp.searchTasks(status = ('New', 'Incomplete', 'Confirmed', 
'Triaged', 'In Progress', 'Fix Committed'), order_by='id')
+liststr = '['
+for task in tasklist:
+bug = task.bug
+liststr += '#%d - %s', % (bug.id, bug.title.replace('\'', 
'\'\''))
+liststr += ']'
+vim.command('silent let bug_lines = %s' % liststr.encode('utf-8'))
+except HTTPError:
+pass
 except ImportError:
-vim.command('echoerr \'python-launchpad-bugs needs to be installed to use 
Launchpad bug completion\'')
+vim.command('echoerr \'python-launchpadlib = 1.5.4 needs to be installed 
to use Launchpad bug completion\'')
 EOF
 else
   if ! filereadable('/usr/sbin/apt-listbugs')



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566842: vim: Don't use MODLIBS when building with python interpreter support and python2.6

2010-01-25 Thread Michael Bienia
Package: vim
Version: 2:7.2.330-1
Severity: normal
Tags: patch

Hello,

vim's configure script uses MODLIBS from python's config/Makefile to
know to link against which libs. As MODLIBS uses LOCALMODLIBS for it's
value and as I got told that LOCALMODLIBS shouldn't be used for python
extensions (I assume this applies here too as vim links against
libpython2.x like python extensions):

LOCALMODLIBS is a macro used to link the python binary against libaries
needed for the binary. It must not used to link extensions.
(from Matthias Klose in an Ubuntu bug)

Without the attached patch, I got a problem building vim with python
interpreter support when doing a merge for Ubuntu (the sanity check for
python failed as it tried to link against -lssl and libssl-dev is not in
the build dependencies). With this change the python interpreter support
got build and as far as I can tell works too (checked with my changes to
ftplugin/debchangelog.vim (see other bug)).

Michael

--- vim-7.2.330.orig/src/configure.in
+++ vim-7.2.330/src/configure.in
@@ -709,7 +709,7 @@
tmp_mkf=$pwd/config-PyMake$$
cat -- ${PYTHON_CONFDIR}/Makefile - 'eof' ${tmp_mkf}
 __:
-   @echo python_MODLIBS='$(MODLIBS)'
+   @echo python_BASEMODLIBS='$(BASEMODLIBS)'
@echo python_LIBS='$(LIBS)'
@echo python_SYSLIBS='$(SYSLIBS)'
@echo python_LINKFORSHARED='$(LINKFORSHARED)'
@@ -726,7 +726,7 @@
  else
  vi_cv_path_python_plibs=-L${PYTHON_CONFDIR} 
-lpython${vi_cv_var_python_version}
  fi
- vi_cv_path_python_plibs=${vi_cv_path_python_plibs} 
${python_MODLIBS} ${python_LIBS} ${python_SYSLIBS} ${python_LINKFORSHARED}
+ vi_cv_path_python_plibs=${vi_cv_path_python_plibs} 
${python_BASEMODLIBS} ${python_LIBS} ${python_SYSLIBS} ${python_LINKFORSHARED}
  dnl remove -ltermcap, it can conflict with an earlier -lncurses
  vi_cv_path_python_plibs=`echo $vi_cv_path_python_plibs | sed 
s/-ltermcap//`
fi
--- vim-7.2.330.orig/src/auto/configure
+++ vim-7.2.330/src/auto/configure
@@ -5153,7 +5153,7 @@
tmp_mkf=$pwd/config-PyMake$$
cat -- ${PYTHON_CONFDIR}/Makefile - 'eof' ${tmp_mkf}
 __:
-   @echo python_MODLIBS='$(MODLIBS)'
+   @echo python_BASEMODLIBS='$(BASEMODLIBS)'
@echo python_LIBS='$(LIBS)'
@echo python_SYSLIBS='$(SYSLIBS)'
@echo python_LINKFORSHARED='$(LINKFORSHARED)'
@@ -5169,7 +5169,7 @@
  else
  vi_cv_path_python_plibs=-L${PYTHON_CONFDIR} 
-lpython${vi_cv_var_python_version}
  fi
- vi_cv_path_python_plibs=${vi_cv_path_python_plibs} 
${python_MODLIBS} ${python_LIBS} ${python_SYSLIBS} ${python_LINKFORSHARED}
+ vi_cv_path_python_plibs=${vi_cv_path_python_plibs} 
${python_BASEMODLIBS} ${python_LIBS} ${python_SYSLIBS} ${python_LINKFORSHARED}
  vi_cv_path_python_plibs=`echo $vi_cv_path_python_plibs | 
sed s/-ltermcap//`
fi




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566328: jadetex: Versioned build dependency on texlive-base-bin

2010-01-22 Thread Michael Bienia
Package: jadetex
Version: 3.13-10
Severity: normal

Hello,

jadetex has a versioned build dependency (Build-Depends-Indep) on
texlive-base-bin. As texlive-base-bin is now only provided by
texlive-binaries, jadetex's build dependency should be updated to
texlive-binaries.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566340: derivations: Versioned build dependency on texlive-base-bin

2010-01-22 Thread Michael Bienia
Package: derivations
Version: 0.51.20080419-2
Severity: normal

Hello,

derivations as a versioned build dependency (Build-Depends-Indep) on
texlive-base-bin. As texlive-base-bin is now only provided by
texlive-binaries, derivations' build dependency should be updated to
texlive-binaries.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#566109: Cannot fetch changelog from packages.debian.org

2010-01-21 Thread Michael Bienia
On 2010-01-21 11:28:48 +0100, Artur R. Czechowski wrote:
 requestsync -d unstable --lp rrdcollect lucid
 requestsync -d unstable rrdcollect lucid
 
 result with following error:
 
 Unable to connect to packages.debian.org: HTTP Error 404: Not Found
 E: Did not retrieve any changelog entries. Was the package recently
 uploaded? (check http://packages.debian.org/changelogs/)
 
 however http://packages.debian.org/changelogs/pool/main/r/rrdcollect/
 contains entry for latest 0.2.6-1 release.

The directory for 0.2.6-1 is there but it's empty (checked at 2010-01-21
11:41:30).  requestsync tries to download changelog.txt from that
directory but as it's missing a 404 gets returned.

BTW: The PTS page for rrdcollect shows a similar problem if you try the
changelog link. As I'm not familiar with the Debian infrastructure I
don't who can look why the changelog extraction isn't working properly.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#560484: Patch to fix the invalid conversion from 'const char*' to 'char*' error

2010-01-20 Thread Michael Bienia
Hello,

here is the patch I applied to the Ubuntu package to fix it.

Michael

--- smilutils-0.3.2+cvs20070731.orig/libkino/gnome-stubs.cc
+++ smilutils-0.3.2+cvs20070731/libkino/gnome-stubs.cc
@@ -24,16 +24,16 @@
 
 char *gnome_config_get_string( const char *path )
 {
-   char *ptr = strchr( path, '=' );
+   const char *ptr = strchr( path, '=' );
if ( ptr != NULL )
return strdup( ptr + 1 );
else
return NULL;
 }
 
-char *gnome_config_get_safe( const char *path )
+const char *gnome_config_get_safe( const char *path )
 {
-   char *ptr = strchr( path, '=' );
+   const char *ptr = strchr( path, '=' );
if ( ptr != NULL )
return ptr + 1;
else



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#563534: cbedic: FTBFS with gcc-4.4 (error: invalid conversion from 'const char*' to 'char*')

2010-01-03 Thread Michael Bienia
Package: cbedic
Version: 4.0-2
Severity: normal
Tags: patch

Hi,

a rebuild of cbedic with gcc-4.4 (in Ubuntu) failed to build:
,[ 
http://people.ubuntuwire.org/~lucas/ubuntu-nbs/32/cbedic_4.0-2build2_llucid32.buildlog
 ]-
| engine/translator.cpp: In member function 'bool 
Translator::isLatinInput(const char*, char*, bool)':
| engine/translator.cpp:300: error: invalid conversion from 'const char*' to 
'char*'
| engine/translator.cpp: In member function 'char* Translator::toLatin(const 
char*, char*, bool)':
| engine/translator.cpp:328: error: invalid conversion from 'const char*' to 
'char*'
`

Attached is a patch to fix it.

Michael

--- cbedic-4.0.orig/src/engine/translator.cpp
+++ cbedic-4.0/src/engine/translator.cpp
@@ -291,7 +291,7 @@
 //
 bool Translator::isLatinInput(const char *word, char *buf, const bool 
ignoreSpace) {
int i = 0;
-   char *p;
+   const char *p;
while (word[i] != '\0') {
if ((ignoreSpace)  (word[i] == ' ')) {
buf[i] = word[i];
@@ -321,7 +321,7 @@
 char *Translator::toLatin(const char *word, char *buf, const bool 
legalLatinInput) {
int i = 0;
int j = 0;
-   char *p;
+   const char *p;
char c;
while ((c = word[i]) != '\0') {
if (legalLatinInput) {



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#563281: test_pbuilder-checkparams fails it pbuilder is not installed

2010-01-01 Thread Michael Bienia
Package: pbuilder
Version: 0.195
Severity: important
Tags: patch
Justification: fails to build from source

Hello,

test_pbuilder-checkparams fails if pbuilder is not installed (e.g. when
building pbuilder inside a pbuilder or on the buildds).
The reason is that pbuilder-checkparams source files from
/usr/lib/pbuilder which aren't there (as pbuilder isn't installed). One
sees the error if the redirection of stderr in
test_pbuilder-checkparams:test_debuildopts() is temporarily commented out:
,
| ./pbuilder-checkparams: line 24: /usr/lib/pbuilder/pbuilder-loadconfig: No 
such file or directory
| ./pbuilder-checkparams: line 25: /usr/lib/pbuilder/pbuilder-modules: No such 
file or directory
| ./pbuilder-checkparams: line 257: BUILDPLACE: Build root directory is not 
defined
`
It seems to be sufficient to define BUILDPLACE before
pbuilder-checkparams is sourced (and continue to ignoring the error
about the non-existant files). With this change the testsuite works (and
succeeds).

Michael

diff -Nru pbuilder-0.195/test_pbuilder-checkparams 
pbuilder-0.195ubuntu1/test_pbuilder-checkparams
--- pbuilder-0.195/test_pbuilder-checkparams2009-12-14 15:05:38.0 
+0100
+++ pbuilder-0.195ubuntu1/test_pbuilder-checkparams 2010-01-01 
19:13:35.0 +0100
@@ -7,6 +7,7 @@
 test_debuildopts() {
 # stderr is redirected to /dev/null to avoid failing if ~/.pbuilderrc is
 # missing (this outputs a warning which is not expected in the output)
+BUILDPLACE=
 . ./pbuilder-checkparams 2/dev/null
 echo $DEBBUILDOPTS
 }



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#561805: python-debian: Please implement __unicode__() for Changelog (and ChangeBlock)

2009-12-20 Thread Michael Bienia
Package: python-debian
Version: 0.1.14
Severity: wishlist

Please implement __unicode__() for Changelog and ChangeBlock to be able
to convert a changelog or a block into a unicode string.

 from debian_bundle.changelog import Changelog
 c = Changelog(open('changelog.txt', 'r'))
 unicode(c)
Traceback (most recent call last):
  File stdin, line 1, in module
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 220: 
ordinal not in range(128)
 unicode(c._blocks[1])
Traceback (most recent call last):
  File stdin, line 1, in module
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 62: 
ordinal not in range(128)
 str(c._blocks[1])
'haskell-src-exts (1.1.4-1) unstable; urgency=low\n\n\n  [ Marco T\xc3\xbalio 
Gontijo e Silva ]\n\n  * debian/control: Use one package for line in 
dependencies.\n\n  * debian/rules: Make build depend only in build-arch to 
avoid building\n\nindep packages in all architectures.\n\n  * 
debian/control: Use more sintetic name for Vcs-Darcs.\n\n\n\n  [ Joachim 
Breitner ]\n\n  * New upstream version \n\n  * Put haddoc et. al. into 
Build-Depends\n\n  * Bump standards version\n\n\n\n -- Joachim Breitner 
nome...@debian.org  Tue, 08 Sep 2009 18:13:15 +0200\n\n\n\n'

A work-around for now is:
 unicode(str(c._blocks[1]).decode('utf-8'))
 unicode(str(c).decode('utf-8'))

The used changelog is
http://packages.debian.org/changelogs/pool/main/h/haskell-src-exts/current/changelog.txt


Thanks,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#560758: False: The versions in Debian and Ubuntu are the same already during requestsync

2009-12-20 Thread Michael Bienia
On 2009-12-11 23:59:49 +0100, Artur R. Czechowski wrote:
 When requesting sync without using the LP extension:
 requestsync -d unstable rrdcollect lucid
 I can see an error:
 E: The versions in Debian and Ubuntu are the same already (0.2.4-2). Aborting.
 
 Please note that launchpad knows about new version:
 https://launchpad.net/debian/sid/+source/rrdcollect
 shows 0.2.5.1

This information from Launchpad is only used with the --lp option.
Without this option requestsync relies completely on rmadison output.

 maxb suggested on IRC that it could be related to rmadison output:
 
 artu...@szczaw:~$ rmadison -u debian -a source -s unstable rrdcollect
 rrdcollect |0.2.4-2 |  unstable | source, hurd-i386
 rrdcollect |  0.2.5.1-1 |  unstable | source, alpha, amd64, armel, hppa, 
 i386, ia64, kfreebsd-amd64, kfreebsd-i386, mips, mipsel, powerpc, s390, sparc

This was a bug in requestsync's parsing of the rmadison output. It was
using the first line contain 'source' and not the most recent one. This
got fixed in r546[0] which will be part of ubuntu-dev-tools 0.86 (not
released yet).

Michael

0: http://bazaar.launchpad.net/~ubuntu-dev/ubuntu-dev-tools/trunk/revision/546



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#559877: qpdf: FTBFS with g++ 4.4 (missing include)

2009-12-07 Thread Michael Bienia
Package: qpdf
Version: 2.1-1
Severity: normal
Tags: patch

Hello,

trying to build qpdf in Ubuntu Lucid with g++ 4.4 failed with the
following error:
, [ 
http://launchpadlibrarian.net/36005866/buildlog_ubuntu-lucid-amd64.qpdf_2.1-1_FAILEDTOBUILD.txt.gz
 ]
| In file included from libqpdf/Pl_AES_PDF.cc:1:
| libqpdf/qpdf/Pl_AES_PDF.hh:45: error: 'uint32_t' does not name a type
| libqpdf/Pl_AES_PDF.cc: In constructor 'Pl_AES_PDF::Pl_AES_PDF(const char*, 
Pipeline*, bool, const unsigned char*)':
| libqpdf/Pl_AES_PDF.cc:28: error: 'class Pl_AES_PDF' has no member named 'rk'
| libqpdf/Pl_AES_PDF.cc:30: error: 'class Pl_AES_PDF' has no member named 'rk'
| libqpdf/Pl_AES_PDF.cc:30: error: 'class Pl_AES_PDF' has no member named 'rk'
| libqpdf/Pl_AES_PDF.cc:36: error: 'class Pl_AES_PDF' has no member named 'rk'
| libqpdf/Pl_AES_PDF.cc:40: error: 'class Pl_AES_PDF' has no member named 'rk'
| libqpdf/Pl_AES_PDF.cc: In member function 'void Pl_AES_PDF::flush(bool)':
| libqpdf/Pl_AES_PDF.cc:180: error: 'class Pl_AES_PDF' has no member named 'rk'
| libqpdf/Pl_AES_PDF.cc:188: error: 'class Pl_AES_PDF' has no member named 'rk'
`
The patch at the end of this mail fixes it.

Regards,
Michael

--- qpdf-2.1.orig/libqpdf/qpdf/Pl_AES_PDF.hh
+++ qpdf-2.1/libqpdf/qpdf/Pl_AES_PDF.hh
@@ -1,6 +1,7 @@
 #ifndef __PL_AES_PDF_HH__
 #define __PL_AES_PDF_HH__
 
+#include stdint.h
 #include qpdf/Pipeline.hh
 #include qpdf/qpdf-config.h
 



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#559889: raul: FTBFS with g++ 4.4 (missing include)

2009-12-07 Thread Michael Bienia
Package: raul
Version: 0.5.1-1
Severity: normal
Tags: patch

Hello,

your package raul failed to build in Ubuntu Lucid with g++ 4.4:
,[ 
http://launchpadlibrarian.net/36473766/buildlog_ubuntu-lucid-i386.raul_0.5.1-1_FAILEDTOBUILD.txt.gz
 ]-
| MFWriter.cpp: In member function 'bool Raul::SMFWriter::start(const 
std::string, Raul::TimeStamp)':
| SMFWriter.cpp:72: error: 'fopen' was not declared in this scope
| SMFWriter.cpp: In member function 'virtual void 
Raul::SMFWriter::write_event(Raul::TimeStamp, size_t, const unsigned char*)':
| SMFWriter.cpp:108: error: 'fseek' was not declared in this scope
| SMFWriter.cpp:120: error: 'fwrite' was not declared in this scope
| SMFWriter.cpp:127: error: 'fwrite' was not declared in this scope
| [...]
`
The patch at the end of this mail fixes it.

Regards,
Michael

--- raul-0.5.1.orig/src/SMFWriter.cpp
+++ raul-0.5.1/src/SMFWriter.cpp
@@ -16,6 +16,7 @@
  */
 
 #include stdint.h
+#include cstdio
 #include cassert
 #include cstring
 #include iostream



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#559898: flowcanvas: FTBFS with g++ 4.4 (missing include)

2009-12-07 Thread Michael Bienia
Package: flowcanvas
Version: 0.5.1-1
Severity: normal
Tags: patch

Hello,

your package flowcanvas failed to build in Ubuntu Lucid with g++ 4.4:
,[ 
http://launchpadlibrarian.net/36472094/buildlog_ubuntu-lucid-i386.flowcanvas_0.5.1-1_FAILEDTOBUILD.txt.gz
 ]-
| In file included from /build/buildd/flowcanvas-0.5.1/flowcanvas/Canvas.hpp:25,
|  from Canvas.cpp:27:
| /build/buildd/flowcanvas-0.5.1/flowcanvas/Connection.hpp:43: error: 
'uint32_t' has not been declared
| /build/buildd/flowcanvas-0.5.1/flowcanvas/Connection.hpp:61: error: 
'uint32_t' has not been declared
| /build/buildd/flowcanvas-0.5.1/flowcanvas/Connection.hpp:86: error: 
'uint32_t' does not name a type
`
At the end of this mail is a patch to fix it.

Regards,
Michael

--- flowcanvas-0.5.1.orig/flowcanvas/Connection.hpp
+++ flowcanvas-0.5.1/flowcanvas/Connection.hpp
@@ -19,6 +19,7 @@
 #define FLOWCANVAS_CONNECTION_HPP
 
 #include list
+#include stdint.h
 #include boost/weak_ptr.hpp
 #include libgnomecanvasmm.h
 #include libgnomecanvasmm/bpath.h



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#539540: FTBFS with GCC 4.4: missing #include

2009-12-07 Thread Michael Bienia
On 2009-08-01 21:21:13 +0200, Martin Michlmayr wrote:
  In file included from FTutils.cpp:25:
  FTutils.hpp:125: error: 'int32_t' does not name a type
  FTutils.hpp: In function 'float flush_to_zero(float)':
  FTutils.hpp:136: error: 'union ls_pcast32' has no member named 'i'

Attached is a patch to fix it.

Regards,
Michael
--- freqtweak-0.7.2.orig/debian/patches/04_missing_include.diff
+++ freqtweak-0.7.2/debian/patches/04_missing_include.diff
@@ -0,0 +1,11 @@
+diff -Nur -x '*.orig' -x '*~' freqtweak-0.7.2/src/FTutils.hpp 
freqtweak-0.7.2.new/src/FTutils.hpp
+--- freqtweak-0.7.2/src/FTutils.hpp2009-07-11 23:15:00.0 +0200
 freqtweak-0.7.2.new/src/FTutils.hpp2009-12-07 18:05:09.0 
+0100
+@@ -23,6 +23,7 @@
+ #include iostream
+ #include sstream
+ #include cmath
++#include stdint.h
+ 
+ //#include wx/string.h
+ 


Bug#558411: template-new: Buffer overflow caused by Debian changes

2009-11-28 Thread Michael Bienia
Package: template-new
Version: 1.3.8-1.1
Severity: normal
Tags: patch

Hello,

template-new failed to build in Ubuntu as gcc caught the following
buffer overflow:
,
| In function 'strcpy',
| inlined from 'write_file' at new_proj.c:196:
| /usr/include/bits/string3.h:106: error: call to __builtin___strcpy_chk will 
always overflow destination buffer
`

This is caused by Debian's renaming of the binary name from new to
template-new without making the destination buffer larger while
contructing the new command line.
Attached is a patch to fix it.

Michael

diff -u template-new-1.3.8/src/new_proj.c template-new-1.3.8/src/new_proj.c
--- template-new-1.3.8/src/new_proj.c
+++ template-new-1.3.8/src/new_proj.c
@@ -192,7 +192,7 @@
 fmt_ptrn_init(x);
 fmt_ptrn_update_kv(x, strdup(PROJ_NAME),
strdup(settings.proj_name));
-argv[argc] = (char *) malloc (4);
+argv[argc] = (char *) malloc (13);
 strcpy (argv[argc++], template-new);
 argv[argc] = (char *) malloc (3);
 strcpy (argv[argc++], -s);



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#546171: cwidget: FTBFS with g++ 4.4 (error: invalid conversion from 'const char*' to 'char*')

2009-09-11 Thread Michael Bienia
Package: cwidget
Version: 0.5.12-4
Severity: normal
Tags: patch

Hello,

during a rebuild test in Ubuntu, cwidget (0.5.12-4ubuntu2) failed to
build with the following error (g++-4.4 4.4.1-3ubuntu3):

,
| fragment.cc: In function 'cwidget::fragment* cwidget::fragf(const char*, 
...)':
| fragment.cc:1348: error: invalid conversion from 'const char*' to 'char*'
| fragment.cc:1410: error: invalid conversion from 'const char*' to 'char*'
| fragment.cc:1421: error: invalid conversion from 'const char*' to 'char*'
| fragment.cc:1489: error: invalid conversion from 'const char*' to 'char*'
| fragment.cc:1524: error: invalid conversion from 'const char*' to 'char*'
| fragment.cc:1683: error: invalid conversion from 'const char*' to 'char*'
`
http://launchpadlibrarian.net/31610844/buildlog_ubuntu-karmic-amd64.cwidget_0.5.12-4ubuntu2_FAILEDTOBUILD.txt.gz

At the end of this mail is a patch to fix this. From a look at cwidget 0.5.13 I
assume it will fail with the same error too (didn't check yet) as it
contains the same code.

Regards,
Michael

diff -u cwidget-0.5.12/src/cwidget/fragment.cc 
cwidget-0.5.12/src/cwidget/fragment.cc
--- cwidget-0.5.12/src/cwidget/fragment.cc
+++ cwidget-0.5.12/src/cwidget/fragment.cc
@@ -1345,7 +1345,7 @@
 
 const char *start=format;
 // find all the arguments.
-char *nextpercent=strchr(start, '%');
+const char *nextpercent=strchr(start, '%');
 
 // loop 1: count the arguments.
 while(nextpercent!=NULL)



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#546175: aspectc++: FTBFS with g++-4.4 (error: invalid conversion from 'const char*' to 'char*')

2009-09-11 Thread Michael Bienia
Package: aspectc++
Version: 1.0pre4~svn.20090615-1
Severity: normal
Tags: patch

Hello,

during a rebuild test in Ubuntu, aspectc++ failed to build with the
following error:
,
| ACWeaver.cc: In member function 'Puma::Unit* 
ACWeaver::translate(Puma::VerboseMgr, const char*, Transformer)':
| ACWeaver.cc:506: error: invalid conversion from 'const char*' to 'char*'
`
http://launchpadlibrarian.net/31585541/buildlog_ubuntu-karmic-i386.aspectc%2B%2B_1.0pre4~svn.20090615-1_FAILEDTOBUILD.txt.gz

Here is a patch to fix this.

Regards,
Michael

--- aspectc++-1.0pre4~svn.20090615.orig/AspectC++/ACWeaver.cc
+++ aspectc++-1.0pre4~svn.20090615/AspectC++/ACWeaver.cc
@@ -503,7 +503,7 @@
  Transformer transformer)
  {
vm  Handling Translation Unit `;
-   char *fname = strrchr (file, (int)'/');
+   const char *fname = strrchr (file, (int)'/');
vm  (fname ? ++fname : file)  '.  endvm;
vm++;
vm  Path \  file  \  endvm;



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#545150: zodb: Bashism in debian/rules

2009-09-05 Thread Michael Bienia
Package: zodb
Version: 3.8.3-1
Severity: normal
Tags: patch

Hello,

there is a bashism in debian/rules which makes the package FTBFS if dash
is used (e.g. on Ubuntu). Attached is a patch to fix it.

Michael
diff -u zodb-3.8.3/debian/rules zodb-3.8.3/debian/rules
--- zodb-3.8.3/debian/rules
+++ zodb-3.8.3/debian/rules
@@ -10 +10,2 @@
-	sed -s -i -e 's/python2.3/python/g' debian/$(package)/$(call py_libdir,$*)/{ZEO,ZODB}/scripts/*.py
+	sed -s -i -e 's/python2.3/python/g' debian/$(package)/$(call py_libdir,$*)/ZEO/scripts/*.py
+	sed -s -i -e 's/python2.3/python/g' debian/$(package)/$(call py_libdir,$*)/ZODB/scripts/*.py


Bug#543216: gnupg: udev acl rules to give gpg access to some smartcard readers

2009-08-23 Thread Michael Bienia
Package: gnupg
Version: 1.4.9-4
Severity: wishlist
Tags: patch

Hello,

here are some udev rules to give gpg access to some SCM smartcard
readers for the currently logged-in user. This patch is already tested
on the gnupg package in Ubuntu karmic and it needs the udev version from
experimental to work properly (applying the ACLs).
Similar rules exist in gnupg2 to let the ACL get set when either gnupg
or gnupg2 is installed.

Michael

diff -u gnupg-1.4.9/debian/rules gnupg-1.4.9/debian/rules
--- gnupg-1.4.9/debian/rules
+++ gnupg-1.4.9/debian/rules
@@ -99,6 +99,7 @@
dh_movefiles -pgpgv --sourcedir=debian/gnupg
dh_install
dh_installman
+   dh_installudev
dh_link
dh_strip
dh_compress
--- gnupg-1.4.9.orig/debian/gnupg.udev
+++ gnupg-1.4.9/debian/gnupg.udev
@@ -0,0 +1,12 @@
+# do not edit this file, it will be overwritten on update
+
+SUBSYSTEM!=usb, GOTO=gnupg_rules_end
+ACTION!=add, GOTO=gnupg_rules_end
+
+# USB SmartCard Readers
+## SCM readers (SCR335, SPR532,  Co)
+ATTR{idVendor}==04e6, ATTR{idProduct}==e001,
ENV{ID_SMARTCARD_READER}=1, ENV{ID_SMARTCARD_READER_DRIVER}=gnupg
+ATTR{idVendor}==04e6, ATTR{idProduct}==e003,
ENV{ID_SMARTCARD_READER}=1, ENV{ID_SMARTCARD_READER_DRIVER}=gnupg
+ATTR{idVendor}==04e6, ATTR{idProduct}==5115,
ENV{ID_SMARTCARD_READER}=1, ENV{ID_SMARTCARD_READER_DRIVER}=gnupg
+
+LABEL=gnupg_rules_end



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#543217: gnupg2: udev acl rules to give gpg access to some smartcard readers

2009-08-23 Thread Michael Bienia
Package: gnupg2
Version: 2.0.11-1
Severity: wishlist
Tags: patch

Hello,

here are some udev rules to give gpg2/scdaemon access to some SCM
smartcard readers for the currently logged-in user. This patch is
already tested on the gnupg2 package in Ubuntu karmic and it needs the
udev version from experimental to work properly (applying the ACLs).
Similar rules exist in gnupg2 to let the ACL get set when either gnupg
or gnupg2 is installed.

Michael

diff -u gnupg2-2.0.11/debian/rules gnupg2-2.0.11/debian/rules
--- gnupg2-2.0.11/debian/rules
+++ gnupg2-2.0.11/debian/rules
@@ -114,6 +114,7 @@
dh_installexamples -a
dh_installinfo -a
dh_installchangelogs -a
+   dh_installudev -a
 
$(install_dir) debian/gnupg2/usr/share/lintian/overrides/
$(install_file) debian/gnupg2.lintian 
debian/gnupg2/usr/share/lintian/overrides/gnupg2
--- gnupg2-2.0.11.orig/debian/gnupg2.udev
+++ gnupg2-2.0.11/debian/gnupg2.udev
@@ -0,0 +1,12 @@
+# do not edit this file, it will be overwritten on update
+
+SUBSYSTEM!=usb, GOTO=gnupg_rules_end
+ACTION!=add, GOTO=gnupg_rules_end
+
+# USB SmartCard Readers
+## SCM readers (SCR335, SPR532,  Co)
+ATTR{idVendor}==04e6, ATTR{idProduct}==e001, ENV{ID_SMARTCARD_READER}=1, 
ENV{ID_SMARTCARD_READER_DRIVER}=gnupg
+ATTR{idVendor}==04e6, ATTR{idProduct}==e003, ENV{ID_SMARTCARD_READER}=1, 
ENV{ID_SMARTCARD_READER_DRIVER}=gnupg
+ATTR{idVendor}==04e6, ATTR{idProduct}==5115, ENV{ID_SMARTCARD_READER}=1, 
ENV{ID_SMARTCARD_READER_DRIVER}=gnupg
+
+LABEL=gnupg_rules_end



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#543229: zope.testbrowser: FTBFS with python 2.6

2009-08-23 Thread Michael Bienia
Package: zope.testbrowser
Version: 3.5.1-1
Severity: wishlist
Tags: patch

Hello,

here is a patch (based on the same changes to zope.interface) to let
zope.testbrowser build with python2.6.

Michael
diff -u zope.testbrowser-3.5.1/debian/changelog 
zope.testbrowser-3.5.1/debian/changelog
--- zope.testbrowser-3.5.1/debian/changelog
+++ zope.testbrowser-3.5.1/debian/changelog
@@ -1,3 +1,10 @@
+zope.testbrowser (3.5.1-2) unstable; urgency=low
+
+  * debian/rules:
+Determine the correct library path for python 2.5 and python 2.6. 
+
+ -- Michael Bienia ge...@ubuntu.com  Fri, 21 Aug 2009 17:44:49 +0200
+
 zope.testbrowser (3.5.1-1) unstable; urgency=low
 
   * Convert to use python-van.pydeb for dependency management.
diff -u zope.testbrowser-3.5.1/debian/rules zope.testbrowser-3.5.1/debian/rules
--- zope.testbrowser-3.5.1/debian/rules
+++ zope.testbrowser-3.5.1/debian/rules
@@ -6,4 +6,4 @@
-   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info 
--exclude-extra test)  debian/$(package).substvars
-   (echo -n 'setuptools:Provides='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info 
--exclude-extra test)  debian/$(package).substvars
+   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --exclude-extra 
test)  debian/$(package).substvars
+   (echo -n 'setuptools:Provides='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --exclude-extra 
test)  debian/$(package).substvars
# docs and test to suggests
-   (echo -n 'setuptools:Suggests='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info --extra 
test)  debian/$(package).substvars
+   (echo -n 'setuptools:Suggests='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --extra test)  
debian/$(package).substvars


Bug#543230: zope.testing: FTBFS with python 2.6

2009-08-23 Thread Michael Bienia
Package: zope.testing
Version: 3.7.4-4
Severity: wishlist
Tags: patch

Hello,

here is a patch (based on the same changes to zope.interface) to let
zope.testing build with python2.6.

Michael
diff -u zope.testing-3.7.4/debian/changelog zope.testing-3.7.4/debian/changelog
--- zope.testing-3.7.4/debian/changelog
+++ zope.testing-3.7.4/debian/changelog
@@ -1,3 +1,10 @@
+zope.testing (3.7.4-5) unstable; urgency=low
+
+  * debian/rules:
+Determine the correct library path for python 2.5 and python 2.6.
+
+ -- Michael Bienia ge...@ubuntu.com  Mon, 20 Jul 2009 16:35:21 +0200
+
 zope.testing (3.7.4-4) unstable; urgency=low
 
   * Port revision 101026 from upstream: add a zope-testrunner binary.
diff -u zope.testing-3.7.4/debian/rules zope.testing-3.7.4/debian/rules
--- zope.testing-3.7.4/debian/rules
+++ zope.testing-3.7.4/debian/rules
@@ -3,8 +3,8 @@
 include /usr/share/python-van.pydeb/rules_templates/rules.1.mk
 
 dependencies-python%:
-   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info 
--exclude-extra zope_tracebacks)  debian/$(package).substvars
-   (echo -n 'setuptools:Recommends='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info --extra 
zope_tracebacks)  debian/$(package).substvars
+   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --exclude-extra 
zope_tracebacks)  debian/$(package).substvars
+   (echo -n 'setuptools:Recommends='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --extra 
zope_tracebacks)  debian/$(package).substvars
 
 post-install-python%:
mv debian/$(package)/usr/bin/zope-testrunner 
debian/$(package)/usr/bin/zope-testrunner$*


Bug#543231: zope.schema: FTBFS with python 2.6

2009-08-23 Thread Michael Bienia
Package: zope.schema
Version: 3.5.4-2
Severity: wishlist
Tags: patch

Hello,

here is a patch (based on the same changes to zope.interface) to let
zope.schema build with python2.6.

Michael
diff -u zope.schema-3.5.4/debian/changelog zope.schema-3.5.4/debian/changelog
--- zope.schema-3.5.4/debian/changelog
+++ zope.schema-3.5.4/debian/changelog
@@ -1,3 +1,10 @@
+zope.schema (3.5.4-3) unstable; urgency=low
+
+  * debian/rules:
+Determine the correct library path for python 2.5 and python 2.6.
+
+ -- Michael Bienia ge...@ubuntu.com  Mon, 20 Jul 2009 16:31:00 +0200
+
 zope.schema (3.5.4-2) unstable; urgency=low
 
   * Test depend on python-zope.testing as we need it for the test runner
diff -u zope.schema-3.5.4/debian/rules zope.schema-3.5.4/debian/rules
--- zope.schema-3.5.4/debian/rules
+++ zope.schema-3.5.4/debian/rules
@@ -6,4 +6,4 @@
-   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info 
--exclude-extra hook --exclude-extra docs)  debian/$(package).substvars
-   (echo -n 'setuptools:Provides='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info 
--exclude-extra hook --exclude-extra docs)  debian/$(package).substvars
+   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --exclude-extra 
hook --exclude-extra docs)  debian/$(package).substvars
+   (echo -n 'setuptools:Provides='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --exclude-extra 
hook --exclude-extra docs)  debian/$(package).substvars
# docs and test to suggests
-   (echo -n 'setuptools:Suggests='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info --extra 
docs)  debian/$(package).substvars
+   (echo -n 'setuptools:Suggests='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --extra docs)  
debian/$(package).substvars


Bug#540068: libgnupg-perl: Missing build-dependency on gnupg

2009-08-05 Thread Michael Bienia
Package: libgnupg-perl
Version: 0.10-1
Severity: normal

Hello,

libgnupg-perl failed to build on the Ubuntu buildd because of missing
gnupg for the test suite:
,
|dh_auto_test
| make[1]: Entering directory `/build/buildd/libgnupg-perl-0.10'
| PERL_DL_NONLAZY=1 /usr/bin/perl -Iblib/lib -Iblib/arch test.pl
| 1..25
| Couldn't find gpg in PATH 
(/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/X11R6/bin:/usr/games)
|  at test.pl line 56
| # Looks like your test died before it could output anything.
| make[1]: *** [test_dynamic] Error 255
| make[1]: Leaving directory `/build/buildd/libgnupg-perl-0.10'
| dh_auto_test: make returned exit code 2
| make: *** [build] Error 1
| dpkg-buildpackage: error: debian/rules build gave error exit status 2
| **
`
Complete log at 
http://launchpadlibrarian.net/29642520/buildlog_ubuntu-karmic-i386.libgnupg-perl_0.10-1_FAILEDTOBUILD.txt.gz

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#539316: python-debian: Parsing an existing changelog duplicates newlines

2009-07-30 Thread Michael Bienia
Package: python-debian
Version: 0.1.14
Severity: normal

Hello,

I was looking if I can use the Changelog class to access changelog
entries from a existing changelogs. The parsing of a existing changelog
went fine but then I found out that there are no methods to iterate over
(or access) existing changelog blocks (I'd should file a wishlist bug
about this instead of accessing internal structures) but the more
important problem was that I got to many newlines when I tried to get a
string representation of a changelog block.
(Note: this was with pyhton 2.6 on Ubuntu if it's important)

 c = debian_bundle.changelog.Changelog(file('changelog.txt', 'r'))
 print c

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#539334: python-debian: Please add methods to Changelog to navigate through existing changelog blocks

2009-07-30 Thread Michael Bienia
Package: python-debian
Version: 0.1.14
Severity: wishlist

Hello,

I'm trying to improve some parts from Ubuntu's requestsync script (in
ubuntu-dev-tools) and was looking if the Changelog class from
python-debian is usable to replace the simple code from requestsync to
copy new changelog entries from a existing Debian changelog into the
sync bug.
Unfortunately the Changelog class does only support adding new blocks
but nothing to access existing changelog blocks or to iterate over the
changelog blocks. I'd like to do something like this:
,
| new_entries = 
| for block in changelog:
|   if block.version = current_ubuntu_version:
| new_entries += str(block)
`

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#537581: zope.interface: FTBFS with python 2.6

2009-07-19 Thread Michael Bienia
Package: zope.interface
Version: 3.5.1-2
Severity: wishlist
Tags: patch

Hello,

I tried building zope.interface in Ubuntu karmic which has already
python 2.6 as the default python version. The build failed with the
following error:
,
| /usr/bin/make -f debian/rules post-install-python2.6
| make[2]: Entering directory `/tmp/buildd/zope.interface-3.5.1'
| install -D -m 644 src/zope/__init__.py \
|   
debian/python-zope.interface/usr/lib/python2.6/site-packages/zope/__init__.py
| (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/python-zope.interface/usr/lib/python2.6/site-packages/zope.interface.egg-info
 --exclude-extra docs)  debian/python-zope.interface.substvars
| Traceback (most recent call last):
|   File /usr/bin/van-pydeb, line 8, in module
| load_entry_point('van.pydeb==1.2.1', 'console_scripts', 'van-pydeb')()
|   File /usr/lib/python2.6/dist-packages/van/pydeb/__init__.py, line 35, in 
main
| return command(argv)
|   File /usr/lib/python2.6/dist-packages/van/pydeb/__init__.py, line 170, in 
_depends_or_provides
| assert os.path.exists(options.egg_info), Does not exist: %s % 
options.egg_info
| AssertionError: Does not exist: 
debian/python-zope.interface/usr/lib/python2.6/site-packages/zope.interface.egg-info
| make[2]: *** [post-install-python2.6] Error 1
| make[2]: Leaving directory `/tmp/buildd/zope.interface-3.5.1'
| make[1]: *** [install-python2.6] Error 2
| make[1]: Leaving directory `/tmp/buildd/zope.interface-3.5.1'
| make: *** [install] Error 2
| dpkg-buildpackage: error: fakeroot debian/rules binary gave error exit status 
2
`
Attached is a debdiff which uses /usr/share/python/python.mk to
determine the correct libary paths for python 2.5 and python 2.6 and
bumps the needed version of python-all-dev. With this change the package
builds successfully for both python 2.5 and python 2.6.

Regards
Michael
diff -u zope.interface-3.5.1/debian/changelog 
zope.interface-3.5.1/debian/changelog
--- zope.interface-3.5.1/debian/changelog
+++ zope.interface-3.5.1/debian/changelog
@@ -1,3 +1,13 @@
+zope.interface (3.5.1-3) unstable; urgency=low
+
+  * debian/rules:
+Use the macros from python.mk to determine the library path for python 2.5
+and python 2.6.
+  * debian/control:
+Bump the needed version of python-all-dev for this.
+
+ -- Michael Bienia ge...@ubuntu.com  Sun, 19 Jul 2009 16:47:04 +0200
+
 zope.interface (3.5.1-2) unstable; urgency=low
 
   * Fix maintainers/upload.
diff -u zope.interface-3.5.1/debian/rules zope.interface-3.5.1/debian/rules
--- zope.interface-3.5.1/debian/rules
+++ zope.interface-3.5.1/debian/rules
@@ -4,6 +4,7 @@
+include /usr/share/python/python.mk
 
 post-install-python%:
install -D -m 644 src/zope/__init__.py \
-   debian/$(package)/usr/lib/python$*/site-packages/zope/__init__.py
-   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info 
--exclude-extra docs)  debian/$(package).substvars
-   (echo -n 'setuptools:Suggests='  van-pydeb depends --egg-info 
debian/$(package)/usr/lib/python$*/site-packages/$(py_package).egg-info --extra 
docs)  debian/$(package).substvars
+   debian/$(package)/$(call py_libdir,$*)/zope/__init__.py
+   (echo -n 'setuptools:Depends='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --exclude-extra 
docs)  debian/$(package).substvars
+   (echo -n 'setuptools:Suggests='  van-pydeb depends --egg-info 
debian/$(package)/$(call py_libdir,$*)/$(py_package).egg-info --extra docs)  
debian/$(package).substvars
diff -u zope.interface-3.5.1/debian/control zope.interface-3.5.1/debian/control
--- zope.interface-3.5.1/debian/control
+++ zope.interface-3.5.1/debian/control
@@ -3,7 +3,7 @@
 Priority: extra
 Maintainer: Debian/Ubuntu Zope Team 
pkg-zope-develop...@lists.alioth.debian.org
 Uploaders: Brian Sutherland br...@vanguardistas.net, Fabio Tranchitella 
kob...@debian.org
-Build-Depends: debhelper (= 5.0.38), python-all-dev (= 2.3.5-11), 
python-central (= 0.5.6), python-setuptools (= 0.6b3), python-van.pydeb
+Build-Depends: debhelper (= 5.0.38), python-all-dev (= 2.5.4-1~), 
python-central (= 0.5.6), python-setuptools (= 0.6b3), python-van.pydeb
 Standards-Version: 3.8.2
 XS-Python-Version: all
 XS-Vcs-Svn: svn://svn.debian.org/pkg-zope/zope.interface/trunk


Bug#534800: SyntaxError: keyword argument repeated in repoze.who.plugins.openid/setup.py

2009-06-28 Thread Michael Bienia
On 2009-06-28 10:42:41 +0200, Stefano Zacchiroli wrote:
 According to the error message, it is surely the proper fix.
 What puzzles me, is that I don't get the error in Debian, can you
 please tell me more about your configuration? In particular, which
 Python version are you using during building? Also, it would be nice
 if you mention the version of all the build-dependencies of the
 package.

It's current Ubuntu karmic. The versions are as follows:
python 2.6.2-0ubuntu1
python2.6 2.6.2-0ubuntu2
debhelper 7.2.14ubuntu1
cdbs 0.4.56ubuntu5
python-dev 2.6.2-0ubuntu1
python-support 1.0.3ubuntu1
python-setuptools 0.6c9-0ubuntu4

I've also attached a log from trying to build it in my karmic pbuilder
in case you need more information.

Michael
I: using fakeroot in build.
I: Current time: Sun Jun 28 11:20:26 CEST 2009
I: pbuilder-time-stamp: 1246180826
I: Building the build Environment
I: extracting base tarball [/var/cache/pbuilder/karmic/base.tgz]
I: creating local configuration
I: copying local configuration
W: --override-config is not set; not updating apt.conf Read the manpage for 
details.
I: mounting /proc filesystem
I: mounting /dev/pts filesystem
I: policy-rc.d already exists
I: Installing the build-deps
W: no hooks of type D found -- ignoring
/usr/lib/python2.6/dist-packages/GDebi/DebPackage.py:73: DeprecationWarning: 
Accessed deprecated property Package.installedVersion, please see the Version 
class for alternatives.
  instver = inst.installedVersion
/usr/lib/python2.6/dist-packages/GDebi/DebPackage.py:417: DeprecationWarning: 
Accessed deprecated property Package.candidateOrigin, please see the Version 
class for alternatives.
  for origin in pkg.candidateOrigin:
Reading package lists...
Building dependency tree...
Reading state information...
Suggested packages:
  autoconf2.13 autobook autoconf-archive gnu-standards autoconf-doc libtool
  wamerican wordlist whois vacation doc-base dh-make cvs gettext-doc groff
  libdata-dump-perl libcrypt-ssleay-perl libio-socket-ssl-perl less
  www-browser libmail-box-perl python-doc python-tk python-profiler
  python2.6-doc python2.6-profiler
Recommended packages:
  curl wget lynx libglib2.0-data shared-mime-info libmailtools-perl
  libhtml-format-perl xml-core libmail-sendmail-perl
The following NEW packages will be installed:
  autoconf automake autotools-dev bsdmainutils cdbs debhelper fdupes file
  gettext gettext-base groff-base html2text intltool intltool-debian libcroco3
  libexpat1 libglib2.0-0 libhtml-parser-perl libhtml-tagset-perl
  libhtml-tree-perl libmagic1 libpcre3 libpython2.6 libsqlite3-0 liburi-perl
  libwww-perl libxml-parser-perl libxml2 m4 man-db mime-support po-debconf
  python python-central python-dev python-pkg-resources python-setuptools
  python-support python2.6 python2.6-dev
0 upgraded, 40 newly installed, 0 to remove and 0 not upgraded.
Need to get 20.9MB of archives.
After this operation, 66.4MB of additional disk space will be used.
Get:1 http://archive.ubuntu.com karmic/main libmagic1 5.03-1ubuntu1 [389kB]
Get:2 http://archive.ubuntu.com karmic/main file 5.03-1ubuntu1 [47.5kB]
Get:3 http://archive.ubuntu.com karmic/main libsqlite3-0 3.6.14.2-1 [298kB]
Get:4 http://archive.ubuntu.com karmic/main mime-support 3.46-1 [33.8kB]
Get:5 http://archive.ubuntu.com karmic/main python2.6 2.6.2-0ubuntu2 [2530kB]
Get:6 http://archive.ubuntu.com karmic/main python 2.6.2-0ubuntu1 [141kB]
Get:7 http://archive.ubuntu.com karmic/main python-support 1.0.3ubuntu1 [31.5kB]
Get:8 http://archive.ubuntu.com karmic/main bsdmainutils 6.1.10ubuntu4 [182kB]
Get:9 http://archive.ubuntu.com karmic/main gettext-base 0.17-6ubuntu2 [80.2kB]
Get:10 http://archive.ubuntu.com karmic/main groff-base 1.18.1.1-22build1 
[886kB]
Get:11 http://archive.ubuntu.com karmic/main libexpat1 2.0.1-4 [137kB]
Get:12 http://archive.ubuntu.com karmic/main libhtml-tagset-perl 3.20-2 [13.5kB]
Get:13 http://archive.ubuntu.com karmic/main liburi-perl 1.37+dfsg-1ubuntu1 
[88.6kB]
Get:14 http://archive.ubuntu.com karmic/main libhtml-parser-perl 3.61-1 [112kB]
Get:15 http://archive.ubuntu.com karmic/main libhtml-tree-perl 3.23-1 [209kB]
Get:16 http://archive.ubuntu.com karmic/main libwww-perl 5.826-1 [381kB]
Get:17 http://archive.ubuntu.com karmic/main libxml-parser-perl 2.36-1.1build2 
[324kB]
Get:18 http://archive.ubuntu.com karmic/main libxml2 2.7.3.dfsg-1ubuntu1 [860kB]
Get:19 http://archive.ubuntu.com karmic/main man-db 2.5.5-2 [1400kB]
Get:20 http://archive.ubuntu.com karmic/main python-central 0.6.11ubuntu7 
[45.4kB]
Get:21 http://archive.ubuntu.com karmic/main m4 1.4.13-1 [287kB]
Get:22 http://archive.ubuntu.com karmic/main autoconf 2.63-3ubuntu1 [508kB]
Get:23 http://archive.ubuntu.com karmic/main autotools-dev 20090427.1 [63.7kB]
Get:24 http://archive.ubuntu.com karmic/main automake 1:1.10.2-1ubuntu2 [527kB]
Get:25 http://archive.ubuntu.com karmic/main html2text 1.3.2a-14 [104kB]
Get:26 http://archive.ubuntu.com karmic/main libpcre3 7.8-2ubuntu1 [215kB]
Get:27 

Bug#534800: SyntaxError: keyword argument repeated in repoze.who.plugins.openid/setup.py

2009-06-27 Thread Michael Bienia
Package: python-repoze.who-plugins
Version: 20090530-2
Severity: normal

Hello,

I'm trying to sync turbogears2 from experimental to Ubuntu karmic and
I'm currently working myself through the stack and checking which
package else I need to get synced. While trying to build
python-repoze.who-plugins I got the following error:
,
|   File setup.py, line 44
| test_suite='repoze.who.plugins.openid.tests'
| SyntaxError: keyword argument repeated
| ERROR: setup.py in subdir repoze.who.plugins.openid failed
| make: *** [debian/python-module-stampdir/python-repoze.who-plugins] Error 1
`

A look in epoze.who.plugins.openid/setup.py shows that it contains two
test_suite lines. Leaving only the second one in place made the
package build successfully, but I'm not sure if it's the proper fix for
it.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#534692: powder: Calling a bash-script with sh fails if sh is not bash

2009-06-26 Thread Michael Bienia
Package: powder
Version: 111-1
Severity: normal
Tags: patch

Hello,

building powder in Ubuntu karmic failed as debian/rules tries to execute
a bash-script with sh which is dash on Ubuntu.
Here is the part from the build log:

,
| sh -ex ./buildall.sh
| + [ -z -g -O2 ]
| + PREFIX_DOC=/usr/share/doc/powder
| + PREFIX_BIN=/opt/bin
| + function usage {
| ./buildall.sh: 1: function: not found
| make: *** [build-stamp] Error 127
`

The easy fix is to use bash to execute this script. From a quick look at
the script it looks easy to make it a sh-script alternatively.

Regards,
Michael

diff -u powder-111/debian/rules powder-111/debian/rules
--- powder-111/debian/rules
+++ powder-111/debian/rules
@@ -6,7 +6,7 @@
 
 build-stamp:
dh_testdir
-   sh -ex ./buildall.sh
+   bash -ex ./buildall.sh
touch $@
 
 clean:



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#534696: liburi-find-perl: libmodule-build-perl provided by perl-modules is too old for this package to build

2009-06-26 Thread Michael Bienia
Package: liburi-find-perl
Version: 20090319-1
Severity: normal

Hello,

trying to build liburi-find-perl in Ubuntu karmic failed as the
libmodule-build-perl provided by perl-modules isn't recent enough for
this package. There is already a versioned build-dependency on
libmodule-build-perl but as it's only an alternative to perl-modules it
doesn't have any effect. Dropping the build-dependeny on perl-modules
makes the package buildable again.

,
|  debian/rules build
| dh_testdir
| # Add commands to compile the package here
| /usr/bin/perl Build.PL installdirs=vendor
| Module::Build version 0.3 required--this is only version 0.280801 at Build.PL 
line 3.
| BEGIN failed--compilation aborted at Build.PL line 3.
| make: *** [build-stamp] Error 9
| dpkg-buildpackage: failure: debian/rules build gave error exit status 2
`

Regards
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#487503: Missing format argument in lockfile-progs.c:usage()

2009-06-19 Thread Michael Bienia
On 2009-06-17 21:21:05 -0700, Rob Browning wrote:
 Michael Bienia mich...@vorlon.ping.de writes:
 
  As Ubuntu's gcc has some hardening patches applied, lockfile-progs
  0.1.11-0.1 fails to build in Ubuntu intrepid with the following error
  message:
  ,
  | lockfile-progs.c: In function 'usage':
  | lockfile-progs.c:79: error: format not a string literal and no format 
  arguments
  `
  I hope your are interested in a patch fixing it (patch attached).
 
 Hmm.  Perhaps the usage_str should be a const char*, but is it actually a
 bug for a format string style function to not have any arguments?

I guess the problem is that gcc can't check if the passed string
contains any format specifier or not (see -Wformat-security in the gcc
manpage. This option is enabled in Ubuntu's gcc by default).

 I'd be happy to change it to a const char* if that helps.

I've checked this and making usage_str a const char * isn't enough and
still needs the %s in the msg() call. But what works is replacing the
assignments to usage_str with calls to msg() (and making so usage_str
unnecessary).

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#532768: gtest: Missing build-dependency on python

2009-06-15 Thread Michael Bienia
On 2009-06-14 11:39:20 -0500, Steve M. Robbins wrote:
Hello,

 On Thu, Jun 11, 2009 at 03:00:40PM +0200, Michael Bienia wrote:
  trying to build your package on Ubuntu karmic it FTBFS because of a
  missing build-dependency on python. 
 
 What confuses me about this report is that it *does* build on Debian
 without using python [1].  The configure script probes for python and
 omits some tests if not found.  What is different about the Ubuntu
 build environment?

Ubuntu has python-minimal marked as Essential, so it's installed
everywhere. Therefore the configure scripts detects the python binary,
but python-minimal doesn't include all the needed modules for the test
suite and a standard python is needed for it.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#532768: gtest: Missing build-dependency on python

2009-06-11 Thread Michael Bienia
Package: gtest
Version: 1.3.0-1
Severity: normal

Hello,

trying to build your package on Ubuntu karmic it FTBFS because of a
missing build-dependency on python. Here is a relevant part of the
build log:

,
| Traceback (most recent call last):
|   File ./test/gtest_break_on_failure_unittest.py, line 43, in module
| import gtest_test_utils
|   File /build/buildd/gtest-1.3.0/test/gtest_test_utils.py, line 38, in 
module
| import unittest
| ImportError: No module named unittest
| FAIL: test/gtest_break_on_failure_unittest.py
| Traceback (most recent call last):
|   File ./test/gtest_color_test.py, line 36, in module
| import gtest_test_utils
|   File /build/buildd/gtest-1.3.0/test/gtest_test_utils.py, line 38, in 
module
| import unittest
| ImportError: No module named unittest
| FAIL: test/gtest_color_test.py
| Traceback (most recent call last):
|   File ./test/gtest_env_var_test.py, line 36, in module
| import gtest_test_utils
|   File /build/buildd/gtest-1.3.0/test/gtest_test_utils.py, line 38, in 
module
| import unittest
| ImportError: No module named unittest
| FAIL: test/gtest_env_var_test.py
| Traceback (most recent call last):
|   File ./test/gtest_filter_unittest.py, line 48, in module
| import sets
| ImportError: No module named sets
| FAIL: test/gtest_filter_unittest.py
`
(it repeats for some more tests).

Adding python to Build-Depends fixes it.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#530861: dietlibc: undefined reference to `__builtin_stdarg_start' with gcc 4.4

2009-05-28 Thread Michael Bienia
Package: dietlibc
Version: 0.31-1.2
Severity: normal
Tags: patch

Hello,

this is a forward of the Ubuntu bug
https://bugs.launchpad.net/ubuntu/+source/dietlibc/+bug/381215

While trying to build mksh in Ubuntu karmic (with gcc 4.4) it failed at
the linking stage with undefined reference to
`__builtin_stdarg_start'.

Here is a test-case (copied from the bug):

tgl...@tglase:~$ gcc test.c
tgl...@tglase:~$ ./a.out
'./a.out' called with 1 args
tgl...@tglase:~$ rm a.out
tgl...@tglase:~$ diet gcc test.c
/usr/lib/diet/lib-i386/libc.a(vprintf.o): In function `vprintf':
vprintf.c:(.text+0x20): warning: warning: the printf functions add
several kilobytes of bloat.
/tmp/cccpAo38.o: In function `printfoo':
test.c:(.text+0x14): undefined reference to `__builtin_stdarg_start'
collect2: ld returned 1 exit status
tgl...@tglase:~$ cat test.c
#include stdarg.h
#include stdio.h
#include stdlib.h

void printfoo(const char *, ...);

void
printfoo(const char *fmt, ...)
{
va_list va;

va_start(va, fmt);
if (vprintf(fmt, va)  0)
abort();
va_end(va);
}

int
main(int argc, char *argv[])
{
printfoo('%s' called with %d args\n, argv[0], argc);
}

And here the proposed patch for it:

--- dietlibc-0.31.orig/debian/diff/0901-fix-stdarg-gcc44.diff
+++ dietlibc-0.31/debian/diff/0901-fix-stdarg-gcc44.diff
@@ -0,0 +1,15 @@
+diff -pruN dietlibc-0.31.orig/include/stdarg.h
dietlibc-0.31/include/stdarg.h
+--- dietlibc-0.31.orig/include/stdarg.h2009-05-28
10:53:06.0 +0200
 dietlibc-0.31/include/stdarg.h 2009-05-28 10:55:12.0
+0200
+@@ -9,7 +9,11 @@ __BEGIN_DECLS
+ #if (__GNUC__  2) || ((__GNUC__ == 2)  (__GNUC_MINOR__ = 96))
+ 
+ typedef __builtin_va_list va_list;
++#if (__GNUC__  4) || ((__GNUC__ == 4)  (__GNUC_MINOR__ = 4))
++#define va_start(v,l) __builtin_va_start((v),l)
++#else
+ #define va_start(v,l) __builtin_stdarg_start((v),l)
++#endif
+ #define va_end__builtin_va_end
+ #define va_arg__builtin_va_arg
+ #define __va_copy(d,s)__builtin_va_copy((d),(s))

(skipped the changelog entry from
http://launchpadlibrarian.net/27233139/dietlibc_0.31-1.2ubuntu2.debdiff)

After applying this patch to dietlibc mksh built fine in my Ubuntu
karmic pbuilder.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#528675: libnet-arp-perl: Buffer overflow in get_mac_linux()

2009-05-14 Thread Michael Bienia
Package: libnet-arp-perl
Version: 1.0.3-2
Severity: normal
Tags: patch

Hello,

libnet-arp-perl fails to build in Ubuntu karmic because of a buffer
overflow in get_mac_linux():

t/get_mac*** buffer overflow detected ***: /usr/bin/perl terminated
=== Backtrace: =
/lib/libc.so.6(__fortify_fail+0x4b)[0x4014c97b]
/lib/libc.so.6[0x4014a9c0]
/lib/libc.so.6(__strcpy_chk+0x44)[0x40149cf4]
/build/buildd/libnet-arp-perl-1.0.3/blib/arch/auto/Net/ARP/ARP.so(get_mac_linux+0x7c)[0x401e2afc]
/build/buildd/libnet-arp-perl-1.0.3/blib/arch/auto/Net/ARP/ARP.so(XS_Net__ARP_get_mac+0x1d5)[0x401e1d15]
/usr/bin/perl(Perl_pp_entersub+0x552)[0x80b3c92]
/usr/bin/perl(Perl_runops_standard+0x19)[0x80b2069]
/usr/bin/perl(perl_run+0x2e0)[0x80b04d0]
/usr/bin/perl(main+0xed)[0x8063ebd]
/lib/libc.so.6(__libc_start_main+0xe5)[0x4007c775]
/usr/bin/perl[0x8063d31]

The upstream Changes file talks about a fix for a buffer overflow in
get_mac_linux() but when I compared the version 1.0.2 and 1.0.3 (or even
1.0.4) I couldn't find any changes for this.

Attached is a small patch which fixes this by ensuring that not more
data is copied into the interface name buffer than fits in.

Regards,
Michael
diff -u libnet-arp-perl-1.0.3/debian/patches/series 
libnet-arp-perl-1.0.3/debian/patches/series
--- libnet-arp-perl-1.0.3/debian/patches/series
+++ libnet-arp-perl-1.0.3/debian/patches/series
@@ -6,0 +7 @@
+get_mac_linux_bufferoverflow.patch
only in patch2:
unchanged:
--- libnet-arp-perl-1.0.3.orig/debian/patches/get_mac_linux_bufferoverflow.patch
+++ libnet-arp-perl-1.0.3/debian/patches/get_mac_linux_bufferoverflow.patch
@@ -0,0 +1,14 @@
+Index: libnet-arp-perl-1.0.3/get_mac_linux.c
+===
+--- libnet-arp-perl-1.0.3.orig/get_mac_linux.c 2009-05-14 17:52:35.0 
+0200
 libnet-arp-perl-1.0.3/get_mac_linux.c  2009-05-14 17:55:59.0 
+0200
+@@ -43,7 +43,8 @@
+   if(strlen(dev) == 0)
+ return -1;
+   
+-  strcpy(iface.ifr_name,dev);
++  strncpy(iface.ifr_name, dev, IFNAMSIZ);
++  iface.ifr_name[IFNAMSIZ] = '\0';
+   
+   // Open a socket
+   if((sock = socket(AF_INET, SOCK_DGRAM, 0))  0)


Bug#526623: html2text: Bashism in debian/tests/runtests

2009-05-02 Thread Michael Bienia
Package: html2text
Version: 1.3.2a-13
Severity: normal
Tags: patch

Hello,

there is a bashism in debian/tests/runtests which makes this package
FTBFS on systems where /bin/sh is dash (e.g. Ubuntu).
Here is the relevant log from an Ubuntu build:

cd debian/tests/  sh runtests
runtests: 7: function: not found

Attached is a patch which fixes this.

Regards
Michael
diff -u html2text-1.3.2a/debian/tests/runtests 
html2text-1.3.2a/debian/tests/runtests
--- html2text-1.3.2a/debian/tests/runtests
+++ html2text-1.3.2a/debian/tests/runtests
@@ -4,7 +4,7 @@
 
 # $1 - html file name
 # $2 - html2text param
-function check
+check()
 {
../../html2text $2 $1  $1.result
cmp $1.result $1.supposed


Bug#526455: canlock: Bashism in debian/rules

2009-05-01 Thread Michael Bienia
Package: canlock
Version: 2b-5
Severity: normal
Tags: patch

Hello,

your package contains a bashism in debian/rules which makes it FTBFS on
system where /bin/sh is dash (like e.g. Ubuntu). Attached is a patch to
fix this.

Regards
Michael
diff -u canlock-2b/debian/rules canlock-2b/debian/rules
--- canlock-2b/debian/rules
+++ canlock-2b/debian/rules
@@ -62,7 +62,8 @@
$(MAKE) test
 
# Add here commands to install the package into debian/tmp
-   mkdir -p $(CURDIR)/debian/tmp/usr/{lib,include}
+   mkdir -p $(CURDIR)/debian/tmp/usr/lib
+   mkdir -p $(CURDIR)/debian/tmp/usr/include
$(MAKE) install DESTDIR=$(CURDIR)/debian/tmp
 
 


Bug#502558: FTBFS: fails to build when either libcurl4-*-dev is installed

2009-04-29 Thread Michael Bienia
On 2008-10-17 16:08:15 -0400, Stephen Depooter wrote:
 [snip]
 gcc  -g -O2 -Wall -Wno-pointer-sign   -o gpgkeys_finger 
 gpgkeys_finger-gpgkeys_finger.o gpgkeys_finger-ksutil.o ../util/libutil.a
 gcc -DHAVE_CONFIG_H -I. -I.. -I../../keyserver -I../../include -I../../intl   
  -g -O2 -Wall -Wno-pointer-sign -MT gpgkeys_curl-gpgkeys_curl.o -MD -MP -MF 
 .deps/gpgkeys_curl-gpgkeys_curl.Tpo -c -o gpgkeys_curl-gpgkeys_curl.o `test 
 -f 'gpgkeys_curl.c' || echo '../../keyserver/'`gpgkeys_curl.c
 ../../keyserver/gpgkeys_curl.c: In function 'main':
 ../../keyserver/gpgkeys_curl.c:304: error: 'typeof' applied to a bit-field
 ../../keyserver/gpgkeys_curl.c:304: error: 'typeof' applied to a bit-field

http://lists.gnupg.org/pipermail/gnupg-devel/2008-April/024344.html
contains a patch to fix this.

Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#517993: urlgrabber: Patch for md5 deprecation warning from python 2.6

2009-03-03 Thread Michael Bienia
Package: urlgrabber
Version: 3.1.0-4
Severity: normal
Tags: patch

Hello,

as Ubuntu is already transitioning to python 2.6 here is a patch which
fixes a deprecation warning about the md5 module. It also contains a
change for the python 2.6 transition and removes the python version
from the shebang in /usr/bin/urlgrabber so the package can depend on
python (instead of python2.5).

Regards
Michael
diff -u urlgrabber-3.1.0/debian/patches/00list 
urlgrabber-3.1.0/debian/patches/00list
--- urlgrabber-3.1.0/debian/patches/00list
+++ urlgrabber-3.1.0/debian/patches/00list
@@ -1,0 +2 @@
+002_md5_deprecation
diff -u urlgrabber-3.1.0/debian/rules urlgrabber-3.1.0/debian/rules
--- urlgrabber-3.1.0/debian/rules
+++ urlgrabber-3.1.0/debian/rules
@@ -9,6 +9,7 @@
 # Uncomment this to turn on verbose mode.
 #export DH_VERBOSE=1
 
+include /usr/share/python/python.mk
 PYVERS = $(shell pyversions -r)
 
 CFLAGS = -Wall -g
@@ -63,10 +64,12 @@
dh_installdirs
# Add here commands to install the package into debian/urlgrabber.
for py in $(PYVERS); do \
-   $$py setup.py install 
--root=$(CURDIR)/debian/python-urlgrabber; \
+   $$py setup.py install --root=$(CURDIR)/debian/python-urlgrabber 
$(py_setup_install_args); \
done
# remove unneeded documents installed by setup.py
-rm -rf $(CURDIR)/debian/python-urlgrabber/usr/share/doc/urlgrabber-3*
+   # use an unversioned python interpreter
+   sed -ri 's,^#![[:space:]]*/usr/bin/python.\..,#! /usr/bin/python,' 
$(CURDIR)/debian/python-urlgrabber/usr/bin/*
 
 # Build architecture-independent files here.
 binary-indep: build install
only in patch2:
unchanged:
--- urlgrabber-3.1.0.orig/debian/patches/002_md5_deprecation.dpatch
+++ urlgrabber-3.1.0/debian/patches/002_md5_deprecation.dpatch
@@ -0,0 +1,46 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## 002_md5_deprecation.dpatch by Michael Bienia ge...@ubuntu.com
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: Fix deprecation warning from python 2.6 about the md5 module
+
+...@dpatch@
+diff -urNad urlgrabber-3.1.0~/urlgrabber/keepalive.py 
urlgrabber-3.1.0/urlgrabber/keepalive.py
+--- urlgrabber-3.1.0~/urlgrabber/keepalive.py  2006-09-22 02:58:05.0 
+0200
 urlgrabber-3.1.0/urlgrabber/keepalive.py   2009-03-03 12:54:14.0 
+0100
+@@ -485,7 +485,7 @@
+ keepalive_handler.close_all()
+ 
+ def continuity(url):
+-import md5
++from hashlib import md5
+ format = '%25s: %s'
+ 
+ # first fetch the file with the normal http handler
+@@ -494,7 +494,7 @@
+ fo = urllib2.urlopen(url)
+ foo = fo.read()
+ fo.close()
+-m = md5.new(foo)
++m = md5(foo)
+ print format % ('normal urllib', m.hexdigest())
+ 
+ # now install the keepalive handler and try again
+@@ -504,7 +504,7 @@
+ fo = urllib2.urlopen(url)
+ foo = fo.read()
+ fo.close()
+-m = md5.new(foo)
++m = md5(foo)
+ print format % ('keepalive read', m.hexdigest())
+ 
+ fo = urllib2.urlopen(url)
+@@ -514,7 +514,7 @@
+ if f: foo = foo + f
+ else: break
+ fo.close()
+-m = md5.new(foo)
++m = md5(foo)
+ print format % ('keepalive readline', m.hexdigest())
+ 
+ def comp(N, url):


Bug#517831: dput: Fix for deprecation warning when used with python 2.6

2009-03-02 Thread Michael Bienia
Package: dput
Version: 0.9.2.37
Severity: normal
Tags: patch

Hello,

as Ubuntu is currently switching to python 2.6 (I guess Debian will
follow in the near future), dput produces the following warnings:
,
| /usr/bin/dput:21: DeprecationWarning: the md5 module is deprecated; use 
hashlib instead
|   import pwd, stat, md5, sha, rfc822
| /usr/bin/dput:21: DeprecationWarning: the sha module is deprecated; use the 
hashlib module instead
|   import pwd, stat, md5, sha, rfc822
`
Attached is a patch which makes dput use the hashlib module.

Regards
Michael
diff -Nru dput-0.9.2.37ubuntu2/dput dput-0.9.2.37ubuntu3/dput
--- dput-0.9.2.37ubuntu2/dput	2009-02-10 16:04:34.0 +0100
+++ dput-0.9.2.37ubuntu3/dput	2009-03-02 11:37:39.0 +0100
@@ -18,7 +18,8 @@
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
 import os, sys, string, re, glob, signal
-import pwd, stat, md5, sha, rfc822
+import pwd, stat, rfc822
+from hashlib import md5, sha1
 
 # Now import our modules
 import ConfigParser
@@ -170,9 +171,9 @@
 if hash == 'md5':
 hash_type = md5
 else:
-hash_type = sha
+hash_type = sha1
 
-check_obj = hash_type.new()
+check_obj = hash_type()
 
 for line in file_to_test.readlines():
 check_obj.update(line)


Bug#515305: libperl-minimumversion-perl: Missing build-dependency on libfile-find-rule-perl-perl

2009-02-15 Thread Michael Bienia
Package: libperl-minimumversion-perl
Version: 1.19-1
Severity: normal
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu jaunty

Hello,

while trying to get libperl-minimumversion-perl get build in Ubuntu
jaunty (a small bootstrapping was needed as libperl-minimumversion-perl
build-depends on libtest-minimumversion-perl which depends on
libperl-minimumversion-perl), I discovered that
libperl-minimumversion-perl is missing a build-dependency on
libfile-find-rule-perl-perl making the testsuite fail.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#515178: ocamlweb: Fails to install if texlive is not installed (update-updmap: not found)

2009-02-14 Thread Michael Bienia
Package: ocamlweb
Version: 1.37-9
Severity: normal

Hello,

while trying to resolve an other build problem on Ubuntu, I came across
this problem with your package. As texlive-latex-extra is now in
Recommends and Recommends don't get installed on buildds, ocamlweb fails
to install in such cases.

,
| Setting up ocamlweb (1.37-9) ...
| /var/lib/dpkg/info/ocamlweb.postinst: 62: update-updmap: not found
| dpkg: error processing ocamlweb (--configure):
|  subprocess post-installation script returned error exit status 127
| Errors were encountered while processing:
|  ocamlweb
`

Looking at the postinst, the relevant code is inserted by dh_installtex.
Perhaps this is not a problem with your package directy but in
dh_installtex not coping well if tex-common is not installed.

Regards,
Michael



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#515002: onetime: Bashism in check.sh and missing build-dependency on python

2009-02-12 Thread Michael Bienia
Package: onetime
Version: 1.73-1
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu jaunty ubuntu-patch

Hello,

when trying to build your package on Ubuntu jaunty it FTBFS because of a
bashism in check.sh. Your package is also missing a build-dependency on
python.

Attached is the patch I applied to the Ubuntu package to make it build
again.

Regards,
Michael
diff -u onetime-1.73/debian/control onetime-1.73/debian/control
--- onetime-1.73/debian/control
+++ onetime-1.73/debian/control
@@ -2,7 +2,7 @@
 Section: misc
 Priority: extra
 Maintainer: Karl Fogel kfo...@red-bean.com
-Build-Depends: debhelper (= 5)
+Build-Depends: debhelper (= 5), python
 Standards-Version: 3.8.0
 Homepage: http://www.red-bean.com/onetime/
 Vcs-Svn: http://svn.red-bean.com/repos/onetime/trunk/
diff -u onetime-1.73/debian/changelog onetime-1.73/debian/changelog
--- onetime-1.73/debian/changelog
+++ onetime-1.73/debian/changelog
@@ -1,3 +1,10 @@
+onetime (1.73-1.1) unstable; urgency=low
+
+  * Fix bashism in check.sh.
+  * Add python to Build-Depends.
+
+ -- Michael Bienia ge...@ubuntu.com  Wed, 11 Feb 2009 18:08:34 +0100
+
 onetime (1.73-1) unstable; urgency=low
 
   * Initial release (Closes: #490240)
only in patch2:
unchanged:
--- onetime-1.73.orig/check.sh
+++ onetime-1.73/check.sh
@@ -62,7 +62,7 @@
 
 # Print the (string) first argument, then display all pad lengths.
 # NOTE: Deactivated by default.  Change 'false' to 'true' to turn on.
-function show_lengths()
+show_lengths()
 {
if false; then
  echo ${1}


Bug#505261: audit: Bashism in debian/rules

2008-11-11 Thread Michael Bienia
Package: audit
Version: 1.7.9-1
Severity: normal
Tags: patch

Hello,

your package FTBFS on Ubuntu (jaunty) as it contains some bashism in
debian/rules. Attached is a patch to fix it.

Thanks.
Michael
diff -u audit-1.7.9/debian/rules audit-1.7.9/debian/rules
--- audit-1.7.9/debian/rules
+++ audit-1.7.9/debian/rules
@@ -138,10 +138,12 @@
dh_compress
dh_fixperms
#chmod o-wx debian/auditd/usr/bin/aulastlog
-   #chmod o-wx debian/auditd/sbin/{auditctl,auditd,autrace}
+   #chmod o-wx debian/auditd/sbin/auditctl
+   #chmod o-wx debian/auditd/sbin/auditd
+   #chmod o-wx debian/auditd/sbin/autrace
#chmod 750 debian/auditd/sbin/audispd
chmod -R o-rwx debian/auditd/var/log/audit
-   chmod -R o-rwx debian/auditd/etc/audit 
debian/auditd/etc/audisp{,/plugins.d}
+   chmod -R o-rwx debian/auditd/etc/audit debian/auditd/etc/audisp 
debian/auditd/etc/audisp/plugins.d
dh_pysupport
dh_makeshlibs
dh_installdeb


Bug#486250: buildd.emdebian.org: gnupg fails to crossbuild - problem with curl?

2008-07-20 Thread Michael Bienia
On 2008-06-14 18:23:07 +0100, Neil Williams wrote:

Hello,

 *** ../emdebian-tail.log
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field
 .../../keyserver/gpgkeys_curl.c:304: error: ?typeof? applied to a bit-field

I've run into the same problem while merging gnupg for Ubuntu intrepid.
This is a problem how gnupg passes its arguments to libcurl. It was
already discussed on the gnupg-devel mailing list[0] and contains also a
patch for it[1].

Regards,
Michael

0: http://lists.gnupg.org/pipermail/gnupg-devel/2008-April/024334.html
1: http://lists.gnupg.org/pipermail/gnupg-devel/2008-April/024344.html



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



Bug#490064: libjdic-java: Missing build-dependency on java-gcj-compat-dev

2008-07-09 Thread Michael Bienia
Package: libjdic-java
Version: 0.0.20060613-8
Severity: normal
User: [EMAIL PROTECTED]
Usertags: origin-ubuntu intrepid

Hello,

trying to build libjdic-java in Ubuntu intrepid fails with the
following error:

ant -f jdic/build.xml buildunixjni buildunixembed
Unable to locate tools.jar. Expected to find it in 
/usr/lib/jvm/java-1.5.0-gcj-4.3-1.5.0.0/lib/tools.jar
Buildfile: jdic/build.xml

init:
 [echo] 
 [echo] + JDIC API build version 2006 release 0308
 [echo] + The distribution directory is: ./dist/linux/
 [echo] 

buildunixjni:
 [exec] make[1]: Entering directory 
`/tmp/buildd/libjdic-java-0.0.20060613/jdic/src/unix/native/jni'
 [exec] g++ -c -fPIC -I/include -I/include/linux `pkg-config --cflags 
glib-2.0` `pkg-config --cflags libgnome-2.0` `pkg-config --cflags 
gnome-vfs-2.0` `pkg-config --cflags gnome-vfs-module-2.0` `pkg-config --cflags 
bonobo-activation-2.0` `pkg-config --cflags libbonobo-2.0` `pkg-config --cflags 
ORBit-2.0` `pkg-config --cflags gconf-2.0` -o GnomeBrowserService.o 
GnomeBrowserService.cpp
 [exec] GnomeBrowserService.cpp:21:17: error: jni.h: No such file or 
directory
 [exec] In file included from GnomeBrowserService.cpp:22:
 [exec] GnomeBrowserService.h:16: error: 'JNIEXPORT' does not name a type
 [exec] make[1]: Leaving directory 
`/tmp/buildd/libjdic-java-0.0.20060613/jdic/src/unix/native/jni'
 [exec] GnomeBrowserService.cpp:30: error: 'JNIEXPORT' does not name a type
 [exec] make[1]: *** [GnomeBrowserService.o] Error 1

BUILD FAILED
/tmp/buildd/libjdic-java-0.0.20060613/jdic/build.xml:341: exec returned: 2

I could fix it by adding java-gcj-compat-dev to build-depends.

Regards
Michael



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



  1   2   >