the fix looks good to me. -- Igor
> On Jan 26, 2018, at 3:25 PM, Andrey Nazarov <andrey.x.naza...@oracle.com> > wrote: > > Added test from duplicated issue > http://cr.openjdk.java.net/~anazarov/JDK-8179294/webrev.01/ > <http://cr.openjdk.java.net/~anazarov/JDK-8179294/webrev.01/> > > —Andrei > >> On 26 Jan 2018, at 15:17, Andrey Nazarov <andrey.x.naza...@oracle.com> wrote: >> >> Hi, >> >> Please review fix which adds absent module dependencies in Pack200 and Jlink >> tests. >> >> http://cr.openjdk.java.net/~anazarov/JDK-8179294/webrev.00 >> https://bugs.openjdk.java.net/browse/JDK-8179294 >> <https://bugs.openjdk.java.net/browse/JDK-8179294> >> >> >> —Thanks, >> Andrei >
