Matthias,
There is
jdk.incubator.jpackage.internal.LinuxRpmBundler.TOOL_RPMBUILD_MIN_VERSION
constant that is currently set to "4.0". Feel free to file a CR and bump
it up to "4.10"
- Alexey
On 2/19/2020 3:05 AM, Baesken, Matthias wrote:
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
