[Bug 3166] invalid zip timestamp handling leads to error building bootstrap-javac

[bugzilla-daemon at icedtea.classpath.org]

http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=3166

--- Comment #5 from Andrew John Hughes <gnu.andrew at redhat.com> ---
(In reply to Ricardo Ribalda from comment #4)
> >As I said, I agree and it will be fixed in the next release.
> >
> >But you do need to look at why you're using such an old unsupported version in a >broken environment.
> 
> 
> I think that I found what broke down my environment. 
> 
> glibj.zip (rt.jar) is build from classpath0.99 this way:
> 
> /var/lib/jenkins/workspace/qt5022-open-cesium/build/tmp/sysroots/x86_64-
> linux/usr/bin/fastjar cf glibj.zip gnu java javax org sun META-INF && 
> 
> fastjar generates the timestamps using time(NULL), with returns the time in
> UTC:
> 
> https://github.com/FauxFaux/fastjar/blob/master/jartool.c#L2444
> 
> while unzip/zip uses the TZ variable.
> 
> Problem solved (at least located) :)
> 

Ah, looks like it hasn't had a release since 2009:

http://download.savannah.gnu.org/releases/fastjar/

Is that part of the GNU Classpath build or a manual invocation? Given it's not
generating a manifest, zip would just as well there.

> 
> Regarding the old version: it is what is provided by meta-java :(. Updating
> the whole layer and testing it with every architecture is way beyond the
> time I have allocated for this task. But when I am done with the deliverable
> I would love to revisit this issue.

I just wonder what you plan to do with the result, because a version that old
is going to be full of known security issues.

> 
> Andrew, do you need the patch in another format? main/git send-mail.... Or
> you take care from here?
> 

It's fine as is. I'll include it (with credit) in the next release.

> Thanks for your help!

No problem.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20160913/962f40d0/attachment.html>