Skip Menu |

This queue is for tickets about the Compress-Raw-Lzma CPAN distribution.

Report information
The Basics
Id: 57080
Status: resolved
Priority: 0/
Queue: Compress-Raw-Lzma

People
Owner: Nobody in particular
Requestors: paul [...] city-fan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 2.027
Fixed in: 2.028



Subject: [PATCH] "persistent" flag removed from XZ API
The module fails to build with current upstream xz as they removed the "persistent" flag in commit 418d64a32e8144210f98a810738fed5a897e8367 on 14th November 2009. There is a new function lzma_filters_update() that replaces it apparently. Attached patch just removes references to the persistent flag. The test suite still passes, as does the IO::Compress::Lzma one.
Subject: Compress-Raw-Lzma-2.027-persistent.patch
Upstream xz removed the "persistent" flag in commit 418d64a32e8144210f98a810738fed5a897e8367 on 14th November 2009. There is a new function lzma_filters_update() that replaces it, but its absence doesn't break the test suite so far. --- Compress-Raw-Lzma-2.027/Lzma.xs.orig 2010-01-08 11:33:52.000000000 +0000 +++ Compress-Raw-Lzma-2.027/Lzma.xs 2010-04-30 00:04:06.527163321 +0100 @@ -93,7 +93,6 @@ (options)->lc = LZMA_LC_DEFAULT; \ (options)->lp = LZMA_LP_DEFAULT; \ (options)->pb = LZMA_PB_DEFAULT; \ - (options)->persistent = FALSE; \ (options)->mode = LZMA_MODE_NORMAL; \ (options)->nice_len = 64; \ (options)->mf = LZMA_MF_BT4; \ @@ -1261,7 +1260,6 @@ p->lc = lc ; p->lp = lp ; p->pb = pb ; - p->persistent = persistent ; p->mode = mode ; p->nice_len = nice_len ; p->mf = mf ;
On Fri Apr 30 09:07:13 2010, paul@city-fan.org wrote: Show quoted text
> The module fails to build with current upstream xz as they removed the > "persistent" flag in commit 418d64a32e8144210f98a810738fed5a897e8367 > on 14th November 2009. There is a new function lzma_filters_update() > that replaces it apparently. > > Attached patch just removes references to the persistent flag. The test > suite still passes, as does the IO::Compress::Lzma one.
Thanks for bringing that to my attention. I'll get a new version onto CPAN as soon as I can. cheers Paul
The fix for this is included in version 2.028, whicj I've just uploaded to CPAN.
From: paul [...] city-fan.org
On Mon May 03 05:29:30 2010, PMQS wrote: Show quoted text
> The fix for this is included in version 2.028, whicj I've just uploaded > to CPAN.
Works for me; thanks.