We discovered that the last webrev subtly changed the behavior of jar tool with respect to the JDK 8 jar tool, so that was fixed, along with some more simplification, and additional test cases were added to demonstrate consistent behavior across releases. Here is the newest webrev.
http://cr.openjdk.java.net/~sdrach/8165944/webrev.06/ <http://cr.openjdk.java.net/~sdrach/8165944/webrev.06/> > On Sep 27, 2016, at 12:31 PM, Steve Drach <steve.dr...@oracle.com> wrote: > > After a discussion with Paul Sandoz, I’ve simplified and, hopefully, thus > clarified the changeset. The new webrev is > > http://cr.openjdk.java.net/~sdrach/8165944/webrev.01/ > <http://cr.openjdk.java.net/~sdrach/8165944/webrev.01/> > >> On Sep 26, 2016, at 12:31 PM, Steve Drach <steve.dr...@oracle.com >> <mailto:steve.dr...@oracle.com>> wrote: >> >> Hi, >> >> Please review these changes to the jar tool to fix a capability regression I >> introduced in an earlier revision. The issue is that this >> >> $ jar -cf test.jar -C test1 . -C test2 . >> >> only puts the files under test1 in the jar and ignores the files under >> test2. The DoubleCs test verified the problem and the solution. >> >> issue: https://bugs.openjdk.java.net/browse/JDK-8165944 >> <https://bugs.openjdk.java.net/browse/JDK-8165944> >> <https://bugs.openjdk.java.net/browse/JDK-8165944 >> <https://bugs.openjdk.java.net/browse/JDK-8165944>> >> webrev: http://cr.openjdk.java.net/~sdrach/8165944/webrev.00/ >> <http://cr.openjdk.java.net/~sdrach/8165944/webrev.00/> >> <http://cr.openjdk.java.net/~sdrach/8165944/webrev.00/ >> <http://cr.openjdk.java.net/~sdrach/8165944/webrev.00/>> >> >> Thanks, >> Steve >> >
