Bug#686502: pxz produces archives broken for busybox's unxz

2013-01-07 Thread Abou Al Montacir
On Sun, 2013-01-06 at 23:32 +0100, Bastian Blank wrote: > On Thu, Dec 27, 2012 at 10:08:07PM +0100, Abou Al Montacir wrote: > > + if (r == XZ_STREAM_END) { > > + /* Eat padding. Stream never starts with zeros, and > >

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-27 Thread Abou Al Montacir
On Thu, 2012-12-27 at 08:38 -0800, Jonathan Nieder wrote: > Abou Al Montacir wrote: > > > Hover, I assume we can save this extra code as soon as we don't loose > > data. > > That's fine with me. All you'd need to do is error out if there is > anything a

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-27 Thread Abou Al Montacir
On Mon, 2012-12-24 at 17:11 -0800, Jonathan Nieder wrote: > Abou Al Montacir wrote: > > On Sat, 2012-12-22 at 10:21 -0800, Jonathan Nieder wrote: > > >> What happens if a stream ends at a buffer boundary, followed by > >> padding? Or if padding doesn't fit

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-24 Thread Abou Al Montacir
urgency=low + + * Fix decompression of multi stream XZ compressed files +(Closes: Bug#bug#686502) + + -- Abou Al Montacir Thu, 21 Dec 2012 22:00:00 +0100 + busybox (1:1.20.0-7) unstable; urgency=low * set CONFIG_FEATURE_COPYBUF_KB from 4 to 64 for all flavours. This diff -Nru busybox-1.20.0/

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-24 Thread Abou Al Montacir
On Sat, 2012-12-22 at 10:21 -0800, Jonathan Nieder wrote: > What happens if a stream ends at a buffer boundary, followed by > padding? Or if padding doesn't fit in the buffer, for that > matter? That make very low probability but could happe indeed. I will upload a new patch which fixes this case

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-21 Thread Abou Al Montacir
On Fri, 2012-12-21 at 14:34 +0100, Abou Al Montacir wrote: > > Add stream padding as specified in the spec. > I'll provide a new patch for eating zeros and fixing issue pointed by Michael Please find attached new patch handling padding and fixing issue highlighted by Michael, Chee

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-21 Thread Abou Al Montacir
On Fri, 2012-12-21 at 14:24 +0100, Bastian Blank wrote: > On Fri, Dec 21, 2012 at 02:06:31PM +0100, Abou Al Montacir wrote: > > On Thu, 2012-12-20 at 23:08 +0100, Bastian Blank wrote: > > > On Thu, Dec 20, 2012 at 10:42:41PM +0100, Abou Al Montacir wrote: > > > > Ca

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-21 Thread Abou Al Montacir
On Fri, 2012-12-21 at 17:13 +0400, Michael Tokarev wrote: > 21.12.2012 17:06, Abou Al Montacir wrote: > > On Thu, 2012-12-20 at 23:08 +0100, Bastian Blank wrote: > >> On Thu, Dec 20, 2012 at 10:42:41PM +0100, Abou Al Montacir wrote: > >>> Can you please test the att

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-21 Thread Abou Al Montacir
On Thu, 2012-12-20 at 23:08 +0100, Bastian Blank wrote: > On Thu, Dec 20, 2012 at 10:42:41PM +0100, Abou Al Montacir wrote: > > Can you please test the attached patch > > How does it implement stream padding? Hi Bastian, As it is implemented, it will iterate until end of stream

Bug#686502: pxz produces archives broken for busybox's unxz

2012-12-20 Thread Abou Al Montacir
:1.20.0-7.1) unstable; urgency=low + + * Fix decompression of multi stream XZ files + + -- Abou Al Montacir Thu, 20 Dec 2012 22:04:00 +0100 + busybox (1:1.20.0-7) unstable; urgency=low * set CONFIG_FEATURE_COPYBUF_KB from 4 to 64 for all flavours. This diff -Nru busybox-1.20.0/debian