Bug#559814: ITA: hamlib / new version for upload

2010-02-17 Thread Stefano Zacchiroli
On Tue, Feb 16, 2010 at 07:53:53PM -0800, Kamal Mostafa wrote:
 http://www.whence.com/debian/proposed/hamlib_1.2.10-1.dsc

Thanks a lot, I'll review this shortly and get back to you.

Cheers.

-- 
Stefano Zacchiroli -o- PhD in Computer Science \ PostDoc @ Univ. Paris 7
z...@{upsilon.cc,pps.jussieu.fr,debian.org} -- http://upsilon.cc/zack/
Dietro un grande uomo c'è ..|  .  |. Et ne m'en veux pas si je te tutoie
sempre uno zaino ...| ..: | Je dis tu à tous ceux que j'aime



--
To UNSUBSCRIBE, email to debian-qa-packages-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20100217083217.ga12...@usha.takhisis.invalid



Bug#559814: ITA: hamlib / new version for upload

2010-02-17 Thread Stefano Zacchiroli
On Tue, Feb 16, 2010 at 07:53:53PM -0800, Kamal Mostafa wrote:
 I wasn't sure what to do with the debian/control Uploaders: field.  As
 a placeholder, I set it to the debian-h...@lists.debian.org list address
 -- I expect that it will be changed by the actual uploader:
 
   Uploaders: Debian Hamradio Maintainers debian-h...@lists.debian.org
 
 This is my first experience with maintenance of a package at Debian, so
 I am not at all sure that I'm following the proper procedures here.  Any
 guidance will be much appreciated.

So, I've just checked your package and it looks generally OK to me. I'll
proceed to upload it to DELAYED/1, just to give other people a bit of
time to react in case there are reservations.

The few comments I've are as follows, please consider them in future
uploads:

- I've switched your Maintainer/Uploaders line, to match the best
  practices on that. Maintainer is the official contact address (the
  mailing list) and Uploaders is the list of people usually *working*
  (not just doing the final upload, despite the name) on the package,
  hence you in this case. Please port the change to your working copy of
  the packaging.

- You install a couple more .so files than in the past, however these
  new .so are not versioned, I believe it is an upstream choice, but you
  should discuss with them whether this is really intended or not.

- As hamlib has a homepage (the sourceforce page), you should declare it
  in debian/control using the Homepage field, check the developer's
  reference for details. If you use some version control system to
  maintain the packaging, you should declare that too using the Vcs-*
  fields (again, devref has details about that).

- There are quite a lot of lintian warnings, you should fix them. Most
  of them are missing ${misc:Depends} substvar that are recommended if
  you use debhelper (as you do) in the packaging; please add them.

Thanks for your work!

-- 
Stefano Zacchiroli -o- PhD in Computer Science \ PostDoc @ Univ. Paris 7
z...@{upsilon.cc,pps.jussieu.fr,debian.org} -- http://upsilon.cc/zack/
Dietro un grande uomo c'è ..|  .  |. Et ne m'en veux pas si je te tutoie
sempre uno zaino ...| ..: | Je dis tu à tous ceux que j'aime



--
To UNSUBSCRIBE, email to debian-qa-packages-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20100217102837.gb22...@usha.takhisis.invalid



Bug#559814: ITA: hamlib / new version for upload

2010-02-16 Thread Kamal Mostafa
After discussion with Steve Conklin (who filed an ITA for 'hamlib' a few
months ago) we have determined that I will volunteer to adopt and
maintain 'hamlib' instead of him.

For reference, I am a ham radio operator (KA6MAL), I do use hamlib
regularly, and I have developed applications which use the library.  

I have packaged the new upstream release (hamlib 1.2.10) which I propose
for review and upload by a kindly DD:

http://www.whence.com/debian/proposed/hamlib_1.2.10-1.dsc

This package includes my fix for bug 559814 (CVE-2009-3736 local libltdl
privilege escalation), and minor Python 2.6 changes from Ubuntu.

I wasn't sure what to do with the debian/control Uploaders: field.  As
a placeholder, I set it to the debian-h...@lists.debian.org list address
-- I expect that it will be changed by the actual uploader:

  Uploaders: Debian Hamradio Maintainers debian-h...@lists.debian.org

This is my first experience with maintenance of a package at Debian, so
I am not at all sure that I'm following the proper procedures here.  Any
guidance will be much appreciated.

Thanks,

 -Kamal Mostafa ka...@whence.com



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