Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.bugs.rc > #283219

Bug#933031: pristine-tar: unable to unpack some deltas of version 2

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Control: tags -1 +patch

On Tue, Jul 28, 2020 at 04:17:08PM +0100, Steve McIntyre wrote:
>Control: severity -1 grave

...

>IMHO this has to be a grave bug - without reimporting this repo we
>can't get our older revisions back. Then I'm worried that this will
>break things if we need to go back and rebuild older versions (using
>older tooling). pristine-tar needs to be safe for data here. :-(
>
>I've done a git bisect of the pristine-tar repo to see where things
>broke, and that clearly tells me:
>
>...
>f287f10b48c3ca5a30db13dbf3ba918c47a943bb is the first bad commit
>commit f287f10b48c3ca5a30db13dbf3ba918c47a943bb
>Author: Tomasz Buchert <tomasz@debian.org>
>Date:   Fri Jan 13 23:28:04 2017 +0100
>
>    fix #851286
>
>:100755 100755 1a34e953ba363ae1e284b97f83f63f6ef80ea544 3ba76f79d376d011451b8c572b20e185b715bed1 M      pristine-tar
>:040000 040000 84bc809be6b4dcf81cfce71f0550858957a3e92f f8eb72beb458cbd9770f88f0f4efc00f0c75a299 M      test
>bisect run success
>...
>
>I hope that helps the maintainers - shout if you'd like more help debugging.

I've carried on and looked at the code. The attached patch makes
things work for me, at the expense of YA thing added to the @try
loop. But I'll take that over failure!

Having looked at this code a little, I'm now also a little concerned
about the way the fallbacks work, i.e. applied one at a time
independently. I'm thinking that there could well be cases where a
stored tarball might need more than one of the workaround/retry
options. Maybe I'm over-thinking it.

Also submitted this as an MR:

  https://salsa.debian.org/debian/pristine-tar/-/merge_requests/7

in case you'd prefer that.

-- 
Steve McIntyre, Cambridge, UK.                                steve@einval.com
< sladen> I actually stayed in a hotel and arrived to find a post-it
          note stuck to the mini-bar saying "Paul: This fridge and
          fittings are the correct way around and do not need altering"

Back to linux.debian.bugs.rc | Previous | Next | Find similar | Unroll thread

Thread

Bug#933031: pristine-tar: unable to unpack some deltas of version 2 Steve McIntyre <steve@einval.com> - 2020-07-29 13:10 +0200

csiph-web