Merge ~vpa1977/ubuntu/+source/asm:asm-upgrade into ubuntu/+source/asm:ubuntu/devel

Proposed by Vladimir Petko
Status: Merged
Merge reported by: Vladimir Petko
Merged at revision: 27a7adbe71e3e52a610b2f1f2960170cff20cb42
Proposed branch: ~vpa1977/ubuntu/+source/asm:asm-upgrade
Merge into: ubuntu/+source/asm:ubuntu/devel
Prerequisite: ~vpa1977/ubuntu/+source/asm:new_upstream_9.6
Diff against target: 288 lines (+55/-39)
12 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/02-ignore-google-java-format-plugin.patch (+3/-4)
debian/patches/03-ignore-jmh-plugin.patch (+1/-1)
debian/patches/04-ignore-sonarqube-plugin.patch (+7/-6)
debian/patches/05-junit-compatibility.patch (+1/-1)
debian/patches/06-disable-retrofitter.patch (+1/-1)
debian/patches/07-restore-asm-all.patch (+1/-1)
debian/patches/08-ignore-sigtest-maven-plugin.patch (+7/-6)
debian/patches/09-ignore-test-coverage.patch (+6/-4)
debian/patches/11-gradle-compatibility.patch (+15/-10)
debian/patches/12-ignore-java-platform-plugin.patch (+4/-4)
Reviewer Review Type Date Requested Status
Dan Bungert Pending
git-ubuntu import Pending
Review via email: mp+459679@code.launchpad.net

Description of the change

Changes:
 - new upstream release, refresh patches.

PPA: ppa:vpa1977/gettext [1]

Testing:
 - piuparts install/purge test[2]
 - rebuild reverse build dependenices in ppa[3]
   Note: there are existing build failures in asm-java reverse build dependencies:
    - android-platform-tools-base: FTBFS: SignatureExtension.java:32: error: DEROutputStream is not public in org.bouncycastle.asn1; cannot be accessed from outside package[4]
    - tika: FTBFS cannot find symbols[5]

[1] https://launchpad.net/~vpa1977/+archive/ubuntu/gettext
[2] https://bugs.launchpad.net/ubuntu/+source/asm/+bug/2051746/comments/2
[3] https://launchpad.net/~vpa1977/+archive/ubuntu/reverse-depends
[4] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1058331
[5] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1011492

To post a comment you must log in.
Revision history for this message
Dan Bungert (dbungert) wrote :

Looks like this was already uploaded

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 403f5ce..0e67ad3 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+asm (9.6~us1-0ubuntu1) noble; urgency=medium
7+
8+ * New upstream release (LP: #2051746):
9+ - Refreshed the patches
10+
11+ -- Vladimir Petko <vladimir.petko@canonical.com> Thu, 04 Jan 2024 10:46:07 +1300
12+
13 asm (9.5-1) unstable; urgency=medium
14
15 * New upstream release
16diff --git a/debian/control b/debian/control
17index 69977ee..8e3542b 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -1,7 +1,8 @@
21 Source: asm
22 Section: java
23 Priority: optional
24-Maintainer: Debian Java Maintainers <pkg-java-maintainers@lists.alioth.debian.org>
25+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
26+XSBC-Original-Maintainer: Debian Java Maintainers <pkg-java-maintainers@lists.alioth.debian.org>
27 Uploaders:
28 Aditya Vaidya <kroq.gar78@gmail.com>,
29 Emmanuel Bourg <ebourg@apache.org>
30diff --git a/debian/patches/02-ignore-google-java-format-plugin.patch b/debian/patches/02-ignore-google-java-format-plugin.patch
31index fd9f1b5..ba8ffb8 100644
32--- a/debian/patches/02-ignore-google-java-format-plugin.patch
33+++ b/debian/patches/02-ignore-google-java-format-plugin.patch
34@@ -9,16 +9,15 @@ Forwarded: not-needed
35
36 -plugins { id 'com.github.sherter.google-java-format' version '0.9' apply false }
37 plugins { id 'me.champeau.jmh' version '0.6.8' apply false }
38- plugins { id 'org.sonarqube' version '3.5.0.2730' apply false }
39+ plugins { id 'org.sonarqube' version '4.3.1.3277' apply false }
40
41-@@ -181,10 +180,6 @@
42+@@ -181,9 +180,6 @@
43 // All projects are checked with googleJavaFormat, Checkstyle and PMD,
44 // and tested with :asm-test and JUnit.
45 subprojects {
46 - apply plugin: 'com.github.sherter.google-java-format'
47 - googleJavaFormat.toolVersion = '1.15.0'
48 - googleJavaFormat.exclude 'src/resources/java/**/*'
49--
50+
51 // Check the coding style with Checkstyle. Fail in case of error or warning.
52 apply plugin: 'checkstyle'
53- checkstyle.configFile = file("${rootDir}/tools/checkstyle.xml")
54diff --git a/debian/patches/03-ignore-jmh-plugin.patch b/debian/patches/03-ignore-jmh-plugin.patch
55index 3ead24f..9358352 100644
56--- a/debian/patches/03-ignore-jmh-plugin.patch
57+++ b/debian/patches/03-ignore-jmh-plugin.patch
58@@ -8,7 +8,7 @@ Forwarded: not-needed
59 }
60
61 -plugins { id 'me.champeau.jmh' version '0.6.8' apply false }
62- plugins { id 'org.sonarqube' version '3.5.0.2730' apply false }
63+ plugins { id 'org.sonarqube' version '4.3.1.3277' apply false }
64
65 description = 'ASM, a very small and fast Java bytecode manipulation framework'
66 @@ -118,10 +117,8 @@
67diff --git a/debian/patches/04-ignore-sonarqube-plugin.patch b/debian/patches/04-ignore-sonarqube-plugin.patch
68index a1dc7f7..6a0cb0e 100644
69--- a/debian/patches/04-ignore-sonarqube-plugin.patch
70+++ b/debian/patches/04-ignore-sonarqube-plugin.patch
71@@ -3,22 +3,23 @@ Author: Emmanuel Bourg <ebourg@apache.org>
72 Forwarded: not-needed
73 --- a/build.gradle
74 +++ b/build.gradle
75-@@ -31,7 +31,6 @@
76+@@ -31,8 +31,6 @@
77 dependencies { classpath 'org.netbeans.tools:sigtest-maven-plugin:1.5' }
78 }
79
80--plugins { id 'org.sonarqube' version '3.5.0.2730' apply false }
81-
82+-plugins { id 'org.sonarqube' version '4.3.1.3277' apply false }
83+-
84 description = 'ASM, a very small and fast Java bytecode manipulation framework'
85
86-@@ -305,13 +304,6 @@
87+ apply plugin: 'java-platform'
88+@@ -311,13 +309,6 @@
89 }
90 }
91
92 - // Apply the SonarQube plugin to monitor the code quality of the project.
93-- // Use with 'gradlew sonarqube -Dsonar.host.url=https://sonarqube.ow2.org'.
94+- // Use with 'gradlew sonar -Dsonar.host.url=https://sonarqube.ow2.org'.
95 - apply plugin: 'org.sonarqube'
96-- sonarqube {
97+- sonar {
98 - properties { property 'sonar.projectKey', "ASM:${project.name}" }
99 - }
100 -
101diff --git a/debian/patches/05-junit-compatibility.patch b/debian/patches/05-junit-compatibility.patch
102index 7895071..45705b0 100644
103--- a/debian/patches/05-junit-compatibility.patch
104+++ b/debian/patches/05-junit-compatibility.patch
105@@ -3,7 +3,7 @@ Author: Emmanuel Bourg <ebourg@apache.org>
106 Forwarded: not-needed
107 --- a/build.gradle
108 +++ b/build.gradle
109-@@ -53,7 +53,6 @@
110+@@ -52,7 +52,6 @@
111 apply plugin: 'jacoco'
112 sourceCompatibility = '1.8'
113 targetCompatibility = '1.8'
114diff --git a/debian/patches/06-disable-retrofitter.patch b/debian/patches/06-disable-retrofitter.patch
115index 603e608..9b1fff5 100644
116--- a/debian/patches/06-disable-retrofitter.patch
117+++ b/debian/patches/06-disable-retrofitter.patch
118@@ -3,7 +3,7 @@ Author: Emmanuel Bourg <ebourg@apache.org>
119 Forwarded: not-needed
120 --- a/build.gradle
121 +++ b/build.gradle
122-@@ -222,22 +222,6 @@
123+@@ -227,22 +227,6 @@
124 }
125 check.dependsOn jacocoTestCoverageVerification
126
127diff --git a/debian/patches/07-restore-asm-all.patch b/debian/patches/07-restore-asm-all.patch
128index 4206b75..f6c9f30 100644
129--- a/debian/patches/07-restore-asm-all.patch
130+++ b/debian/patches/07-restore-asm-all.patch
131@@ -3,7 +3,7 @@ Author: Emmanuel Bourg <ebourg@apache.org>
132 Forwarded: not-needed
133 --- a/build.gradle
134 +++ b/build.gradle
135-@@ -74,6 +74,10 @@
136+@@ -73,6 +73,10 @@
137 // Project descriptions
138 // -----------------------------------------------------------------------------
139
140diff --git a/debian/patches/08-ignore-sigtest-maven-plugin.patch b/debian/patches/08-ignore-sigtest-maven-plugin.patch
141index 8a5d363..30b9d89 100644
142--- a/debian/patches/08-ignore-sigtest-maven-plugin.patch
143+++ b/debian/patches/08-ignore-sigtest-maven-plugin.patch
144@@ -3,7 +3,7 @@ Author: Emmanuel Bourg <ebourg@apache.org>
145 Forwarded: not-needed
146 --- a/build.gradle
147 +++ b/build.gradle
148-@@ -233,17 +233,6 @@
149+@@ -238,35 +238,9 @@
150 task "${f.name}"(dependsOn: 'classes') {
151 inputs.files(f, sourceSets.main.java)
152 outputs.file("${buildDir}/${f.name}")
153@@ -21,10 +21,10 @@ Forwarded: not-needed
154 }
155 check.dependsOn f.name
156 }
157-@@ -251,16 +240,6 @@
158- task "buildSigtest"(dependsOn: 'classes') {
159- inputs.files(sourceSets.main.java)
160- outputs.file("src/test/resources/sigtest-${version}.txt")
161+- // Define a task to create a sigtest file for the current version.
162+- task "buildSigtest"(dependsOn: 'classes') {
163+- inputs.files(sourceSets.main.java)
164+- outputs.file("src/test/resources/sigtest-${version}.txt")
165 - doLast {
166 - def setup = new com.sun.tdk.signaturetest.Setup()
167 - def args = ['-ApiVersion', version, '-FileName', outputs.getFiles()[0],
168@@ -35,6 +35,7 @@ Forwarded: not-needed
169 - setup.run(args as String[], new PrintWriter(System.err, true), null)
170 - if (!setup.isPassed()) throw new GradleException()
171 - }
172- }
173+- }
174 }
175
176+ jar.manifest.attributes(
177diff --git a/debian/patches/09-ignore-test-coverage.patch b/debian/patches/09-ignore-test-coverage.patch
178index bd66086..79cf2d2 100644
179--- a/debian/patches/09-ignore-test-coverage.patch
180+++ b/debian/patches/09-ignore-test-coverage.patch
181@@ -3,12 +3,12 @@ Author: Emmanuel Bourg <ebourg@apache.org>
182 Forwarded: not-needed
183 --- a/build.gradle
184 +++ b/build.gradle
185-@@ -214,18 +214,6 @@
186+@@ -215,20 +215,6 @@
187 // checked for code coverage and backward compatibility, retrofited to Java 1.5,
188 // and packaged with generated module-info classes.
189 configure(subprojects.findAll{it.provides}) {
190 - // Code coverage configuration.
191-- jacoco.toolVersion = '0.8.8'
192+- jacoco.toolVersion = '0.8.10'
193 - jacocoTestReport {
194 - reports { xml.required = true }
195 - classDirectories.setFrom(sourceSets.main.output.classesDirs)
196@@ -16,9 +16,11 @@ Forwarded: not-needed
197 - jacocoTestCoverageVerification {
198 - classDirectories.setFrom(sourceSets.main.output.classesDirs)
199 - violationRules.rule { limit { minimum = 0.95; counter = 'INSTRUCTION' } }
200+- dependsOn ':asm:jar'
201+- dependsOn ':asm-tree:jar'
202+- dependsOn ':asm-commons:jar'
203 - }
204 - check.dependsOn jacocoTestCoverageVerification
205--
206+
207 // Create one backward compatibility checking task for each 'sigtest-*' file
208 // in test/resources, and make the 'check' task depend on all these tasks.
209- if (file('src/test/resources/').exists()) {
210diff --git a/debian/patches/11-gradle-compatibility.patch b/debian/patches/11-gradle-compatibility.patch
211index aa2d7d3..195cd7e 100644
212--- a/debian/patches/11-gradle-compatibility.patch
213+++ b/debian/patches/11-gradle-compatibility.patch
214@@ -3,27 +3,32 @@ Author: Emmanuel Bourg <ebourg@apache.org>
215 Forwarded: not-needed
216 --- a/build.gradle
217 +++ b/build.gradle
218-@@ -202,7 +202,7 @@
219+@@ -203,7 +203,7 @@
220 }
221-
222+
223 // Produce byte-for-byte reproducible archives.
224 - tasks.withType(AbstractArchiveTask).configureEach {
225 + tasks.withType(AbstractArchiveTask) {
226 preserveFileTimestamps = false
227 reproducibleFileOrder = true
228 dirMode = 0775
229-@@ -279,10 +279,6 @@
230- from sourceSets.main.allSource
231- classifier 'sources'
232+@@ -270,15 +270,9 @@
233+ // together with the main jar (containing the compiled code).
234+ task javadocJar(type: Jar, dependsOn: 'javadoc') {
235+ from javadoc.destinationDir
236+- archiveClassifier = 'javadoc'
237 }
238+ task sourcesJar(type: Jar, dependsOn: 'classes') {
239+ from sourceSets.main.allSource
240+- archiveClassifier = 'sources'
241+- }
242 - java {
243 - withJavadocJar()
244 - withSourcesJar()
245-- }
246+ }
247 }
248-
249- // Configure the root project, and those with a non-empty 'provides' property,
250-@@ -290,6 +286,7 @@
251+
252+@@ -287,6 +281,7 @@
253 configure([rootProject] + subprojects.findAll { it.provides }) {
254 apply plugin: 'maven-publish'
255 apply plugin: 'signing'
256@@ -31,7 +36,7 @@ Forwarded: not-needed
257 publishing {
258 repositories {
259 maven {
260-@@ -382,4 +379,5 @@
261+@@ -379,4 +374,5 @@
262 sign publishing.publications.maven
263 }
264 tasks.withType(GenerateModuleMetadata) { enabled = false }
265diff --git a/debian/patches/12-ignore-java-platform-plugin.patch b/debian/patches/12-ignore-java-platform-plugin.patch
266index fb82387..3fbeeee 100644
267--- a/debian/patches/12-ignore-java-platform-plugin.patch
268+++ b/debian/patches/12-ignore-java-platform-plugin.patch
269@@ -3,10 +3,10 @@ Author: Emmanuel Bourg <ebourg@apache.org>
270 Forwarded: not-needed
271 --- a/build.gradle
272 +++ b/build.gradle
273-@@ -34,14 +34,6 @@
274-
275+@@ -33,14 +33,6 @@
276+
277 description = 'ASM, a very small and fast Java bytecode manipulation framework'
278-
279+
280 -apply plugin: 'java-platform'
281 -dependencies {
282 - constraints {
283@@ -17,4 +17,4 @@ Forwarded: not-needed
284 -
285 allprojects {
286 group = 'org.ow2.asm'
287- version = '9.5' + (rootProject.hasProperty('release') ? '' : '-SNAPSHOT')
288+ version = '9.6' + (rootProject.hasProperty('release') ? '' : '-SNAPSHOT')

Subscribers

People subscribed via source and target branches