[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Replacing libio-compress-bzip2-perl, libcompress-zlib-perl, libio-compress-zlib-perl, and libio-compress-base-perl with libio-compress-perl



On Wed, 22 Jul 2009 17:26:31 -0400, Scott Kitterman wrote:

> As I previously mentioned on the debian-perl list, upstream has combined 
> libio-compress-bzip2-perl, libcompress-zlib-perl, libio-compress-zlib-perl, 
> and libio-compress-base-perl into libio-compress-perl.  I'm working on 
> updating this in Ubuntu and want to contribute the work back to Debian.

Great!
 
> The draft package is in the pkg-perl svn (since they are maintainer of all but 
> one of the constituent modules).  I also added anyone who was in Uploaders or 
> Maintainer for any of these packages to Uploaders for the combined package.
> 
> http://svn.debian.org/viewsvn/pkg-perl/trunk/libio-compress-perl/debian/

Some comments:
* debian/watch:
  please drop the _ to avoid matching developer releases
  (that "tradition" is rather new)
* please rename debian/libio-compress-perl.docs to
  debian/libio-compress-perl.examples and change the contents to
  "examples/*"
* debian/changelog:
  why is there the whole changelog of libio-compress-base-perl?
  libio-compress-perl is a new package so it should only have one
  changelog stanza with "Initial release (closes: #nnnnnn)." (and
  maybe a mentioning of the merge for users and ftp-masters). That
  also implies filing an ITP.
* debian/copyright:
  please convert to the new format -- and I think it can be greatly
  simplified. IMO there's no reason to mention stuff from older
  packages.
* debian/rules:
  this is the debian/rules file from libio-compress-base-perl ?!
  I guess it could be simplified but haven't looked in detail
* debian/control:
  - there's "Build-Depends-Indep: [..], libcompress-raw-bzip2-perl
    (>=2.020), libcompress-raw-zlib-perl (>= 2.020)" but those are
    not in Depends
  - we need transitional dummy packages for the 4 "old" packages,
    depending on the new libio-compress-perl, and
  - Replaces and Conflicts have then to be versioned as "(<< 2.020-1)"
 
> The package builds and is Lintian clean (source and binary), but before it can 
> be uploaded, libcompress-raw-bzip2-perl and libcompress-raw-zlib-perl will 
> have to be updated.  

Right, quoting META.yml:
requires:
    Compress::Raw::Bzip2:  2.020
    Compress::Raw::Zlib:  2.020

> As soon as this is done, libio-compress-bzip2-perl, 
> libcompress-zlib-perl, libio-compress-zlib-perl, and libio-compress-base-perl 
> will be uninstallable.  Given the need for a trip through New, I'm not sure 
> what the best way to manage that is.

The new libio-compress-perl binary packages needs to go through NEW,
so the then dummy "old" packages wil also be in NEW, but I don't see
any problems for the packages in the archive at the moment or the
ones depending on them.

I haven't built te package now because of the build dependency on
libcompress-raw-bzip2-perl (>=2.020), libcompress-raw-zlib-perl (>=
2.020). libcompress-raw-bzip2-perl is already at 2.020-1, but
libcompress-raw-zlib-perl is at 2.015-2. It has the following notes
in changelog in SVN:

  TODO:
  wait for 2.015-2 to hit testing

  WARNING:
  uploading 2.020 would break libio-compress-zlib-perl, which depends on
  libcompress-raw-zlib-perl (>= 2.015), libcompress-raw-zlib-perl (<< 2.016)

The first point has already happend, the second needs to be sorted
out before ...


Disclaimer: all this stuff is hurting my brain, so other thoughts are
highly appreciated!


@Bas:
At the moment libio-compress-perl is in the pkg-perl repo and I guess
we can maintain it there, but I think there are no objections if you
want to take care of it :)

Cheers,
gregor 
-- 
 .''`.   http://info.comodo.priv.at/ -- GPG Key IDs: 0x00F3CFE4, 0x8649AA06
 : :' :  Debian GNU/Linux user, admin, & developer - http://www.debian.org/
 `. `'   Member of VIBE!AT, SPI Inc., fellow of FSFE | http://got.to/quote/
   `-    BOFH excuse #324:  Your packets were eaten by the terminator 

Attachment: signature.asc
Description: Digital signature


Reply to: