Bug#581268: libdata-visitor-perl: move away from libdata-alias-perl

[Niko Tyni]

tag 581268 patch
tag 581268 - help
thanks

On Tue, May 18, 2010 at 10:29:36AM +0300, Niko Tyni wrote:
> On Wed, May 12, 2010 at 10:49:54AM +0300, Niko Tyni wrote:
> > On Mon, Apr 26, 2010 at 09:57:30PM +0300, Niko Tyni wrote:
> 
> > > B: remove libdata-alias-perl from the archive: either
> > >   B1. remove the libdata-visitor-perl dependency on libdata-alias-perl
> > >     and hope that no libdata-visitor-perl reverse dependencies need
> > >     the removed functionality
> 
> It turns out that Data::Alias is only needed for currently undocumented 
> functionality, enabled with the 'weaken' attribute of Data::Visitor objects.
> 
> Therefore I think this solution is acceptable and even preferred.
> 
> I've glanced through the six reverse dependencies of libdata-visitor-perl,
> and I don't see any indication that the undocumented weak reference
> support in Data::Visitor is needed.

I've now rebuilt all the reverse dependencies (along with a lot of other
packages) against a local version of libdata-visitor-perl that has the
libdata-alias-perl dependency removed. No regressions were found.

> I propose we demote libdata-alias-perl to Recommends: or even
> Suggests: for now. I very much doubt anything is going to break.

I think even Recommends is too much for undocumented functionality, and
I'm inclined to just drop the libdata-alias-perl dependencies altogether.

Trivial patch attached.
-- 
Niko Tyni   ntyni at debian.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 581268.patch
Type: text/x-diff
Size: 1070 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20100524/ea65204a/attachment.patch>