Re: Patch: add support for -i in fastjar

[Dalibor Topic <robilad at kaffe dot org>]

Tom Tromey <tromey <at> redhat.com> writes:

> 
> >>>>> "Olivier" == Olivier Jolly <olivier.jolly <at> pcedev.com> writes:
> 
> Olivier> here is a patch submition which enables fastjar to update an
> Olivier> existing .jar with an META-INF/INDEX.LIST aka -i
> Olivier> support. Still, it doesn't process the MANIFEST.MF to index
> Olivier> files coming from dependant archives.
> 
> There's now a new fastjar project on savannah.  If you make an
> account there, I will give you cvs write access and you can check in
> the patch there.  Thanks!

I've done it now for a slightly modified version, with the following 
ChangeLog:

2007-02-04  Dalibor Topic  <robilad@kaffe.org>

        * jartool.c (build_index): Initialize
        a ziplistentry's next_element with NULL
        before passing it to add_list_entry to
        fix a crash. Don't re-assign filename
        of a zip entry with the index name, as
        that turns directories into files, and
        breaks the zip files.

2007-02-04  Olivier Jolly <olivier.jolly@pcedev.com>

        * jartool.c: Enable fastjar to update an
        existing .jar with an META-INF/INDEX.LIST
        aka -i support. Still, it doesn't process
        the MANIFEST.MF to index files coming from
        dependant archives.

cheers,
dalibor topic