Merge ~vpa1977/ubuntu/+source/bazel-bootstrap:ubuntu/devel into ubuntu/+source/bazel-bootstrap:ubuntu/devel

Proposed by Vladimir Petko
Status: Merged
Merge reported by: Vladimir Petko
Merged at revision: 5bccf3802a96c1d0caec8c20f5dc595191bcc60c
Proposed branch: ~vpa1977/ubuntu/+source/bazel-bootstrap:ubuntu/devel
Merge into: ubuntu/+source/bazel-bootstrap:ubuntu/devel
Diff against target: 74 lines (+40/-1)
4 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/handle_synthetic_method_parameters_entries_that_dont_have_names.patch (+30/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Canonical Foundations Team Pending
Ubuntu Sponsors Pending
git-ubuntu import Pending
Review via email: mp+460544@code.launchpad.net

Description of the change

This MP resolves Java 21 ftbfs.

PPA: ppa:vpa1977/gettext [1]

Changes:
 - Cherry-pick upstream patch to resolve Java 21 ftbfs

Testing:
 - rebuild in default Java 21 ppa[2]
 - piuparts test [3]

[1] https://launchpad.net/~vpa1977/+archive/ubuntu/gettext
[2] https://launchpad.net/~vpa1977/+archive/ubuntu/gettext-21/+sourcepub/15788140/+listing-archive-extra
[3] https://bugs.launchpad.net/ubuntu/+source/bazel-bootstrap/+bug/2053202/comments/1

To post a comment you must log in.

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 04ce192..153fcd9 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
1bazel-bootstrap (4.2.3+ds-9ubuntu1) noble; urgency=medium
2
3 * d/p/handle_synthetic_method_parameters_that_dont_have_names.patch:
4 apply upstream patch to resolve Java 21 ftbfs (LP: #2053202).
5
6 -- Vladimir Petko <vladimir.petko@canonical.com> Fri, 05 Jan 2024 15:57:39 +1300
7
1bazel-bootstrap (4.2.3+ds-9) unstable; urgency=high8bazel-bootstrap (4.2.3+ds-9) unstable; urgency=high
29
3 [ Markus Koschany ]10 [ Markus Koschany ]
diff --git a/debian/control b/debian/control
index ef19ccf..a6f2767 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: bazel-bootstrap1Source: bazel-bootstrap
2Section: devel2Section: devel
3Priority: optional3Priority: optional
4Maintainer: Debian Bazel Team <bazel-team@lists.launchpad.net>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Debian Bazel Team <bazel-team@lists.launchpad.net>
5Uploaders:6Uploaders:
6 Debian Bazel Team <team+bazel@tracker.debian.org>,7 Debian Bazel Team <team+bazel@tracker.debian.org>,
7 Olek Wojnar <olek@debian.org>8 Olek Wojnar <olek@debian.org>
diff --git a/debian/patches/handle_synthetic_method_parameters_entries_that_dont_have_names.patch b/debian/patches/handle_synthetic_method_parameters_entries_that_dont_have_names.patch
8new file mode 1006449new file mode 100644
index 0000000..199d9e1
--- /dev/null
+++ b/debian/patches/handle_synthetic_method_parameters_entries_that_dont_have_names.patch
@@ -0,0 +1,30 @@
1Description: Handle synthetic method parameters entries that don't have names
2 > If the value of the name_index item is zero, then this parameters element indicates a formal parameter with no name.
3 https://docs.oracle.com/javase/specs/jvms/se20/html/jvms-4.html#jvms-4.7.24
4Author: Liam Miller-Cushon <cushon@google.com>
5Origin: upstream, https://github.com/bazelbuild/bazel/commit/3954a18fa6b0e3d1a1005cc3409ebc95f6adf3af
6Forwarded: not-needed
7Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1057490
8Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/bazel-bootstrap/+bug/2053202
9Last-Update: 2024-01-05
10--- a/third_party/ijar/classfile.cc
11+++ b/third_party/ijar/classfile.cc
12@@ -1168,7 +1168,8 @@
13 u1 parameters_count = get_u1(p);
14 for (int ii = 0; ii < parameters_count; ++ii) {
15 MethodParameter* parameter = new MethodParameter;
16- parameter->name_ = constant(get_u2be(p));
17+ int name_id = get_u2be(p);
18+ parameter->name_ = name_id == 0 ? NULL : constant(name_id);
19 parameter->access_flags_ = get_u2be(p);
20 attr->parameters_.push_back(parameter);
21 }
22@@ -1180,7 +1181,7 @@
23 u1 *payload_start = p - 4;
24 put_u1(p, parameters_.size());
25 for (MethodParameter* parameter : parameters_) {
26- put_u2be(p, parameter->name_->slot());
27+ put_u2be(p, parameter->name_ == NULL ? 0 : parameter->name_->slot());
28 put_u2be(p, parameter->access_flags_);
29 }
30 put_u4be(payload_start, p - 4 - payload_start); // backpatch length
diff --git a/debian/patches/series b/debian/patches/series
index 2ca3fef..6a3cef0 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -31,3 +31,4 @@ grpc-absl_synchronization.patch
31exclude_build_data.patch31exclude_build_data.patch
32fix-install_base_key-generation.patch32fix-install_base_key-generation.patch
33javax.annotations.patch33javax.annotations.patch
34handle_synthetic_method_parameters_entries_that_dont_have_names.patch

Subscribers

People subscribed via source and target branches