Merge ~vpa1977/ubuntu/+source/openjdk-17:lp2003820-native-debug into ubuntu/+source/openjdk-17:ubuntu/devel

Proposed by Vladimir Petko
Status: Merged
Merged at revision: be71f7cb953b1f258f91b90bf7a6b2ca38fcc37e
Proposed branch: ~vpa1977/ubuntu/+source/openjdk-17:lp2003820-native-debug
Merge into: ubuntu/+source/openjdk-17:ubuntu/devel
Prerequisite: ~vpa1977/ubuntu/+source/openjdk-17:lp2011749-repack
Diff against target: 30 lines (+3/-2)
2 files modified
debian/changelog (+2/-1)
debian/rules (+1/-1)
Reviewer Review Type Date Requested Status
Steve Langasek (community) Approve
git-ubuntu import Pending
Review via email: mp+439104@code.launchpad.net

Description of the change

This is a final merge proposal of the series [1][2][3]

Summary of changes:
 * d/rules: use --with-debug-symbols=none (LP: #2003820).

This is required for building chiselled images as embedded debug symbols both bloat JDK image to ~0.5G and are unnecessary.

PPA: ppa:vpa1977/openjdk-drop-patches [4]

Steps to test:

sudo apt install openjdk-17-jdk openjdk-17-dbg
echo "public class infinite { public static void main(String[] args) { while (true){} } }" > infinite.java
/usr/lib/jvm/java-17-openjdk-amd64/bin/javac infinite.java
/usr/lib/jvm/java-17-openjdk-amd64/bin/java -cp . infinite&
[1] 6100
sudo /usr/lib/jvm/java-17-openjdk-amd64/bin/jhsdb jmap --heap --pid 6100
-----cut------------------------------------------------------
Attaching to process ID 6100, please wait...
Debugger attached successfully.
Server compiler detected.
JVM version is 17.0.6+10-Ubuntu-1ubuntu2ppa1

using thread-local object allocation.
Garbage-First (G1) GC with 2 thread(s)

Heap Configuration:
   MinHeapFreeRatio = 40
   MaxHeapFreeRatio = 70
   MaxHeapSize = 1029701632 (982.0MB)
   NewSize = 1363144 (1.2999954223632812MB)
   MaxNewSize = 617611264 (589.0MB)
   OldSize = 5452592 (5.1999969482421875MB)
   NewRatio = 2

-----------cut----------------------------------------------------

[1] https://code.launchpad.net/~vpa1977/ubuntu/+source/openjdk-17/+git/openjdk-17/+merge/439093
[2] https://code.launchpad.net/~vpa1977/ubuntu/+source/openjdk-17/+git/openjdk-17/+merge/439105
[3] https://code.launchpad.net/~vpa1977/ubuntu/+source/openjdk-17/+git/openjdk-17/+merge/439104
[4] https://launchpad.net/~vpa1977/+archive/ubuntu/openjdk-drop-patches

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) :
review: Approve

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 fd2d0d9..dbb07dd 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,10 +1,11 @@
6-openjdk-17 (17.0.6+10-1ubuntu2) UNRELEASED; urgency=medium
7+openjdk-17 (17.0.6+10-1ubuntu2) lunar; urgency=medium
8
9 * d/p: drop obsolete patches (LP: #2011653).
10 - workaround_expand_exec_shield_cs_limit.diff: obsoleted by
11 hotspot-disable-exec-shield-workaround.diff.
12 - generated-headers.patch: include is already added by openjdk makefile.
13 * d/copyright, d/watch: implement uscan repackaging (LP: #2011749).
14+ * d/rules: use --with-debug-symbols=none (LP: #2003820).
15
16 -- Vladimir Petko <vladimir.petko@canonical.com> Thu, 16 Mar 2023 15:04:36 +1300
17
18diff --git a/debian/rules b/debian/rules
19index bd64964..be080b5 100755
20--- a/debian/rules
21+++ b/debian/rules
22@@ -514,7 +514,7 @@ COMMON_CONFIGURE_ARGS += \
23
24 COMMON_CONFIGURE_ARGS += \
25 --with-debug-level=release \
26- --with-native-debug-symbols=internal \
27+ --with-native-debug-symbols=none \
28 --enable-unlimited-crypto \
29 --with-zlib=system \
30 --with-giflib=system \

Subscribers

People subscribed via source and target branches