On 26/01/2012 10:37 PM, Rickard Bäckman wrote:
Karen,
that makes sense, I'll push to 8 as well.
Recall that I plan to change all this by moving it into make/closed for
8 (and then 7u). So this could be cleaned up then.
David
-----
Thanks
/R
On 01/26/2012 01:20 PM, Karen Kinnear wrote:
Thanks Rickard.
My guess is that if it works with all version of jar which you've
tested, then it will
be easier going forward to maintain the file consistently, and it
causes no harm.
thanks,
Karen
On Jan 26, 2012, at 7:03 AM, Rickard Bäckman wrote:
On 01/26/2012 12:53 PM, Karen Kinnear wrote:
Code change looks good.
I assume you want to put this fix back to JDK8 as well.
thank you for the quick fix!
Karen
Thanks for the review Karen,
I'm not sure about JDK8, I think it build with JDK1.7.0? Which has a
fixed version of jar.
I'll try to find out if we need to fix it in 8 as well.
/R
On Jan 26, 2012, at 4:44 AM, Rickard Bäckman wrote:
Hi,
We have a problem with some versions of jar reporting errors when
trying to run jar cf<some.jar> com/test com/test/foo
This fix removes the redundant subdirectories from the command.
Webrev is here:
http://cr.openjdk.java.net/~rbackman/7133124/webrev/
Thanks
/Rickard
