Re: RFS: l2tp-ipsec-vpn
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I addressed all the issues mentioned below (hopefully got it right) and uploaded it to mentors.debian.net. Still looking for somebody to sponsor it. Thanks for the thorough review. Werner On 06/19/2011 07:34 PM, Arno Töll wrote: > Hi Werner, > > I was taking a look into your package, here are some comments on it > (as usual, I am pointing out I can't sponsor you, as I'm not > developer). > > On 19.06.2011 07:55, Werner Jaeger wrote: >> I am looking for a sponsor for my package "l2tp-ipsec-vpn". > > * In debian/changelog - just a minor stylistic note: it is > considered best practice to denote bug numbers in parenthesis e.g. > "(Closes: #602504)" [see also 1]. * Please remove > debian/changelog.org from your source package. This seems to be > your Ubuntu changelog. You don't have to remove it entirely, just > make sure it does not appear in Debian source packages to avoid > confusion. Same holds for control.org. * debian/compat + > debian/control indicate debhelper compatibility 7. We're on version > 8 now, please bump both to 8.0 (see debhelper(1)). * Your standards > version is not entirely up to date. Please check whether your > package complies with 3.9.2 and bump it in your control file > afterwards [4]. * It is best practice to denote VCS fields in your > control file. Learn more on [5] * Your synopsis line reads fine, > the long description however is a bit unclear. Imagine a user wants > to find out, what your package is about and the first thing he > learns is "This GUI is not a network manager plug-in". You don't > have to describe every feature either, especially not every button > available in your GUI. Just make sure you provide a comprehensive, > but short description what your program is about in first place. * > In debian/copyright the "Format" link is broken. No, that's not > your fault, this is due to the recent Alioth migration. Perhaps > update it anyway. * You certainly can, but its not required to > point the exact release in "Source" fields in debian/copyright. If > you do, make sure you don't forget to update this link for every > Debian release though. * Your postinst script looks a bit > problematic. First of all, you silently assume gksu is installed > but you don't depend on it. Moreover I'm not entirely sure you > should really (re-)start your application in postinst. Since it is > a GUI application there are plenty of corner cases where the user > you think would be the GUI user, actually isn't. If you really want > to do such magic, perhaps restart your application for the user > owning the old PID. * What's your rationale to delete > /usr/share/applications/L2TP-Ipsec-VPN-Manager.desktop in postrm? > Shoudln't you ship this desktop file with your package and thus > letting dpkg take care of it? The deletion of > /etc/xdg/autostart/L2tpIPsecVpn.desktop is also problematic (or > even more). /etc isn't considered a package's domain, but of the > site administrator. You shall not alter any files there without > explicit permission of the administrator. This includes deletion of > files, see §10.7.3 [2][3]. Here I advocate again to let dpkg handle > treatment of this file. Ultimately its the user's choice whether he > wants to keep this file or purge the package. * In debian/rules > please don't set DH_VERBOSE on release packages. Its fine to do for > your debugging purposes, but comment it out on packages you want to > have uploaded. Note, some people don't agree here, and this is a > cosmetic change after all. * Your package fails to build from > source in a clean chroot: > > g++ -c -m64 -pipe -O2 -Wall -W -D_REENTRANT -DQT_NO_DEBUG > -DQT_GUI_LIB -DQT_NETWORK_LIB -DQT_CORE_LIB -DQT_SHARED > -I/usr/share/qt4/mkspecs/linux-g++-64 -Inbproject > -I/usr/include/qt4/QtCore -I/usr/include/qt4/QtNetwork > -I/usr/include/qt4/QtGui -I/usr/include/qt4 -Isrc > -I/usr/include/qt4 -Isrc/generated -Isrc/generated -I. -o > build/Release/GNU-Linux-x86/OpensslConfWriter.o > src/conf/OpensslConfWriter.cpp In file included from > /usr/include/ctemplate/template.h:50:0, from > src/conf/OpensslConfWriter.cpp:25: > /usr/include/ctemplate/template_dictionary.h:73:11: error: > 'ptrdiff_t' does not name a type make[2]: *** > [build/Release/GNU-Linux-x86/OpensslConfWriter.o] Error 1 make[2]: > Leaving directory `/tmp/buildd/l2tp-ipsec-vpn-0.9.9' make[1]: *** > [override_dh_auto_build] Error 2 make[1]: Leaving directory > `/tmp/buildd/l2tp-ipsec-vpn-0.9.9' make: *** [build] Error 2 > dpkg-buildpackage: error: debian/rules build gave error exit status > 2 > > I'm not entirely sure, whether YOU are to blame here. You have to > find out though :> > > >> This is not my first request but so far I did not get any > response. Is there anything else I >> should be doing ? > > Unfortunately not. Just keep it trying every once in a while until > you either find a sponsor or you are scared off. You can improve > your chances to find a sponsor by asking maintainer o
RFS: acsccid (Updated)
Dear mentors, I am looking for a sponsor for my package "acsccid". Actually, I posted the request about 1 month ago and I did not receive any response here. The package had been updated and fix the following problems: 1. The project is hosted in sourceforge now and watch file is updated. 2. Remove dh_makeshlibs from rules file. 3. Fix non-redistributable reference manual problem. * Package name: acsccid Version : 1.0.2-1 Upstream Author : Advanced Card Systems Ltd. * URL : http://acsccid.sourceforge.net/ * License : LGPL-2.1 Section : libs It builds these binary packages: libacsccid - PC/SC driver for ACS USB CCID smart card readers The package appears to be lintian clean. The upload would fix these bugs: 627038 My motivation for maintaining this package is: This package contains the official PC/SC driver for ACS USB CCID smart card readers. Compared with libccid, it supports many latest readers from ACS. Users of these readers are required to install this driver. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/a/acsccid - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/a/acsccid/acsccid_1.0.2-1.dsc This is my first package and I would be glad if someone reviewed and uploaded this package for me. Kind regards Godfrey Chung
RFS: mosh
Dear mentors, [Please read long description under this template] * Package name: mosh Version : 0.2.7-1 Upstream Author : Taro Minowa * URL : http://code.google.com/p/mosh-scheme/ * License : BSD-2-clause Section : lisp It builds these binary packages: mosh - fast R6RS Scheme interpreter mosh-doc - reference documentation for Mosh The package appears to be lintian clean. Pedantic shows a few warnings about duplicated files in the docs package - this is not a problem in my opinion, as the files are small autogenerated index entries and semantically meaningful in their own right. I have overridden unusual-interpreter. The upload would fix these bugs: 631139, 537776 (archived) My motivation for maintaining this package is: I use this interpreter and have submitted several patches upstream. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/m/mosh - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/m/mosh/mosh_0.2.7-1.dsc Before you review it, let me point out several issues that may interest you. This is a rather tough package, for two main reasons: #1. It requires an embedded copy of the Boehm GC. dmoerner previously attempted to unbundle this and it did work, however a recent change in Mosh relies on a non-default compile time configuration of the GC, and also bugfixes which are only present in the CVS version. As such it's quite impractical to unbundle the GC library at the moment, and the upstream bug is marked WONTFIX. See: http://code.google.com/p/mosh-scheme/issues/detail?id=156 For reference, I have attempted to build against a libgc using a default configuration and it breaks badly at runtime. #2. psyntax-mosh requires several Scheme sources to be compiled into a single 'binary' (which is actually text, but not human-editable). However, the build script requires a previous version of Mosh. Releases are distributed with a precompiled version so the users doesn't need an older version. I asked about this on IRC, and it seems it's unacceptable to use the precompiled file in the final build, so two solutions were suggested. One is to initially build using the precompiled file and then rebuild over the top using the now-bootstrapped version (The version doesn't necessarily need to be older.) The other method is to split the source package into two packages, mosh-bootstrap and mosh, where mosh-bootstrap is arch-independent and mosh arch-dependent. Neither of these are clean but that is probably unavoidable. As a result of this I am not immediately expecting a sponsor (though hope springs eternal), but rather looking for comments and some advice on how to proceed. Cheers, -- David Banks -- 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/itsgii$d4g$1...@dough.gmane.org
RFS: gogglesmm
Dear mentors, I am looking for a sponsor for my package "gogglesmm". * Package name: gogglesmm Version : 0.12.2-5 Upstream Author : Sander Jansen * URL : http://code.google.com/p/gogglesmm/ * License : GPLv3 Section : sound It builds these binary packages: gogglesmm - Goggles Music Manager The package appears to be lintian clean. My motivation for maintaining this package is: Goggles Music Manager is a music collection manager and player. It is very lightweight and fast, supports Ogg Vorbis , FLAC, MP3 , MP4 , ASF and Musepack files, has support for AlbumArt and a very nice Browser for your music collection. Furthermore it has last.fm and libre.fm audio scrobbler support. It has already been mentioned in several magazines > "Keeping it simple is probably GMM's biggest selling point" - Andy Hudson in > Linux Format Issue 123, October 2009 > > Scratching that itch: Sander Jansen - Joe Barr on Linux.com / Newsforge- June > 2006 (http://www.linux.com/archive/articles/54944) > > "Goggles is my candidate for a lightweight music player" - Bruce > Byfield on Linux Magazine Blog > (http://www.linux-magazine.com/Online/Blogs/Off-the-Beat-Bruce-Byfield-s-Blog/Goggle-is-my-candidate-for-a-lightweight-music-player) I am using Goggles for at least a year now and I am very happy with it. I would be happy to see this Software in the Debian official repositories. Screenshots: http://picasaweb.google.com/s.jansen/GogglesMusicManager08 The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/g/gogglesmm - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/g/gogglesmm/gogglesmm_0.12.2-5.dsc I would be glad if someone uploaded this package for me. Kind regards Hendrik Rittich -- Hendrik Rittich signature.asc Description: PGP signature
Re: RFS: lugaru (new package)
Am Dienstag, den 21.06.2011, 23:27 +0200 schrieb Arand Nash: > On 21/06/11 18:09, Bruno Kleinert wrote: > > Am Dienstag, den 21.06.2011, 13:32 +0200 schrieb Arand Nash: > >> On 08/06/11 15:27, Bruno Kleinert wrote: > >>> I always omit this field in my .desktop files ;) The specification says > >>> "Path to an executable file on disk used to determine if the program is > >>> actually installed." As far as I understand this, > >>> specifying /usr/lib/games/lugaru/lugaru isn't correct and in this case > >>> TryExec doesn't make any sense at all, because the .desktop file isn't > >>> installed as long as lugaru isn't installed. I'd suggest to omit the > >>> TryExec field. > >> > >> I fixed this in a new upload to mentors. > >> I'd be glad if someone could sponsor it, or if it's not ready, hint on > >> how to make it. > > One final issue: Please remove "Closes: #123" from the changelog in > > lugaru-data, it's only lugaru that closes the ITP. > > > > Cheers - Fuddl > > Done and re-uploaded to mentors, I assume you meant to simply remove the > Closes: stanza. Yes, I meant exactly that. Are you sure you uploaded it? When I dget http://mentors.debian.net/debian/pool/contrib/l/lugaru-data/lugaru-data_0~20110520.1+hge4354-1.dsc the closes: stanza is still there? Greetings - Fuddl signature.asc Description: This is a digitally signed message part
Re: RFS: lugaru (new package)
On 22/06/11 15:55, Bruno Kleinert wrote: Am Dienstag, den 21.06.2011, 23:27 +0200 schrieb Arand Nash: On 21/06/11 18:09, Bruno Kleinert wrote: Am Dienstag, den 21.06.2011, 13:32 +0200 schrieb Arand Nash: I fixed this in a new upload to mentors. I'd be glad if someone could sponsor it, or if it's not ready, hint on how to make it. One final issue: Please remove "Closes: #123" from the changelog in lugaru-data, it's only lugaru that closes the ITP. Cheers - Fuddl Done and re-uploaded to mentors, I assume you meant to simply remove the Closes: stanza. Yes, I meant exactly that. Are you sure you uploaded it? When I dget http://mentors.debian.net/debian/pool/contrib/l/lugaru-data/lugaru-data_0~20110520.1+hge4354-1.dsc the closes: stanza is still there? Greetings - Fuddl Ah, that's the old version, since before I changed to place it in non-free. Annoying that those still remain on the server unreplaced... - dget http://mentors.debian.net/debian/pool/non-free/l/lugaru-data/lugaru-data_0~20110520.1+hge4354-1.dsc should hopefully get you the current version -- 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/4e022292.3040...@gmail.com
RFS: dhcpd-pools
Dear mentors, I am looking for a sponsor for my package "dhcpd-pools". * Package name: dhcpd-pools Version : 2.15-1 Upstream Author : Sami Kerola * URL : http://dhcpd-pools.sourceforge.net/ * License : GPL-3.0+ Section : net It builds these binary packages: dhcpd-pools - ISC dhcpd lease analysis and reporting The package appears to be lintian clean. The upload would fix these bugs: 631229 My motivation for maintaining this package is: I have a client that needed this package, and I believe that it is generic enough that it should available in main. It is a good candidate for my first official Debian package, due to its simplicity (pure autoreconf, and currently no additional patches). The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/d/dhcpd-pools - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/d/dhcpd-pools/dhcpd-pools_2.15-1.dsc - git: http://git.asbjorn.biz/?p=debian/dhcpd-pools.git I would be glad if someone uploaded this package for me. $ git diff --stat upstream...master debian/changelog |5 + debian/compat |1 + debian/control| 21 + debian/copyright | 29 + debian/docs |4 debian/rules |8 debian/source/format |1 + debian/watch | 23 +++ 8 files changed, 92 insertions(+), 0 deletions(-) -- Kind regards Asbjørn Sloth Tønnesen -- 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/4e024317.4010...@asbjorn.biz