306463 - make an 'unwrap_full_update.sh' script to unmar and unbzip2 a complete patch .mar file

Status: RESOLVED FIXED

(Toolkit :: Application Update, defect)

Description

[Chase Phillips]

Let's make an unwrap_full_update.sh script in tools/update-packaging that will
unmar and unbzip2 a .mar (reversing the process that make_full_update.sh) does.
 This will make creating partial patches easier since they are based on complete
patches.

Comment 1

[Darin Fisher]

Attachment: v1 patch [checked in]

Comment 2

[Chase Phillips]

Comment on attachment 194363 [details] [diff] [review]
v1 patch [checked in]

This is a great first cut.  Can you land it on the trunk so I can use it in my
patch packaging system?

The only piece not in this version is the optional specification of a directory
into which the mar would be extracted.	But that can be handled in other ways
and it's not a good reason to hold from getting this in.

Comment 3

[Chase Phillips]

Comment on attachment 194363 [details] [diff] [review]
v1 patch [checked in]

Darin, I checked this in on the trunk on your behalf so I could get unblocked
for creating partial patches.  I hope that's okay.  Thanks again for the patch.

RCS file: /cvsroot/mozilla/tools/update-packaging/unwrap_full_update.sh,v
done
Checking in unwrap_full_update.sh;
/cvsroot/mozilla/tools/update-packaging/unwrap_full_update.sh,v  <-- 
unwrap_full_update.sh
initial revision: 1.1
done

Comment 4

[Darin Fisher]

OK, thanks chase.  I imagined using the -C argument to the mar executable to
specify where to extract the MAR file.  I may do that as a follow-up patch.

Comment 5

[Chase Phillips]

(In reply to comment #4)
> OK, thanks chase.  I imagined using the -C argument to the mar executable to
> specify where to extract the MAR file.  I may do that as a follow-up patch.

Oddest thing.. I couldn't get -C to work for me locally so I've just been
altering my working directory before calling mar.  I seem to recall having a
similar problem a ways back but don't have any data points on that.  I wish I
could give you more info but that's all I've got atm.

Comment 6

[Darin Fisher]

Yeah, it turns out that -C won't work with extraction unless the path to the
archive file is absolute.  I need to fix the mar executable so that the -C
argument can be used even when the MAR file is specified using a relative path.

Comment 7

[Chase Phillips]

(In reply to comment #6)
> Yeah, it turns out that -C won't work with extraction unless the path to the
> archive file is absolute.  I need to fix the mar executable so that the -C
> argument can be used even when the MAR file is specified using a relative path.

Aha!  Gotcha.

unwrap_full_update.sh works well enough that my partial patch sizes make sense
now.  Good times.