Bug#1035977: unblock: maven-debian-helper/2.6.3
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock
X-Debbugs-Cc: maven-debian-helper@packages.debian.org
Control: affects -1 + src:maven-debian-helper
Please unblock package maven-debian-helper
This update is required to fix the build failure of src:tiles with OpenJDK 17
(#1011597). The FTBFS is caused by an old Maven plugin (maven-autotag-plugin)
that doesn't run with OpenJDK 17 unless an extra JVM argument is passed
to Maven (--add-opens). There is no mechanism in maven-debian-helper do
to that though, the Maven launcher supports a MAVEN_OPTS environment variable
to set extra arguments but maven-debian-helper doesn't use it. This update
implements MAVEN_OPTS for maven-debian-helper.
Thank you,
Emmanuel Bourg
unblock maven-debian-helper/2.6.3
diff -Nru maven-debian-helper-2.6.2/debian/changelog maven-debian-helper-2.6.3/debian/changelog
--- maven-debian-helper-2.6.2/debian/changelog 2023-02-26 21:33:00.000000000 +0100
+++ maven-debian-helper-2.6.3/debian/changelog 2023-05-12 01:20:16.000000000 +0200
@@ -1,3 +1,11 @@
+maven-debian-helper (2.6.3) unstable; urgency=medium
+
+ * Support the MAVEN_OPTS environment variable to pass JVM arguments
+ (such as --add-opens/--add-exports required to run old plugins on recent
+ JDKs)
+
+ -- Emmanuel Bourg <ebourg@apache.org> Fri, 12 May 2023 01:20:16 +0200
+
maven-debian-helper (2.6.2) unstable; urgency=medium
* Team upload.
diff -Nru maven-debian-helper-2.6.2/share/perl/maven.pm maven-debian-helper-2.6.3/share/perl/maven.pm
--- maven-debian-helper-2.6.2/share/perl/maven.pm 2023-02-26 19:48:03.000000000 +0100
+++ maven-debian-helper-2.6.3/share/perl/maven.pm 2023-05-12 01:16:05.000000000 +0200
@@ -40,6 +40,9 @@
if (-e "$this->{cwd}/debian/maven.properties") {
push (@jvmopts, "-Dproperties.file.manual=$this->{cwd}/debian/maven.properties");
}
+ if (exists $ENV{MAVEN_OPTS}) {
+ push (@jvmopts, split(/ /, $ENV{MAVEN_OPTS}));
+ }
@{$this->{maven_cmd}} = ($java_home . '/bin/java',
@jvmopts,
Reply to: