Re: RFS: hwinfo (updated package)

2011-06-27 Thread William Vera
Hello!

On Thu, Jun 16, 2011 at 11:05 AM, Michael Biebl bi...@debian.org wrote:
 Am 16.06.2011 13:48, schrieb Kilian Krause:
 Hi William,


 Simply changing the build-depends from libhal-dev to libudev-dev does not
 magically port an application from HAL to udev.

 Btw, the URL debian/copyright [1] does give a 404. You should update that.

Done


 The latest version I could find is 18.5 via [2]. The spec file indicates, that
 this version no longer uses hal:

 * Fri May 21 2010 snw...@suse.de
 - detect formfactor without hal (bnc #591703)
 * Wed Mar 17 2010 snw...@suse.de
 - remove hal dependency
 - removed VERSION

I said the same in my last email


 Please consider updating the hwinfo package in Debian accordingly.

I have some issues about the package:

kbd.c: In function 'add_serial_console':
kbd.c:105:27: warning: unused variable 'tty_minor' [-Wunused-variable]
kbd.c:105:12: warning: unused variable 'tty_major' [-Wunused-variable]
gcc -c -O2 -Wall -Wno-pointer-sign -pipe -g -fPIC  -I../../src/hd -I
/usr/include/dbus-1.0 -I /usr/lib64/dbus-1.0/include -I
/usr/lib/dbus-1.0/include klog.c
gcc -c -O2 -Wall -Wno-pointer-sign -pipe -g -fPIC  -I../../src/hd -I
/usr/include/dbus-1.0 -I /usr/lib64/dbus-1.0/include -I
/usr/lib/dbus-1.0/include manual.c
gcc -c -O2 -Wall -Wno-pointer-sign -pipe -g -fPIC  -I../../src/hd -I
/usr/include/dbus-1.0 -I /usr/lib64/dbus-1.0/include -I
/usr/lib/dbus-1.0/include mdt.c
mdt.c:20:20: fatal error: x86emu.h: No such file or directory
compilation terminated.
make[3]: *** [mdt.o] Error 1
make[3]: Leaving directory `/home/billy/dev/hwinfo/hwinfo-18.5/src/hd'
make[2]: *** [subdirs] Error 2
make[2]: Leaving directory `/home/billy/dev/hwinfo/hwinfo-18.5/src'
make[1]: *** [subdirs] Error 2
make[1]: Leaving directory `/home/billy/dev/hwinfo/hwinfo-18.5'
dh_auto_build: make -j1 returned exit code 2
make: *** [build] Error 2
dpkg-buildpackage: error: debian/rules build gave error exit status 2
billy@skynet:~/dev/hwinfo/hwinfo-18.5$

Then I looking for x86emu.h:

billy@skynet:~/dev/hwinfo/hwinfo-18.5$ locate x86emu.h
/home/billy/dev/hwinfo/hwinfo-16.0/src/x86emu/include/x86emu.h
billy@skynet:~/dev/hwinfo/hwinfo-18.5$

Is only in the old version (16.0), if I see the RPM depends:

billy@skynet:~$ rpm -qpR hwinfo-18.5-2.2.src.rpm
warning: hwinfo-18.5-2.2.src.rpm: Header V3 RSA/SHA256 Signature, key
ID 3dbdc284: NOKEY
doxygen
flex
perl-XML-Parser
udev
perl-XML-Writer
libx86emu-devel
rpmlib(CompressedFileNames) = 3.0.4-1
billy@skynet:~$

It show a libx86emu-devel as depend obviously, and at the new project's URL:

http://download.opensuse.org/factory/repo/src-oss/suse/src/

We can found at libx86emu-1.1-12.1.src.rpm, as I see x86emu.h is
packaged as different software and is no longer included in the
sources of hwinfo.

I tried to find a package to provide x86emu.h unsuccessfully:

billy@skynet:~$ apt-cache search x86emu
billy@skynet:~$ dpkg -S x86emu
dpkg-query: no path found matching pattern *x86emu*.
billy@skynet:~$

I need make a package for x86emu.h or I can make some patch for this?

I would appreciate your comments and advices.


 Cheers,
 Michael

 [1]
 http://mirrors.kernel.org/pub/opensuse/distribution/SL-OSS-factory/inst-source/suse/src
 [2] 
 http://software.opensuse.org/search?q=hwinfobaseproject=openSUSE%3AFactory
 --
 Why is it that all of the instruments seeking intelligent life in the
 universe are pointed away from Earth?



Regards

-- 
William Vera bi...@billy.com.mx
PGP Key: 1024D/F5CC22A4
Fingerprint: 3E73 FA1F 5C57 6005 0439  4D75 1FD2 BF96 F5CC 22A4


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/banlktikjmxhnedlpfhfzhkyclu9vaka...@mail.gmail.com



Re: RFS: hwinfo (updated package)

2011-06-27 Thread Kilian Krause
William,

On Mon, 2011-06-27 at 01:34 -0500, William Vera wrote:
-(snip)-
 I need make a package for x86emu.h or I can make some patch for this?
 
 I would appreciate your comments and advices.

from a first look at [1] this seems to be a kFreeBSD issue so it may be
safe to patch the x86emu.h out for Linux.

[1]:
http://packages.debian.org/search?searchon=contentskeywords=x86emu.hmode=exactfilenamesuite=unstablearch=any

-- 
Best regards,
Kilian


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


Re: RFS: arp-scan (updated package, new maintainer)

2011-06-27 Thread Kilian Krause
William,

On Sun, 2011-06-26 at 20:29 -0500, William Vera wrote:
 I am looking for a sponsor for the new version 1.6-3
 of my package arp-scan.
 
 It builds these binary packages:
 arp-scan   - arp scanning and fingerprinting tool
 
 The package appears to be lintian clean.
 
 The upload would fix these bugs: 630563, 631213
 
 The package can be found on mentors.debian.net:
 - URL: http://mentors.debian.net/debian/pool/main/a/arp-scan
 - Source repository: deb-src http://mentors.debian.net/debian unstable
 main contrib non-free
 - dget 
 http://mentors.debian.net/debian/pool/main/a/arp-scan/arp-scan_1.6-3.dsc
 
 I would be glad if someone uploaded this package for me.

thanks for your work and for stepping up as new maintainer of this
package. Here's some question's I'd like answered before finally putting
this into the archive:

1. you introduced debian/watch and latest version according to uscan is
1.8. Yet you still uploaded a 1.6-2. Why? Is 1.8 not yet ready?

2. You bump debhelper build-depends to 7.0.50~ but leave debian/compat
at 5. I think both should be 8.

3. debian/patches/debian-changes-1.6-2 was removed with no obvious
reason why it was needed in the first place and now isn't anymore. To me
this patch smells like a backport from a newer upstream release. Please
put some sort of comment into the changelog if this was intentionally
dropped with a short explanation (obviously cannot be fixed upstream
with same upstream version packaged).

4. arp-scan-1.6-2/debian/patches/fixmanpages.patch has no comment this
was forwarded upstream. I think it should be though.

5. lrwxrwxrwx  root/root   /usr/share/doc/arp-scan/ChangeLog.gz -
changelog.gz

is a funny artefact of dh_installchangelogs -k  I think going
without the -k would be better unless you say something is relying on
the name ChangeLog.gz in mixed spelling.

6. dpkg-gencontrol: warning: package arp-scan: unused substitution
variable ${perl:Depends}

should also be fixed I guess.

-- 
Best regards,
Kilian


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


Re: RFS: arp-scan (updated package, new maintainer)

2011-06-27 Thread William Vera
Hi!

On Mon, Jun 27, 2011 at 2:04 AM, Kilian Krause kil...@debian.org wrote:

 thanks for your work and for stepping up as new maintainer of this
 package. Here's some question's I'd like answered before finally putting
 this into the archive:

 1. you introduced debian/watch and latest version according to uscan is
 1.8. Yet you still uploaded a 1.6-2. Why? Is 1.8 not yet ready?

I recently downloaded and at first look, I have this error:

Checking custom ARP request packet with VLAN tag and LLC/SNAP framing
against ./pkt-custom-request-vlan-llc.dat ...
FAILED
FAIL: check-packet
Checking simple ARP response packet decode using ./pkt-simple-response.pcap ...
ok
Checking padded ARP response packet decode using ./pkt-padding-response.pcap ...
ok
Checking 802.1Q ARP response packet decode using ./pkt-vlan-response.pcap ...
ok
Checking LLC/SNAP ARP response packet decode using ./pkt-llc-response.pcap ...
ok
Checking 192.168.1.0/24 ARP response packet decode using
./pkt-net1921681-response.pcap ...
ok
Checking IP range ARP response packet decode using
./pkt-simple-response.pcap ...
ok
Checking IP net:mask ARP response packet decode using
./pkt-simple-response.pcap ...
ok
Checking trailer ARP response packet decode using
./pkt-trailer-response.pcap ...
ok
Checking 802.1Q LLC ARP response packet decode using
./pkt-vlan-llc-response.pcap ...
ok
PASS: check-decode
Checking host list creation using ./pkt-net1921681-response.pcap ...
ok
PASS: check-host-list
=
1 of 4 tests failed
Please report to arp-s...@nta-monitor.com
=
make[2]: *** [check-TESTS] Error 1
make[2]: Leaving directory `/home/billy/dev/arp-scan/arp-scan-1.8'
make[1]: *** [check-am] Error 2
make[1]: Leaving directory `/home/billy/dev/arp-scan/arp-scan-1.8'
dh_auto_test: make -j1 check returned exit code 2
make: *** [build] Error 29
dpkg-buildpackage: error: debian/rules build gave error exit status 2

I'm a little tired now (it's 3am here) but as I said, at first look
its a bite weird.


 2. You bump debhelper build-depends to 7.0.50~ but leave debian/compat
 at 5. I think both should be 8.

Updated.


 3. debian/patches/debian-changes-1.6-2 was removed with no obvious
 reason why it was needed in the first place and now isn't anymore. To me
 this patch smells like a backport from a newer upstream release. Please
 put some sort of comment into the changelog if this was intentionally
 dropped with a short explanation (obviously cannot be fixed upstream
 with same upstream version packaged).


It's not from original source is because my dirty workspace I just
removed the directory and uncompressed again to cleanup the diff.

 4. arp-scan-1.6-2/debian/patches/fixmanpages.patch has no comment this
 was forwarded upstream. I think it should be though.

Yes I did.


 5. lrwxrwxrwx  root/root   /usr/share/doc/arp-scan/ChangeLog.gz -
 changelog.gz

 is a funny artefact of dh_installchangelogs -k  I think going
 without the -k would be better unless you say something is relying on
 the name ChangeLog.gz in mixed spelling.


It's according the man page: man dh_installchangelogs

 6. dpkg-gencontrol: warning: package arp-scan: unused substitution
 variable ${perl:Depends}

I'm a little confused, how I can clean this warning?


 should also be fixed I guess.


Sure, I'm working on it!

 --
 Best regards,
 Kilian


Thanks!
Cheers

-- 
William Vera bi...@billy.com.mx
PGP Key: 1024D/F5CC22A4
Fingerprint: 3E73 FA1F 5C57 6005 0439  4D75 1FD2 BF96 F5CC 22A4


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/BANLkTi=drsibf1g9heqf2zdzsq_kpga...@mail.gmail.com



Re: RFS: arp-scan (updated package, new maintainer)

2011-06-27 Thread Kilian Krause
William,

On Mon, 2011-06-27 at 03:05 -0500, William Vera wrote:
 Hi!
 
 On Mon, Jun 27, 2011 at 2:04 AM, Kilian Krause kil...@debian.org wrote:
 
  thanks for your work and for stepping up as new maintainer of this
  package. Here's some question's I'd like answered before finally putting
  this into the archive:
 
  1. you introduced debian/watch and latest version according to uscan is
  1.8. Yet you still uploaded a 1.6-2. Why? Is 1.8 not yet ready?
 
 I recently downloaded and at first look, I have this error:
 
 Checking custom ARP request packet with VLAN tag and LLC/SNAP framing
 against ./pkt-custom-request-vlan-llc.dat ...
 FAILED
 FAIL: check-packet
 Checking simple ARP response packet decode using ./pkt-simple-response.pcap 
 ...
 ok
 Checking padded ARP response packet decode using ./pkt-padding-response.pcap 
 ...
 ok
 Checking 802.1Q ARP response packet decode using ./pkt-vlan-response.pcap ...
 ok
 Checking LLC/SNAP ARP response packet decode using ./pkt-llc-response.pcap ...
 ok
 Checking 192.168.1.0/24 ARP response packet decode using
 ./pkt-net1921681-response.pcap ...
 ok
 Checking IP range ARP response packet decode using
 ./pkt-simple-response.pcap ...
 ok
 Checking IP net:mask ARP response packet decode using
 ./pkt-simple-response.pcap ...
 ok
 Checking trailer ARP response packet decode using
 ./pkt-trailer-response.pcap ...
 ok
 Checking 802.1Q LLC ARP response packet decode using
 ./pkt-vlan-llc-response.pcap ...
 ok
 PASS: check-decode
 Checking host list creation using ./pkt-net1921681-response.pcap ...
 ok
 PASS: check-host-list
 =
 1 of 4 tests failed
 Please report to arp-s...@nta-monitor.com
 =
 make[2]: *** [check-TESTS] Error 1
 make[2]: Leaving directory `/home/billy/dev/arp-scan/arp-scan-1.8'
 make[1]: *** [check-am] Error 2
 make[1]: Leaving directory `/home/billy/dev/arp-scan/arp-scan-1.8'
 dh_auto_test: make -j1 check returned exit code 2
 make: *** [build] Error 29
 dpkg-buildpackage: error: debian/rules build gave error exit status 2
 
 I'm a little tired now (it's 3am here) but as I said, at first look
 its a bite weird.

I guess you should try to do what it reads. Ask 
arp-s...@nta-monitor.com and see what they tell about why this is
happening - and how to fix it properly.



  2. You bump debhelper build-depends to 7.0.50~ but leave debian/compat
  at 5. I think both should be 8.
 
 Updated.
 
 
  3. debian/patches/debian-changes-1.6-2 was removed with no obvious
  reason why it was needed in the first place and now isn't anymore. To me
  this patch smells like a backport from a newer upstream release. Please
  put some sort of comment into the changelog if this was intentionally
  dropped with a short explanation (obviously cannot be fixed upstream
  with same upstream version packaged).
 
 
 It's not from original source is because my dirty workspace I just
 removed the directory and uncompressed again to cleanup the diff.

I seem to have put it confusingly. You dropped a patch that was in
1.6-1. Where has that patch gone? Why is it dropped and no longer
needed? 


  4. arp-scan-1.6-2/debian/patches/fixmanpages.patch has no comment this
  was forwarded upstream. I think it should be though.
 
 Yes I did.

Great!


  5. lrwxrwxrwx  root/root   /usr/share/doc/arp-scan/ChangeLog.gz -
  changelog.gz
 
  is a funny artefact of dh_installchangelogs -k  I think going
  without the -k would be better unless you say something is relying on
  the name ChangeLog.gz in mixed spelling.
 
 
 It's according the man page: man dh_installchangelogs

Which part of:
  Keep the original name of the upstream changelog. This will be
  accomplished by installing the upstream changelog as
changelog, and
  making a symlink from that to the original name of the
changelog
  file. This can be useful if the upstream changelog has an
unusual
  name, or if other documentation in the package refers to the
  changelog file.

are you referring to? ChangeLog is not a unusual name. Thus IMHO the -k
can still be dropped.


  6. dpkg-gencontrol: warning: package arp-scan: unused substitution
  variable ${perl:Depends}
 
 I'm a little confused, how I can clean this warning?

Feed the perl file that resides out of place to dh_perl on the command
line or drop the ${perl:Depends} if it's not needed.


  should also be fixed I guess.
 
 
 Sure, I'm working on it!

Thanks!

-- 
Best regards,
Kilian


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


Re: RFS: cfortran (updated package)

2011-06-27 Thread Kilian Krause
Hi Bastien,

On Sun, 2011-06-26 at 21:08 +0200, Bastien ROUCARIES wrote:
 I am looking for a sponsor for the new version 1:20110621-1
 of my package cfortran.

actually why would you want that? The latest version in unstable has
DM-Upload-Allowed set to yes already.

Anyhow here goes some comments.

 It builds these binary packages:
 cfortran   - Header file permitting Fortran routines to be called in C/C++
 
 The package appears to be lintian clean.

Yes. Very nice. Even though --pedantic still complains about:
P: cfortran source: unneeded-build-dep-on-quilt
P: cfortran: no-upstream-changelog
I: cfortran: conflicts-with-version libcfitsio-dev ( 2.440-1)
I: cfortran: possible-documentation-but-no-doc-base-registration

Regarding the first two I would agree this should be fixed with one of
the next uploads. 


 In order to be more robust I have switched this package to arch all to arch 
 any in order to run testsuite

Not really sure this is a good rationale even though I like the idea.


 Notice that I am maintening it upstream.
 
 The package can be found on mentors.debian.net:
 - URL: http://mentors.debian.net/debian/pool/main/c/cfortran
 - Source repository: deb-src http://mentors.debian.net/debian unstable main 
 contrib non-free
 - dget 
 http://mentors.debian.net/debian/pool/main/c/cfortran/cfortran_20110621-1.dsc
 
 I would be glad if someone uploaded this package for me.

1. uscan gives:
uscan warning: In watchfile debian/watch, reading webpage
  http://qa.debian.org/watch/sf.php/cfortran/ failed: 404 File Not Found

2. Especially since you're upstream yourself I'd favour not doing
autoreconf during the actual build. This should be done prior to
generating the orig.tar.gz IMHO as it usually causes more problems than
it solves. dh-autoreconf should thus be dropped from the B-Depends.

3. debian/copyright isn't yet DEP-5 format.

4. You include /usr/share/quilt/quilt.make in debian/rules to do the
Git-Packaging I guess. As said, I myself would rather export something
stable as orig.tar.gz and work from there. How you script your
get-orig-source target is another thing and can use
quilt/git/dpatch/whatever...

5. You dropped /usr/include/cfortran.h from your deb which may or may
not be required by other packages. I hope this was intentional - cannot
find any mentioning in the changelog.

Anyway, it's your package and someone has already granted you
DM-Upload-Allowed for a reason. Thus if you think this is the way your
package works best, why not upload it yourself?

-- 
Best regards,
Kilian


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


Re: RFS: dsniff (updated package)

2011-06-27 Thread Kilian Krause
Hi William,

On Sun, 2011-06-26 at 01:43 -0500, William Vera wrote:
 Hello Mentors!
 I am still looking for a sponsor, your help is greatly appreciated.
 Thanks
  http://mentors.debian.net/debian/pool/main/d/dsniff/

1. README.source still talks about dpatch despite changelog reads
package converted.

2. libx11-6 is now always pulled in via Depends. You can safely edit
README.Debian to reflect this or rather remove it entirely.

3. in debian/rules why is
override_dh_auto_configure:
dh_auto_configure -- --host=$(DEB_HOST_GNU_TYPE) --build=
$(DEB_BUILD_GNU_TYPE) --prefix=/usr --mandir=\$${prefix}/share/man
--infodir=\$${prefix}/share/info

needed and different from the debhelper7 automagic configuration?

Apart from this fine for upload into Debian IMHO.

-- 
Cheers,
Kilian


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/1309165634.4072.45.ca...@rusty.rus.uni-stuttgart.de



Re: RFS: s3ql (new python application)

2011-06-27 Thread Kilian Krause
Hi Nikolaus,

On Sat, 2011-06-25 at 21:05 -0400, Nikolaus Rath wrote:
 I am looking for a sponsor for my package s3ql.
 
 * Package name: s3ql
   Version : 1.0.1-1
   Upstream Author : me (Nikolaus Rath nikol...@rath.org)
 * URL : http://code.google.com/p/s3ql/
 * License : GPLv3
   Section : misc
 
 It builds these binary packages:
 s3ql   - Full-featured file system for online data storage
 
 The package appears to be lintian clean.
 
 ITP: 626651
 
 I'd be happy to join the python-apps-team and have this team maintained,
 but so far I did not get any response on my request to join the team on
 Alioth. Should I be doing anything else?
 
 
 At the moment, the package can be found on mentors.debian.net:
 - URL: http://mentors.debian.net/debian/pool/main/s/s3ql
 - Source repository: deb-src http://mentors.debian.net/debian unstable main 
 contrib non-free
 - dget http://mentors.debian.net/debian/pool/main/s/s3ql/s3ql_1.0.1-1.dsc
 
 I would be glad if someone uploaded this package for me.

Thanks for your work. Here goes my comments:

1. You use quilt as build-depends. This should not be neccessary with
dpkg-source v3.

2. fix-sys-path.patch feels like you should split out the python modules
into the standard python path. That may also cause ${python:Depends} to
match further packages directly without having them explicitly spelled
out on the Depends line in debian/control

Setting --install-lib=/usr/share/s3ql to something more sensible
(according to [1] would be /usr/lib/s3ql) will help here.

3. debian/README.Debian doesn't need to be in *.docs. See man
dh_installdocs:
 Each of these files is automatically installed if present for a
 package.

4. Downloading
http://pypi.python.org/packages/source/d/distribute/distribute-0.6.14.tar.gz
Extracting in /tmp/tmpeY0G4V

OUCH! buildds do not all have internet access. Your package should build
fine without that. Moreover the contents of the pulled in tarball could
be anything. Thus ensuring DFSG-free sources is virtually impossible
when external sources are pulled in.

5. Your package doesn't build in pbuilder:

dpkg-source: warning: ignoring deletion of directory doc
dpkg-source: error: cannot represent change to
s3ql-1.0.1/distribute-0.6.14.tar.gz: binary file contents changed
dpkg-source: error: add distribute-0.6.14.tar.gz in
debian/source/include-binaries if you want to store the modified binary
in the debian tarball
dpkg-source: error: cannot represent change to
s3ql-1.0.1/distribute-0.6.14-py2.6.egg: binary file contents changed
dpkg-source: error: add distribute-0.6.14-py2.6.egg in
debian/source/include-binaries if you want to store the modified binary
in the debian tarball
dpkg-source: error: cannot represent change to
s3ql-1.0.1/distribute-0.6.14-py2.7.egg: binary file contents changed
dpkg-source: error: add distribute-0.6.14-py2.7.egg in
debian/source/include-binaries if you want to store the modified binary
in the debian tarball
dpkg-source: error: unrepresentable changes to source
dpkg-buildpackage: error: dpkg-source -b s3ql-1.0.1 gave error exit
status 2


[1]:
http://www.debian.org/doc/packaging-manuals/python-policy/ch-module_packages.html

Sorry!

-- 
Regards,
Kilian


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


Re: RFS: oggfix

2011-06-27 Thread Michael Tautschnig
Hi,

[...]
 
  Ok, I wouldn't bother much about the above, but one thing does stop me from
  uploading: why are you setting the version number in such a way that Debian 
  is
  upstream? This doesn't seem to be accurate. IMHO you should be using 0.9.1-1
  as
  version number!?
 Never found out what the version number ought to be.
 0.9.1-1 sounds exactly right. Thanks for that, too!
  

This is explained in full detail in the Debian Policy [1].

Hope this helps,
Michael

[1] http://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Version



pgpjqeXLda4PY.pgp
Description: PGP signature


Re: RFS: jappix

2011-06-27 Thread Michael Tautschnig
Hi,

[...]
  
  The package looks pretty fine from my POV, hence built and uploaded. One 
  thing
  to be fixed by upstream, however, is the lack of license and copyright
  information in source files. Given that this is what will be shipped plus 
  the
  interspersed BSD and GPL files such information would really be valuable.
  
  Furthermore, in future uploads please address the following:
  
  W: jappix: description-synopsis-starts-with-article
  W: jappix: embedded-javascript-library usr/share/jappix/js/jquery.form.js
  
  
  Best regards,
  Michael
  
  PS.: And yes, there's a new upstream version out :-)
 
 Thanks Michael for taking the time to review this package.
 
 I started to set up the packaging in the pkg-xmpp team Git repository. I
 am also in the process of making a package with the latest (0.7)
 version. Marcelo Jorge Viera helped me to find the issues with the
 initial package.
 
 Most of the problems you mentioned should be fixed in the Git
 repository. Do you want me to copy the new version to
 mentors.debian.net?
 

Not necessarily, no, but a pointer to the git repository would have been nice
:-) Anyway, I figured it must be anonscm.debian.org/git/pkg-xmpp/jappix.git/ and
did a git-buildpackage on that one. The HEAD appears to be a somewhat outdated
ed8c3e4ab8ee2992bd4987f14e37ea01450df957, however, and I couldn't find any of my
above concerns addressed in this version!?

Best,
Michael



pgpxYvYjlDpej.pgp
Description: PGP signature


Re: RFS: s3ql (new python application)

2011-06-27 Thread Jakub Wilk

* Kilian Krause kil...@debian.org, 2011-06-27, 11:20:
2. fix-sys-path.patch feels like you should split out the python 
modules into the standard python path.


No. Putting private modules into, well, a private directory is the best 
practice.



Setting --install-lib=/usr/share/s3ql to something more sensible
(according to [1] would be /usr/lib/s3ql) will help here.


Again, no. Even the Python Policy chapter you quotes explicitly says 
/usr/share/s3ql is all right:



http://www.debian.org/doc/packaging-manuals/python-policy/ch-module_packages.html


The more relevant Python Policy chapter would be
http://www.debian.org/doc/packaging-manuals/python-policy/ch-programs.html
however.

--
Jakub Wilk


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110627095400.ga9...@jwilk.net



Re: RFS: nglogc (updated package)

2011-06-27 Thread Michael Tautschnig
Hi,

 Dear mentors,
 
 I am looking for a sponsor for my package nglogc.
 
 * Package name: nglogc
   Version : 1.0.0-1
   Upstream Author : Dennis Krzyzaniak ebros...@netgarage.org
 * URL : http://code.google.com/p/nglogc/downloads/list
 * License : GNU LGPL
   Section : libs
 
 It builds these binary packages:
 nglogc-dev - Flexible C logging API
 nglogc0- Flexible C logging API
 
 The package appears to be lintian clean.
 
[...]

Sorry, but that's simply not true. I'm finding 7 warnings and 2 errors. Please
re-try...

Anyway, here are the more detailed comments, which would also solve some of
those issues found by lintian:

- debian/control: why build for architecture i386 only?
- debian/copyright: please use DEP-5 format.
- debian/rules: please consider using the debhelper 7 simplified style.
- no debian/watch file

Hope this helps,
Michael



pgpxJ2olng4FH.pgp
Description: PGP signature


Re: RFS: s3ql (new python application)

2011-06-27 Thread Kilian Krause
Jakub,

On Mon, 2011-06-27 at 11:54 +0200, Jakub Wilk wrote:
 * Kilian Krause kil...@debian.org, 2011-06-27, 11:20:
 2. fix-sys-path.patch feels like you should split out the python 
 modules into the standard python path.
 
 No. Putting private modules into, well, a private directory is the best 
 practice.
 
 Setting --install-lib=/usr/share/s3ql to something more sensible
 (according to [1] would be /usr/lib/s3ql) will help here.
 
 Again, no. Even the Python Policy chapter you quotes explicitly says 
 /usr/share/s3ql is all right:
 
 http://www.debian.org/doc/packaging-manuals/python-policy/ch-module_packages.html
 
 The more relevant Python Policy chapter would be
 http://www.debian.org/doc/packaging-manuals/python-policy/ch-programs.html
 however.

thanks for double checking. I was somewhat under the impression that
these modules might be used sort of as a library by other programs too.
But I admit that I had a too brief look and should have checked closer.

In any case if upstream foresees this as lib for other python scripts
to use (as is suggested by the configure parameter) then it should be
consequently installed as public lib. If it's private then upstream
should foresee that option consistently in the configure script and
sources. The correct path set for configure PLUS patching still feels
wrong.

-- 
Best regards,
Kilian


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


Re: RFS: debconf-kde (ping)

2011-06-27 Thread Michael Tautschnig
Hi,

 Dear mentors,
 
 I am looking for a sponsor for my package debconf-kde.
 
 * Package name: debconf-kde
   Version : 0.1+git20101228-1
   Upstream Author : Daniel Nicoletti dantti85...@yahoo.com.br
 * URL :
 https://projects.kde.org/projects/extragear/sysadmin/libdebconf-kde/
 * License : LGPL-2+
   Section : kde
 
 It builds these binary packages:
 debconf-kde-dbg - Debconf KDE debugging symbols
 debconf-kde-helper - Debconf KDE GUI frontend tool
 libdebconf-kde-dev - Development headers for the Debconf KDE library
 libdebconf-kde0 - Debconf KDE GUI library
 
[...]

It seems that no one has found the time to sponsor this package so far, hence I
just did that and uploaded it. It looks mostly ok, apart from a few smaller
issues, which should be addressed in future versions:

- There is no Homepage field and also no watch file.
- There is a symbols file, but that has a two problems: it's called
  libdebconf-kde.symbols, and not libdebconf-kde0.symbols; second, you should
  preferably use the c++ pattern of dpkg-gensymbols, which was introduced in
  dpkg-dev 1.5.18.
- Lintian will also tell you the following:
I: debconf-kde-dbg: capitalization-error-in-description debian Debian
  (and similarly for all other packages)

Best regards,
Michael



pgpENSrEI9LJU.pgp
Description: PGP signature


Re: RFS: hiredis - Minimalistic C client library for Redis

2011-06-27 Thread Alessandro Ghedini
On Sat, Jun 25, 2011 at 06:27:50PM +0100, Michael Tautschnig wrote:
 Hi,

Hello,

 Sorry for getting back to this with such a large delay.

No problem.

 [...]
   just one major problem: all files in adapter/ and
   the example* and test.c files lack both copyright and license information.
   Please persuade upstream to get this fixed.
  
  I'm going to contact upstream as soon as possible. Thanks for the pointer.
  What would be a viable solution? I mean, is a new upstream release needed,
  or just a statement from the upstream author (e.g. on the project's 
  bugtracker, to which d/copyright will point) is enough? The second solution
  does not exlude the first of course (the files will be fixed in a 
  subsequent release) but would permit the package to enter Debian without 
  waiting and to not block other packages that depend on this (e.g. webdis 
  which is an ITP).
  
 
 I've uploaded your package as-is. Yet it would be nice if upstream could, 
 with a
 future new upstream release, add all this information to the source files.

Great, thank you very much. Regarding the missing copyright and license 
issue, I have opened a ticket upstream, and, hopefully, it will get a fix
soon.

Thank you again

-- 
perl -E'$_=q;$/= @{[@_]};and s;\S+;inidehG ordnasselA;eg;say~~reverse'


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110627102735.gc1...@pc-ale.fastwebnet.it



Re: RFS: dsniff (updated package)

2011-06-27 Thread William Vera
Hi!

On Mon, Jun 27, 2011 at 4:07 AM, Kilian Krause k...@verfaction.de wrote:
 Hi William,


 1. README.source still talks about dpatch despite changelog reads
 package converted.

Updated for quilt


 2. libx11-6 is now always pulled in via Depends. You can safely edit
 README.Debian to reflect this or rather remove it entirely.

It's in Recommends because it's used by 'webspy' not properly by 'dnisff'
IMHO it's fine how is in README.Debian


 3. in debian/rules why is
 override_dh_auto_configure:
        dh_auto_configure -- --host=$(DEB_HOST_GNU_TYPE) --build=
 $(DEB_BUILD_GNU_TYPE) --prefix=/usr --mandir=\$${prefix}/share/man
 --infodir=\$${prefix}/share/info

 needed and different from the debhelper7 automagic configuration?

Not really, droped


 Apart from this fine for upload into Debian IMHO.


Thanks for you review, the packages is updated:
http://mentors.debian.net/debian/pool/main/d/dsniff/dsniff_2.4b1+debian-21.dsc

 --
 Cheers,
 Kilian



Regards

-- 
William Vera bi...@billy.com.mx
PGP Key: 1024D/F5CC22A4
Fingerprint: 3E73 FA1F 5C57 6005 0439  4D75 1FD2 BF96 F5CC 22A4


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/banlktikctxkcxny3knjzslhcyli_ger...@mail.gmail.com



Re: RFS: arp-scan (updated package, new maintainer)

2011-06-27 Thread William Vera
Hi

On Mon, Jun 27, 2011 at 3:29 AM, Kilian Krause kil...@debian.org wrote:
 William,

 On Mon, 2011-06-27 at 03:05 -0500, William Vera wrote:

 I guess you should try to do what it reads. Ask
 arp-s...@nta-monitor.com and see what they tell about why this is
 happening - and how to fix it properly.

Exactly, I already sent a email


 I seem to have put it confusingly. You dropped a patch that was in
 1.6-1. Where has that patch gone? Why is it dropped and no longer
 needed?

Was not a patch on intentionally, was a self-generated file, the
package did not use quilt until I implemented it, the file
autogenerated showed differences between diff because of a dirty
environment.



 Which part of:
          Keep the original name of the upstream changelog. This will be
          accomplished by installing the upstream changelog as
 changelog, and
          making a symlink from that to the original name of the
 changelog
          file. This can be useful if the upstream changelog has an
 unusual
          name, or if other documentation in the package refers to the
          changelog file.

 are you referring to? ChangeLog is not a unusual name. Thus IMHO the -k
 can still be dropped.


Droped


  6. dpkg-gencontrol: warning: package arp-scan: unused substitution
  variable ${perl:Depends}

 I'm a little confused, how I can clean this warning?

 Feed the perl file that resides out of place to dh_perl on the command
 line or drop the ${perl:Depends} if it's not needed.


I don't understood, I don't have ${perl:Depends} in my control file,
or which file are you talking about?



 Thanks!


Thanks to you for your review

 --
 Best regards,
 Kilian


Cheers

-- 
William Vera bi...@billy.com.mx
PGP Key: 1024D/F5CC22A4
Fingerprint: 3E73 FA1F 5C57 6005 0439  4D75 1FD2 BF96 F5CC 22A4


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/BANLkTi=yxc4unHqBNU9h=G2N0m_S9zX5=g...@mail.gmail.com



Re: RFS: dsniff (updated package)

2011-06-27 Thread Kilian Krause
William,

On Mon, 2011-06-27 at 08:22 -0500, William Vera wrote:
 On Mon, Jun 27, 2011 at 4:07 AM, Kilian Krause k...@verfaction.de wrote:
  Hi William,
 
 
  1. README.source still talks about dpatch despite changelog reads
  package converted.
 
 Updated for quilt

Good.

 
  2. libx11-6 is now always pulled in via Depends. You can safely edit
  README.Debian to reflect this or rather remove it entirely.
 
 It's in Recommends because it's used by 'webspy' not properly by 'dnisff'
 IMHO it's fine how is in README.Debian

Your resulting deb is:
$ dpkg -I dsniff_2.4b1+debian-21_amd64.deb|grep libx11
 Depends: libc6 (= 2.7), libdb5.1, libice6 (= 1:1.0.0), libnet1 (=
1.1.2.1), libnids1.21, libpcap0.8 (= 0.9.8), libsm6, libssl1.0.0 (=
1.0.0), libx11-6, libxmu6, openssl
   (requires libx11-6 installed).
$ 

obviously setting Depends, not Recommends. Thus your README.Debian is
kinda bogusly telling the user to install something she already has
installed.

 
  3. in debian/rules why is
  override_dh_auto_configure:
 dh_auto_configure -- --host=$(DEB_HOST_GNU_TYPE) --build=
  $(DEB_BUILD_GNU_TYPE) --prefix=/usr --mandir=\$${prefix}/share/man
  --infodir=\$${prefix}/share/info
 
  needed and different from the debhelper7 automagic configuration?
 
 Not really, droped

Good.

 
  Apart from this fine for upload into Debian IMHO.
 
 
 Thanks for you review, the packages is updated:
 http://mentors.debian.net/debian/pool/main/d/dsniff/dsniff_2.4b1+debian-21.dsc

built, signed, uploaded.

Thanks!

-- 
Cheers,
Kilian


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


Re: RFS: arp-scan (updated package, new maintainer)

2011-06-27 Thread Kilian Krause
William,

On Mon, 2011-06-27 at 09:01 -0500, William Vera wrote:
 On Mon, Jun 27, 2011 at 3:29 AM, Kilian Krause kil...@debian.org wrote:
  William,
 
  On Mon, 2011-06-27 at 03:05 -0500, William Vera wrote:
 
  I guess you should try to do what it reads. Ask
  arp-s...@nta-monitor.com and see what they tell about why this is
  happening - and how to fix it properly.
 
 Exactly, I already sent a email

Good. ;-)


 
  I seem to have put it confusingly. You dropped a patch that was in
  1.6-1. Where has that patch gone? Why is it dropped and no longer
  needed?
 
 Was not a patch on intentionally, was a self-generated file, the
 package did not use quilt until I implemented it, the file
 autogenerated showed differences between diff because of a dirty
 environment.

Ah, I see.


  Which part of:
   Keep the original name of the upstream changelog. This will be
   accomplished by installing the upstream changelog as
  changelog, and
   making a symlink from that to the original name of the
  changelog
   file. This can be useful if the upstream changelog has an
  unusual
   name, or if other documentation in the package refers to the
   changelog file.
 
  are you referring to? ChangeLog is not a unusual name. Thus IMHO the -k
  can still be dropped.
 
 
 Droped
 
 
   6. dpkg-gencontrol: warning: package arp-scan: unused substitution
   variable ${perl:Depends}
 
  I'm a little confused, how I can clean this warning?
 
  Feed the perl file that resides out of place to dh_perl on the command
  line or drop the ${perl:Depends} if it's not needed.
 
 
 I don't understood, I don't have ${perl:Depends} in my control file,
 or which file are you talking about?

Right. Weird. My pbuilder does throw that though. I think it can be
safely ignored then.


 
 
  Thanks!
 
 
 Thanks to you for your review

Np ;-)

-- 
Best regards,
Kilian


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


Re: RFS: dsniff (updated package)

2011-06-27 Thread William Vera
Hi Kilian

On Mon, Jun 27, 2011 at 9:10 AM, Kilian Krause kil...@debian.org wrote:
 William,

 It's in Recommends because it's used by 'webspy' not properly by 'dnisff'
 IMHO it's fine how is in README.Debian

 Your resulting deb is:
 $ dpkg -I dsniff_2.4b1+debian-21_amd64.deb|grep libx11
  Depends: libc6 (= 2.7), libdb5.1, libice6 (= 1:1.0.0), libnet1 (=
 1.1.2.1), libnids1.21, libpcap0.8 (= 0.9.8), libsm6, libssl1.0.0 (=
 1.0.0), libx11-6, libxmu6, openssl
               (requires libx11-6 installed).
 $

 obviously setting Depends, not Recommends. Thus your README.Debian is
 kinda bogusly telling the user to install something she already has
 installed.


Touché ... I'll fix it in the next release


 built, signed, uploaded.

 Thanks!

Thanks!


 --
 Cheers,
 Kilian


Regards

-- 
William Vera bi...@billy.com.mx
PGP Key: 1024D/F5CC22A4
Fingerprint: 3E73 FA1F 5C57 6005 0439  4D75 1FD2 BF96 F5CC 22A4


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/BANLkTi[-Scm=qlzs_abvt003z6j3m...@mail.gmail.com



Re: RFS: debconf-kde (ping)

2011-06-27 Thread Matthias Klumpp
 [...]
 
 It seems that no one has found the time to sponsor this package so far,
 hence I
 just did that and uploaded it. It looks mostly ok, apart from a few
smaller
 issues, which should be addressed in future versions:
 
 - There is no Homepage field and also no watch file.
 - There is a symbols file, but that has a two problems: it's called
   libdebconf-kde.symbols, and not libdebconf-kde0.symbols; second, you
   should
   preferably use the c++ pattern of dpkg-gensymbols, which was
introduced
   in
   dpkg-dev 1.5.18.
 - Lintian will also tell you the following:
 I: debconf-kde-dbg: capitalization-error-in-description debian Debian
   (and similarly for all other packages)
Hi!
Wow, great! Thanks for sponsoring this package! I'm currently working with
pkg-kde on maintaining it under the pkg-kde-team umbrella, all the issues
you mentioned above have been fixed in our Git branch :)
Anyway, this means Debian is ready for Muon and Apper now ;) (as well as
for a generic KDE Debconf-frontend)
Cheers,
   Matthias




-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/e1108e324798373d911d4918d866d...@mb8-2.1blu.de



How to patch a file with changing path?

2011-06-27 Thread Christian Welzel
Hi there,

currently i'm trying to package the new alpha of typo3, but there is a
problem: the new upstream tar.gz contains the core of typo3 file in
a directory that has the version in its name
(eg: [/]typo3_src-4.6.0alpha2/t3lib/GPL.txt). There are also some
symlinks into this directory tslib - typo3_src-4.6.0alpha2/t3lib.
So if i want to patch GPL.txt, i cannot patch trough the symlink
as quilt wont let me, but i do not want to change all patches every
release to adjust the path of the files...

I'm not sure, how to handle this and need counsel.

There is also a zip available, that does not contain either symlinks
or versioned directories, but i cannot use zip as upstream package.

I use 3.0(quilt) format.

Please set me to CC as i'm not subscribed to the list.


-- 
 MfG, Christian Welzel

  GPG-Key: http://www.camlann.de/de/pgpkey.html
  Fingerprint: 4F50 19BF 3346 36A6 CFA9 DBDC C268 6D24 70A1 AD15


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4e08b622.8030...@camlann.de



Re: How to patch a file with changing path?

2011-06-27 Thread Kilian Krause
Hi Christian,

On Mon, Jun 27, 2011 at 06:56:02PM +0200, Christian Welzel wrote:
 currently i'm trying to package the new alpha of typo3, but there is a
 problem: the new upstream tar.gz contains the core of typo3 file in
 a directory that has the version in its name
 (eg: [/]typo3_src-4.6.0alpha2/t3lib/GPL.txt). There are also some
 symlinks into this directory tslib - typo3_src-4.6.0alpha2/t3lib.
 So if i want to patch GPL.txt, i cannot patch trough the symlink
 as quilt wont let me, but i do not want to change all patches every
 release to adjust the path of the files...

interesting problem. ;-)

 I'm not sure, how to handle this and need counsel.
 
 There is also a zip available, that does not contain either symlinks
 or versioned directories, but i cannot use zip as upstream package.
 
 I use 3.0(quilt) format.

As I see things you've got 3 options:
- talk to upstream and make them release a tarball without that versioned
  dir

- introduce a sed line (or perl/python/whatever) in the clean target using a
  fancy regexp on debian/patches/... and fixing the path according to the
  version number of your deb or even better with a path that you just found
  via e.g. ls -1

- Why you can't use the zip I don't see. You can easily repack and note that
  in README.source. Use the get-orig-source target in debian/rules for that
  and possibly use a ~debian version number. The ~debian is not neccessary
  though IMHO in this case.

-- 
Best regards,
Kilian


signature.asc
Description: Digital signature


Re: RFS: s3ql (new python application)

2011-06-27 Thread Nikolaus Rath
Kilian Krause kilian-8fiuurrzop0dnm+yrof...@public.gmane.org writes:
 Hi Nikolaus,

 On Sat, 2011-06-25 at 21:05 -0400, Nikolaus Rath wrote:
 I am looking for a sponsor for my package s3ql.
 
 * Package name: s3ql
   Version : 1.0.1-1
   Upstream Author : me (Nikolaus Rath 
 nikolaus-bth8mxji...@public.gmane.org)
 * URL : http://code.google.com/p/s3ql/
 * License : GPLv3
   Section : misc
 
 It builds these binary packages:
 s3ql   - Full-featured file system for online data storage
 
 The package appears to be lintian clean.
 
 ITP: 626651
 
 I'd be happy to join the python-apps-team and have this team maintained,
 but so far I did not get any response on my request to join the team on
 Alioth. Should I be doing anything else?
 
 
 At the moment, the package can be found on mentors.debian.net:
 - URL: http://mentors.debian.net/debian/pool/main/s/s3ql
 - Source repository: deb-src http://mentors.debian.net/debian unstable main 
 contrib non-free
 - dget http://mentors.debian.net/debian/pool/main/s/s3ql/s3ql_1.0.1-1.dsc
 
 I would be glad if someone uploaded this package for me.

 Thanks for your work. Here goes my comments:

 1. You use quilt as build-depends. This should not be neccessary with
 dpkg-source v3.

It seems to be. Without quilt:

dh clean --with python2,quilt
dh: unable to load addon quilt: Can't locate Debian/Debhelper/Sequence/quilt.pm 
in @INC (@INC contains: /etc/perl /usr/local/lib/perl/5.12.3 
/usr/local/share/perl/5.12.3 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.12 
/usr/share/perl/5.12 /usr/local/lib/site_perl .) at (eval 23) line 2.
BEGIN failed--compilation aborted at (eval 23) line 2.


 3. debian/README.Debian doesn't need to be in *.docs. See man
 dh_installdocs:

Thanks, fixed.

 4. Downloading
 http://pypi.python.org/packages/source/d/distribute/distribute-0.6.14.tar.gz
 Extracting in /tmp/tmpeY0G4V

 OUCH! buildds do not all have internet access. Your package should build
 fine without that. Moreover the contents of the pulled in tarball could
 be anything. Thus ensuring DFSG-free sources is virtually impossible
 when external sources are pulled in.

 5. Your package doesn't build in pbuilder:

 dpkg-source: warning: ignoring deletion of directory doc
 dpkg-source: error: cannot represent change to
 s3ql-1.0.1/distribute-0.6.14.tar.gz: binary file contents changed

That's weird, it works just fine here, and it also does not download
anything but uses the existing python-setuptools. Can you tell me what
commands you used to get this?


Best,

   -Nikolaus

-- 
 »Time flies like an arrow, fruit flies like a Banana.«

  PGP fingerprint: 5B93 61F8 4EA2 E279 ABF6  02CF A9AD B7F8 AE4E 425C


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/87iprrozkq@inspiron.ap.columbia.edu



Re: RFS: s3ql (new python application)

2011-06-27 Thread Arno Töll
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi Nikolaus,

On 27.06.2011 21:15, Nikolaus Rath wrote:
 It seems to be. Without quilt:
 
 dh clean --with python2,quilt
 dh: unable to load addon quilt: Can't locate 
 Debian/Debhelper/Sequence/quilt.pm in @INC (@INC contains: /etc/perl 
 /usr/local/lib/perl/5.12.3 /usr/local/share/perl/5.12.3 /usr/lib/perl5 
 /usr/share/perl5 /usr/lib/perl/5.12 /usr/share/perl/5.12 
 /usr/local/lib/site_perl .) at (eval 23) line 2.
 BEGIN failed--compilation aborted at (eval 23) line 2.

this is, because you configured:

%:
dh $@ --with python2,quilt

which runs dh_quilt_patch upon creation of the source directory. This is
not necessary when using the new quilt 3.0 format which does this
implicitly.

That said, you should be able to remove the quilt addon from dh and
everything should still work (untested).

- -- 
with kind regards,
Arno Töll
IRC: daemonkeeper on Freenode/OFTC
GnuPG Key-ID: 0x9D80F36D
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJOCOHFAAoJEMcrUe6dgPNt+HYP/2ZuwNIS7dQTslXxd+LWBpdd
8N+9AHYSOyfimdL8aM5LhXqNsasd7J8fqDWabYVOfL/HSh3RkHJ58ITZrqLBdaoL
qRjok8G76RAcYStba+LHv+rUI4v3YI5nAZVRi/+nZ43dHLWHHFRsHQp02kzB8VQp
PZAqoSSAw6uxK+bekq36yVlmJE6efezuPa+T7Q4WdPmSadJRbKu0ry7/rK5qTvbF
eLGISJSzvpoIUmanxRLjS5a1zoHe258tWU0q/wjXDJ5x/AOtVkGQpoT9P+rLGwoE
LrbOzy0ZTPj+x85cLt/X7CusNFAlbB5BvrlDHzD/3wrQt3J3OfOG4iP2Uw9cIngY
OoxS7lbN8s6sdKWMLueKb6m+tH2gTTuX+oohtPge0I4LCKxsP/f5bfYTrtv23CJQ
NDiW05uYXXglZAaIh56vmpR/UBdkkjk11s0+VAho3azbzbLJb5ZwtVMtO/4FPAp+
Sd5YTwGKFOxFqLm4SeJGkonjqd0jENYrOhkVvhkrFXKqUYXJudklQJaGzdQsBvgK
bZMIhD6EvFpZTUNFX14ajfjXysPSoojiAeq4az0bepgLn9KNdLNuSG6cki7inIYw
WsOwuorL6oSyABMRkkegeYP0TSXmKkVjIr5SQ0p5ATLX0OlqdH/jdTBGvsYFHRur
rVbpBs1NdZDLs8zs9X3D
=t7TR
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4e08e1c6.5040...@toell.net



Re: RFS: s3ql (new python application)

2011-06-27 Thread Kilian Krause
Hi Nikolaus,

On Mon, Jun 27, 2011 at 03:15:49PM -0400, Nikolaus Rath wrote:
  1. You use quilt as build-depends. This should not be neccessary with
  dpkg-source v3.
 
 It seems to be. Without quilt:
 
 dh clean --with python2,quilt
 dh: unable to load addon quilt: Can't locate 
 Debian/Debhelper/Sequence/quilt.pm in @INC (@INC contains: /etc/perl 
 /usr/local/lib/perl/5.12.3 /usr/local/share/perl/5.12.3 /usr/lib/perl5 
 /usr/share/perl5 /usr/lib/perl/5.12 /usr/share/perl/5.12 
 /usr/local/lib/site_perl .) at (eval 23) line 2.
 BEGIN failed--compilation aborted at (eval 23) line 2.

my suggestion would be the same as Arno already pointed out. The --with
quilt part should not be needed - I hope.


  3. debian/README.Debian doesn't need to be in *.docs. See man
  dh_installdocs:
 
 Thanks, fixed.
 
  4. Downloading
  http://pypi.python.org/packages/source/d/distribute/distribute-0.6.14.tar.gz
  Extracting in /tmp/tmpeY0G4V
 
  OUCH! buildds do not all have internet access. Your package should build
  fine without that. Moreover the contents of the pulled in tarball could
  be anything. Thus ensuring DFSG-free sources is virtually impossible
  when external sources are pulled in.
 
  5. Your package doesn't build in pbuilder:
 
  dpkg-source: warning: ignoring deletion of directory doc
  dpkg-source: error: cannot represent change to
  s3ql-1.0.1/distribute-0.6.14.tar.gz: binary file contents changed
 
 That's weird, it works just fine here, and it also does not download
 anything but uses the existing python-setuptools. Can you tell me what
 commands you used to get this?

dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild

which spawns a fresh unstable pbuilder chroot and installs your listed
build-deps.

-- 
Best regards,
Kilian


signature.asc
Description: Digital signature


Re: RFS: s3ql (new python application)

2011-06-27 Thread Nikolaus Rath
On 06/27/2011 05:32 PM, Kilian Krause wrote:
 5. Your package doesn't build in pbuilder:

 dpkg-source: warning: ignoring deletion of directory doc
 dpkg-source: error: cannot represent change to
 s3ql-1.0.1/distribute-0.6.14.tar.gz: binary file contents changed

 That's weird, it works just fine here, and it also does not download
 anything but uses the existing python-setuptools. Can you tell me what
 commands you used to get this?
 
 dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild
 
 which spawns a fresh unstable pbuilder chroot and installs your listed
 build-deps.

Hmm. On my system:

dget http://mentors.debian.net/debian/pool/main/s/s3ql/s3ql_1.0.1-1.dsc
dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild

runs perfectly. I do not have any extra stuff installed in my chroot
except for vim.

The complete build output is at

http://pastebin.com/3na83pQT

I don't quite know what to do here... do you have any idea? Could you
post the complete output on your system as well?


Best,

   -Nikolaus

-- 
 »Time flies like an arrow, fruit flies like a Banana.«

  PGP fingerprint: 5B93 61F8 4EA2 E279 ABF6  02CF A9AD B7F8 AE4E 425C


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4e090860.8010...@rath.org



Re: RFS: s3ql (new python application)

2011-06-27 Thread Jakub Wilk

* Nikolaus Rath nikol...@rath.org, 2011-06-27, 18:46:

5. Your package doesn't build in pbuilder:

dpkg-source: warning: ignoring deletion of directory doc
dpkg-source: error: cannot represent change to
s3ql-1.0.1/distribute-0.6.14.tar.gz: binary file contents changed


That's weird, it works just fine here, and it also does not download
anything but uses the existing python-setuptools. Can you tell me 
what commands you used to get this?


dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild

which spawns a fresh unstable pbuilder chroot and installs your 
listed build-deps.


Hmm. On my system:

dget http://mentors.debian.net/debian/pool/main/s/s3ql/s3ql_1.0.1-1.dsc
dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild

runs perfectly. I do not have any extra stuff installed in my chroot 
except for vim.


My wild[0] guess is that bad thing happen _outside_ chroot: pdebuild 
runs the clean target without build-dependencies installed.



[0] I didn't look at the package lately and I've never user pdebuild.

--
Jakub Wilk


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110627231458.ga7...@jwilk.net



RFS: kst (updated package)

2011-06-27 Thread Ruben Molina
Dear mentors,

I am looking for a sponsor for the new version 2.0.3-1
of my package kst.

It builds these binary packages:
kst- scientific data plotting tool
kst-data   - set of data files for kst
kst-doc- set of tutorials for kst
libkst2core2 - kst core library
libkst2math2 - kst math library
libkst2widgets2 - kst widgets library

The package appears to be lintian clean.

The upload would fix these bugs: 553793, 589685, 604335, 615738

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/k/kst
- Source repository: deb-src http://mentors.debian.net/debian unstable main 
contrib non-free
- dget http://mentors.debian.net/debian/pool/main/k/kst/kst_2.0.3-1.dsc

I would be glad if someone uploaded this package for me.

Kind regards
 Ruben Molina


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


RFS: calamaris (updated package)

2011-06-27 Thread Daniel Echeverry
Dear mentors,

I am looking for a sponsor for the new version 2.99.4.0-14
of my package calamaris.

It builds these binary packages:
calamaris  - log analyzer for Squid or Oops proxy log files

The package appears to be lintian clean.

The upload would fix these bugs: 624120, 626242, 627467, 627708

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/c/calamaris
- Source repository: deb-src http://mentors.debian.net/debian unstable main
contrib non-free
- dget
http://mentors.debian.net/debian/pool/main/c/calamaris/calamaris_2.99.4.0-14.dsc

I would be glad if someone uploaded this package for me.

Kind regards
 daniel echeverry

-- 
Epsilon
http://www.rinconinformatico.net
http://www.fitnessdeportes.com
http://www.dragonjar.org
Linux user: #477840
Debian user


Re: RFS: s3ql (new python application)

2011-06-27 Thread Kilian Krause
Nikolaus,

On Mon, 2011-06-27 at 18:46 -0400, Nikolaus Rath wrote:
 On 06/27/2011 05:32 PM, Kilian Krause wrote:
  5. Your package doesn't build in pbuilder:
 
  dpkg-source: warning: ignoring deletion of directory doc
  dpkg-source: error: cannot represent change to
  s3ql-1.0.1/distribute-0.6.14.tar.gz: binary file contents changed
 
  That's weird, it works just fine here, and it also does not download
  anything but uses the existing python-setuptools. Can you tell me what
  commands you used to get this?
  
  dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild
  
  which spawns a fresh unstable pbuilder chroot and installs your listed
  build-deps.
 
 Hmm. On my system:
 
 dget http://mentors.debian.net/debian/pool/main/s/s3ql/s3ql_1.0.1-1.dsc
 dpkg-source -x s3ql_1.0.1-1.dsc;cd s3ql-1.0.1;pdebuild
 
 runs perfectly. I do not have any extra stuff installed in my chroot
 except for vim.
 
 The complete build output is at
 
 http://pastebin.com/3na83pQT
 
 I don't quite know what to do here... do you have any idea? Could you
 post the complete output on your system as well?

the missing build-deps on my system are:
dpkg-checkbuilddeps: Unmet build dependencies: python-setuptools (=
0.6.14) python-apsw (= 3.7.0) python-sphinx python-argparse
python-llfuse (= 0.31) python-pycryptopp python-paramiko python-lzma
(= 0.5.3)

full build log is at:
http://people.debian.org/~kilian/s3ql_1.0.1-1_amd64.build

-- 
Regards,
Kilian


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