[jira] [Updated] (MSITE-945) Remove dependency on Commons IO

2023-04-28 Thread Michael Osipov (Jira)


 [ 
https://issues.apache.org/jira/browse/MSITE-945?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Osipov updated MSITE-945:
-
Fix Version/s: 4.0.0-M8

> Remove dependency on Commons IO
> ---
>
> Key: MSITE-945
> URL: https://issues.apache.org/jira/browse/MSITE-945
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Elliotte Rusty Harold
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 4.0.0-M8
>
>
> All uses look easily replaceable by standard JDK methods these days



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MSITE-945) Remove dependency on Commons IO

2023-04-28 Thread Michael Osipov (Jira)


 [ 
https://issues.apache.org/jira/browse/MSITE-945?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Osipov updated MSITE-945:
-
Summary: Remove dependency on Commons IO  (was: Remove dependency on 
commons IO)

> Remove dependency on Commons IO
> ---
>
> Key: MSITE-945
> URL: https://issues.apache.org/jira/browse/MSITE-945
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Elliotte Rusty Harold
>Assignee: Michael Osipov
>Priority: Major
>
> All uses look easily replaceable by standard JDK methods these days



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MSHARED-953) trimming can remove all whitespace

2023-04-28 Thread Slawomir Jaranowski (Jira)


 [ 
https://issues.apache.org/jira/browse/MSHARED-953?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Slawomir Jaranowski updated MSHARED-953:

Fix Version/s: maven-shared-utils-3.4.1

> trimming can remove all whitespace
> --
>
> Key: MSHARED-953
> URL: https://issues.apache.org/jira/browse/MSHARED-953
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-shared-utils
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
> Fix For: maven-shared-utils-3.4.1
>
>
> The trim parameter in buildDom methods in Xpp3DomBuilder is based on a 
> serious misunderstanding of how the characters method in SAX works. As 
> currently written, trim can potentially remove all whitespace in text content 
> from a document. A little more commonly, it will randomly remove or not 
> remove whitespace depending on how the SAX parser chooses to split text nodes 
> when it reports them.
> It might be possible to implement this correctly at the cost of a lot of code 
> complexity if anyone is using this. More easily, we could make the parameter 
> a no-op and never trim. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Comment Edited] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717722#comment-17717722
 ] 

Gary D. Gregory edited comment on MSHARED-1247 at 4/28/23 8:50 PM:
---

[~sjaranowski]

WRT ossfuzz/issue51980/Test.class:

A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an {{IOException}}, 
then any error is logged and does not blow up the Maven run (the method already 
throws {{IOException}}).


was (Author: garydgregory):
WRT ossfuzz/issue51980/Test.class:

A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an {{IOException}}, 
then any error is logged and does not blow up the Maven run (the method already 
throws {{IOException}}).

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at 

[jira] [Commented] (MRESOLVER-358) Upgrade Guava, Maven, mockito, Plexus Utils

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717800#comment-17717800
 ] 

ASF GitHub Bot commented on MRESOLVER-358:
--

slawekjaranowski commented on code in PR #284:
URL: https://github.com/apache/maven-resolver/pull/284#discussion_r1180770118


##
maven-resolver-demos/pom.xml:
##
@@ -38,7 +38,7 @@
   
 
   
-3.8.4
+3.8.8

Review Comment:
   There is IT in resolver-demo-maven-plugin which will fail when Maven < 3.8.8 
will be used ...
   But we enforce minimal version as 3.8.4





> Upgrade Guava, Maven, mockito, Plexus Utils
> ---
>
> Key: MRESOLVER-358
> URL: https://issues.apache.org/jira/browse/MRESOLVER-358
> Project: Maven Resolver
>  Issue Type: Dependency upgrade
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Upgrades
>  * guava to 31.1-jre (was 30.1-jre)
>  * mockito-core to 4.11.0 (was 4.8.1)
>  * plexus-utils to 3.5.1 (was 3.5.0)
>  * maven deps (used in demos only) to 3.8.8 (was 3.8.4)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] slawekjaranowski commented on a diff in pull request #284: [MRESOLVER-358] Upgrade some dependencies

2023-04-28 Thread via GitHub


slawekjaranowski commented on code in PR #284:
URL: https://github.com/apache/maven-resolver/pull/284#discussion_r1180770118


##
maven-resolver-demos/pom.xml:
##
@@ -38,7 +38,7 @@
   
 
   
-3.8.4
+3.8.8

Review Comment:
   There is IT in resolver-demo-maven-plugin which will fail when Maven < 3.8.8 
will be used ...
   But we enforce minimal version as 3.8.4



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MENFORCER-479) Version enforcer rules print single versions when a range is expected

2023-04-28 Thread Slawomir Jaranowski (Jira)


[ 
https://issues.apache.org/jira/browse/MENFORCER-479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717795#comment-17717795
 ] 

Slawomir Jaranowski commented on MENFORCER-479:
---

>From enforcer perspective {{3.8.4}} is equal to range {{[3.8.4,)}} it is 
>different than standard Maven behavior where {{3.8.4}} means - recommended 
>version.

Current messages are:

{noformat}
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-enforcer-plugin:3.3.0:enforce 
(enforce-maven-version) on project apache: 
[ERROR] Rule 0: org.apache.maven.enforcer.rules.version.RequireMavenVersion 
failed with message:
[ERROR] Detected Maven Version: 3.9.1 is not in the allowed range [3.10.1,).

[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-enforcer-plugin:3.3.0:enforce 
(enforce-java-version) on project apache: 
[ERROR] Rule 0: org.apache.maven.enforcer.rules.version.RequireJavaVersion 
failed with message:
[ERROR] Detected JDK version 20 
(JAVA_HOME=/usr/local/Cellar/openjdk/20/libexec/openjdk.jdk/Contents/Home) is 
not in the allowed range [22.2.1,).
{noformat}


> Version enforcer rules print single versions when a range is expected
> -
>
> Key: MENFORCER-479
> URL: https://issues.apache.org/jira/browse/MENFORCER-479
> Project: Maven Enforcer Plugin
>  Issue Type: Bug
>  Components: Rule API
>Reporter: Elliotte Rusty Harold
>Priority: Major
>
> AbstractStandardEnforcerRule incorrectly assumes that the toString method of 
> a VersionRange will be a version range. However it can be just a single 
> version rather than the entire range. E.g. 3.8.4 instead [3.8.0, 3.9)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717791#comment-17717791
 ] 

ASF GitHub Bot commented on MRESOLVER-359:
--

cstamas commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180733890


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)

Review Comment:
   Hm, wait a second... seems am wrong: "[enforcer] redefined as a minimum 
version.". So did I misinterpret then?
   
   Still, I think if we provide (semantically) equivalent value, but in form of 
(maven and not "redefined enforcer") range, we are somehow clearer what we 
_really mean_ here.





> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work. CI 
> "answered" this for me, it uses Java 8u362
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] cstamas commented on a diff in pull request #285: [MRESOLVER-359] Make build be explicit about build time requirements

2023-04-28 Thread via GitHub


cstamas commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180733890


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)

Review Comment:
   Hm, wait a second... seems am wrong: "[enforcer] redefined as a minimum 
version.". So did I misinterpret then?
   
   Still, I think if we provide (semantically) equivalent value, but in form of 
(maven and not "redefined enforcer") range, we are somehow clearer what we 
_really mean_ here.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717790#comment-17717790
 ] 

ASF GitHub Bot commented on MRESOLVER-359:
--

cstamas commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180731785


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)
+[1.8.0-362,)

Review Comment:
   Same as above, 1.8.0-362 is NOT the recommended Java 8, just _minimal_ we 
allow to build with.





> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work. CI 
> "answered" this for me, it uses Java 8u362
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] cstamas commented on a diff in pull request #285: [MRESOLVER-359] Make build be explicit about build time requirements

2023-04-28 Thread via GitHub


cstamas commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180731785


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)
+[1.8.0-362,)

Review Comment:
   Same as above, 1.8.0-362 is NOT the recommended Java 8, just _minimal_ we 
allow to build with.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717789#comment-17717789
 ] 

ASF GitHub Bot commented on MRESOLVER-359:
--

cstamas commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180731396


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)

Review Comment:
   Is not, see 
https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html
   
   "3.8.4" was "x >= 3.8.4 but 3.8.4 RECOMMENDED". This is not (semantically) 
what we _mean_.
   What we mean is "use minimally 3.8.4 or ANY NEWER".





> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work. CI 
> "answered" this for me, it uses Java 8u362
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] cstamas commented on a diff in pull request #285: [MRESOLVER-359] Make build be explicit about build time requirements

2023-04-28 Thread via GitHub


cstamas commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180731396


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)

Review Comment:
   Is not, see 
https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html
   
   "3.8.4" was "x >= 3.8.4 but 3.8.4 RECOMMENDED". This is not (semantically) 
what we _mean_.
   What we mean is "use minimally 3.8.4 or ANY NEWER".



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717787#comment-17717787
 ] 

ASF GitHub Bot commented on MRESOLVER-359:
--

slawekjaranowski commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180727977


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)
+[1.8.0-362,)

Review Comment:
   Here can also be simple `1.8.0-362`



##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)

Review Comment:
   It is exactly the same - what is the reason for such change?





> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work. CI 
> "answered" this for me, it uses Java 8u362
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] slawekjaranowski commented on a diff in pull request #285: [MRESOLVER-359] Make build be explicit about build time requirements

2023-04-28 Thread via GitHub


slawekjaranowski commented on code in PR #285:
URL: https://github.com/apache/maven-resolver/pull/285#discussion_r1180727977


##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)
+[1.8.0-362,)

Review Comment:
   Here can also be simple `1.8.0-362`



##
pom.xml:
##
@@ -93,7 +93,8 @@
 30.1-jre
 1.0.1
 1.7.36
-3.8.4
+[3.8.4,)

Review Comment:
   It is exactly the same - what is the reason for such change?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MENFORCER-479) Version enforcer rules print single versions when a range is expected

2023-04-28 Thread Elliotte Rusty Harold (Jira)


[ 
https://issues.apache.org/jira/browse/MENFORCER-479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717754#comment-17717754
 ] 

Elliotte Rusty Harold commented on MENFORCER-479:
-

Apparently 3.8.4 is interpreted as [3.8.4,) That might make sense for authoring 
purposes but I still think that's unclear output here. [3.8.4,) would be 
clearer UI, and "3.8.4 or later" would be better still. The purpose of this 
message is to communicate with the human developer, not to set a value. 

> Version enforcer rules print single versions when a range is expected
> -
>
> Key: MENFORCER-479
> URL: https://issues.apache.org/jira/browse/MENFORCER-479
> Project: Maven Enforcer Plugin
>  Issue Type: Bug
>  Components: Rule API
>Reporter: Elliotte Rusty Harold
>Priority: Major
>
> AbstractStandardEnforcerRule incorrectly assumes that the toString method of 
> a VersionRange will be a version range. However it can be just a single 
> version rather than the entire range. E.g. 3.8.4 instead [3.8.0, 3.9)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MENFORCER-479) Version enforcer rules print single versions when a range is expected

2023-04-28 Thread Elliotte Rusty Harold (Jira)


[ 
https://issues.apache.org/jira/browse/MENFORCER-479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717753#comment-17717753
 ] 

Elliotte Rusty Harold commented on MENFORCER-479:
-

Whichever version is pulled in by head in maven-resolver right now, though 
looking at the source code for the enforcer rules and the Version Range class 
I'm pretty sure this is true at head unless someone fixed it today. 

> Version enforcer rules print single versions when a range is expected
> -
>
> Key: MENFORCER-479
> URL: https://issues.apache.org/jira/browse/MENFORCER-479
> Project: Maven Enforcer Plugin
>  Issue Type: Bug
>  Components: Rule API
>Reporter: Elliotte Rusty Harold
>Priority: Major
>
> AbstractStandardEnforcerRule incorrectly assumes that the toString method of 
> a VersionRange will be a version range. However it can be just a single 
> version rather than the entire range. E.g. 3.8.4 instead [3.8.0, 3.9)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Resolved] (MNG-7729) Deprecate Xpp3Dom.isNonEmpty and isEmpty

2023-04-28 Thread Elliotte Rusty Harold (Jira)


 [ 
https://issues.apache.org/jira/browse/MNG-7729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Elliotte Rusty Harold resolved MNG-7729.

Resolution: Fixed

> Deprecate Xpp3Dom.isNonEmpty and isEmpty
> 
>
> Key: MNG-7729
> URL: https://issues.apache.org/jira/browse/MNG-7729
> Project: Maven
>  Issue Type: Task
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Minor
>
> And yet again copy pasta of random string utility methods that are public and 
> should never have been here



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Closed] (MNG-7729) Deprecate Xpp3Dom.isNonEmpty and isEmpty

2023-04-28 Thread Elliotte Rusty Harold (Jira)


 [ 
https://issues.apache.org/jira/browse/MNG-7729?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Elliotte Rusty Harold closed MNG-7729.
--

> Deprecate Xpp3Dom.isNonEmpty and isEmpty
> 
>
> Key: MNG-7729
> URL: https://issues.apache.org/jira/browse/MNG-7729
> Project: Maven
>  Issue Type: Task
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Minor
>
> And yet again copy pasta of random string utility methods that are public and 
> should never have been here



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-mvnd] tbw777 commented on issue #793: java.lang.UnsatisfiedLinkError: Can't load library: jansi.dll

2023-04-28 Thread via GitHub


tbw777 commented on issue #793:
URL: https://github.com/apache/maven-mvnd/issues/793#issuecomment-1527809399

   1.0m6
   
   C:\Users\Андрей>c:\mvnd\bin\mvnd.cmd -v
   Failed to load native library:jansi-2.4.0-7a1b3e95bab9e2b8-jansi.dll. 
osinfo: Windows/x86_64
   java.lang.UnsatisfiedLinkError: Can't load library: 
C:\Users\A4F7~1\AppData\Local\Temp\jansi-2.4.0-7a1b3e95bab9e2b8-jansi.dll
   Exception in thread "main" java.lang.UnsatisfiedLinkError: 
org.fusesource.jansi.internal.CLibrary.isatty(I)I [symbol: 
Java_org_fusesource_jansi_internal_CLibrary_isatty or 
Java_org_fusesource_jansi_internal_CLibrary_isatty__I]
   at 
org.graalvm.nativeimage.builder/com.oracle.svm.core.jni.access.JNINativeLinkage.getOrFindEntryPoint(JNINativeLinkage.java:152)
   at 
org.graalvm.nativeimage.builder/com.oracle.svm.core.jni.JNIGeneratedMethodSupport.nativeCallAddress(JNIGeneratedMethodSupport.java:53)
   at org.fusesource.jansi.internal.CLibrary.isatty(CLibrary.java)
   at 
org.mvndaemon.mvnd.client.DefaultClient.main(DefaultClient.java:103)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-359:
--
Description: 
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.

Questions:
 * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
and then even after a year later (when possibly new Java 8 versions may be 
released), it is expected that u371 and later ones will still work. CI 
"answered" this for me, it uses Java 8 361

List of Java 8 releases 
[https://www.oracle.com/java/technologies/javase/8all-relnotes.html]

  was:
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.

Questions:
 * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
and then even after a year later (when possibly new Java 8 versions may be 
released), it is expected that u371 and later ones will still work.

List of Java 8 releases 
[https://www.oracle.com/java/technologies/javase/8all-relnotes.html]


> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work. CI 
> "answered" this for me, it uses Java 8 361
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-359:
--
Description: 
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.

Questions:
 * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
and then even after a year later (when possibly new Java 8 versions may be 
released), it is expected that u371 and later ones will still work. CI 
"answered" this for me, it uses Java 8u362

List of Java 8 releases 
[https://www.oracle.com/java/technologies/javase/8all-relnotes.html]

  was:
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.

Questions:
 * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
and then even after a year later (when possibly new Java 8 versions may be 
released), it is expected that u371 and later ones will still work. CI 
"answered" this for me, it uses Java 8 361

List of Java 8 releases 
[https://www.oracle.com/java/technologies/javase/8all-relnotes.html]


> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work. CI 
> "answered" this for me, it uses Java 8u362
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Comment Edited] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717722#comment-17717722
 ] 

Gary D. Gregory edited comment on MSHARED-1247 at 4/28/23 3:22 PM:
---

WRT ossfuzz/issue51980/Test.class:

A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an {{IOException}}, 
then any error is logged and does not blow up the Maven run (the method already 
throws {{IOException}}).


was (Author: garydgregory):
A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an {{IOException}}, 
then any error is logged and does not blow up the Maven run (the method already 
throws {{IOException}}).

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.r

[jira] [Comment Edited] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717722#comment-17717722
 ] 

Gary D. Gregory edited comment on MSHARED-1247 at 4/28/23 3:22 PM:
---

A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an {{IOException}}, 
then any error is logged and does not blow up the Maven run (the method already 
throws {{IOException}}).


was (Author: garydgregory):
A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an IOException, then 
any error is logged and does not blow up the Maven run (the method already 
throws {{{}IOException{}}}).

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> 

[jira] [Comment Edited] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717722#comment-17717722
 ] 

Gary D. Gregory edited comment on MSHARED-1247 at 4/28/23 3:21 PM:
---

A related problem is that 
{{ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}} throws a 
{{RuntimeException}} which is an anti-pattern; as opposed to an better 
unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an IOException, then 
any error is logged and does not blow up the Maven run (the method already 
throws {{{}IOException{}}}).


was (Author: garydgregory):
A related problem is that 
{{org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}}
 throws a {{RuntimeException}} which is an anti-pattern; as opposed to an 
better unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an IOException, then 
any error is logged and does not blow up the Maven run (the method already 
throws {{{}IOException{}}}).

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.Mojo

[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717722#comment-17717722
 ] 

Gary D. Gregory commented on MSHARED-1247:
--

A related problem is that 
{{org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences(ByteBuffer)}}
 throws a {{RuntimeException}} which is an anti-pattern; as opposed to an 
better unchecked exception like {{{}IllegalArgumentException{}}}, 
{{{}IllegalStateException{}}}, or {{{}UncheckedIOException{}}}.

IIRC, if in the PR, I change {{ClassFileVisitorUtils.acceptDirectory(File, 
ClassFileVisitor)}} to rethrow the {{RuntimeException}} as an IOException, then 
any error is logged and does not blow up the Maven run (the method already 
throws {{{}IOException{}}}).

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
> (DefaultMojosExecutionStrategy.java:39)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:160)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:105)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:73)
> at 
> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
>  (SingleThreadedBuilder.java:53)
> at org.apache

[jira] [Updated] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-357?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-357:
--
Priority: Blocker  (was: Major)

> ConflictResolver STANDARD verbosity misbehaves
> --
>
> Key: MRESOLVER-357
> URL: https://issues.apache.org/jira/browse/MRESOLVER-357
> Project: Maven Resolver
>  Issue Type: Bug
>  Components: Resolver
>Affects Versions: 1.9.8, 1.9.9
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Blocker
> Fix For: 1.9.10
>
>
> As per user report, the 1.9.8 and 1.9.9 versions produce different "verbose 
> tree" than previous versions:
> {noformat}
> Input node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  +- some-group:c:jar:1.0 [compile]
> |  +- some-group:c:jar:2.0 [compile]
> |  \- some-group:d:jar:1.0 [compile]
> +- some-group:c:jar:2.0 [compile]
> +- some-group:c:jar:1.0 [compile]
> \- some-group:d:jar:2.0 [compile]
> Transformed node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  \- some-group:c:jar:2.0 [compile] (nearer exists)
> +- some-group:c:jar:2.0 [compile]
> \- some-group:d:jar:2.0 [compile] {noformat}
> The {{some-group:d:jar:1.0 [compile]}} should not be removed.
> Seems that code comment "leaving only 1 loser" is present in comments only, 
> but the actual code does not deliver this promise.
> In fact, the check is wrong: it should check for same GA child count and not 
> ANY child count.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717718#comment-17717718
 ] 

Gary D. Gregory commented on MSHARED-1247:
--

Hi [~sjaranowski] 

Thank you for the quick feedback.

I added a test to the PR.

Yeah, {{cyclonedx}}  I suppose a downstream project can depend on any jar 
files test or not, and end up delivering it. That's the only reason I can 
imagine for analyzing test code and resources.

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
> (DefaultMojosExecutionStrategy.java:39)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:160)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:105)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:73)
> at 
> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
>  (SingleThreadedBuilder.java:53)
> at org.apache.maven.lifecycle.internal.LifecycleStarter.execute 
> (LifecycleStarter.java:118)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
> at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
> at org.apache.maven.cli.

[jira] [Updated] (MRESOLVER-358) Upgrade Guava, Maven, mockito, Plexus Utils

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-358?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-358:
--
Component/s: Resolver

> Upgrade Guava, Maven, mockito, Plexus Utils
> ---
>
> Key: MRESOLVER-358
> URL: https://issues.apache.org/jira/browse/MRESOLVER-358
> Project: Maven Resolver
>  Issue Type: Dependency upgrade
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Upgrades
>  * guava to 31.1-jre (was 30.1-jre)
>  * mockito-core to 4.11.0 (was 4.8.1)
>  * plexus-utils to 3.5.1 (was 3.5.0)
>  * maven deps (used in demos only) to 3.8.8 (was 3.8.4)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-359:
--
Component/s: Resolver

> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work.
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Assigned] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak reassigned MRESOLVER-359:
-

Assignee: Tamas Cservenak

> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work.
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Resolved] (MSHARED-953) trimming can remove all whitespace

2023-04-28 Thread Elliotte Rusty Harold (Jira)


 [ 
https://issues.apache.org/jira/browse/MSHARED-953?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Elliotte Rusty Harold resolved MSHARED-953.
---
Resolution: Fixed

> trimming can remove all whitespace
> --
>
> Key: MSHARED-953
> URL: https://issues.apache.org/jira/browse/MSHARED-953
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-shared-utils
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
>
> The trim parameter in buildDom methods in Xpp3DomBuilder is based on a 
> serious misunderstanding of how the characters method in SAX works. As 
> currently written, trim can potentially remove all whitespace in text content 
> from a document. A little more commonly, it will randomly remove or not 
> remove whitespace depending on how the SAX parser chooses to split text nodes 
> when it reports them.
> It might be possible to implement this correctly at the cost of a lot of code 
> complexity if anyone is using this. More easily, we could make the parameter 
> a no-op and never trim. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSHARED-953) trimming can remove all whitespace

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-953?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717715#comment-17717715
 ] 

ASF GitHub Bot commented on MSHARED-953:


elharo closed pull request #125: [MSHARED-953] Trim at end of element
URL: https://github.com/apache/maven-shared-utils/pull/125




> trimming can remove all whitespace
> --
>
> Key: MSHARED-953
> URL: https://issues.apache.org/jira/browse/MSHARED-953
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-shared-utils
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
>
> The trim parameter in buildDom methods in Xpp3DomBuilder is based on a 
> serious misunderstanding of how the characters method in SAX works. As 
> currently written, trim can potentially remove all whitespace in text content 
> from a document. A little more commonly, it will randomly remove or not 
> remove whitespace depending on how the SAX parser chooses to split text nodes 
> when it reports them.
> It might be possible to implement this correctly at the cost of a lot of code 
> complexity if anyone is using this. More easily, we could make the parameter 
> a no-op and never trim. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSHARED-953) trimming can remove all whitespace

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-953?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717714#comment-17717714
 ] 

ASF GitHub Bot commented on MSHARED-953:


elharo merged PR #124:
URL: https://github.com/apache/maven-shared-utils/pull/124




> trimming can remove all whitespace
> --
>
> Key: MSHARED-953
> URL: https://issues.apache.org/jira/browse/MSHARED-953
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-shared-utils
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
>
> The trim parameter in buildDom methods in Xpp3DomBuilder is based on a 
> serious misunderstanding of how the characters method in SAX works. As 
> currently written, trim can potentially remove all whitespace in text content 
> from a document. A little more commonly, it will randomly remove or not 
> remove whitespace depending on how the SAX parser chooses to split text nodes 
> when it reports them.
> It might be possible to implement this correctly at the cost of a lot of code 
> complexity if anyone is using this. More easily, we could make the parameter 
> a no-op and never trim. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Closed] (MSHARED-953) trimming can remove all whitespace

2023-04-28 Thread Elliotte Rusty Harold (Jira)


 [ 
https://issues.apache.org/jira/browse/MSHARED-953?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Elliotte Rusty Harold closed MSHARED-953.
-

> trimming can remove all whitespace
> --
>
> Key: MSHARED-953
> URL: https://issues.apache.org/jira/browse/MSHARED-953
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-shared-utils
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
>
> The trim parameter in buildDom methods in Xpp3DomBuilder is based on a 
> serious misunderstanding of how the characters method in SAX works. As 
> currently written, trim can potentially remove all whitespace in text content 
> from a document. A little more commonly, it will randomly remove or not 
> remove whitespace depending on how the SAX parser chooses to split text nodes 
> when it reports them.
> It might be possible to implement this correctly at the cost of a lot of code 
> complexity if anyone is using this. More easily, we could make the parameter 
> a no-op and never trim. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717716#comment-17717716
 ] 

ASF GitHub Bot commented on MRESOLVER-359:
--

cstamas opened a new pull request, #285:
URL: https://github.com/apache/maven-resolver/pull/285

   Fixes:
   * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
   * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
   
   ---
   
   https://issues.apache.org/jira/browse/MRESOLVER-359




> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work.
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-shared-utils] elharo closed pull request #125: [MSHARED-953] Trim at end of element

2023-04-28 Thread via GitHub


elharo closed pull request #125: [MSHARED-953] Trim at end of element
URL: https://github.com/apache/maven-shared-utils/pull/125


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [maven-resolver] cstamas opened a new pull request, #285: [MRESOLVER-359] Make build be explicit about build time requirements

2023-04-28 Thread via GitHub


cstamas opened a new pull request, #285:
URL: https://github.com/apache/maven-resolver/pull/285

   Fixes:
   * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
   * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
   
   ---
   
   https://issues.apache.org/jira/browse/MRESOLVER-359


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [maven-shared-utils] elharo merged pull request #124: [MSHARED-953] don't trim

2023-04-28 Thread via GitHub


elharo merged PR #124:
URL: https://github.com/apache/maven-shared-utils/pull/124


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-359:
--
Description: 
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.

Questions:
 * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
and then even after a year later (when possibly new Java 8 versions may be 
released), it is expected that u371 and later ones will still work.

List of Java 8 releases 
[https://www.oracle.com/java/technologies/javase/8all-relnotes.html]

  was:
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-352,)" we mean "recent" Java 8.

Questions:

* why "[1.8.0-352,)"? List of Java 8 releases 
https://www.oracle.com/java/technologies/javase/8all-relnotes.html


> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-371,)" we mean "recent" Java 8.
> Questions:
>  * why "[1.8.0-371,)"? I think as in this moment use of latest Java 8 works, 
> and then even after a year later (when possibly new Java 8 versions may be 
> released), it is expected that u371 and later ones will still work.
> List of Java 8 releases 
> [https://www.oracle.com/java/technologies/javase/8all-relnotes.html]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MNG-7729) Deprecate Xpp3Dom.isNonEmpty and isEmpty

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7729?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717713#comment-17717713
 ] 

ASF GitHub Bot commented on MNG-7729:
-

elharo merged PR #136:
URL: https://github.com/apache/maven-shared-utils/pull/136




> Deprecate Xpp3Dom.isNonEmpty and isEmpty
> 
>
> Key: MNG-7729
> URL: https://issues.apache.org/jira/browse/MNG-7729
> Project: Maven
>  Issue Type: Task
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Minor
>
> And yet again copy pasta of random string utility methods that are public and 
> should never have been here



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-shared-utils] elharo merged pull request #136: [MNG-7729] deprecate questionable IsEmpty/IsNotEmpty methods

2023-04-28 Thread via GitHub


elharo merged PR #136:
URL: https://github.com/apache/maven-shared-utils/pull/136


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-359:
--
Description: 
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-352,)" we mean "recent" Java 8.

Questions:

* why "[1.8.0-352,)"? List of Java 8 releases 
https://www.oracle.com/java/technologies/javase/8all-relnotes.html

  was:
Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-352,)" we mean "recent" Java 8.


> Make build be explicit about build time requirements
> 
>
> Key: MRESOLVER-359
> URL: https://issues.apache.org/jira/browse/MRESOLVER-359
> Project: Maven Resolver
>  Issue Type: Task
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Current state of affairs:
>  * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
> rule) set in resolver TLP contains "3.8.4", that according to enforcer 
> "Version Range Specification" means "3.8.4 or later but 3.8.4 is 
> RECOMMENDED". This is NOT what we mean.
>  * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
> rule) inherited from parent contains "1.8" which again, is NOT what we mean, 
> as we know build will not work on some obsolete Java 8 versions (ie. 
> Java8u172 from 2018)
> Fixes:
>  * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
> "maintained" Maven version.
>  * make minimalJavaBuildVersion be "[1.8.0-352,)" we mean "recent" Java 8.
> Questions:
> * why "[1.8.0-352,)"? List of Java 8 releases 
> https://www.oracle.com/java/technologies/javase/8all-relnotes.html



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (MRESOLVER-359) Make build be explicit about build time requirements

2023-04-28 Thread Tamas Cservenak (Jira)
Tamas Cservenak created MRESOLVER-359:
-

 Summary: Make build be explicit about build time requirements
 Key: MRESOLVER-359
 URL: https://issues.apache.org/jira/browse/MRESOLVER-359
 Project: Maven Resolver
  Issue Type: Task
Reporter: Tamas Cservenak
 Fix For: 1.9.10


Current state of affairs:
 * property minimalMavenBuildVersion (passed to requireMavenVersion enforcer 
rule) set in resolver TLP contains "3.8.4", that according to enforcer "Version 
Range Specification" means "3.8.4 or later but 3.8.4 is RECOMMENDED". This is 
NOT what we mean.
 * property minimalJavaBuildVersion (passed to requireJavaVersion enforcer 
rule) inherited from parent contains "1.8" which again, is NOT what we mean, as 
we know build will not work on some obsolete Java 8 versions (ie. Java8u172 
from 2018)

Fixes:
 * make minimalMavenBuildVersion be "[3.8.4,)", we mean "recent" or maybe 
"maintained" Maven version.
 * make minimalJavaBuildVersion be "[1.8.0-352,)" we mean "recent" Java 8.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MRESOLVER-358) Upgrade Guava, Maven, mockito, Plexus Utils

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717706#comment-17717706
 ] 

ASF GitHub Bot commented on MRESOLVER-358:
--

cstamas opened a new pull request, #284:
URL: https://github.com/apache/maven-resolver/pull/284

   Upgrades
   * guava to 31.1-jre (was 30.1-jre)
   * mockito-core to 4.11.0 (was 4.8.1)
   * plexus-utils to 3.5.1 (was 3.5.0)
   * maven deps (used in demos only) to 3.8.8 (was 3.8.4)
   
   ---
   
   https://issues.apache.org/jira/browse/MRESOLVER-358




> Upgrade Guava, Maven, mockito, Plexus Utils
> ---
>
> Key: MRESOLVER-358
> URL: https://issues.apache.org/jira/browse/MRESOLVER-358
> Project: Maven Resolver
>  Issue Type: Dependency upgrade
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Upgrades
>  * guava to 31.1-jre (was 30.1-jre)
>  * mockito-core to 4.11.0 (was 4.8.1)
>  * plexus-utils to 3.5.1 (was 3.5.0)
>  * maven deps (used in demos only) to 3.8.8 (was 3.8.4)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Assigned] (MRESOLVER-358) Upgrade Guava, Maven, mockito, Plexus Utils

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-358?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak reassigned MRESOLVER-358:
-

Assignee: Tamas Cservenak

> Upgrade Guava, Maven, mockito, Plexus Utils
> ---
>
> Key: MRESOLVER-358
> URL: https://issues.apache.org/jira/browse/MRESOLVER-358
> Project: Maven Resolver
>  Issue Type: Dependency upgrade
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> Upgrades
>  * guava to 31.1-jre (was 30.1-jre)
>  * mockito-core to 4.11.0 (was 4.8.1)
>  * plexus-utils to 3.5.1 (was 3.5.0)
>  * maven deps (used in demos only) to 3.8.8 (was 3.8.4)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Slawomir Jaranowski (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717707#comment-17717707
 ] 

Slawomir Jaranowski commented on MSHARED-1247:
--

We have 

{noformat}
java.lang.IllegalArgumentException: Unsupported class file major version 1025
{noformat}

for class:

{noformat}
ossfuzz/issue51980/Test.class
{noformat}

As I see {{ossfuzz}} contains corrupted classes for tests - [~ggregory] 
questions is why {{cyclonedx}} try to analyze also test resources 


> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
> (DefaultMojosExecutionStrategy.java:39)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:160)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:105)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:73)
> at 
> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
>  (SingleThreadedBuilder.java:53)
> at org.apache.maven.lifecycle.internal.LifecycleStarter.execute 
> (LifecycleStarter.java:118)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
> at org.apache.maven.DefaultMaven.execute (DefaultMave

[GitHub] [maven-resolver] cstamas opened a new pull request, #284: [MRESOLVER-358] Upgrade some dependencies

2023-04-28 Thread via GitHub


cstamas opened a new pull request, #284:
URL: https://github.com/apache/maven-resolver/pull/284

   Upgrades
   * guava to 31.1-jre (was 30.1-jre)
   * mockito-core to 4.11.0 (was 4.8.1)
   * plexus-utils to 3.5.1 (was 3.5.0)
   * maven deps (used in demos only) to 3.8.8 (was 3.8.4)
   
   ---
   
   https://issues.apache.org/jira/browse/MRESOLVER-358


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (MRESOLVER-358) Upgrade Guava, Maven, mockito, Plexus Utils

2023-04-28 Thread Tamas Cservenak (Jira)
Tamas Cservenak created MRESOLVER-358:
-

 Summary: Upgrade Guava, Maven, mockito, Plexus Utils
 Key: MRESOLVER-358
 URL: https://issues.apache.org/jira/browse/MRESOLVER-358
 Project: Maven Resolver
  Issue Type: Dependency upgrade
Reporter: Tamas Cservenak
 Fix For: 1.9.10


Upgrades
 * guava to 31.1-jre (was 30.1-jre)
 * mockito-core to 4.11.0 (was 4.8.1)
 * plexus-utils to 3.5.1 (was 3.5.0)
 * maven deps (used in demos only) to 3.8.8 (was 3.8.4)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MNG-7769) Upgrade to Resolver 1.9.10

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MNG-7769?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MNG-7769:
-
Description: 
As resolver 1.9.8 suffers from event bug MRESOLVER-352 that may break 
extensions relying on events.

And resolver 1.9.9 suffers from verbose tree bug MRESOLVER-357 that may not 
detect some cases of divergence.

  was:As resolver 1.9.8 suffers from event bug MRESOLVER-352 that may break 
extensions relying on events.


> Upgrade to Resolver 1.9.10
> --
>
> Key: MNG-7769
> URL: https://issues.apache.org/jira/browse/MNG-7769
> Project: Maven
>  Issue Type: Dependency upgrade
>  Components: Dependencies
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 3.9.2, 4.0.0-alpha-6, 4.0.0
>
>
> As resolver 1.9.8 suffers from event bug MRESOLVER-352 that may break 
> extensions relying on events.
> And resolver 1.9.9 suffers from verbose tree bug MRESOLVER-357 that may not 
> detect some cases of divergence.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MNG-7769) Upgrade to Resolver 1.9.10

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MNG-7769?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MNG-7769:
-
Summary: Upgrade to Resolver 1.9.10  (was: Upgrade to Resolver 1.9.9)

> Upgrade to Resolver 1.9.10
> --
>
> Key: MNG-7769
> URL: https://issues.apache.org/jira/browse/MNG-7769
> Project: Maven
>  Issue Type: Dependency upgrade
>  Components: Dependencies
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 3.9.2, 4.0.0-alpha-6, 4.0.0
>
>
> As resolver 1.9.8 suffers from event bug MRESOLVER-352 that may break 
> extensions relying on events.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Slawomir Jaranowski (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717696#comment-17717696
 ] 

Slawomir Jaranowski commented on MSHARED-1247:
--

This one is a problem: 
{{commons-bcel/target/test-classes/issue362/Bcel362.class}}

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
> (DefaultMojosExecutionStrategy.java:39)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:160)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:105)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:73)
> at 
> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
>  (SingleThreadedBuilder.java:53)
> at org.apache.maven.lifecycle.internal.LifecycleStarter.execute 
> (LifecycleStarter.java:118)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
> at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
> at org.apache.maven.cli.MavenCli.execute (MavenCli.java:827)
> at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:272)
> at org.apache.maven.cli.MavenCli.main (MavenCli.java:195)
> at sun.reflect.Nat

[jira] [Commented] (MGPG-97) add pgpverify check to the build

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MGPG-97?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717691#comment-17717691
 ] 

ASF GitHub Bot commented on MGPG-97:


hboutemy merged PR #48:
URL: https://github.com/apache/maven-gpg-plugin/pull/48




> add pgpverify check to the build
> 
>
> Key: MGPG-97
> URL: https://issues.apache.org/jira/browse/MGPG-97
> Project: Maven GPG Plugin
>  Issue Type: Improvement
>Affects Versions: 3.0.1
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.1.0
>
>
> signing is useful only if signature checks are done
> let's apply this best practice to maven-gpg-plugin itself



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-gpg-plugin] hboutemy merged pull request #48: [MGPG-97] use pgpverify plugin to check dependencies signatures

2023-04-28 Thread via GitHub


hboutemy merged PR #48:
URL: https://github.com/apache/maven-gpg-plugin/pull/48


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Slawomir Jaranowski (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717690#comment-17717690
 ] 

Slawomir Jaranowski commented on MSHARED-1247:
--

[~ggregory] thanks for reports. Do you know which class from your project has 
type 17 in constant pool?

I can try to find it, and we can add test - it will be easier to debug.

By the way all versions are affected :)

> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
> (DefaultMojosExecutionStrategy.java:39)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:160)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:105)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:73)
> at 
> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
>  (SingleThreadedBuilder.java:53)
> at org.apache.maven.lifecycle.internal.LifecycleStarter.execute 
> (LifecycleStarter.java:118)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
> at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
> at org.apache.maven.cli.MavenCli.execute (MavenCli.java:827)
> at org.apache

[jira] [Commented] (SUREFIRE-2072) Surefire retains too much heap in some case, leading to OutOfMemory

2023-04-28 Thread Martijn Visser (Jira)


[ 
https://issues.apache.org/jira/browse/SUREFIRE-2072?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717689#comment-17717689
 ] 

Martijn Visser commented on SUREFIRE-2072:
--

Is there any update on this ticket?

> Surefire retains too much heap in some case, leading to OutOfMemory
> ---
>
> Key: SUREFIRE-2072
> URL: https://issues.apache.org/jira/browse/SUREFIRE-2072
> Project: Maven Surefire
>  Issue Type: Bug
>Reporter: Mickael Istria
>Priority: Major
>
> Most likely since SuREFIRE-1845, every test execution can retain 64kB of RAM 
> (as cache). Which caching is welcome, this cannot scale with some big test 
> suites of several thousands of tests and cause OutOfMemoryError.
> This is the case when running Eclipse JDT dom test suite, since 
> tycho-surefire-plugin started to adopt surefire 3.0.0.M6, as described in 
> https://github.com/eclipse/tycho/issues/879#issuecomment-1096454637



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Assigned] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-357?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak reassigned MRESOLVER-357:
-

Assignee: Tamas Cservenak

> ConflictResolver STANDARD verbosity misbehaves
> --
>
> Key: MRESOLVER-357
> URL: https://issues.apache.org/jira/browse/MRESOLVER-357
> Project: Maven Resolver
>  Issue Type: Bug
>  Components: Resolver
>Affects Versions: 1.9.8, 1.9.9
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
> tree" that previous versions:
> {noformat}
> Input node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  +- some-group:c:jar:1.0 [compile]
> |  +- some-group:c:jar:2.0 [compile]
> |  \- some-group:d:jar:1.0 [compile]
> +- some-group:c:jar:2.0 [compile]
> +- some-group:c:jar:1.0 [compile]
> \- some-group:d:jar:2.0 [compile]
> Transformed node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  \- some-group:c:jar:2.0 [compile] (nearer exists)
> +- some-group:c:jar:2.0 [compile]
> \- some-group:d:jar:2.0 [compile] {noformat}
> The {{some-group:d:jar:1.0 [compile]}} should not be removed.
> Seems that code comment "leaving only 1 loser" is present in comments only, 
> but the actual code does not deliver this promise.
> In fact, the check is wrong: it should check for same GA child count and not 
> ANY child count.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-357?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-357:
--
Description: 
As per user report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" than previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The {{some-group:d:jar:1.0 [compile]}} should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise.

In fact, the check is wrong: it should check for same GA child count and not 
ANY child count.

  was:
As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" that previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The {{some-group:d:jar:1.0 [compile]}} should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise.

In fact, the check is wrong: it should check for same GA child count and not 
ANY child count.


> ConflictResolver STANDARD verbosity misbehaves
> --
>
> Key: MRESOLVER-357
> URL: https://issues.apache.org/jira/browse/MRESOLVER-357
> Project: Maven Resolver
>  Issue Type: Bug
>  Components: Resolver
>Affects Versions: 1.9.8, 1.9.9
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> As per user report, the 1.9.8 and 1.9.9 versions produce different "verbose 
> tree" than previous versions:
> {noformat}
> Input node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  +- some-group:c:jar:1.0 [compile]
> |  +- some-group:c:jar:2.0 [compile]
> |  \- some-group:d:jar:1.0 [compile]
> +- some-group:c:jar:2.0 [compile]
> +- some-group:c:jar:1.0 [compile]
> \- some-group:d:jar:2.0 [compile]
> Transformed node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  \- some-group:c:jar:2.0 [compile] (nearer exists)
> +- some-group:c:jar:2.0 [compile]
> \- some-group:d:jar:2.0 [compile] {noformat}
> The {{some-group:d:jar:1.0 [compile]}} should not be removed.
> Seems that code comment "leaving only 1 loser" is present in comments only, 
> but the actual code does not deliver this promise.
> In fact, the check is wrong: it should check for same GA child count and not 
> ANY child count.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717668#comment-17717668
 ] 

Gary D. Gregory commented on MSHARED-1247:
--

Here is an attempt at fixing this issue: 
[https://github.com/apache/maven-dependency-analyzer/pull/88]

It might not be complete or there might be other issues.

If I run "mvn clean install" and then update BCEL's pom.xml to:

{noformat}
diff --git a/pom.xml b/pom.xml
index 7ebef432..4b3bd4e1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -27,7 +27,7 @@
   
 org.apache.commons
 commons-parent
-56
+57^M
   

   org.apache.bcel
@@ -210,6 +210,17 @@
   
 clean verify apache-rat:check japicmp:cmp checkstyle:check 
pmd:check spotbugs:check javadoc:javadoc
 
+  ^M
+org.cyclonedx^M
+cyclonedx-maven-plugin^M
+^M
+  ^M
+org.apache.maven.shared^M
+maven-dependency-analyzer^M
+1.13.2-SNAPSHOT^M
+  ^M
+   ^M
+  ^M
   
 org.apache.felix
 maven-bundle-plugin
{noformat}

I then get a different error that suggests the parsing is still wrong: 
{noformat}
Caused by: java.lang.IllegalArgumentException: Unsupported class file major 
version 1025
at org.objectweb.asm.ClassReader. (ClassReader.java:199)
at org.objectweb.asm.ClassReader. (ClassReader.java:180)
at org.objectweb.asm.ClassReader. (ClassReader.java:166)
at 
org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
 (DependencyClassFileVisitor.java:57)
at 
org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
(ClassFileVisitorUtils.java:114)
at 
org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
(ClassFileVisitorUtils.java:106)
at 
org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
 (ClassFileVisitorUtils.java:97)
at org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
(ClassFileVisitorUtils.java:59)
at 
org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
(ASMDependencyAnalyzer.java:43)
{noformat}

Any suggestions or fix?


> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComp

[jira] [Commented] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MSHARED-1247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717666#comment-17717666
 ] 

ASF GitHub Bot commented on MSHARED-1247:
-

garydgregory opened a new pull request, #88:
URL: https://github.com/apache/maven-dependency-analyzer/pull/88

   - Parses constant pool tag 17 per 
https://docs.oracle.com/javase/specs/jvms/se17/html/jvms-4.html#jvms-4.4.10
   - No unit test :-(




> java.lang.RuntimeException: Unknown constant pool type '17'
> ---
>
> Key: MSHARED-1247
> URL: https://issues.apache.org/jira/browse/MSHARED-1247
> Project: Maven Shared Components
>  Issue Type: Bug
>  Components: maven-dependency-analyzer
>Affects Versions: maven-dependency-analyzer-1.13.1
> Environment: Apache Maven 3.9.1 
> (2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
> Maven home: C:\java\apache-maven-3.9.1
> Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
> Adoptium\jdk-8.0.362.9-hotspot\jre
> Default locale: en_US, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
> Microsoft Windows [Version 10.0.19044.2728]
>Reporter: Gary D. Gregory
>Priority: Blocker
>
> You get the exception:
> {noformat}
> ...
> Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
>  (ConstantPoolParser.java:120)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
>  (ConstantPoolParser.java:104)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
>  (DependencyClassFileVisitor.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:114)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
> (ClassFileVisitorUtils.java:106)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
>  (ClassFileVisitorUtils.java:97)
> at 
> org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
> (ClassFileVisitorUtils.java:59)
> at 
> org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
> (ASMDependencyAnalyzer.java:43)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:206)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
>  (DefaultProjectDependencyAnalyzer.java:200)
> at 
> org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
>  (DefaultProjectDependencyAnalyzer.java:68)
> at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
> (CycloneDxMojo.java:86)
> at 
> org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
> (CycloneDxAggregateMojo.java:130)
> at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
> (BaseCycloneDxMojo.java:258)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
> (DefaultBuildPluginManager.java:126)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
> (MojoExecutor.java:342)
> at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
> (MojoExecutor.java:330)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:213)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:175)
> at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
> (MojoExecutor.java:76)
> at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
> (MojoExecutor.java:163)
> at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
> (DefaultMojosExecutionStrategy.java:39)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
> (MojoExecutor.java:160)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:105)
> at 
> org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
> (LifecycleModuleBuilder.java:73)
> at 
> org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
>  (SingleThreadedBuilder.java:53)
> at org.apache.maven.lifecycle.internal.LifecycleStarter.execute 
> (LifecycleStarter.java:118)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
> at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
> at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
> at org.apache.maven.cli.MavenCli.execute (MavenCli.java:827

[GitHub] [maven-dependency-analyzer] garydgregory opened a new pull request, #88: [MSHARED-1247] java.lang.RuntimeException: Unknown constant pool type

2023-04-28 Thread via GitHub


garydgregory opened a new pull request, #88:
URL: https://github.com/apache/maven-dependency-analyzer/pull/88

   - Parses constant pool tag 17 per 
https://docs.oracle.com/javase/specs/jvms/se17/html/jvms-4.html#jvms-4.4.10
   - No unit test :-(


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (MSHARED-1247) java.lang.RuntimeException: Unknown constant pool type '17'

2023-04-28 Thread Gary D. Gregory (Jira)
Gary D. Gregory created MSHARED-1247:


 Summary: java.lang.RuntimeException: Unknown constant pool type 
'17'
 Key: MSHARED-1247
 URL: https://issues.apache.org/jira/browse/MSHARED-1247
 Project: Maven Shared Components
  Issue Type: Bug
  Components: maven-dependency-analyzer
Affects Versions: maven-dependency-analyzer-1.13.1
 Environment: Apache Maven 3.9.1 
(2e178502fcdbffc201671fb2537d0cb4b4cc58f8)
Maven home: C:\java\apache-maven-3.9.1
Java version: 1.8.0_362, vendor: Temurin, runtime: C:\Program Files\Eclipse 
Adoptium\jdk-8.0.362.9-hotspot\jre
Default locale: en_US, platform encoding: Cp1252
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"

Microsoft Windows [Version 10.0.19044.2728]
Reporter: Gary D. Gregory


You get the exception:
{noformat}
...
Caused by: java.lang.RuntimeException: Unknown constant pool type '17'
at 
org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.parseConstantPoolClassReferences
 (ConstantPoolParser.java:120)
at 
org.apache.maven.shared.dependency.analyzer.asm.ConstantPoolParser.getConstantPoolClassReferences
 (ConstantPoolParser.java:104)
at 
org.apache.maven.shared.dependency.analyzer.asm.DependencyClassFileVisitor.visitClass
 (DependencyClassFileVisitor.java:59)
at 
org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
(ClassFileVisitorUtils.java:114)
at 
org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.visitClass 
(ClassFileVisitorUtils.java:106)
at 
org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.acceptDirectory
 (ClassFileVisitorUtils.java:97)
at org.apache.maven.shared.dependency.analyzer.ClassFileVisitorUtils.accept 
(ClassFileVisitorUtils.java:59)
at 
org.apache.maven.shared.dependency.analyzer.asm.ASMDependencyAnalyzer.analyze 
(ASMDependencyAnalyzer.java:43)
at 
org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildDependencyClasses
 (DefaultProjectDependencyAnalyzer.java:206)
at 
org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.buildTestDependencyClasses
 (DefaultProjectDependencyAnalyzer.java:200)
at 
org.apache.maven.shared.dependency.analyzer.DefaultProjectDependencyAnalyzer.analyze
 (DefaultProjectDependencyAnalyzer.java:68)
at org.cyclonedx.maven.CycloneDxMojo.doProjectDependencyAnalysis 
(CycloneDxMojo.java:86)
at 
org.cyclonedx.maven.CycloneDxAggregateMojo.extractComponentsAndDependencies 
(CycloneDxAggregateMojo.java:130)
at org.cyclonedx.maven.BaseCycloneDxMojo.execute 
(BaseCycloneDxMojo.java:258)
at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo 
(DefaultBuildPluginManager.java:126)
at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 
(MojoExecutor.java:342)
at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute 
(MojoExecutor.java:330)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
(MojoExecutor.java:213)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
(MojoExecutor.java:175)
at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 
(MojoExecutor.java:76)
at org.apache.maven.lifecycle.internal.MojoExecutor$1.run 
(MojoExecutor.java:163)
at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute 
(DefaultMojosExecutionStrategy.java:39)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute 
(MojoExecutor.java:160)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
(LifecycleModuleBuilder.java:105)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject 
(LifecycleModuleBuilder.java:73)
at 
org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build
 (SingleThreadedBuilder.java:53)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute 
(LifecycleStarter.java:118)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:261)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:173)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:827)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:272)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:195)
at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke 
(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke 
(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:498)
at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced 
(Launcher.java:282)
at org.codehaus.plexus.classworlds.launcher.Launcher.launch 
(Launcher.java:225)
at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode

[jira] [Commented] (MNG-7338) Reduce carbon footprint in CI

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717656#comment-17717656
 ] 

ASF GitHub Bot commented on MNG-7338:
-

michael-o commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180344798


##
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##
@@ -444,10 +447,10 @@ private CommandLine cliMerge(CommandLine mavenConfig, 
CommandLine mavenCli) {
  */
 void logging(CliRequest cliRequest) {
 // LOG LEVEL
-cliRequest.verbose = 
cliRequest.commandLine.hasOption(CLIManager.VERBOSE)
-|| cliRequest.commandLine.hasOption(CLIManager.DEBUG);
-cliRequest.quiet = !cliRequest.verbose && 
cliRequest.commandLine.hasOption(CLIManager.QUIET);
-cliRequest.showErrors = cliRequest.verbose || 
cliRequest.commandLine.hasOption(CLIManager.ERRORS);
+CommandLine commandLine = cliRequest.commandLine;
+cliRequest.verbose = commandLine.hasOption(CLIManager.VERBOSE) || 
commandLine.hasOption(CLIManager.DEBUG);

Review Comment:
   Fine





> Reduce carbon footprint in CI
> -
>
> Key: MNG-7338
> URL: https://issues.apache.org/jira/browse/MNG-7338
> Project: Maven
>  Issue Type: Bug
>Reporter: Jörg Hohwiller
>Assignee: Martin Kanters
>Priority: Major
>
> MNG-4198 was closed with a simple workaround to add {{-B}} option.
> However, if you look at the real world you will notice that in travis, 
> circle-ci, jenkins, github-actions, etc. 99% of the builds do not use it (not 
> even by defaults from the makers of build templates) and hence 80% of the log 
> files are pure spam and waste:
> {code}
> Progress (2): 0.9/2.6 MB | 160/502 kB
> Progress (2): 0.9/2.6 MB | 164/502 kB
> Progress (2): 0.9/2.6 MB | 168/502 kB
> Progress (2): 0.9/2.6 MB | 172/502 kB
> Progress (2): 0.9/2.6 MB | 176/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 184/502 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 4.1/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 8.2/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 12/196 kB 
> Progress (3): 1.0/2.6 MB | 184/502 kB | 16/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 20/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 25/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 29/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 33/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 37/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 41/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 45/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 49/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 53/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 57/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 61/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 66/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 70/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 74/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 78/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 82/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 86/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 90/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 94/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 98/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 102/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 106/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 111/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 188/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 193/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 197/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 201/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 205/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 209/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 213/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 217/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 221/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 225/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 229/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 233/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 238/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 242/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 246/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 250/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 254/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 258/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 262/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 266/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 270/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 274/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 279/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB 

[GitHub] [maven] michael-o commented on a diff in pull request #869: [MNG-7338] Automatically activate batch-mode and make output quiet when running in CI.

2023-04-28 Thread via GitHub


michael-o commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180344798


##
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##
@@ -444,10 +447,10 @@ private CommandLine cliMerge(CommandLine mavenConfig, 
CommandLine mavenCli) {
  */
 void logging(CliRequest cliRequest) {
 // LOG LEVEL
-cliRequest.verbose = 
cliRequest.commandLine.hasOption(CLIManager.VERBOSE)
-|| cliRequest.commandLine.hasOption(CLIManager.DEBUG);
-cliRequest.quiet = !cliRequest.verbose && 
cliRequest.commandLine.hasOption(CLIManager.QUIET);
-cliRequest.showErrors = cliRequest.verbose || 
cliRequest.commandLine.hasOption(CLIManager.ERRORS);
+CommandLine commandLine = cliRequest.commandLine;
+cliRequest.verbose = commandLine.hasOption(CLIManager.VERBOSE) || 
commandLine.hasOption(CLIManager.DEBUG);

Review Comment:
   Fine



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MENFORCER-479) Version enforcer rules print single versions when a range is expected

2023-04-28 Thread Slawomir Jaranowski (Jira)


[ 
https://issues.apache.org/jira/browse/MENFORCER-479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717651#comment-17717651
 ] 

Slawomir Jaranowski commented on MENFORCER-479:
---

Which version of plugin?

Did you test with the latest version?

> Version enforcer rules print single versions when a range is expected
> -
>
> Key: MENFORCER-479
> URL: https://issues.apache.org/jira/browse/MENFORCER-479
> Project: Maven Enforcer Plugin
>  Issue Type: Bug
>  Components: Rule API
>Reporter: Elliotte Rusty Harold
>Priority: Major
>
> AbstractStandardEnforcerRule incorrectly assumes that the toString method of 
> a VersionRange will be a version range. However it can be just a single 
> version rather than the entire range. E.g. 3.8.4 instead [3.8.0, 3.9)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MGPG-97) add pgpverify check to the build

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MGPG-97?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717648#comment-17717648
 ] 

ASF GitHub Bot commented on MGPG-97:


hboutemy commented on PR #48:
URL: https://github.com/apache/maven-gpg-plugin/pull/48#issuecomment-1527483398

   this is part for the experience: if we do not check signature, why sign at 
all?




> add pgpverify check to the build
> 
>
> Key: MGPG-97
> URL: https://issues.apache.org/jira/browse/MGPG-97
> Project: Maven GPG Plugin
>  Issue Type: Improvement
>Affects Versions: 3.0.1
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.1.0
>
>
> signing is useful only if signature checks are done
> let's apply this best practice to maven-gpg-plugin itself



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-gpg-plugin] hboutemy commented on pull request #48: [MGPG-97] use pgpverify plugin to check dependencies signatures

2023-04-28 Thread via GitHub


hboutemy commented on PR #48:
URL: https://github.com/apache/maven-gpg-plugin/pull/48#issuecomment-1527483398

   this is part for the experience: if we do not check signature, why sign at 
all?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MENFORCER-479) Version enforcer rules print single versions when a range is expected

2023-04-28 Thread Elliotte Rusty Harold (Jira)


[ 
https://issues.apache.org/jira/browse/MENFORCER-479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717638#comment-17717638
 ] 

Elliotte Rusty Harold commented on MENFORCER-479:
-

Real world example from RequireMavenVersion:


[ERROR] Rule 0: org.apache.maven.plugins.enforcer.RequireMavenVersion
failed with message:
Detected Maven Version: 3.5.0 is not in the allowed range 3.8.4


3.8.4 is a single version, not a range. Either fix VersionRange.toString to 
always return a range or don't use toString here.




> Version enforcer rules print single versions when a range is expected
> -
>
> Key: MENFORCER-479
> URL: https://issues.apache.org/jira/browse/MENFORCER-479
> Project: Maven Enforcer Plugin
>  Issue Type: Bug
>  Components: Rule API
>Reporter: Elliotte Rusty Harold
>Priority: Major
>
> AbstractStandardEnforcerRule incorrectly assumes that the toString method of 
> a VersionRange will be a version range. However it can be just a single 
> version rather than the entire range. E.g. 3.8.4 instead [3.8.0, 3.9)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (MENFORCER-479) Version enforcer rules print single versions when a range is expected

2023-04-28 Thread Elliotte Rusty Harold (Jira)
Elliotte Rusty Harold created MENFORCER-479:
---

 Summary: Version enforcer rules print single versions when a range 
is expected
 Key: MENFORCER-479
 URL: https://issues.apache.org/jira/browse/MENFORCER-479
 Project: Maven Enforcer Plugin
  Issue Type: Bug
  Components: Rule API
Reporter: Elliotte Rusty Harold


AbstractStandardEnforcerRule incorrectly assumes that the toString method of a 
VersionRange will be a version range. However it can be just a single version 
rather than the entire range. E.g. 3.8.4 instead [3.8.0, 3.9)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (MNG-7775) toString method in VersionRange is undocumented and surprising

2023-04-28 Thread Elliotte Rusty Harold (Jira)
Elliotte Rusty Harold created MNG-7775:
--

 Summary: toString method in VersionRange is undocumented and 
surprising
 Key: MNG-7775
 URL: https://issues.apache.org/jira/browse/MNG-7775
 Project: Maven
  Issue Type: Bug
Reporter: Elliotte Rusty Harold


It sometimes returns a single version rather than the full range, though the 
only way to see this is by reading the code. I've seen this create confusing 
end user messages in enforcer rules. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717632#comment-17717632
 ] 

ASF GitHub Bot commented on MRESOLVER-357:
--

cstamas opened a new pull request, #283:
URL: https://github.com/apache/maven-resolver/pull/283

   The check was wrong, it was "has ANY sibling", while it should be "has ANY 
RELATED sibling".
   
   ---
   
   https://issues.apache.org/jira/browse/MRESOLVER-357




> ConflictResolver STANDARD verbosity misbehaves
> --
>
> Key: MRESOLVER-357
> URL: https://issues.apache.org/jira/browse/MRESOLVER-357
> Project: Maven Resolver
>  Issue Type: Bug
>  Components: Resolver
>Affects Versions: 1.9.8, 1.9.9
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
> tree" that previous versions:
> {noformat}
> Input node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  +- some-group:c:jar:1.0 [compile]
> |  +- some-group:c:jar:2.0 [compile]
> |  \- some-group:d:jar:1.0 [compile]
> +- some-group:c:jar:2.0 [compile]
> +- some-group:c:jar:1.0 [compile]
> \- some-group:d:jar:2.0 [compile]
> Transformed node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  \- some-group:c:jar:2.0 [compile] (nearer exists)
> +- some-group:c:jar:2.0 [compile]
> \- some-group:d:jar:2.0 [compile] {noformat}
> The {{some-group:d:jar:1.0 [compile]}} should not be removed.
> Seems that code comment "leaving only 1 loser" is present in comments only, 
> but the actual code does not deliver this promise.
> In fact, the check is wrong: it should check for same GA child count and not 
> ANY child count.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] cstamas opened a new pull request, #283: [MRESOLVER-357] Fix the code to deliver the promise in comment

2023-04-28 Thread via GitHub


cstamas opened a new pull request, #283:
URL: https://github.com/apache/maven-resolver/pull/283

   The check was wrong, it was "has ANY sibling", while it should be "has ANY 
RELATED sibling".
   
   ---
   
   https://issues.apache.org/jira/browse/MRESOLVER-357


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MNG-7714) sp < final

2023-04-28 Thread Elliotte Rusty Harold (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717626#comment-17717626
 ] 

Elliotte Rusty Harold commented on MNG-7714:


Yes, I think that's true. What I haven't figured out yet is what the correct 
logic to fix this is. Any suggestions or PRs you might have are appreciated. 

> sp < final
> --
>
> Key: MNG-7714
> URL: https://issues.apache.org/jira/browse/MNG-7714
> Project: Maven
>  Issue Type: Bug
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
> Fix For: 4.0.0-alpha-6
>
>
> Ported from a comment on https://issues.apache.org/jira/browse/MNG-7701
> The claim is that sp < final, which if true is incorrect according to spec. 
> It is easy to demonstrate that this is not fixed and also not in line with 
> the spec, with just this one important example (yes this does break for us):
> $ jbang org.apache.maven:maven-artifact:3.8.6 1.0.final-redhat-0001 
> 1.0.sp1-redhat-0001
> Display parameters as parsed by Maven (in canonical form and as a list of 
> tokens) and comparison result:
> 1. 1.0.final-redhat-0001 -> 1-redhat-1; tokens: [1, [redhat, [1]]]
>1.0.final-redhat-0001 < 1.0.sp1-redhat-0001
> 2. 1.0.sp1-redhat-0001 -> 1.0.sp-1-redhat-1; tokens: [1, 0, sp, [1, [redhat, 
> [1
> versus
> $ jbang org.apache.maven:maven-artifact:3.8.7 1.0.final-redhat-0001 
> 1.0.sp1-redhat-0001
> Display parameters as parsed by Maven (in canonical form and as a list of 
> tokens) and comparison result:
> 1. 1.0.final-redhat-0001 -> 1-redhat-1; tokens: [1, [redhat, [1]]]
>1.0.final-redhat-0001 > 1.0.sp1-redhat-0001
> 2. 1.0.sp1-redhat-0001 -> 1-sp-1-redhat-1; tokens: [1, [sp, [1, [redhat, 
> [1]
> As you can see, our `sp` release is now ordered after our `final` release 
> despite this clear text in the "spec":
> Non-numeric tokens ("qualifiers") have the alphabetical order, except for 
> the following tokens which come first in this order: "alpha" < "beta" < 
> "milestone" < "rc" = "cr" < "snapshot" < "" = "final" = "ga" < "sp"
> It's clear that this tokenization isn't really correct by any reasonable 
> measurement, and breaking large amounts of (our) existing artifacts in the 
> wild is definitely not OK.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-357?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-357:
--
Description: 
As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" that previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The {{some-group:d:jar:1.0 [compile]}} should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise.

In fact, the check is wrong: it should check for same GA child count and not 
ANY child count.

  was:
As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" that previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The {{some-group:d:jar:1.0 [compile]}} should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise (the check is wrong, it should 
check for same GAs and not ANY child count).


> ConflictResolver STANDARD verbosity misbehaves
> --
>
> Key: MRESOLVER-357
> URL: https://issues.apache.org/jira/browse/MRESOLVER-357
> Project: Maven Resolver
>  Issue Type: Bug
>  Components: Resolver
>Affects Versions: 1.9.8, 1.9.9
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
> tree" that previous versions:
> {noformat}
> Input node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  +- some-group:c:jar:1.0 [compile]
> |  +- some-group:c:jar:2.0 [compile]
> |  \- some-group:d:jar:1.0 [compile]
> +- some-group:c:jar:2.0 [compile]
> +- some-group:c:jar:1.0 [compile]
> \- some-group:d:jar:2.0 [compile]
> Transformed node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  \- some-group:c:jar:2.0 [compile] (nearer exists)
> +- some-group:c:jar:2.0 [compile]
> \- some-group:d:jar:2.0 [compile] {noformat}
> The {{some-group:d:jar:1.0 [compile]}} should not be removed.
> Seems that code comment "leaving only 1 loser" is present in comments only, 
> but the actual code does not deliver this promise.
> In fact, the check is wrong: it should check for same GA child count and not 
> ANY child count.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread Tamas Cservenak (Jira)
Tamas Cservenak created MRESOLVER-357:
-

 Summary: ConflictResolver STANDARD verbosity misbehaves
 Key: MRESOLVER-357
 URL: https://issues.apache.org/jira/browse/MRESOLVER-357
 Project: Maven Resolver
  Issue Type: Bug
  Components: Resolver
Affects Versions: 1.9.9, 1.9.8
Reporter: Tamas Cservenak
 Fix For: 1.9.10


As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" that previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The d:1.0 should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise (the check is wrong, it should 
check for same GAs and not ANY child count).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (MRESOLVER-357) ConflictResolver STANDARD verbosity misbehaves

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MRESOLVER-357?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak updated MRESOLVER-357:
--
Description: 
As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" that previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The {{some-group:d:jar:1.0 [compile]}} should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise (the check is wrong, it should 
check for same GAs and not ANY child count).

  was:
As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
tree" that previous versions:
{noformat}
Input node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  +- some-group:c:jar:1.0 [compile]
|  +- some-group:c:jar:2.0 [compile]
|  \- some-group:d:jar:1.0 [compile]
+- some-group:c:jar:2.0 [compile]
+- some-group:c:jar:1.0 [compile]
\- some-group:d:jar:2.0 [compile]

Transformed node:
some-group:a:jar:1.0 [compile]
+- some-group:b:jar:1.0 [compile]
|  \- some-group:c:jar:2.0 [compile] (nearer exists)
+- some-group:c:jar:2.0 [compile]
\- some-group:d:jar:2.0 [compile] {noformat}
The d:1.0 should not be removed.

Seems that code comment "leaving only 1 loser" is present in comments only, but 
the actual code does not deliver this promise (the check is wrong, it should 
check for same GAs and not ANY child count).


> ConflictResolver STANDARD verbosity misbehaves
> --
>
> Key: MRESOLVER-357
> URL: https://issues.apache.org/jira/browse/MRESOLVER-357
> Project: Maven Resolver
>  Issue Type: Bug
>  Components: Resolver
>Affects Versions: 1.9.8, 1.9.9
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.9.10
>
>
> As per use report, the 1.9.8 and 1.9.9 versions produce different "verbose 
> tree" that previous versions:
> {noformat}
> Input node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  +- some-group:c:jar:1.0 [compile]
> |  +- some-group:c:jar:2.0 [compile]
> |  \- some-group:d:jar:1.0 [compile]
> +- some-group:c:jar:2.0 [compile]
> +- some-group:c:jar:1.0 [compile]
> \- some-group:d:jar:2.0 [compile]
> Transformed node:
> some-group:a:jar:1.0 [compile]
> +- some-group:b:jar:1.0 [compile]
> |  \- some-group:c:jar:2.0 [compile] (nearer exists)
> +- some-group:c:jar:2.0 [compile]
> \- some-group:d:jar:2.0 [compile] {noformat}
> The {{some-group:d:jar:1.0 [compile]}} should not be removed.
> Seems that code comment "leaving only 1 loser" is present in comments only, 
> but the actual code does not deliver this promise (the check is wrong, it 
> should check for same GAs and not ANY child count).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MGPG-97) add pgpverify check to the build

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MGPG-97?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717591#comment-17717591
 ] 

ASF GitHub Bot commented on MGPG-97:


hboutemy commented on code in PR #48:
URL: https://github.com/apache/maven-gpg-plugin/pull/48#discussion_r1180202813


##
pom.xml:
##
@@ -194,6 +194,14 @@ under the License.
   maven-invoker-plugin
   3.5.1
 
+
+  org.simplify4u.plugins
+  pgpverify-maven-plugin
+  1.11.0

Review Comment:
   updated, thank you





> add pgpverify check to the build
> 
>
> Key: MGPG-97
> URL: https://issues.apache.org/jira/browse/MGPG-97
> Project: Maven GPG Plugin
>  Issue Type: Improvement
>Affects Versions: 3.0.1
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.1.0
>
>
> signing is useful only if signature checks are done
> let's apply this best practice to maven-gpg-plugin itself



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-gpg-plugin] hboutemy commented on a diff in pull request #48: [MGPG-97] use pgpverify plugin to check dependencies signatures

2023-04-28 Thread via GitHub


hboutemy commented on code in PR #48:
URL: https://github.com/apache/maven-gpg-plugin/pull/48#discussion_r1180202813


##
pom.xml:
##
@@ -194,6 +194,14 @@ under the License.
   maven-invoker-plugin
   3.5.1
 
+
+  org.simplify4u.plugins
+  pgpverify-maven-plugin
+  1.11.0

Review Comment:
   updated, thank you



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Closed] (MGPG-96) add INFO message when signing

2023-04-28 Thread Herve Boutemy (Jira)


 [ 
https://issues.apache.org/jira/browse/MGPG-96?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Herve Boutemy closed MGPG-96.
-
  Assignee: Herve Boutemy
Resolution: Fixed

> add INFO message when signing
> -
>
> Key: MGPG-96
> URL: https://issues.apache.org/jira/browse/MGPG-96
> Project: Maven GPG Plugin
>  Issue Type: Improvement
>Affects Versions: 3.0.1
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.1.0
>
>
> currently, gpg:sign does not display any INFO, which makes feel that nothing 
> may happen:
> {noformat}
> [INFO] --- gpg:3.1.0-SNAPSHOT:sign (sign-release-artifacts) @ 
> maven-gpg-plugin ---
> [INFO] {noformat}
> Displaying most important info, like number of files being signed, or key 
> configuration options (like optional keyname) would make this output still 
> simple enough but show the action being done, something like:
> {noformat}
> [INFO] --- gpg:3.1.0-SNAPSHOT:sign (sign-release-artifacts) @ 
> maven-gpg-plugin ---
> [INFO] Signing 5 files with default key
> [INFO]{noformat}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MGPG-96) add INFO message when signing

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MGPG-96?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717588#comment-17717588
 ] 

ASF GitHub Bot commented on MGPG-96:


hboutemy merged PR #47:
URL: https://github.com/apache/maven-gpg-plugin/pull/47




> add INFO message when signing
> -
>
> Key: MGPG-96
> URL: https://issues.apache.org/jira/browse/MGPG-96
> Project: Maven GPG Plugin
>  Issue Type: Improvement
>Affects Versions: 3.0.1
>Reporter: Herve Boutemy
>Priority: Major
> Fix For: 3.1.0
>
>
> currently, gpg:sign does not display any INFO, which makes feel that nothing 
> may happen:
> {noformat}
> [INFO] --- gpg:3.1.0-SNAPSHOT:sign (sign-release-artifacts) @ 
> maven-gpg-plugin ---
> [INFO] {noformat}
> Displaying most important info, like number of files being signed, or key 
> configuration options (like optional keyname) would make this output still 
> simple enough but show the action being done, something like:
> {noformat}
> [INFO] --- gpg:3.1.0-SNAPSHOT:sign (sign-release-artifacts) @ 
> maven-gpg-plugin ---
> [INFO] Signing 5 files with default key
> [INFO]{noformat}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-gpg-plugin] hboutemy merged pull request #47: [MGPG-96] add INFO message

2023-04-28 Thread via GitHub


hboutemy merged PR #47:
URL: https://github.com/apache/maven-gpg-plugin/pull/47


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MNG-7338) Reduce carbon footprint in CI

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717569#comment-17717569
 ] 

ASF GitHub Bot commented on MNG-7338:
-

MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180163974


##
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##
@@ -444,10 +447,10 @@ private CommandLine cliMerge(CommandLine mavenConfig, 
CommandLine mavenCli) {
  */
 void logging(CliRequest cliRequest) {
 // LOG LEVEL
-cliRequest.verbose = 
cliRequest.commandLine.hasOption(CLIManager.VERBOSE)
-|| cliRequest.commandLine.hasOption(CLIManager.DEBUG);
-cliRequest.quiet = !cliRequest.verbose && 
cliRequest.commandLine.hasOption(CLIManager.QUIET);
-cliRequest.showErrors = cliRequest.verbose || 
cliRequest.commandLine.hasOption(CLIManager.ERRORS);
+CommandLine commandLine = cliRequest.commandLine;
+cliRequest.verbose = commandLine.hasOption(CLIManager.VERBOSE) || 
commandLine.hasOption(CLIManager.DEBUG);

Review Comment:
   Hmm, I think it's a nice improvement, but since it's just formatting 
related, I would rather just ship it with this PR. 





> Reduce carbon footprint in CI
> -
>
> Key: MNG-7338
> URL: https://issues.apache.org/jira/browse/MNG-7338
> Project: Maven
>  Issue Type: Bug
>Reporter: Jörg Hohwiller
>Assignee: Martin Kanters
>Priority: Major
>
> MNG-4198 was closed with a simple workaround to add {{-B}} option.
> However, if you look at the real world you will notice that in travis, 
> circle-ci, jenkins, github-actions, etc. 99% of the builds do not use it (not 
> even by defaults from the makers of build templates) and hence 80% of the log 
> files are pure spam and waste:
> {code}
> Progress (2): 0.9/2.6 MB | 160/502 kB
> Progress (2): 0.9/2.6 MB | 164/502 kB
> Progress (2): 0.9/2.6 MB | 168/502 kB
> Progress (2): 0.9/2.6 MB | 172/502 kB
> Progress (2): 0.9/2.6 MB | 176/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 184/502 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 4.1/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 8.2/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 12/196 kB 
> Progress (3): 1.0/2.6 MB | 184/502 kB | 16/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 20/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 25/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 29/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 33/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 37/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 41/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 45/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 49/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 53/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 57/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 61/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 66/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 70/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 74/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 78/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 82/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 86/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 90/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 94/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 98/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 102/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 106/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 111/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 188/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 193/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 197/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 201/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 205/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 209/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 213/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 217/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 221/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 225/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 229/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 233/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 238/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 242/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 246/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 250/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 254/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 258/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 262/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 266/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 270/502 kB | 115/196 kB
> Progress (

[GitHub] [maven] MartinKanters commented on a diff in pull request #869: [MNG-7338] Automatically activate batch-mode and make output quiet when running in CI.

2023-04-28 Thread via GitHub


MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180163974


##
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##
@@ -444,10 +447,10 @@ private CommandLine cliMerge(CommandLine mavenConfig, 
CommandLine mavenCli) {
  */
 void logging(CliRequest cliRequest) {
 // LOG LEVEL
-cliRequest.verbose = 
cliRequest.commandLine.hasOption(CLIManager.VERBOSE)
-|| cliRequest.commandLine.hasOption(CLIManager.DEBUG);
-cliRequest.quiet = !cliRequest.verbose && 
cliRequest.commandLine.hasOption(CLIManager.QUIET);
-cliRequest.showErrors = cliRequest.verbose || 
cliRequest.commandLine.hasOption(CLIManager.ERRORS);
+CommandLine commandLine = cliRequest.commandLine;
+cliRequest.verbose = commandLine.hasOption(CLIManager.VERBOSE) || 
commandLine.hasOption(CLIManager.DEBUG);

Review Comment:
   Hmm, I think it's a nice improvement, but since it's just formatting 
related, I would rather just ship it with this PR. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MNG-7714) sp < final

2023-04-28 Thread Zhongming Hua (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717556#comment-17717556
 ] 

Zhongming Hua commented on MNG-7714:


[~elharo] 

I find that this problem is related to the following logic. Because treat *.x* 
as {*}-x{*}, 1.0.sp is resolved as [1,[sp]], not [1,sp].

 
{code:java}
// 1.0.0.X1 < 1.0.0-X2
// treat .X as -X for any string qualifier X
if (!list.isEmpty()) {
list.add(list = new ListItem());
stack.push(list);
} {code}
 

 

 

 

> sp < final
> --
>
> Key: MNG-7714
> URL: https://issues.apache.org/jira/browse/MNG-7714
> Project: Maven
>  Issue Type: Bug
>Reporter: Elliotte Rusty Harold
>Assignee: Elliotte Rusty Harold
>Priority: Major
> Fix For: 4.0.0-alpha-6
>
>
> Ported from a comment on https://issues.apache.org/jira/browse/MNG-7701
> The claim is that sp < final, which if true is incorrect according to spec. 
> It is easy to demonstrate that this is not fixed and also not in line with 
> the spec, with just this one important example (yes this does break for us):
> $ jbang org.apache.maven:maven-artifact:3.8.6 1.0.final-redhat-0001 
> 1.0.sp1-redhat-0001
> Display parameters as parsed by Maven (in canonical form and as a list of 
> tokens) and comparison result:
> 1. 1.0.final-redhat-0001 -> 1-redhat-1; tokens: [1, [redhat, [1]]]
>1.0.final-redhat-0001 < 1.0.sp1-redhat-0001
> 2. 1.0.sp1-redhat-0001 -> 1.0.sp-1-redhat-1; tokens: [1, 0, sp, [1, [redhat, 
> [1
> versus
> $ jbang org.apache.maven:maven-artifact:3.8.7 1.0.final-redhat-0001 
> 1.0.sp1-redhat-0001
> Display parameters as parsed by Maven (in canonical form and as a list of 
> tokens) and comparison result:
> 1. 1.0.final-redhat-0001 -> 1-redhat-1; tokens: [1, [redhat, [1]]]
>1.0.final-redhat-0001 > 1.0.sp1-redhat-0001
> 2. 1.0.sp1-redhat-0001 -> 1-sp-1-redhat-1; tokens: [1, [sp, [1, [redhat, 
> [1]
> As you can see, our `sp` release is now ordered after our `final` release 
> despite this clear text in the "spec":
> Non-numeric tokens ("qualifiers") have the alphabetical order, except for 
> the following tokens which come first in this order: "alpha" < "beta" < 
> "milestone" < "rc" = "cr" < "snapshot" < "" = "final" = "ga" < "sp"
> It's clear that this tokenization isn't really correct by any reasonable 
> measurement, and breaking large amounts of (our) existing artifacts in the 
> wild is definitely not OK.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MNG-7338) Reduce carbon footprint in CI

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717553#comment-17717553
 ] 

ASF GitHub Bot commented on MNG-7338:
-

MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180126561


##
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##
@@ -38,6 +38,10 @@ public class CLIManager {
 
 public static final char BATCH_MODE = 'B';
 
+public static final String NON_INTERACTIVE = "ni";
+
+public static final String FORCE_INTERACTIVE = "fi";

Review Comment:
   I agree, I've removed them





> Reduce carbon footprint in CI
> -
>
> Key: MNG-7338
> URL: https://issues.apache.org/jira/browse/MNG-7338
> Project: Maven
>  Issue Type: Bug
>Reporter: Jörg Hohwiller
>Assignee: Martin Kanters
>Priority: Major
>
> MNG-4198 was closed with a simple workaround to add {{-B}} option.
> However, if you look at the real world you will notice that in travis, 
> circle-ci, jenkins, github-actions, etc. 99% of the builds do not use it (not 
> even by defaults from the makers of build templates) and hence 80% of the log 
> files are pure spam and waste:
> {code}
> Progress (2): 0.9/2.6 MB | 160/502 kB
> Progress (2): 0.9/2.6 MB | 164/502 kB
> Progress (2): 0.9/2.6 MB | 168/502 kB
> Progress (2): 0.9/2.6 MB | 172/502 kB
> Progress (2): 0.9/2.6 MB | 176/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 184/502 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 4.1/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 8.2/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 12/196 kB 
> Progress (3): 1.0/2.6 MB | 184/502 kB | 16/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 20/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 25/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 29/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 33/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 37/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 41/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 45/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 49/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 53/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 57/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 61/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 66/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 70/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 74/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 78/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 82/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 86/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 90/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 94/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 98/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 102/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 106/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 111/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 188/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 193/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 197/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 201/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 205/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 209/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 213/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 217/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 221/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 225/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 229/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 233/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 238/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 242/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 246/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 250/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 254/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 258/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 262/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 266/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 270/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 274/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 279/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 283/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 287/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 291/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 295/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 299/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 303/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 307/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 311/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 311/502 kB | 115/196 kB
> Progress (4): 1.0/2.6 MB | 

[jira] [Commented] (MNG-7338) Reduce carbon footprint in CI

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717551#comment-17717551
 ] 

ASF GitHub Bot commented on MNG-7338:
-

MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180126255


##
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##
@@ -173,7 +177,17 @@ public CLIManager() {
 .build());
 options.addOption(Option.builder(Character.toString(BATCH_MODE))
 .longOpt("batch-mode")
-.desc("Run in non-interactive (batch) mode (disables output 
color)")
+.desc(
+"Run in non-interactive (batch) mode (disables output 
color) - alias for --non-interactive (kept for backwards compatability)")
+.build());
+options.addOption(Option.builder(NON_INTERACTIVE)
+.longOpt("non-interactive")
+.desc("Run in non-interactive (batch) mode (disables output 
color) - alias for --batch-mode")
+.build());
+options.addOption(Option.builder(FORCE_INTERACTIVE)
+.longOpt("force-interactive")
+.desc(
+"Run in interactive mode - even when the environment 
variable CI is set to true and --non-interactive or --batch-mode are set")

Review Comment:
   I'm not sure how to pass this information to Jansi. `isatty(2)` will atm 
just return 1/true. Also if `--batch-mode` is set. 
   Do you think a change like this should be part of this PR? 





> Reduce carbon footprint in CI
> -
>
> Key: MNG-7338
> URL: https://issues.apache.org/jira/browse/MNG-7338
> Project: Maven
>  Issue Type: Bug
>Reporter: Jörg Hohwiller
>Assignee: Martin Kanters
>Priority: Major
>
> MNG-4198 was closed with a simple workaround to add {{-B}} option.
> However, if you look at the real world you will notice that in travis, 
> circle-ci, jenkins, github-actions, etc. 99% of the builds do not use it (not 
> even by defaults from the makers of build templates) and hence 80% of the log 
> files are pure spam and waste:
> {code}
> Progress (2): 0.9/2.6 MB | 160/502 kB
> Progress (2): 0.9/2.6 MB | 164/502 kB
> Progress (2): 0.9/2.6 MB | 168/502 kB
> Progress (2): 0.9/2.6 MB | 172/502 kB
> Progress (2): 0.9/2.6 MB | 176/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 184/502 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 4.1/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 8.2/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 12/196 kB 
> Progress (3): 1.0/2.6 MB | 184/502 kB | 16/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 20/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 25/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 29/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 33/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 37/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 41/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 45/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 49/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 53/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 57/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 61/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 66/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 70/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 74/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 78/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 82/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 86/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 90/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 94/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 98/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 102/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 106/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 111/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 188/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 193/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 197/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 201/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 205/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 209/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 213/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 217/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 221/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 225/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 229/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 233/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 238/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 242/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB

[jira] [Commented] (MNG-7338) Reduce carbon footprint in CI

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717554#comment-17717554
 ] 

ASF GitHub Bot commented on MNG-7338:
-

MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180127108


##
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##
@@ -173,7 +177,17 @@ public CLIManager() {
 .build());
 options.addOption(Option.builder(Character.toString(BATCH_MODE))
 .longOpt("batch-mode")
-.desc("Run in non-interactive (batch) mode (disables output 
color)")
+.desc(
+"Run in non-interactive (batch) mode (disables output 
color) - alias for --non-interactive (kept for backwards compatability)")

Review Comment:
   Yeah, that's true. I copied the original description of --batch-mode, but I 
agree that it doesn't cover the functionally anymore. I've rewritten them.





> Reduce carbon footprint in CI
> -
>
> Key: MNG-7338
> URL: https://issues.apache.org/jira/browse/MNG-7338
> Project: Maven
>  Issue Type: Bug
>Reporter: Jörg Hohwiller
>Assignee: Martin Kanters
>Priority: Major
>
> MNG-4198 was closed with a simple workaround to add {{-B}} option.
> However, if you look at the real world you will notice that in travis, 
> circle-ci, jenkins, github-actions, etc. 99% of the builds do not use it (not 
> even by defaults from the makers of build templates) and hence 80% of the log 
> files are pure spam and waste:
> {code}
> Progress (2): 0.9/2.6 MB | 160/502 kB
> Progress (2): 0.9/2.6 MB | 164/502 kB
> Progress (2): 0.9/2.6 MB | 168/502 kB
> Progress (2): 0.9/2.6 MB | 172/502 kB
> Progress (2): 0.9/2.6 MB | 176/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 0.9/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 180/502 kB
> Progress (2): 1.0/2.6 MB | 184/502 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 4.1/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 8.2/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 12/196 kB 
> Progress (3): 1.0/2.6 MB | 184/502 kB | 16/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 20/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 25/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 29/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 33/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 37/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 41/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 45/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 49/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 53/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 57/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 61/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 66/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 70/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 74/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 78/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 82/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 86/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 90/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 94/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 98/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 102/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 106/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 111/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 184/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 188/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 193/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 197/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 201/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 205/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 209/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 213/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 217/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 221/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 225/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 229/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 233/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 238/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 242/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 246/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 250/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 254/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 258/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 262/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 266/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 270/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 274/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 279/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 283/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 287/502 kB | 115/196 kB
> Progress (3): 1.0/2.6 MB | 291/502 k

[GitHub] [maven] MartinKanters commented on a diff in pull request #869: [MNG-7338] Automatically activate batch-mode and make output quiet when running in CI.

2023-04-28 Thread via GitHub


MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180127108


##
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##
@@ -173,7 +177,17 @@ public CLIManager() {
 .build());
 options.addOption(Option.builder(Character.toString(BATCH_MODE))
 .longOpt("batch-mode")
-.desc("Run in non-interactive (batch) mode (disables output 
color)")
+.desc(
+"Run in non-interactive (batch) mode (disables output 
color) - alias for --non-interactive (kept for backwards compatability)")

Review Comment:
   Yeah, that's true. I copied the original description of --batch-mode, but I 
agree that it doesn't cover the functionally anymore. I've rewritten them.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [maven] MartinKanters commented on a diff in pull request #869: [MNG-7338] Automatically activate batch-mode and make output quiet when running in CI.

2023-04-28 Thread via GitHub


MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180126561


##
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##
@@ -38,6 +38,10 @@ public class CLIManager {
 
 public static final char BATCH_MODE = 'B';
 
+public static final String NON_INTERACTIVE = "ni";
+
+public static final String FORCE_INTERACTIVE = "fi";

Review Comment:
   I agree, I've removed them



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [maven] MartinKanters commented on a diff in pull request #869: [MNG-7338] Automatically activate batch-mode and make output quiet when running in CI.

2023-04-28 Thread via GitHub


MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180126255


##
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##
@@ -173,7 +177,17 @@ public CLIManager() {
 .build());
 options.addOption(Option.builder(Character.toString(BATCH_MODE))
 .longOpt("batch-mode")
-.desc("Run in non-interactive (batch) mode (disables output 
color)")
+.desc(
+"Run in non-interactive (batch) mode (disables output 
color) - alias for --non-interactive (kept for backwards compatability)")
+.build());
+options.addOption(Option.builder(NON_INTERACTIVE)
+.longOpt("non-interactive")
+.desc("Run in non-interactive (batch) mode (disables output 
color) - alias for --batch-mode")
+.build());
+options.addOption(Option.builder(FORCE_INTERACTIVE)
+.longOpt("force-interactive")
+.desc(
+"Run in interactive mode - even when the environment 
variable CI is set to true and --non-interactive or --batch-mode are set")

Review Comment:
   I'm not sure how to pass this information to Jansi. `isatty(2)` will atm 
just return 1/true. Also if `--batch-mode` is set. 
   Do you think a change like this should be part of this PR? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Closed] (MSITE-964) Fix failure of 'site-jar' IT on Java 17

2023-04-28 Thread Michael Osipov (Jira)


 [ 
https://issues.apache.org/jira/browse/MSITE-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Osipov closed MSITE-964.

Resolution: Fixed

Fixed with 
[48f4155af2069e61c15399a475a26d2467749347|https://gitbox.apache.org/repos/asf?p=maven-site-plugin.git;a=commit;h=48f4155af2069e61c15399a475a26d2467749347].

> Fix failure of 'site-jar' IT on Java 17
> ---
>
> Key: MSITE-964
> URL: https://issues.apache.org/jira/browse/MSITE-964
> Project: Maven Site Plugin
>  Issue Type: Sub-task
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
>
> See: 
> https://github.com/apache/maven-site-plugin/pull/148#issuecomment-1525468715



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Closed] (MSITE-960) CI is broken

2023-04-28 Thread Michael Osipov (Jira)


 [ 
https://issues.apache.org/jira/browse/MSITE-960?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Osipov closed MSITE-960.

Resolution: Fixed

All issues done.

> CI is broken
> 
>
> Key: MSITE-960
> URL: https://issues.apache.org/jira/browse/MSITE-960
> Project: Maven Site Plugin
>  Issue Type: Bug
>Reporter: Elliotte Rusty Harold
>Assignee: Michael Osipov
>Priority: Blocker
> Attachments: screenshot-1.png
>
>
>  Error:  Failed to execute goal 
> org.apache.maven.plugins:maven-site-plugin:3.12.1:site (default-site) on 
> project maven-site-plugin: RendererException while preparing context for 
> skin: Cannot use skin: has 2.0.0-M5 Doxia Sitetools prerequisite, but current 
> is 1.11.1 -> [Help 1]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MSITE-964) Fix failure of 'site-jar' IT on Java 17

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MSITE-964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717529#comment-17717529
 ] 

ASF GitHub Bot commented on MSITE-964:
--

asfgit closed pull request #150: [MSITE-964] Fix failure of 'site-jar' IT on 
Java 17
URL: https://github.com/apache/maven-site-plugin/pull/150




> Fix failure of 'site-jar' IT on Java 17
> ---
>
> Key: MSITE-964
> URL: https://issues.apache.org/jira/browse/MSITE-964
> Project: Maven Site Plugin
>  Issue Type: Sub-task
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
>
> See: 
> https://github.com/apache/maven-site-plugin/pull/148#issuecomment-1525468715



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-site-plugin] asfgit closed pull request #150: [MSITE-964] Fix failure of 'site-jar' IT on Java 17

2023-04-28 Thread via GitHub


asfgit closed pull request #150: [MSITE-964] Fix failure of 'site-jar' IT on 
Java 17
URL: https://github.com/apache/maven-site-plugin/pull/150


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Assigned] (MSITE-964) Fix failure of 'site-jar' IT on Java 17

2023-04-28 Thread Michael Osipov (Jira)


 [ 
https://issues.apache.org/jira/browse/MSITE-964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Osipov reassigned MSITE-964:


Assignee: Michael Osipov

> Fix failure of 'site-jar' IT on Java 17
> ---
>
> Key: MSITE-964
> URL: https://issues.apache.org/jira/browse/MSITE-964
> Project: Maven Site Plugin
>  Issue Type: Sub-task
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
>
> See: 
> https://github.com/apache/maven-site-plugin/pull/148#issuecomment-1525468715



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MRESOLVER-304) DirectoryUtils should consider interpolation

2023-04-28 Thread Tamas Cservenak (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717510#comment-17717510
 ] 

Tamas Cservenak commented on MRESOLVER-304:
---

Maven 3.9.2 targeted issue and related work in MNG-7774 solves this.

> DirectoryUtils should consider interpolation
> 
>
> Key: MRESOLVER-304
> URL: https://issues.apache.org/jira/browse/MRESOLVER-304
> Project: Maven Resolver
>  Issue Type: Improvement
>  Components: Resolver
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 1.10.0
>
>
> As currently it cannot be interpolated, and it applies this logic:
>  * if "relative path" it resolves against local repo
>  * if "absolute path" it resolves against path
> what would be ideal to introduce somehow 3rd case:
>  * resolve against local repo
>  * resolve against maven.multiModuleProjectDirectory
>  * resolve against absolute path
> As new 2nd option would allow to check in filter config along with sources.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Assigned] (MNG-7774) Maven config and command line interpolation

2023-04-28 Thread Tamas Cservenak (Jira)


 [ 
https://issues.apache.org/jira/browse/MNG-7774?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tamas Cservenak reassigned MNG-7774:


Assignee: Tamas Cservenak

> Maven config and command line interpolation
> ---
>
> Key: MNG-7774
> URL: https://issues.apache.org/jira/browse/MNG-7774
> Project: Maven
>  Issue Type: Task
>  Components: Command Line
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 3.9.2
>
>
> Note: this issue "mildly follows" functionality done in Maven 4 as part of 
> MNG-7038 and MNG-6303, with partial implementation only (is not full 
> back-port), hence "mild introduction".
> Rationale: many new resolver features are akin to be able to store their 
> configuration and input along SCM tracked Maven project sources. Sadly, 
> currently this is not possible. The two issues targeting Maven4 (MNG-7038 and 
> MNG-6303) with introduction of session and project "topDirectory" and 
> "rootDirectory" properties solve this. Still, Maven 3.9.x does not need all 
> of this.
> So, idea is following:
>  * extend MavenCli to interpolate Maven arguments
>  * introduce two special properties {{session.rootDirectory}} and 
> {{session.topDirectory}} in interpolation. They are special, as they may not 
> be defined, and IF they are not defined, their use in Maven argument 
> interpolation causes error (Maven will not boot).
>  * limit the use of the two new properties to Maven CLI interpolation only. 
> This makes projects relying on Maven CLI interpolation "permeable" between 
> Maven 3.9.x and Maven 4, while all the whistle and bells (exposing this via 
> session or project) will be available in Maven 4 only.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MNG-7774) Maven config and command line interpolation

2023-04-28 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17717506#comment-17717506
 ] 

ASF GitHub Bot commented on MNG-7774:
-

cstamas opened a new pull request, #1097:
URL: https://github.com/apache/maven/pull/1097

   This is "mild introduction" of subset of new Maven4 features into Maven 
3.9.x land, with single intent: be able to store resolver configuration along 
with checked in project sources in SCM.
   
   Changes:
   * interpolate properties and arguments
   * introduce two new special properties "session.rootDirectory" and 
"session.topDirectory"
   * these two properties may be undefined (the .mvn may not be found)
   * these two properties if undefined, but referenced during interpolation, 
causes Maven to bail out (error out), and will prevent starting.
   * some other minor cosmetic cleanup
   
   ---
   
   https://issues.apache.org/jira/browse/MNG-7774




> Maven config and command line interpolation
> ---
>
> Key: MNG-7774
> URL: https://issues.apache.org/jira/browse/MNG-7774
> Project: Maven
>  Issue Type: Task
>  Components: Command Line
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 3.9.2
>
>
> Note: this issue "mildly follows" functionality done in Maven 4 as part of 
> MNG-7038 and MNG-6303, with partial implementation only (is not full 
> back-port), hence "mild introduction".
> Rationale: many new resolver features are akin to be able to store their 
> configuration and input along SCM tracked Maven project sources. Sadly, 
> currently this is not possible. The two issues targeting Maven4 (MNG-7038 and 
> MNG-6303) with introduction of session and project "topDirectory" and 
> "rootDirectory" properties solve this. Still, Maven 3.9.x does not need all 
> of this.
> So, idea is following:
>  * extend MavenCli to interpolate Maven arguments
>  * introduce two special properties {{session.rootDirectory}} and 
> {{session.topDirectory}} in interpolation. They are special, as they may not 
> be defined, and IF they are not defined, their use in Maven argument 
> interpolation causes error (Maven will not boot).
>  * limit the use of the two new properties to Maven CLI interpolation only. 
> This makes projects relying on Maven CLI interpolation "permeable" between 
> Maven 3.9.x and Maven 4, while all the whistle and bells (exposing this via 
> session or project) will be available in Maven 4 only.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven] cstamas opened a new pull request, #1097: [MNG-7774] Maven config and command line interpolation

2023-04-28 Thread via GitHub


cstamas opened a new pull request, #1097:
URL: https://github.com/apache/maven/pull/1097

   This is "mild introduction" of subset of new Maven4 features into Maven 
3.9.x land, with single intent: be able to store resolver configuration along 
with checked in project sources in SCM.
   
   Changes:
   * interpolate properties and arguments
   * introduce two new special properties "session.rootDirectory" and 
"session.topDirectory"
   * these two properties may be undefined (the .mvn may not be found)
   * these two properties if undefined, but referenced during interpolation, 
causes Maven to bail out (error out), and will prevent starting.
   * some other minor cosmetic cleanup
   
   ---
   
   https://issues.apache.org/jira/browse/MNG-7774


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (MNG-7774) Maven config and command line interpolation

2023-04-28 Thread Tamas Cservenak (Jira)
Tamas Cservenak created MNG-7774:


 Summary: Maven config and command line interpolation
 Key: MNG-7774
 URL: https://issues.apache.org/jira/browse/MNG-7774
 Project: Maven
  Issue Type: Task
  Components: Command Line
Reporter: Tamas Cservenak
 Fix For: 3.9.2


Note: this issue "mildly follows" functionality done in Maven 4 as part of 
MNG-7038 and MNG-6303, with partial implementation only (is not full 
back-port), hence "mild introduction".

Rationale: many new resolver features are akin to be able to store their 
configuration and input along SCM tracked Maven project sources. Sadly, 
currently this is not possible. The two issues targeting Maven4 (MNG-7038 and 
MNG-6303) with introduction of session and project "topDirectory" and 
"rootDirectory" properties solve this. Still, Maven 3.9.x does not need all of 
this.

So, idea is following:
 * extend MavenCli to interpolate Maven arguments
 * introduce two special properties {{session.rootDirectory}} and 
{{session.topDirectory}} in interpolation. They are special, as they may not be 
defined, and IF they are not defined, their use in Maven argument interpolation 
causes error (Maven will not boot).
 * limit the use of the two new properties to Maven CLI interpolation only. 
This makes projects relying on Maven CLI interpolation "permeable" between 
Maven 3.9.x and Maven 4, while all the whistle and bells (exposing this via 
session or project) will be available in Maven 4 only.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


  1   2   >