[ I apologize for the private mail to Gregor, it was an error of mine. ] Hi Gregor!
On Fri, Sep 26, 2008 at 03:44:09PM +0200, gregor herrmann wrote:
> On Fri, 26 Sep 2008 14:55:44 +0200, Cristian Greco wrote:
> > I would be glad if someone reviewed and uploaded this package for me.
>
> Just a few remarks:
> * I would not install (i.e. remove in debian/rules)
> /etc/pacpl/modules/{,sample.m}
Acknowledged, with some other changes to the package:
- delete /etc/pacpl/modules
- add pacpl.install to create two subdirs in /usr/share/doc/pacpl:
+ modules/: sample module and readme file
+ pacpl-gui/: kommander gui and readme file
- add README.Debian to explain where to place the user modules.
> * debian/copyright:
> - the upstream license should be GPL-3+ (and not GPL-3)
Right, done.
> - "Files: *" should be before "Files: debian/*", if I understand
> the section about "Match order" correctly on
> http://wiki.debian.org/Proposals/CopyrightFormat
Done.
> - any specific reason why you license debian/* under GPL-2+ and not
> GPL-3+ like upstream?
No specific reason, just my usual license for all of my packages (except
perl modules). There is no problem to conform with upstream choice, so I
updated license to GPL-3+.
A new package is available on mentors:
dget http://mentors.debian.net/debian/pool/main/p/pacpl/pacpl_4.0.3-1.dsc
Thanks,
Cristian
signature.asc
Description: Digital signature
