Hi, Still no tests when directory where files are extracted is not empty. It very common case.
—Andrey > On 8 Dec 2016, at 16:09, Chris Hegarty <chris.hega...@oracle.com> wrote: > > >> On 8 Dec 2016, at 00:44, Mandy Chung <mandy.ch...@oracle.com> wrote: >> >> >>> On Dec 6, 2016, at 2:46 AM, Chris Hegarty <chris.hega...@oracle.com> wrote: >>> >>> This change adds a basic option to the jmod tool to extract all its >>> contents to >>> the current working directory, 8166568 [1]. Additionally, there is a bug >>> fix for >>> a public mutable static, 8169492 [2]. >>> >>> http://cr.openjdk.java.net/~chegar/8166568_8169492.00/ >> >> >> Looks good. I agree with Alan and it’d be good to take the destination >> directory to which the contents are written. FYI. jimage extract command >> takes —-dir option. > > Updated webrev contain a target destination dir: > http://cr.openjdk.java.net/~chegar/8166568_8169492.01/ > > -Chris >
