Merge ~vpa1977/ubuntu/+source/libsvm:java21-ftbfs into ubuntu/+source/libsvm:ubuntu/devel

Proposed by Vladimir Petko
Status: Merged
Merged at revision: 71d061f6abd4e80c0421ac0969c8773f3c38dc40
Proposed branch: ~vpa1977/ubuntu/+source/libsvm:java21-ftbfs
Merge into: ubuntu/+source/libsvm:ubuntu/devel
Diff against target: 86 lines (+28/-3)
4 files modified
debian/changelog (+15/-0)
debian/control (+3/-1)
debian/patches/target-source.patch (+8/-1)
debian/rules (+2/-1)
Reviewer Review Type Date Requested Status
Vladimir Petko (community) Abstain
Zixing Liu (community) Approve
Pushkar Kulkarni (community) Approve
git-ubuntu import Pending
Review via email: mp+460734@code.launchpad.net

Description of the change

This MP resolves Java 21 ftbfs.

PPA: ppa:vpa1977/gettext [1]

Changes:
 - Use java_compat_level variable provided by java-common to adjust -source/-target level to the minimum required by the default Java

Testing:
 - rebuild in default Java 21 ppa[2]
 - piuparts test [3]
 - autopkgtest in Java 17[4] and Java 21[5] ppa

[1] https://launchpad.net/~vpa1977/+archive/ubuntu/gettext
[2] https://launchpad.net/~vpa1977/+archive/ubuntu/gettext-21/+sourcepub/15798730/+listing-archive-extra
[3] https://bugs.launchpad.net/ubuntu/+source/libsvm/+bug/2054242/comments/1
[4] https://autopkgtest.ubuntu.com/results/autopkgtest-noble-vpa1977-gettext/noble/amd64/libs/libsvm/20240219_010026_7f556@/log.gz
[5] https://autopkgtest.ubuntu.com/results/autopkgtest-noble-vpa1977-gettext-21/noble/amd64/libs/libsvm/20240219_010103_ecbb4@/log.gz

To post a comment you must log in.
Revision history for this message
Pushkar Kulkarni (pushkarnk) :
review: Approve
Revision history for this message
Vladimir Petko (vpa1977) wrote :

Uploaded, thank you

Revision history for this message
Vladimir Petko (vpa1977) wrote :

Build failed for non-amd64

review: Needs Fixing
Revision history for this message
Vladimir Petko (vpa1977) wrote :
Revision history for this message
Zixing Liu (liushuyu-011) :
Revision history for this message
Vladimir Petko (vpa1977) wrote :
Revision history for this message
Zixing Liu (liushuyu-011) wrote :

LGTM

review: Approve
Revision history for this message
Vladimir Petko (vpa1977) :
review: Abstain
Revision history for this message
Vladimir Petko (vpa1977) wrote :

Uploaded, thank you!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index 2d4242e..77179de 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,18 @@
1libsvm (3.24+ds-6ubuntu2) noble; urgency=medium
2
3 * d/control: add java-common to Build-Depends to resolve build
4 failure on platforms other than amd64.
5
6 -- Vladimir Petko <vladimir.petko@canonical.com> Thu, 22 Feb 2024 16:33:36 +1300
7
8libsvm (3.24+ds-6ubuntu1) noble; urgency=medium
9
10 * d/rules, d/target-source.patch: use java_compat_level variable
11 provided by java-common to adjust -source/-target level to the
12 minimum required by the default Java (LP: #2054242).
13
14 -- Vladimir Petko <vladimir.petko@canonical.com> Mon, 19 Feb 2024 10:34:10 +1300
15
1libsvm (3.24+ds-6) unstable; urgency=medium16libsvm (3.24+ds-6) unstable; urgency=medium
217
3 * d/test/control: Add python3-all to python-simple's Depends:18 * d/test/control: Add python3-all to python-simple's Depends:
diff --git a/debian/control b/debian/control
index 5361a28..750fff8 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: libsvm1Source: libsvm
2Section: devel2Section: devel
3Priority: optional3Priority: optional
4Maintainer: Debian Science Maintainers <debian-science-maintainers@lists.alioth.debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Debian Science Maintainers <debian-science-maintainers@lists.alioth.debian.org>
5Uploaders:6Uploaders:
6 Chen-Tse Tsai <ctse.tsai@gmail.com>,7 Chen-Tse Tsai <ctse.tsai@gmail.com>,
7 Christian Kastner <ckk@debian.org>,8 Christian Kastner <ckk@debian.org>,
@@ -11,6 +12,7 @@ Build-Depends:
11 libtool,12 libtool,
12 python3-all:native,13 python3-all:native,
13 dh-python,14 dh-python,
15 java-common,
14Build-Depends-Indep:16Build-Depends-Indep:
15 default-jdk17 default-jdk
16Rules-Requires-Root: no18Rules-Requires-Root: no
diff --git a/debian/patches/target-source.patch b/debian/patches/target-source.patch
index 302d64f..abdaa42 100644
--- a/debian/patches/target-source.patch
+++ b/debian/patches/target-source.patch
@@ -8,7 +8,14 @@ Bug-Debian: https://bugs.debian.org/893291
88
9--- a/java/Makefile9--- a/java/Makefile
10+++ b/java/Makefile10+++ b/java/Makefile
11@@ -11,7 +11,7 @@ JAVAC = javac11@@ -5,13 +5,13 @@
12 svm_train.class svm_predict.class svm_toy.class svm_scale.class
13
14 #JAVAC = jikes
15-JAVAC_FLAGS = -target 1.7 -source 1.7
16+JAVAC_FLAGS = -target $(JAVA_COMPAT_LEVEL) -source $(JAVA_COMPAT_LEVEL)
17 JAVAC = javac
18 # JAVAC_FLAGS =
12 export CLASSPATH := .:$(CLASSPATH)19 export CLASSPATH := .:$(CLASSPATH)
13 20
14 all: $(FILES)21 all: $(FILES)
diff --git a/debian/rules b/debian/rules
index f18e5ad..26a7c4b 100755
--- a/debian/rules
+++ b/debian/rules
@@ -3,6 +3,7 @@
33
4# Uncomment this to turn on verbose mode.4# Uncomment this to turn on verbose mode.
5#export DH_VERBOSE=15#export DH_VERBOSE=1
6include /usr/share/java/java_defaults.mk
67
7# For DEB_VERSION_UPSTREAM8# For DEB_VERSION_UPSTREAM
8include /usr/share/dpkg/pkg-info.mk9include /usr/share/dpkg/pkg-info.mk
@@ -53,7 +54,7 @@ override_dh_auto_build-arch:
5354
54override_dh_auto_build-indep:55override_dh_auto_build-indep:
55 dh_auto_build -i56 dh_auto_build -i
56 $(MAKE) -C java57 $(MAKE) JAVA_COMPAT_LEVEL=$(java_compat_level) -C java
5758
58override_dh_auto_install:59override_dh_auto_install:
59 dh_auto_install -- libdir=/usr/lib/$(DEB_HOST_MULTIARCH)/60 dh_auto_install -- libdir=/usr/lib/$(DEB_HOST_MULTIARCH)/

Subscribers

People subscribed via source and target branches