Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread gregor herrmann
On Wed, 21 Jun 2017 23:28:52 +0100, Dominic Hargreaves wrote:

> > > My reading is that no change in IO-Compress was needed once
> > > Compress-Raw-Zlib was fixed. And the CPAN bug was resolved with
> > > no code change in IO-Compress.
> > In any case, both are updated to the latest version in unstable (and
> > they are released in lockstep), so I assume no problems there.
> This is about the version bundled with perl itself which is not yet fixed
> in unstable. But good to point out that the separate packaged versions
> don't need a separate bug report :)

Oops; I know that the original bug report was about the bundled
modules but for whatever reason (ETOOLATE maybe) I thought the
discussion had changed to the separate packages :)
 

Cheers,
gregor

-- 
 .''`.  https://info.comodo.priv.at/ - Debian Developer https://www.debian.org
 : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D  85FA BB3A 6801 8649 AA06
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   NP: Tracy Chapman: Telling Stories


signature.asc
Description: Digital Signature


Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread Dominic Hargreaves
On Thu, Jun 22, 2017 at 12:25:16AM +0200, gregor herrmann wrote:
> On Wed, 21 Jun 2017 20:17:48 +0100, Dominic Hargreaves wrote:
> 
> > > > Just to be clear, this is in Compress-Raw-Zlib that has already 
> > > > (according to https://rt.cpan.org/Public/Bug/Display.html?id=119762)
> > > > been fixed in 2.74, which is already in blead and perl 5.26 (which is
> > > > in experimental).
> > > The Ubuntu patches and the second bug also reference IO-Compress, not
> > > sure if that's upstream yet.
> > My reading is that no change in IO-Compress was needed once
> > Compress-Raw-Zlib was fixed. And the CPAN bug was resolved with
> > no code change in IO-Compress.
> 
> In any case, both are updated to the latest version in unstable (and
> they are released in lockstep), so I assume no problems there.

This is about the version bundled with perl itself which is not yet fixed
in unstable. But good to point out that the separate packaged versions
don't need a separate bug report :)

https://anonscm.debian.org/cgit/perl/perl.git/tree/cpan/Compress-Raw-Zlib/lib/Compress/Raw/Zlib.pm#n12



Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread gregor herrmann
On Wed, 21 Jun 2017 20:17:48 +0100, Dominic Hargreaves wrote:

> > > Just to be clear, this is in Compress-Raw-Zlib that has already 
> > > (according to https://rt.cpan.org/Public/Bug/Display.html?id=119762)
> > > been fixed in 2.74, which is already in blead and perl 5.26 (which is
> > > in experimental).
> > The Ubuntu patches and the second bug also reference IO-Compress, not
> > sure if that's upstream yet.
> My reading is that no change in IO-Compress was needed once
> Compress-Raw-Zlib was fixed. And the CPAN bug was resolved with
> no code change in IO-Compress.

In any case, both are updated to the latest version in unstable (and
they are released in lockstep), so I assume no problems there.
 

Cheers,
gregor

-- 
 .''`.  https://info.comodo.priv.at/ - Debian Developer https://www.debian.org
 : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D  85FA BB3A 6801 8649 AA06
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   NP: Steppenwolf: Born To Be Wild


signature.asc
Description: Digital Signature


Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread Dominic Hargreaves
On Wed, Jun 21, 2017 at 10:02:37PM +0300, Niko Tyni wrote:
> On Wed, Jun 21, 2017 at 07:42:32PM +0100, Dominic Hargreaves wrote:
> > On Wed, Jun 21, 2017 at 08:10:04AM +, Gianfranco Costamagna wrote:
> > > Source: perl
> > > Version: 5.24.1-4
> > > Severity: important
> > > 
> > > Hello, with the new zlib > 1.2.9 the testsuite fails,
> > > (e.g. you can look at Ubuntu build failures).
> > > https://launchpad.net/ubuntu/+source/perl/5.24.1-2
> > > 
> > > I found the test failures on rt.cpan.org and I applied to Ubuntu the 
> > > proposed
> > > patches:
> > > https://rt.cpan.org/Public/Bug/Display.html?id=120134
> > > https://rt.cpan.org/Public/Bug/Display.html?id=119762
> 
> > Just to be clear, this is in Compress-Raw-Zlib that has already 
> > (according to https://rt.cpan.org/Public/Bug/Display.html?id=119762)
> > been fixed in 2.74, which is already in blead and perl 5.26 (which is
> > in experimental).
> 
> The Ubuntu patches and the second bug also reference IO-Compress, not
> sure if that's upstream yet.

My reading is that no change in IO-Compress was needed once
Compress-Raw-Zlib was fixed. And the CPAN bug was resolved with
no code change in IO-Compress.

> > I'm not convinced that we need to spend time on fixing this for 5.24
> > given where we are in the release cycle, but this will depend on
> > when the zlib maintainer (Cced) plans to update to a newer zlib.
> 
> Agreed.



Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread Niko Tyni
On Wed, Jun 21, 2017 at 07:42:32PM +0100, Dominic Hargreaves wrote:
> On Wed, Jun 21, 2017 at 08:10:04AM +, Gianfranco Costamagna wrote:
> > Source: perl
> > Version: 5.24.1-4
> > Severity: important
> > 
> > Hello, with the new zlib > 1.2.9 the testsuite fails,
> > (e.g. you can look at Ubuntu build failures).
> > https://launchpad.net/ubuntu/+source/perl/5.24.1-2
> > 
> > I found the test failures on rt.cpan.org and I applied to Ubuntu the 
> > proposed
> > patches:
> > https://rt.cpan.org/Public/Bug/Display.html?id=120134
> > https://rt.cpan.org/Public/Bug/Display.html?id=119762

> Just to be clear, this is in Compress-Raw-Zlib that has already 
> (according to https://rt.cpan.org/Public/Bug/Display.html?id=119762)
> been fixed in 2.74, which is already in blead and perl 5.26 (which is
> in experimental).

The Ubuntu patches and the second bug also reference IO-Compress, not
sure if that's upstream yet.

> I'm not convinced that we need to spend time on fixing this for 5.24
> given where we are in the release cycle, but this will depend on
> when the zlib maintainer (Cced) plans to update to a newer zlib.

Agreed.
-- 
Niko



Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread Dominic Hargreaves
Control: fixed -1 5.26.0~rc1-1

On Wed, Jun 21, 2017 at 08:10:04AM +, Gianfranco Costamagna wrote:
> Source: perl
> Version: 5.24.1-4
> Severity: important
> 
> Hello, with the new zlib > 1.2.9 the testsuite fails,
> (e.g. you can look at Ubuntu build failures).
> https://launchpad.net/ubuntu/+source/perl/5.24.1-2
> 
> I found the test failures on rt.cpan.org and I applied to Ubuntu the proposed
> patches:
> https://rt.cpan.org/Public/Bug/Display.html?id=120134
> https://rt.cpan.org/Public/Bug/Display.html?id=119762
> 
> 
> I don't know why they are not applied upstream, nor if they are really 
> backward
> compatible with the older zlib
> 
> 
> please consider applying them or upstreaming them (in case cpan is not the 
> upstream
> place).

Just to be clear, this is in Compress-Raw-Zlib that has already 
(according to https://rt.cpan.org/Public/Bug/Display.html?id=119762)
been fixed in 2.74, which is already in blead and perl 5.26 (which is
in experimental).

I'm not convinced that we need to spend time on fixing this for 5.24
given where we are in the release cycle, but this will depend on
when the zlib maintainer (Cced) plans to update to a newer zlib.

Thanks,
Dominic.



Bug#865411: perl: please backport zlib > 1.2.9 patches

2017-06-21 Thread Gianfranco Costamagna
Source: perl
Version: 5.24.1-4
Severity: important

Hello, with the new zlib > 1.2.9 the testsuite fails,
(e.g. you can look at Ubuntu build failures).
https://launchpad.net/ubuntu/+source/perl/5.24.1-2

I found the test failures on rt.cpan.org and I applied to Ubuntu the proposed
patches:
https://rt.cpan.org/Public/Bug/Display.html?id=120134
https://rt.cpan.org/Public/Bug/Display.html?id=119762


I don't know why they are not applied upstream, nor if they are really backward
compatible with the older zlib


please consider applying them or upstreaming them (in case cpan is not the 
upstream
place).

thanks!

Gianfranco