Thank's for the reviews. Do you have a good central place in the existing coding to add a similar rpmbuild version check (e.g. for 4.10 or 4.11 which seem to be reasonable ) ?
Best regards, Matthias > +1 > > - Alexey > > On 2/18/2020 10:56 AM, Langer, Christoph wrote: > > Hi Matthias, > > > > Looks good to me now. > > > > Cheers > > Christoph > > > >> -----Original Message----- > >> From: Baesken, Matthias <matthias.baes...@sap.com> > >> Sent: Dienstag, 18. Februar 2020 16:55 > >> To: Langer, Christoph <christoph.lan...@sap.com>; core-libs- > >> d...@openjdk.java.net; Alexey Semenyuk > <alexey.semen...@oracle.com> > >> Subject: RE: 8238953: tools/jpackage tests do not work on Ubuntu Linux > >> > >> Ok why not, new webrev : > >> > >> http://cr.openjdk.java.net/~mbaesken/webrevs/8238953.2/ > >> > >> Thanks, Matthias > >> > >> > >> > >>> Hi Matthias, > >>> > >>> you could improve the patch a bit by starting TKit:: isUbuntu() with > >>> > >>> if (!isLinux()) { > >>> return false; > >>> } > >>> > >>> And then, in PackageType, you could simply do: > >>> > >>> private final static Set<String> DISABLED_PACKAGERS = > >> Optional.ofNullable( > >>> TKit.tokenizeConfigProperty("disabledPackagers")).orElse( > >>> TKit.isUbuntu() ? Set.of("rpm") : > >>> Collections.emptySet()); > >>> > >>> Best regards > >>> Christoph > >>> > >>>> -----Original Message----- > >>>> From: core-libs-dev <core-libs-dev-boun...@openjdk.java.net> On > >> Behalf > >>>> Of Baesken, Matthias > >>>> Sent: Dienstag, 18. Februar 2020 09:14 > >>>> To: core-libs-dev@openjdk.java.net; Alexey Semenyuk > >>>> <alexey.semen...@oracle.com> > >>>> Subject: [CAUTION] RE: 8238953: tools/jpackage tests do not work on > >>>> Ubuntu Linux > >>>> > >>>> Ping ... are you fine with the latest version ? > >>>> > >>>> Best Regards, Matthias > >>>> > >>>>> Hi Alexey , I like your idea to do the handling in > >>>>> test/jdk/tools/jpackage/helpers/jdk/jpackage/test/PackageType.java > . > >>>>> > >>>>> New webrev : > >>>>> > >>>>> http://cr.openjdk.java.net/~mbaesken/webrevs/8238953.1/ > >>>>> > >>>>> > >>>>> > >>>>> Best regards, Matthias > >>>>> > >>>>>