Re: RFS: triggerhappy [fixed mistake in URL]
Dies schrieb Benoît Knecht (benoit.kne...@fsfe.org): I understand :) But now that you're packaging it, you should let people know by retitling the bug to ITP and setting yourself as the bug owner (otherwise someone else might start working on it too, only to discover the work has already been done). And don't forget to add (Closes: #603842) to your changelog. Hm, for some reason the BTS did not retitle the report, although it processed the ownership change, do you know what I did wrong? -- 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/20101122082009.ga15...@zirkel.wertarbyte.de
Re: RFS: triggerhappy [fixed mistake in URL]
Stefan Tomanek wrote: Dies schrieb Benoît Knecht (benoit.kne...@fsfe.org): I understand :) But now that you're packaging it, you should let people know by retitling the bug to ITP and setting yourself as the bug owner (otherwise someone else might start working on it too, only to discover the work has already been done). And don't forget to add (Closes: #603842) to your changelog. Hm, for some reason the BTS did not retitle the report, although it processed the ownership change, do you know what I did wrong? As a matter of fact, I do :) You retitled it to the exact same title it already has: retitle 603842 RFP: triggerhappy -- a system-wide hotkey and input event daemon Change that to: retitle 603842 ITP: triggerhappy -- a system-wide hotkey and input event daemon and it should work just fine. Cheers, -- Benoît Knecht -- 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/20101122083623.ga21...@debian.lan
Re: RFS: triggerhappy [fixed mistake in URL]
Dies schrieb Benoît Knecht (benoit.kne...@fsfe.org): As a matter of fact, I do :) You retitled it to the exact same title it already has: Indeed, I just noticed. Classic case of copypaste gone wrong. -- 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/20101122084026.gd15...@zirkel.wertarbyte.de
Re: RFS: kstars-data-extra-tycho2 (third try)
On Viernes 12 Noviembre 2010 16:47:24 Benoît Knecht escribió: [...] I'll try and have a look at postinst later, but I'm not very familiar with debconf or the kstarsrc syntax, so I'll be of little assistance I'm afraid. Dear Benoît: Have you been able to do the check? Thanks Noel er Envite signature.asc Description: This is a digitally signed message part.
Re: RFS: webhoneypot
Hi Ansgar, I read the document, and had a look at the policy and decided that I will follow your advice. I will put the virtual host to /etc/webhonepot/webhoneypot.vhost and modify the README. Cu, Chris -- Christian Pohl w...@pohlcity.de -- 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/b3540b6a8c1dc71c46911e107fc75b81.squir...@aringill
RFS: tartarus
Dear mentors, I am looking for a sponsor for my package tartarus. * Package name: tartarus Version : 0.9.4-2 Upstream Author : Stefan Tomanek stefan.tomanek+tarta...@wertarbyte.de * URL : http://wertarbyte.de/tartarus.shtml * License : GPLv3 Section : utils It builds these binary packages: tartarus - script based backup system The package appears to be lintian clean. The upload would fix these bugs: 604447 My motivation for maintaining this package is: I've written tartarus as a simple backup solution for my dedicated server. It uses parts of the unix toolshed that should be available even in the most basic rescue system, making recovery easy in case things go wrong. By using Bash and Perl, the program is easily extendable and offers various hooks that can be used to modify its behaviour, while including features like LVM snapshots, on-the-fly-uploading and compression/encryption out of the box. I created the package to simplify the installation on now roughly 1100 systems using this program (http://wertarbyte.de/stats/tartarus-use.png) on a regular basis and I would really like to this this included into Debian. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/t/tartarus - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/t/tartarus/tartarus_0.9.4-2.dsc I would be glad if someone uploaded this package for me. Kind regards Stefan Tomanek -- 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/20101122105251.gg15...@zirkel.wertarbyte.de
Re: RFS: kstars-data-extra-tycho2 (third try)
Hi Noel, Noel David Torres Taño wrote: On Viernes 12 Noviembre 2010 16:47:24 Benoît Knecht escribió: [...] I'll try and have a look at postinst later, but I'm not very familiar with debconf or the kstarsrc syntax, so I'll be of little assistance I'm afraid. Dear Benoît: Have you been able to do the check? Sorry, I had completely forgotten; thanks for reminding me. So first thing first. I tried installing your package using piuparts, and got the following error while running 'dpkg -i kstars-data-extra-tycho2_1.1r1-2_all.deb': Selecting previously deselected package kstars-data-extra-tycho2. (Reading database ... 6131 files and directories currently installed.) Unpacking kstars-data-extra-tycho2 (from .../kstars-data-extra-tycho2_1.1r1-2_all.deb) ... Setting up kstars-data-extra-tycho2 (1.1r1-2) ... /var/lib/dpkg/info/kstars-data-extra-tycho2.postinst: 39: cannot create /etc/kde4/kstarsrc: Directory nonexistent dpkg: error processing kstars-data-extra-tycho2 (--install): subprocess installed post-installation script returned error exit status 2 Errors were encountered while processing: kstars-data-extra-tycho2 (Since your package doesn't depend on any kde4 stuff, you need to make sure /etc/kde4 exists.) Regarding debconf, you should make sure that you're not prompting the user when it's not necessary, so your postinst should take into account which parameter is passed to it; see [1] for details. [1] http://www.debian.org/doc/debian-policy/ch-binary.html#s-maintscriptprompt Along the same idea (minimizing user prompting), I think it would be a good idea to read kstarsrc and see if action/get_data is already set; if it is, don't prompt the user. Also, it would be better to just edit that file instead of overwriting it. Finally, I think the default for kstars-data-extra/kstarsrc-does-not-exist should be false (you don't want to create that configuration file and lock downloads without the user explicitly deciding to do so). Cheers, -- Benoît Knecht -- 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/20101122111421.gb21...@debian.lan
Re: Debconf in non-interactive mode
On Sat, Nov 20, 2010 at 11:38:08PM +0100, Mats Erik Andersson wrote: I could find good use of some advice on Debconf usage. The recent automated package upgrade tests, conducted by Lucas Nussbaum, hava revealed that my package Webfs fails to upgrade properly from stable (1.21-5.1) to testing (1.21+ds1-6). I have filed an RC bug against myself on this matter, so I will have to solve it. My present analysis indicates that after my adoption of the package, starting at 1.21+ds1-1, I was too keen in letting the postinst script insert some new settings in the configuration file. Thus the script awaits feedback even in non-interactive mode. It's not really about non-interactive mode, that just happened to pinpoint the problem. The issue is that you're modifying a dpkg conffile (/etc/webfsd.conf) in your maintainer scripts. This is a violation of Debian policy 10.7.3 and explicitly listed in the 'Release Critical Issues for Squeeze' document at http://release.debian.org/squeeze/rc_policy.txt so #603926 (cc'd) is definitely RC. You might want to look at using ucf(1) to install the file instead of shipping it as a conffile. -- Niko Tyni nt...@debian.org -- 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/20101122104744.ga4...@madeleine.local.invalid
RFS: ifupdown-scripts-wa
Dear mentors, I am looking for a sponsor for my package ifupdown-scripts-wa. * Package name: ifupdown-scripts-wa Version : 0.4.2 Upstream Author : Stefan Tomanek stefan.tomanek+ifupd...@wertarbyte.de * URL : https://github.com/wertarbyte/ifupdown-scripts/ * License : GPLv3 Section : admin It builds these binary packages: ifupdown-scripts-wa - Additional ifupdown scripts for configuring network devices The package appears to be lintian clean. The upload would fix these bugs: 604462 My motivation for maintaining this package is: The package contains various extension scripts to ifupdown I've written over time. They add the following functionality to the mechanisms provided by Debian's ifupdown: * addresses allows multiple addresses to be added to an network device in a convenient CIDR notation * reachable add additional networks reachable through a configured interface, either directly or by using a connected gateway * conflicts automatically (de)configures an interface when another one comes up or goes down * pan connects your computer to an Bluetooth PAN access point before configuring the interface * vpnc controls a cisco VPN connect through ifupdown * wireless-tools-snooze waits for association with the access point before retrieving an IP address for the wireless network * ifupdown-network pm-utils script that shuts down specified network devices during suspend and restores them during resume The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/i/ifupdown-scripts-wa - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/i/ifupdown-scripts-wa/ifupdown-scripts-wa_0.4.2.dsc I would be glad if someone uploaded this package for me. Kind regards Stefan Tomanek -- 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/20101122120813.gh15...@zirkel.wertarbyte.de
Re: RFS: tartarus
Hi Stefan, 22/11/2010 11:52 +0100, Stefan Tomanek wrote: Dear mentors, I am looking for a sponsor for my package tartarus. * Package name: tartarus Version : 0.9.4-2 Upstream Author : Stefan Tomanek stefan.tomanek+tarta...@wertarbyte.de * URL : http://wertarbyte.de/tartarus.shtml * License : GPLv3 Section : utils It builds these binary packages: tartarus - script based backup system The package appears to be lintian clean. The upload would fix these bugs: 604447 My motivation for maintaining this package is: I've written tartarus as a simple backup solution for my dedicated server. It uses parts of the unix toolshed that should be available even in the most basic rescue system, making recovery easy in case things go wrong. By using Bash and Perl, the program is easily extendable and offers various hooks that can be used to modify its behaviour, while including features like LVM snapshots, on-the-fly-uploading and compression/encryption out of the box. I created the package to simplify the installation on now roughly 1100 systems using this program (http://wertarbyte.de/stats/tartarus-use.png) on a regular basis and I would really like to this this included into Debian. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/t/tartarus - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/t/tartarus/tartarus_0.9.4-2.dsc I would be glad if someone uploaded this package for me. I'm not a DD, but here's my (quick) review for your package. I hope it's useful for you: * Lintian checking (it's not Lintian clean): - I: tartarus source: debian-watch-file-is-missing It's a good practice to add a watch file. - W: tartarus source: out-of-date-standards-version 3.8.0 (current is 3.9.1) You should check [1] and upgrade to Standards-Version to 3.9.1 - I: tartarus: spelling-error-in-manpage usr/share/man/man1/tartarus.1.gz managable manageable You can find this spelling error in $ grep -nR managable * bin/tartarus:26:# configuration from easily managable configuration files. It can store If you are the upstreamer, you can easily correct it in upstream source before packaging. - P: tartarus: no-upstream-changelog It could be nice if upstream have an upstream changelog and it could be accessible from Debian package. * Debian files checking: - debian/changelog: I think you should not add previous changes before entering the package to Debian. So this package version should be 0.9.4-1, altough you had it before in a personal repo. - debian/rules: You can delete commented lines that tell us you are using a template. - debian/menu: Consider to create it. - debian/patches: You have a debian/patches automatic file (debian-changes-0.9.4-2): If this patch is necessary for Debian package, you should comment why in the patch file. If it's fine to have this change in upstream and you're the upstreamer, you can change it in upstream code before packaging. * Ustream source checking: - Upstream source should be licensed. I haven't checked how program works. If any other person from the list sees a mistake in my review, reviews of my review are welcomed! Cheers, Mònica [1] http://www.debian.org/doc/packaging-manuals/upgrading-checklist.txt signature.asc Description: This is a digitally signed message part
Re: RFS: ifupdown-scripts-wa
Hi Stefan, Stefan Tomanek wrote: I am looking for a sponsor for my package ifupdown-scripts-wa. * Package name: ifupdown-scripts-wa Version : 0.4.2 Upstream Author : Stefan Tomanek stefan.tomanek+ifupd...@wertarbyte.de * URL : https://github.com/wertarbyte/ifupdown-scripts/ * License : GPLv3 Section : admin It builds these binary packages: ifupdown-scripts-wa - Additional ifupdown scripts for configuring network devices The package appears to be lintian clean. You should get in touch with the maintainer of ifupdown, Anthony Towns a...@debian.org. And FYI, lintian reports the following: W: ifupdown-scripts-wa source: out-of-date-standards-version 3.8.1 (current is 3.9.1) W: ifupdown-scripts-wa: latest-debian-changelog-entry-changed-to-native Cheers, -- Benoît Knecht -- 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/20101122140856.ga4...@debian.lan
Re: Debconf in non-interactive mode
Dear Niko, måndag den 22 november 2010 klockan 12:47 skrev Niko Tyni detta: On Sat, Nov 20, 2010 at 11:38:08PM +0100, Mats Erik Andersson wrote: I could find good use of some advice on Debconf usage. The recent automated package upgrade tests, conducted by Lucas Nussbaum, hava revealed that my package Webfs fails to upgrade properly from stable (1.21-5.1) to testing (1.21+ds1-6). I have filed an RC bug against myself on this matter, so I will have to solve it. My present analysis indicates that after my adoption of the package, starting at 1.21+ds1-1, I was too keen in letting the postinst script insert some new settings in the configuration file. Thus the script awaits feedback even in non-interactive mode. It's not really about non-interactive mode, that just happened to pinpoint the problem. The issue is that you're modifying a dpkg conffile (/etc/webfsd.conf) in your maintainer scripts. This is a violation of Debian policy 10.7.3 and explicitly listed in the 'Release Critical Issues for Squeeze' document at http://release.debian.org/squeeze/rc_policy.txt so #603926 (cc'd) is definitely RC. This confirms the conclusion I was forced to make in a renewed analysis after sending the message to mentors.d.n. Thank you for the exact pointers. You might want to look at using ucf(1) to install the file instead of shipping it as a conffile. I did implement a solution using ucf(1) for another package and another RC-bug during spring, so I have some distant experience of that. Once more, thank you for the clarification. Regards, Mats E Andersson -- 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/20101122141442.gb...@mea.homelinux.org
Problem testing with piuparts
Hi, What is the proper way to test packages inside chroot? piuparts fails with error that seem unrelated to my package: 0m35.5s ERROR: Broken symlinks: /usr/lib/python2.4/site-packages/python-support.pth - /var/lib/python-support/python2.4/.path Can you recommend any best practices or helper scripts? Please, CC. Thanks. -- anatoly t. -- Forwarded message -- From: anatoly techtonik techto...@gmail.com Date: Mon, Nov 22, 2010 at 9:44 AM Subject: Errors with piuparts To: debian-python debian-pyt...@lists.debian.org Hi, I use pbuilder environment on lenny to test new trac-bitten with piuparts, but the process fails with the broken symlinks error in python-support: $ sudo piuparts -p trac-bitten_0.6b3-1_all.deb -d lenny ... Setting up libjs-jquery (1.2.6-2) ... Setting up trac (0.11.1-2.1) ... Setting up trac-bitten (0.6b3-1) ... Processing triggers for python-support ... 0m34.8s DEBUG: Command ok: ['chroot', '/tmp/tmpg5YLyT', 'apt-get', '-yf', '--no-remove', 'install'] 0m34.8s DEBUG: Starting command: ['chroot', '/tmp/tmpg5YLyT', 'apt-get', 'clean'] 0m34.8s DEBUG: Command ok: ['chroot', '/tmp/tmpg5YLyT', 'apt-get', 'clean'] 0m34.8s DEBUG: Removing /tmp/tmpg5YLyT/tmp/trac-bitten_0.6b3-1_all.deb 0m34.8s DEBUG: Starting command: ['lsof', '-w', '+D', '/tmp/tmpg5YLyT'] 0m35.2s DEBUG: Command failed (status=1), but ignoring error: ['lsof', '-w', '+D', '/tmp/tmpg5YLyT'] 0m35.5s ERROR: Broken symlinks: /usr/lib/python2.4/site-packages/python-support.pth - /var/lib/python-support/python2.4/.path 0m35.5s DEBUG: Starting command: ['chroot', '/tmp/tmpg5YLyT', 'umount', '/proc'] 0m35.5s DEBUG: Command ok: ['chroot', '/tmp/tmpg5YLyT', 'umount', '/proc'] 0m35.8s DEBUG: Removed directory tree at /tmp/tmpg5YLyT How do you test PAPT packages? -- anatoly t. -- 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/aanlktik1n57r0dyzsrwf2k62osoxrqknxhwt2_j=f...@mail.gmail.com
Re: RFS: aqemu (update)
On Sunday 21 November 2010 22:41:07 Ignace Mouzannar wrote: Looks much better now, but I think there's another (hopefully last) problem in d/copyright (yes, that's often the hardest file ...): Definitely.. Ignace, could you please send me your copyright file? It does sound like an interesting test case for my DEP-5 parser based on Config::Model ;-) All the best Dominique -- http://config-model.wiki.sourceforge.net/ -o- http://search.cpan.org/~ddumont/ http://www.ohloh.net/accounts/ddumont -o- http://ddumont.wordpress.com/ -- 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/201011221601.07505.dominique.dum...@hp.com
Re: RFS: aqemu (update)
Hi Dominique, On Mon, Nov 22, 2010 at 16:01, Dominique Dumont dominique.dum...@hp.com wrote: On Sunday 21 November 2010 22:41:07 Ignace Mouzannar wrote: Looks much better now, but I think there's another (hopefully last) problem in d/copyright (yes, that's often the hardest file ...): Definitely.. Ignace, could you please send me your copyright file? You can find the latest version of the debian/copyright file here: http://svn.debian.org/viewsvn/collab-maint/ext-maint/aqemu/trunk/debian/copyright It does sound like an interesting test case for my DEP-5 parser based on Config::Model ;-) I hope it will be helpful. :) Cheers, Ignace M -- 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/aanlktimqcmq7vruw_iqcuofuyxamx7rshvjuav=bt...@mail.gmail.com
Re: RFS: ifupdown-scripts-wa
Dies schrieb Benoît Knecht (benoit.kne...@fsfe.org): You should get in touch with the maintainer of ifupdown, Anthony Towns a...@debian.org. And FYI, lintian reports the following: W: ifupdown-scripts-wa source: out-of-date-standards-version 3.8.1 (current is 3.9.1) W: ifupdown-scripts-wa: latest-debian-changelog-entry-changed-to-native Yes, I know - ifupdown-scripts aren't thatuseful for other distributions I think? -- 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/20101122154713.gi15...@zirkel.wertarbyte.de
Re: Build-Depends-Indep, please review
Roger Leigh rle...@codelibre.net writes:
On Sun, Nov 21, 2010 at 07:09:44PM +0100, Goswin von Brederlow wrote:
Output at http://people.debian.org/~rleigh/build-rule-check.bz2
I haven't had time to analyse this, if someone else wants to,
that would be cool.
Done on lintian.debian.org using the following:
#!/bin/sh
cd /srv/lintian.debian.org/laboratory/source
find . -type f -name rules | grep 'debfiles/rules$' | while read rule
do
egrep -H '^(build(.*%)?|build-(arch|indep)):' $rule
done
Regards,
Roger
Now why does it only list 8k sources if it matches the required
build: target? Are 50% of all sources already dropping through that
grep because they use dh ('%:') or include files?
That's what I think, and a cursory look at a few gives the same
impression. Which is surprisingly good--there's only a single
tool to update to get support added for a huge chunk of the
archive (well, there's cdbs as well of course, so it's two main
ones).
I would think dh already supports them. So we might actuay have 50%
coverage of binay-arch/indep already or verry close to it.
Maybe we can actualy change policy post squeeze because the majority of
sources supports it.
Anyway, grepping for sources that (probably) have build-arch/indep:
% bzcat build-rule-check.bz2 | egrep
'/rules:(build(.*%)+|build-(arch|indep)):' | cut -d: -f1 | sort -u | wc -l
572
Lots of work to do.
Agreed! But, should be an achievable goal for squeeze+1 if we start
on it early.
Regards,
Roger
Totaly.
MfG
Goswin
--
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/87r5edibvo@frosties.localnet
Re: Problem testing with piuparts
Hi, On Montag, 22. November 2010, anatoly techtonik wrote: What is the proper way to test packages inside chroot? you already found it :-) sadly you also found out, that often also problems in dependend packages are found. piuparts fails with error that seem unrelated to my package: 0m35.5s ERROR: Broken symlinks: /usr/lib/python2.4/site-packages/python-support.pth - /var/lib/python-support/python2.4/.path Can you recommend any best practices or helper scripts? a.) file a bug against the dependend package b.) use --warn-on-others which should make piuparts not complain about problems in dependent packages. It doesn't and cannot work for all cases. c.) use --no-symlinks to disable this particular test. d.) RTFM: http://piuparts.debian.org/doc/piuparts.1.html and http://piuparts.debian.org/doc/README.html (in no particular order) btw, what are you doing with python2.4? it's not in squeeze :) cheers, Holger signature.asc Description: This is a digitally signed message part.
Re: RFS: ifupdown-scripts-wa
Hi, On Montag, 22. November 2010, Stefan Tomanek wrote: Dies schrieb Benoît Knecht (benoit.kne...@fsfe.org): You should get in touch with the maintainer of ifupdown, Anthony Towns a...@debian.org. you didn't reply to this important aspect... And FYI, lintian reports the following: W: ifupdown-scripts-wa source: out-of-date-standards-version 3.8.1 (current is 3.9.1) W: ifupdown-scripts-wa: latest-debian-changelog-entry-changed-to-native Yes, I know - ifupdown-scripts aren't thatuseful for other distributions I think? native packages should be refrained upon. always use non-native packages, even for native stuff: with native packages each version always gets a new orig.tar, which is not the cases with non-native packages. native packages have no gain whatsoever neither. cheers, Holger signature.asc Description: This is a digitally signed message part.
Re: RFS: aqemu (update)
On Mon, Nov 22, 2010 at 17:04, Dominique Dumont dominique.dum...@hp.com wrote: On Monday 22 November 2010 16:17:39 Ignace Mouzannar wrote: It does sound like an interesting test case for my DEP-5 parser based on Config::Model ;-) I hope it will be helpful. :) Well, DEP-5 parser found : - french and english for license was mixed up (e.g. licen*c*e instead of licen*s*e). I often do the same mistake ;-) - First paragraph lacks 'Files:*' (I grant you that's a recent requirement) Nice catch! :) IANADD, but I don't think these mistakes requires a new upload. Just fix this for next version. The license file has been updated [1]. Cheers, Ignace M [1] http://svn.debian.org/viewsvn/collab-maint/ext-maint/aqemu/trunk/debian/copyright -- 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/aanlktim1nzyt0tn9ml58kh1idyduurjjjfoofodh0...@mail.gmail.com
Re: RFS: aqemu (update)
On Monday 22 November 2010 16:17:39 Ignace Mouzannar wrote: It does sound like an interesting test case for my DEP-5 parser based on Config::Model ;-) I hope it will be helpful. :) Well, DEP-5 parser found : - french and english for license was mixed up (e.g. licen*c*e instead of licen*s*e). I often do the same mistake ;-) - First paragraph lacks 'Files:*' (I grant you that's a recent requirement) IANADD, but I don't think these mistakes requires a new upload. Just fix this for next version. All the best Dominique -- http://config-model.wiki.sourceforge.net/ -o- http://search.cpan.org/~ddumont/ http://www.ohloh.net/accounts/ddumont -o- http://ddumont.wordpress.com/ -- 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/201011221704.18835.dominique.dum...@hp.com
Re: RFS: tartarus
Thanks for your feedback, I just created a new package version which should take care of most issues described by you. -- 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/20101122162454.gk15...@zirkel.wertarbyte.de
Re: Build-Depends-Indep, please review
On Mon, Nov 22, 2010 at 05:03:39PM +0100, Goswin von Brederlow wrote:
Roger Leigh rle...@codelibre.net writes:
On Sun, Nov 21, 2010 at 07:09:44PM +0100, Goswin von Brederlow wrote:
Now why does it only list 8k sources if it matches the required
build: target? Are 50% of all sources already dropping through that
grep because they use dh ('%:') or include files?
That's what I think, and a cursory look at a few gives the same
impression. Which is surprisingly good--there's only a single
tool to update to get support added for a huge chunk of the
archive (well, there's cdbs as well of course, so it's two main
ones).
I would think dh already supports them. So we might actuay have 50%
coverage of binay-arch/indep already or verry close to it.
Maybe we can actualy change policy post squeeze because the majority of
sources supports it.
I checked, and dh currently doesn't support build-arch and build-indep
at all internally (you need to add the rules yourself). I've made a
patch to add proper support which I'll submit once I've tested it.
I've submitted a proposed change to Policy to require the use
of these targets (#604397), but we need at least 50% coverage first.
But, we'll get that as soon as dh and cdbs support it.
We also need a lintian check to warn if these targets are missing;
but I'm not at all familiar with hacking on lintian.
Regards,
Roger
--
.''`. Roger Leigh
: :' : Debian GNU/Linux http://people.debian.org/~rleigh/
`. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net/
`-GPG Public Key: 0x25BFB848 Please GPG sign your mail.
signature.asc
Description: Digital signature
Re: Build-Depends-Indep, please review
On Mon, Nov 22, 2010 at 04:39:20PM +, Roger Leigh wrote:
I checked, and dh currently doesn't support build-arch and build-indep
at all internally (you need to add the rules yourself). I've made a
patch to add proper support which I'll submit once I've tested it.
FWIW, the current progress on this is attached. No guarantees--it's
totally untested.
Regards,
Roger
--
.''`. Roger Leigh
: :' : Debian GNU/Linux http://people.debian.org/~rleigh/
`. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net/
`-GPG Public Key: 0x25BFB848 Please GPG sign your mail.
diff --git a/dh b/dh
index d2771ab..794ff4f 100755
--- a/dh
+++ b/dh
@@ -16,13 +16,15 @@ Bdh Isequence [B--with Iaddon[B,Iaddon ...]] [B--list] [B--until
=head1 DESCRIPTION
Bdh runs a sequence of debhelper commands. The supported Isequences
-correspond to the targets of a Fdebian/rules file: Bbuild, Bclean,
-Binstall, Bbinary-arch, Bbinary-indep, and Bbinary.
+correspond to the targets of a Fdebian/rules file: Bbuild-arch,
+Bbuild-indep, Bbuild, Bclean, Binstall, Bbinary-arch,
+Bbinary-indep, and Bbinary.
-Commands in the Bbinary-indep sequence are passed the B-i option to ensure
-they only work on binary independent packages, and commands in the
-Bbinary-arch sequences are passed the B-a option to ensure they only work
-on architecture dependent packages.
+Commands in the Bbuild-indep and Bbinary-indep sequences are passed
+the B-i option to ensure they only work on binary independent packages,
+and commands in the Bbuild-arch and Bbinary-arch sequences are
+passed the B-a option to ensure they only work on architecture
+dependent packages.
If Fdebian/rules contains a target with a name like Boverride_Idh_command,
then when it would normally run Idh_command, Bdh will instead call that
@@ -322,6 +324,8 @@ $sequences{build} = [qw{
dh_auto_build
dh_auto_test
}],
+$sequences{'build-indep'} = [...@{$sequences{build}}];
+$sequences{'build-arch'} = [...@{$sequences{build}}];
$sequences{clean} = [qw{
dh_testdir
dh_auto_clean
@@ -367,19 +371,25 @@ $sequences{install} = [...@{$sequences{build}}, qw{
dh_compress
dh_fixperms
}];
+$sequences{install-indep} = [...@{$sequences{build-indep}}, @i];
+$sequences{install-arch} = [...@{$sequences{build-arch}}, @i];
my @b=qw{
dh_installdeb
dh_gencontrol
dh_md5sums
dh_builddeb
};
-$sequences{'binary-indep'} = [...@{$sequences{install}}, @b];
+$sequences{'binary-indep'} = [...@{$sequences{install-indep}}, @b];
$sequences{binary} = [...@{$sequences{install}}, qw{
dh_strip
dh_makeshlibs
dh_shlibdeps
}, @b];
-$sequences{'binary-arch'} = [...@{$sequences{binary}}];
+$sequences{'binary-arch'} = [...@{$sequences{install-arch}}, qw{
+ dh_strip
+ dh_makeshlibs
+ dh_shlibdeps
+}];
# Additional command options
my %command_opts;
@@ -514,14 +524,18 @@ my @packag...@{$dh{dopackages}};
# Get the options to pass to commands in the sequence.
# Filter out options intended only for this program.
my @options;
-if ($sequence eq 'binary-arch') {
+if ($sequence eq 'build-arch' ||
+$sequence eq 'install-arch' ||
+$sequence eq 'binary-arch') {
push @options, -a;
# as an optimisation, remove from the list any packages
# that are not arch dependent
my %arch_packages = map { $_ = 1 } getpackages(arch);
@packages = grep { $arch_packages{$_} } @packages;
}
-elsif ($sequence eq 'binary-indep') {
+elsif ($sequence eq 'build-indep' ||
+ $sequence eq 'install-indep' ||
+ $sequence eq 'binary-indep') {
push @options, -i;
# ditto optimisation for arch indep
my %indep_packages = map { $_ = 1 } getpackages(indep);
signature.asc
Description: Digital signature
Re: RFS: aqemu (update)
On Mon, 22 Nov 2010 17:04:18 +0100, Dominique Dumont wrote: - french and english for license was mixed up (e.g. licen*c*e instead of licen*s*e). I often do the same mistake ;-) Isn't this more like en_GB vs. en_US? (Ok, maybe not in this case :)) - First paragraph lacks 'Files:*' (I grant you that's a recent requirement) Is this really a requirement in the meantime? Ah, you mean the first non-header stanza. - I really missed this one, good to have tools :) Cheers, gregor -- .''`. http://info.comodo.priv.at/ -- GPG key IDs: 0x8649AA06, 0x00F3CFE4 : :' : Debian GNU/Linux user, admin, developer - http://www.debian.org/ `. `' Member of VIBE!AT SPI, fellow of Free Software Foundation Europe `-NP: Richie Havens: Freedom (Live at Woodstock) signature.asc Description: Digital signature
Re: RFS: tartarus
On Mon, 22 Nov 2010 13:57:01 +0100, Mònica Ramírez Arceda wrote: Hi Mònica, I'm not a DD, but here's my (quick) review for your package. I hope it's useful for you: Thanks for doing this diligent review of Stefan's package. Just 2 quick comments: - debian/changelog: I think you should not add previous changes before entering the package to Debian. So this package version should be 0.9.4-1, altough you had it before in a personal repo. I'm not sure there is a consensus on this issue; my personal opinion is that it's better to keep the previous work if older binary packages have been available from non-official repositories, as it makes updates easier for users of those packages. - debian/menu: Consider to create it. I'm not sure if a menu entry for a command-line script is really that helpful. If any other person from the list sees a mistake in my review, reviews of my review are welcomed! :) Cheers, gregor -- .''`. http://info.comodo.priv.at/ -- GPG key IDs: 0x8649AA06, 0x00F3CFE4 : :' : Debian GNU/Linux user, admin, developer - http://www.debian.org/ `. `' Member of VIBE!AT SPI, fellow of Free Software Foundation Europe `-NP: Bruce Springsteen: My father's house signature.asc Description: Digital signature
RFS: php-log (updated package)
Dear mentors, I am looking for a sponsor for the new version 1.12.3-1 of my package php-log. It builds these binary packages: php-log- log module for PEAR The package appears to be lintian clean. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/p/php-log - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/p/php-log/php-log_1.12.3-1.dsc I would be glad if someone uploaded this package for me. Kind regards Guillaume Delacour signature.asc Description: Ceci est une partie de message numériquement signée
RFS: php-net-smtp (updated package)
Dear mentors, I am looking for a sponsor for the new version 1.4.4-1 of my package php-net-smtp. It builds these binary packages: php-net-smtp - PHP PEAR module implementing SMTP protocol The package appears to be lintian clean. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/p/php-net-smtp - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/p/php-net-smtp/php-net-smtp_1.4.4-1.dsc I would be glad if someone uploaded this package for me. Kind regards Guillaume Delacour signature.asc Description: Ceci est une partie de message numériquement signée
RFS: aspell-id
Dear mentors, I am looking for a sponsor for my package aspell-id. * Package name: aspell-id Version : 0.1-1 Upstream Author : Benitius Brevoort benitius at tiscali it * URL : ftp://ftp.gnu.org/gnu/aspell/dict/id * License : gpl Section : text It builds these binary packages: aspell-id - Indonesian (id) dictionary for GNU aspell The package appears to be lintian clean. The upload would fix these bugs: 604557 My motivation for maintaining this package is: learning packaging from debian mentors packaging localization The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/a/aspell-id - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/a/aspell-id/aspell-id_0.1-2.dsc I would be glad if someone uploaded this package for me. Kind regards Mahyuddin Susanto -- 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/aanlktikfvp_qkrdhj+tqa1lcltax-eu2fj2eez4a1...@mail.gmail.com
Re: RFS: aqemu (update)
On 22 November 2010 11:07, gregor herrmann gre...@debian.org wrote: On Mon, 22 Nov 2010 17:04:18 +0100, Dominique Dumont wrote: - french and english for license was mixed up (e.g. licen*c*e instead of licen*s*e). I often do the same mistake ;-) Isn't this more like en_GB vs. en_US? (Ok, maybe not in this case :)) It is. In my en_GB `licence' is a noun and `license' is a verb. (The same with `advice' and `advise', `practice' and `practise') - First paragraph lacks 'Files:*' (I grant you that's a recent requirement) Is this really a requirement in the meantime? Ah, you mean the first non-header stanza. - I really missed this one, good to have tools :) Cheers, gregor -- .''`. http://info.comodo.priv.at/ -- GPG key IDs: 0x8649AA06, 0x00F3CFE4 : :' : Debian GNU/Linux user, admin, developer - http://www.debian.org/ `. `' Member of VIBE!AT SPI, fellow of Free Software Foundation Europe `- NP: Richie Havens: Freedom (Live at Woodstock) -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.10 (GNU/Linux) iQIcBAEBCAAGBQJM6qNTAAoJELs6aAGGSaoGHcwP/2Cai5mgrkeYwSyxoGn1GMAS hLJOfStuh3hbCuX6Bhc8u8IWtvrdR2f907kIkFWBSRsd3+Ss6PmkyC3H0E3yvK5z vk6PwVCBEu2tndY36sWJ7vIws2QhfBJrOw1Fk8rFtR+lcWGLFBkXL+9SlOgCNaxO p95c3a03JpmhMW5Nk788z2JNoDQnWjN8KaS0i8YGL1X8OLw4mVtJ5QxlP/Lp0HDB j2fVKk6BdZ6MveEP38txAnIuV31+dAR3dX9YoredyBOQeDMJfg+xT0Q871XVu0fv O5C4e4G+hAhnBBUb+iYlN0U2Pf+AGKevBPP0E/ugW9M/1S+bw7CU20zToCpY/ouL 5MwzR4Fm7ARVlyPQ2zeyRZm3Rv/QRPfteVheJyQx44t3hu3AmzRVLzyN0aRqWymr QsRtpGKvKVdbpWDd+XO3+h8WptVjeNplybs93ktJz9T47xGmeiqkQq1i+3Q5LFXf h1+PZX9kd2Gb79M9yYh5eXhkoBhKXjS+B8AgpVaGzfcmUOQJPLjOa/uL/2thl9sx 7Zr7s/3X+ZdQwGxoeBS/0ugAnmItcqvCe7CbFxWgvt1Gb1eZi4ZRJ8qToiyI/6z+ PI4+raEvBzqIco1x3nPTJJaAzGceZQ0XjJagmHvNTzkYkmxEMeZgKQ8qwYeRfgGg uhaWIzLNP1NiumS/fc/l =72Op -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/aanlktimo-hxd7-e79asscpvypuxttg5dqyg8izjn_...@mail.gmail.com
Re: RFS: php-net-smtp (updated package)
Hi Guillaume, Guillaume Delacour wrote: php-net-smtp (1.4.4-1) unstable; urgency=low * New upstream release * debian/rules: complete rewrite based on cdbs (dh-make-php template) Sounds like a bad idea. Shouldn't the fix to #602865 get some more testing by sid users first so it can get into squeeze? It might be better to target the new upstream version to experimental. Hope that helps, Jonathan -- 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/2010112955.ga2...@burratino
Re: RFS: aspell-id
Hello, 2010/11/23 Jonathan Nieder jrnie...@gmail.com: Hi, Mahyuddin Susanto wrote: * URL : ftp://ftp.gnu.org/gnu/aspell/dict/id [...] - dget http://mentors.debian.net/debian/pool/main/a/aspell-id/aspell-id_0.1-2.dsc The .orig.tar.gz seems to be a git clone of git://github.com/udienz/ispell-id.git, though the files match aspell5-id-1.2-0.tar.bz2. I would suggest: no.. github.com contain debian directory. i don't know where is original cvs because i see that aspell-id team is now disappear and now debian-id rebuillding team again :). - setting the version number in debian/changelog based on the upstream version: aspell-id (1.2-0-1) unstable; urgency=low * Initial package for Debian (Closes: #604557). * Add simple rules file (using cdbs, based on such-and-such documentation), a control file describing the binary package, and an appropriate compat file. -- Mahyuddin Susanto udi...@gmail.com ... whenever ... done - putting a copy of aspell5-id-1.2-0.tar.bz2 named aspell-id_1.2-0.orig.tar.bz2 in the parent directory to your build dir when building the package. done and lintian says clear Perhaps this package should Provides: aspell-dictionary? Brian Nelson might have more advice; cc-ing him. yes, right this is dictionary for indonesian My motivation for maintaining this package is: learning packaging from debian mentors Mm, that's a lousy reason to introduce a new package. Better to improve the existing ones. i mean i want to more contribute to Debian, not only as user packaging localization But this is a very good reason to introduce a package. Hope that helps. Jonathan Thanks Jonathan for very quick reply and suggest -- 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/aanlktin526e5rhwhoqogofh+kdunrab79pvpf5je-...@mail.gmail.com
Re: RFS: php-log (updated package)
Hi Guillaume, Guillaume Delacour wrote: I am looking for a sponsor for the new version 1.12.3-1 of my package php-log. I am not a cdbs or php expert, but this looks pretty good. Some quick nitpicks: - Do you track your packaging with a vcs? If not, why not? :) (Mini-patches are easier to review.) - This bumps the required debhelper compat level to 7. Probably worth mentioning in the changelog. - I'm assuming the examples are still intended to be installed in /usr/share/doc/php-log/examples, but the changelog item + Delete nonexistent docs/examples/ does not make that so clear. - Probably it would be simplest to combine the changelog entries so there is exactly one entry for each ancestor of the current version released in a Debian-derived distribution. Please feel free to cc me if you have any questions. Regards, Jonathan -- 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/2010114801.ga2...@burratino
Re: RFC: MinGW-w64 toolchain (adoption and new packages)
(+cc: the almost-defunct debian-toolchain@ list, for posterity's sake)
Hi Stephen,
Stephen Kitt wrote:
I'm working on packaging a new version of the MinGW-w64 toolchain, which
allows 32- and 64-bit Windows software to be compiled as a cross-compiler
target using gcc.
Sounds neat. Have there been any efforts on integrating this work into
upstream GCC?
* binutils-mingw-w64, a simple binutils-source-based package providing
binutils targetting MinGW-w64's triplets;
Any examples for how this can be used directly? e.g., can simple
Windows toys in assembler be compiled this way, and can it help with
analyzing binaries from such systems?
* gcc-mingw-w64, a more complex gcc-4.5-source-based package providing gcc
targetting MinGW-w64's triplets;
I don't remember how far the dak work has proceeded in supporting this. :(
* mingw-w64, the MinGW-w64 development headers and libraries.
That means libc, I guess.
MinGW-w64 now has its own official triplets, differing from MinGW's which are
what had been used so far in Debian with the gcc-mingw32 and co. packages.
This is why new packages are needed
Thanks for explaining.
Building the packages is slightly involved:
1. Build binutils-mingw-w64 and install it.
So the first step could be to get this package alone in Debian.
2. Extract gcc-mingw-w64, and change the debian/control and
debian/rules.variant links so they point to debian/control.bootstrap and
debian/rules.bootstrap respectively.
3. Build gcc-mingw-w64 and install the resulting gcc-mingw-w64-bootstrap
package.
4. Build mingw-w64 and install the resulting mingw-w64-dev package.
5. Return to the gcc-mingw-w64 folder, and clean the build
fakeroot debian/rules clean
6. Change the debian/control and debian/rules.variant links so they point to
debian/control.full and debian/rules.full.
7. Build gcc-mingw-w64.
Note that since mingw-w64-dev is Architecture: all, this should only be
required to prepare the first upload.
Yagh. I guess Debian infrastructure does not take care of multi-package
bootstrapping scenarios so well. The general scheme seems sane; I
just have three suggestions:
. It would be simpler to have only one debian/rules file, with a
makefile variable to control which packages get built. See the
binutils package for an example.
. debian/rules is allowed to generate debian/control, so one would only
need one starting debian/control for this. See the binutils package
for an example.
. Maybe an example script in mingw-w64-dev's debian/README.source would
be helpful for people trying to figure out how the bootstrap worked.
It is nice that the dependency loop can be broken by uploading an
Arch: all package.
Also note that while this bootstrapping
sequence follows the existing Debian packaging structure, with separate
binutils, gcc and mingw packages, Dmitrijs Ledkovs (who is assisting me
with this packaging effort) has an alternative scheme with a single
self-bootstrapping package at https://launchpad.net/~mingw-w64
Yes, I tend to prefer the multiple-package method, too.
binutils has manpage errors and spelling errors in its binaries, none of
which I thought really warranted fixing (especially since they're all in
binutils-source anyway).
Please feel free to file a bug, especially if you have time to write a
patch.
gcc emits the following:
* debian-control-file-is-a-symlink
See above.
* copyright-refers-to-symlink-license: this is part of debian/copyright which
is reproduced from gcc-4.5-source's, and justified IMO; the
version-specific licence symlink follows it immediately;
Perhaps this is worth a bug on the lintian package?
* binary-without-manpage: on the todo list, although it doesn't seem
particularly urgent to me; should MinGW-w64-specific manpages be provided,
or would it do to just symlink the (old) gcc manpages?
I think brief mingw-w64-specific manpages would be ideal.
mingw-w64 has a ton of warnings, all instances of non-standard-dir-in-usr or
file-in-unusual-dir because it ships its headers and libraries
in /usr/$target/{include,lib}.
Sounds like a lintian bug.
gcc-mingw-w64 will eventually be split up, to provide smaller packages
providing the various compilers, headers and libraries in a similar fashion
to the mainline gcc package.
By this, you mean split into multiple binary packages, right?
* once the packages have been tested in actual use, we'll determine with Ron
Lee whether it is useful to keep MinGW32 in Debian along with MinGW-w64.
Thanks in advance for any comments!
Thanks for your hard work. Sounds very good.
Regards,
Jonathan
--
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/20101122232241.ga2...@burratino
Re: RFS: ramond
Hi Nicolas, Nicolas Dandrimont wrote: We've been using this software on our network for a few months now and it has proven very useful in killing off the rogue IPv6 routes and notifying our users of their configuration problem. I think including it into Debian would be worthwile. Sounds interesting. This is the first package I intend for inclusion into the main Debian repositories, so there may be some kinks to work out. [...] - dget http://mentors.debian.net/debian/pool/main/r/ramond/ramond_0.4-1.dsc FWIW, I am not a DD, so you have no hope of having this uploaded by me. :) I also do not have an IPv6 network to test it on. Quick tips: . did you know about dh_installexamples? . it might be nice to strip the outdated Installation section from the README at build time, like this: sed '/Installation/, /textproc/ d' README debian/README dh_installdocs debian/README . is it important to say This manual page was written for the Debian distribution because the original program does not have a manual page. so early in the ramond.8 page? If it were me, I'd not mention it at all; the COPYRIGHT section already says this was written for Debian. The packaging is very clean. Thanks for a pleasant read. Regards, Jonathan -- 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/20101122234338.ga3...@burratino
Re: RFS: ramond
Hi Jonathan, * Jonathan Nieder jrnie...@gmail.com [2010-11-22 17:43:38 -0600]: FWIW, I am not a DD, so you have no hope of having this uploaded by me. :) I also do not have an IPv6 network to test it on. Well, I'm glad to get feedback from a packaging point of view anyway. :) Quick tips: . did you know about dh_installexamples? I did not, but it sounds useful. . it might be nice to strip the outdated Installation section from the README at build time, like this: sed '/Installation/, /textproc/ d' README debian/README dh_installdocs debian/README Seems like a good idea. . is it important to say This manual page was written for the Debian distribution because the original program does not have a manual page. so early in the ramond.8 page? If it were me, I'd not mention it at all; the COPYRIGHT section already says this was written for Debian. Well now that I read it back it sounds weird indeed. I'll do another packaging pass tomorrow afternoon taking your remarks into account. The packaging is very clean. Thanks for a pleasant read. Thank you for your review! Best, -- Nicolas Dandrimont If you want to travel around the world and be invited to speak at a lot of different places, just write a Unix operating system. (By Linus Torvalds) signature.asc Description: Digital signature
Re: Build-Depends-Indep, please review
On Mon, Nov 22, 2010 at 04:42:03PM +, Roger Leigh wrote: On Mon, Nov 22, 2010 at 04:39:20PM +, Roger Leigh wrote: I checked, and dh currently doesn't support build-arch and build-indep at all internally (you need to add the rules yourself). I've made a patch to add proper support which I'll submit once I've tested it. FWIW, the current progress on this is attached. No guarantees--it's totally untested. Final tested patch in #604563. -- .''`. Roger Leigh : :' : Debian GNU/Linux http://people.debian.org/~rleigh/ `. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net/ `-GPG Public Key: 0x25BFB848 Please GPG sign your mail. signature.asc Description: Digital signature
Re: RFS: triggerhappy
Hi Stefan, Stefan Tomanek wrote: * URL : https://github.com/wertarbyte/triggerhappy [...] My motivation for maintaining this package is: I've written this small program to fill a need not covered by any existing program, so I think having it in Debian and making it available to others is a good thing. Currently, there is no practical way to configure global hotkeys that act independent from a user and/or X11 session. Buttons like Suspend or Volume Control should be handeled without requiring a user to log into their desktop environment How does this compare to Rick van Rein's funkey[1]? It might make sense to add your app to Andries Brouwer (cc'd)'s keyboard and console HOWTO, though the latter is sadly a bit dated by now. Regards, Jonathan [1] http://rick.vanrein.org/linux/funkey/ -- 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/20101123002327.ga3...@burratino
Re: RFS: zsnes (updated package - ITA)
Hi Etienne, Etienne Millon wrote: I have been using this package for years and it seems to need love. As I am learning how to maintain package, this seems a perfect candidate for adoption. I am preparing an upload fixing a few bugs, and will close this ITA when it's done. These are words I like to hear. - dget http://mentors.debian.net/debian/pool/main/z/zsnes/zsnes_1.510-3.dsc I am not a DD so there is no hope for an upload from me. :) Quick thoughts: + * Remove useless build-deps on xlibmesa-gl-dev and xutils (Closes: #579040) Are you sure the build-deps were useless? It could be that some of them are still needed but were brought in accidentally via libsdl-dev, for example. (See http://bugs.debian.org/123725) $ apt-cache show xutils xlibmesa-gl-dev x11-xfs-utils, x11-utils, x11-xserver-utils, x11-session-utils, xfonts-utils, libgl1-mesa-dev would be a conservative replacement. (Of course, some subset of that, perhaps the empty one, would be the correct replacement.) + * Add debian/README.source to document dpatch. I would have used a symlink or text pointing to /usr/share/doc/dpatch/README.source.gz for easier maintenance. + * Add debian/watch file. Hoorah. :) I haven't checked the build-deps. Everything else looks good. Regards, Jonathan -- 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/20101123010256.ga3...@burratino
Re: [Pkg-php-pear] RFS: php-net-smtp (updated package)
On 11/23/2010 06:29 AM, Jonathan Nieder wrote: Hi Guillaume, Guillaume Delacour wrote: php-net-smtp (1.4.4-1) unstable; urgency=low * New upstream release * debian/rules: complete rewrite based on cdbs (dh-make-php template) Sounds like a bad idea. Shouldn't the fix to #602865 get some more testing by sid users first so it can get into squeeze? It might be better to target the new upstream version to experimental. Hope that helps, Jonathan I agree, now is not the time for rewriting php-net-smtp, we should focus on the Squeeze version, and more over, I don't think rewriting things using CDBS is better. Also, I would have appreciate if you could have care about the issue, you took so long to show you are there! Now, is there an issue with the current version in Testing? Any bugfix that would need to be backported? Thomas -- 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/4ceb610e@goirand.fr
Re: RFS: zsnes (updated package - ITA)
Hello, Thanks for this review. Are you sure the build-deps were useless? It could be that some of them are still needed but were brought in accidentally via libsdl-dev, for example. (See http://bugs.debian.org/123725) The package builds in a clean chroot, so I thought this was good enough. If this is bad practice I will have another look at it. + * Add debian/README.source to document dpatch. I would have used a symlink or text pointing to /usr/share/doc/dpatch/README.source.gz for easier maintenance. Good idea indeed. Thanks again, -- Etienne Millon -- 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/20101123071809.ga2...@john.ssi.corp
Re: RFS: triggerhappy
Dies schrieb Jonathan Nieder (jrnie...@gmail.com): How does this compare to Rick van Rein's funkey[1]? Hm, funkey does require a kernel patch und looks quite dated? Funky Daemon which demonstrates how to parse the /dev/funkey character device. As I said, triggerhappy uses the /dev/input/eventX files, anything that fires SW_, KEY_ or BTN_ events can be used t launch programs. -- 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/20101123073810.gn15...@zirkel.wertarbyte.de
