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
1diff --git a/debian/changelog b/debian/changelog
2index 2d4242e..77179de 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,18 @@
6+libsvm (3.24+ds-6ubuntu2) noble; urgency=medium
7+
8+ * d/control: add java-common to Build-Depends to resolve build
9+ failure on platforms other than amd64.
10+
11+ -- Vladimir Petko <vladimir.petko@canonical.com> Thu, 22 Feb 2024 16:33:36 +1300
12+
13+libsvm (3.24+ds-6ubuntu1) noble; urgency=medium
14+
15+ * d/rules, d/target-source.patch: use java_compat_level variable
16+ provided by java-common to adjust -source/-target level to the
17+ minimum required by the default Java (LP: #2054242).
18+
19+ -- Vladimir Petko <vladimir.petko@canonical.com> Mon, 19 Feb 2024 10:34:10 +1300
20+
21 libsvm (3.24+ds-6) unstable; urgency=medium
22
23 * d/test/control: Add python3-all to python-simple's Depends:
24diff --git a/debian/control b/debian/control
25index 5361a28..750fff8 100644
26--- a/debian/control
27+++ b/debian/control
28@@ -1,7 +1,8 @@
29 Source: libsvm
30 Section: devel
31 Priority: optional
32-Maintainer: Debian Science Maintainers <debian-science-maintainers@lists.alioth.debian.org>
33+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
34+XSBC-Original-Maintainer: Debian Science Maintainers <debian-science-maintainers@lists.alioth.debian.org>
35 Uploaders:
36 Chen-Tse Tsai <ctse.tsai@gmail.com>,
37 Christian Kastner <ckk@debian.org>,
38@@ -11,6 +12,7 @@ Build-Depends:
39 libtool,
40 python3-all:native,
41 dh-python,
42+ java-common,
43 Build-Depends-Indep:
44 default-jdk
45 Rules-Requires-Root: no
46diff --git a/debian/patches/target-source.patch b/debian/patches/target-source.patch
47index 302d64f..abdaa42 100644
48--- a/debian/patches/target-source.patch
49+++ b/debian/patches/target-source.patch
50@@ -8,7 +8,14 @@ Bug-Debian: https://bugs.debian.org/893291
51
52 --- a/java/Makefile
53 +++ b/java/Makefile
54-@@ -11,7 +11,7 @@ JAVAC = javac
55+@@ -5,13 +5,13 @@
56+ svm_train.class svm_predict.class svm_toy.class svm_scale.class
57+
58+ #JAVAC = jikes
59+-JAVAC_FLAGS = -target 1.7 -source 1.7
60++JAVAC_FLAGS = -target $(JAVA_COMPAT_LEVEL) -source $(JAVA_COMPAT_LEVEL)
61+ JAVAC = javac
62+ # JAVAC_FLAGS =
63 export CLASSPATH := .:$(CLASSPATH)
64
65 all: $(FILES)
66diff --git a/debian/rules b/debian/rules
67index f18e5ad..26a7c4b 100755
68--- a/debian/rules
69+++ b/debian/rules
70@@ -3,6 +3,7 @@
71
72 # Uncomment this to turn on verbose mode.
73 #export DH_VERBOSE=1
74+include /usr/share/java/java_defaults.mk
75
76 # For DEB_VERSION_UPSTREAM
77 include /usr/share/dpkg/pkg-info.mk
78@@ -53,7 +54,7 @@ override_dh_auto_build-arch:
79
80 override_dh_auto_build-indep:
81 dh_auto_build -i
82- $(MAKE) -C java
83+ $(MAKE) JAVA_COMPAT_LEVEL=$(java_compat_level) -C java
84
85 override_dh_auto_install:
86 dh_auto_install -- libdir=/usr/lib/$(DEB_HOST_MULTIARCH)/

Subscribers

People subscribed via source and target branches