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 

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 



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


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 
> 
> 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