Re: [PR] [MNG-8750] Delete unused interface [maven]

[via GitHub]

elharo merged PR #2388:
URL: https://github.com/apache/maven/pull/2388


-- 
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

[PR] Bump eu.maveniverse.maven.nisse:extension from 0.4.3 to 0.4.5 [maven-mvnd]

[via GitHub]

dependabot[bot] opened a new pull request, #1335:
URL: https://github.com/apache/maven-mvnd/pull/1335

   Bumps 
[eu.maveniverse.maven.nisse:extension](https://github.com/maveniverse/nisse) 
from 0.4.3 to 0.4.5.
   
   Release notes
   Sourced from https://github.com/maveniverse/nisse/releases";>eu.maveniverse.maven.nisse:extension's
 releases.
   
   0.4.5
   Improvement: added new property, the git commit date but formatted as 
ISO8601. It can be used in POM as:
   ${nisse.jgit.dateIso8601}
   
   Minor dependency updates as well.
   What's Changed
   
   Bump eu.maveniverse.maven.shared:extension from 0.1.5 to 0.1.6 by https://github.com/dependabot";>@​dependabot in https://redirect.github.com/maveniverse/nisse/pull/73";>maveniverse/nisse#73
   Bump org.eclipse.sisu:org.eclipse.sisu.inject from 0.9.0.M3 to 0.9.0.M4 
by https://github.com/dependabot";>@​dependabot in https://redirect.github.com/maveniverse/nisse/pull/69";>maveniverse/nisse#69
   Emit JGit commit date as ISO8601 date by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/maveniverse/nisse/pull/75";>maveniverse/nisse#75
   
   Full Changelog: https://github.com/maveniverse/nisse/compare/release-0.4.4...release-0.4.5";>https://github.com/maveniverse/nisse/compare/release-0.4.4...release-0.4.5
   0.4.4
   Fix classpath issue and add proper requirement checks.
   What's Changed
   
   Fix classpath by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/maveniverse/nisse/pull/67";>maveniverse/nisse#67
   
   Full Changelog: https://github.com/maveniverse/nisse/compare/release-0.4.3...release-0.4.4";>https://github.com/maveniverse/nisse/compare/release-0.4.3...release-0.4.4
   
   
   
   Commits
   
   https://github.com/maveniverse/nisse/commit/f36dd5450ebd916470a831f6833f7c9a686b3896";>f36dd54
 [maven-release-plugin] prepare release release-0.4.5
   https://github.com/maveniverse/nisse/commit/12bd0346db8abce353f042d8282bfe857ac164b5";>12bd034
 Emit JGit commit date as ISO8601 date (https://redirect.github.com/maveniverse/nisse/issues/75";>#75)
   https://github.com/maveniverse/nisse/commit/76dd78028e0c906a3b8c6546141b250e5830c05c";>76dd780
 Bump org.eclipse.sisu:org.eclipse.sisu.inject from 0.9.0.M3 to 0.9.0.M4 (https://redirect.github.com/maveniverse/nisse/issues/69";>#69)
   https://github.com/maveniverse/nisse/commit/d5ff2c1ebbd066650d9a1ae83de774174030eeef";>d5ff2c1
 Bump eu.maveniverse.maven.shared:extension from 0.1.5 to 0.1.6 (https://redirect.github.com/maveniverse/nisse/issues/73";>#73)
   https://github.com/maveniverse/nisse/commit/caf16e52c56680e708749cc8b70e6000d0f1fdb4";>caf16e5
 Update to parent 38
   https://github.com/maveniverse/nisse/commit/fc88ae524cc73e2f62ae1ac78d8f18339f1e2e66";>fc88ae5
 Narrow the CI matrix
   https://github.com/maveniverse/nisse/commit/8099ff1ebf376b4f172f2912675df155d06cd220";>8099ff1
 Update parent to 37
   https://github.com/maveniverse/nisse/commit/340d9009e4be8da36954d54af19e26fefd84d3d3";>340d900
 [maven-release-plugin] prepare for next development iteration
   https://github.com/maveniverse/nisse/commit/2cc3313f87d369cacd7b987e4371d0f60a6314fc";>2cc3313
 [maven-release-plugin] prepare release release-0.4.4
   https://github.com/maveniverse/nisse/commit/d0263fa53621ec4326013356f5314c076b68b19d";>d0263fa
 Fix classpath (https://redirect.github.com/maveniverse/nisse/issues/67";>#67)
   Additional commits viewable in https://github.com/maveniverse/nisse/compare/release-0.4.3...release-0.4.5";>compare
 view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=eu.maveniverse.maven.nisse:extension&package-manager=maven&previous-version=0.4.3&new-version=0.4.5)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of

Re: [PR] Bump eu.maveniverse.maven.nisse:extension from 0.4.3 to 0.4.4 [maven-mvnd]

[via GitHub]

dependabot[bot] commented on PR #1325:
URL: https://github.com/apache/maven-mvnd/pull/1325#issuecomment-2915786853

   Superseded by #1335.


-- 
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

Re: [PR] Bump eu.maveniverse.maven.nisse:extension from 0.4.3 to 0.4.4 [maven-mvnd]

[via GitHub]

dependabot[bot] closed pull request #1325: Bump 
eu.maveniverse.maven.nisse:extension from 0.4.3 to 0.4.4
URL: https://github.com/apache/maven-mvnd/pull/1325


-- 
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-8753) delete DependencyManagementImporter

[Elliotte Rusty Harold (Jira)]

Elliotte Rusty Harold created MNG-8753:
--

 Summary: delete DependencyManagementImporter
 Key: MNG-8753
 URL: https://issues.apache.org/jira/browse/MNG-8753
 Project: Maven
  Issue Type: Improvement
Reporter: Elliotte Rusty Harold


Another case of excess abstraction where several public classes and interfaces 
look able to be reduced to s single private static method. The public API can 
be reintroduced later if there's ever a need for it, but more likely YAGNI



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

[jira] [Updated] (MNG-8753) delete DependencyManagementImporter

[Elliotte Rusty Harold (Jira)]

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

Elliotte Rusty Harold updated MNG-8753:
---
Description: Another case of excess abstraction where several public 
classes and interfaces look able to be reduced to s single private static 
method. The public API can be reintroduced later if there's ever a need for it, 
but more likely YAGNI. It's worth checking this for everything in 
org.apache.maven.api.services.model  (was: Another case of excess abstraction 
where several public classes and interfaces look able to be reduced to s single 
private static method. The public API can be reintroduced later if there's ever 
a need for it, but more likely YAGNI)

> delete DependencyManagementImporter
> ---
>
> Key: MNG-8753
> URL: https://issues.apache.org/jira/browse/MNG-8753
> Project: Maven
>  Issue Type: Improvement
>Reporter: Elliotte Rusty Harold
>Priority: Blocker
>
> Another case of excess abstraction where several public classes and 
> interfaces look able to be reduced to s single private static method. The 
> public API can be reintroduced later if there's ever a need for it, but more 
> likely YAGNI. It's worth checking this for everything in 
> org.apache.maven.api.services.model



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

Re: [PR] avoid false `IO` catch on `mkdirs()` [maven]

[via GitHub]

elharo commented on code in PR #2396:
URL: https://github.com/apache/maven/pull/2396#discussion_r2111553285


##
its/core-it-suite/src/test/resources/mng-2135/plugin/src/main/java/coreit/ItMojo.java:
##
@@ -52,8 +52,8 @@ public class ItMojo extends AbstractMojo {
 private File outputFile;
 
 public void execute() throws MojoExecutionException {

Review Comment:
   Not sure how old this code is, perhaps pre-Java 5. This and a lot of other 
methods are missing @Override annotations. I also see a lot fo old javadoc 
style annotations in these classes. 



##
its/core-it-support/core-it-plugins/maven-it-plugin-uses-wagon/src/main/java/org/apache/maven/plugin/coreit/LookupWagonMojo.java:
##
@@ -93,8 +93,8 @@ public void execute() throws MojoExecutionException, 
MojoFailureException {
 getLog().info("[MAVEN-CORE-IT-LOG] Creating output file " + 
outputFile);
 
 OutputStream out = null;
+outputFile.getParentFile().mkdirs();

Review Comment:
   This code is pretty old and not up to 2025 standards. I think we should 
merge the try with resources before making this change. Otherwise, line 
splitting is making it even less clear. 



-- 
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

Re: [PR] chore: `ArtifactHandlerTest` add assertion to resolve `@SuppressWarnings("checkstyle:UnusedLocalVariable")` [maven]

[via GitHub]

elharo commented on code in PR #2375:
URL: https://github.com/apache/maven/pull/2375#discussion_r2111578166


##
impl/maven-core/src/test/java/org/apache/maven/artifact/handler/ArtifactHandlerTest.java:
##
@@ -77,8 +107,15 @@ void testAptConsistency() throws Exception {
 ArtifactHandler handler =
 
container.lookup(ArtifactHandlerManager.class).getArtifactHandler(type);
 assertEquals(handler.getExtension(), extension, type + " 
extension");
-// Packaging/Directory is Maven1 remnant!!!
-// assertEquals(handler.getPackaging(), packaging, type + " 
packaging");
+assertThat(PACKAGING_TYPES).contains(handler.getPackaging(), 
packaging);
+if (handler.getPackaging().equals("test-jar")) {
+assertEquals("jar", packaging);
+
+} else if (handler.getPackaging().equals("ejb-client")) {
+assertEquals("ejb", packaging);

Review Comment:
   I might be starting to see what's going on here. I think you're hitting a 
weird issue (though WAI) about file extension vs dependency type. See 
https://maven.apache.org/ref/3.9.9/maven-core/artifact-handlers.html and pages 
I can't put my finger on right now, but updated a few months ago.
   
   That being said, I am more convinced now that this approach is incorrect. We 
want one single assertEquals that tests a specific value, not contains 
something from a collection and not "if this assert that". This might mean 
splitting into multiple test methods. DRY does not apply to tests and asserts. 
*Unit* tests should be very straight-forward with no branching and no 
conditionals. 



-- 
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

Re: [I] Update from 3.5.2 to 3.5.3 can break dependencies [maven-surefire]

[via GitHub]

slawekjaranowski commented on issue #850:
URL: https://github.com/apache/maven-surefire/issues/850#issuecomment-2915643670

   dependencies management from surefire project should not have impact on 
resolved version in used project.
   
   As workaround you can also add to plugin dependncies:
   
   ```xml
 
 org.junit.platform
 junit-platform-engine
 older.version
 
   
   ```


-- 
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

Re: [PR] avoid false `IO` catch on `mkdirs()` [maven]

[via GitHub]

Pankraz76 commented on PR #2396:
URL: https://github.com/apache/maven/pull/2396#issuecomment-2915543289

   Caused by: org.apache.maven.plugin.MojoExecutionException: The following 
files had format violations:
   src/main/java/org/apache/maven/plugin/coreit/CleanMojo.java
   @@ -70,7 +70,6 @@

outputFile·=·new·File(project.getBasedir(),·pathname).getAbsoluteFile();
}

   -
outputFile.getParentFile().mkdirs();
try·{
outputFile.createNewFile();
   Run 'mvn spotless:apply' to fix these violations.


-- 
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-8699) Seems inferring fields is not fully covered?

[Guillaume Nodet (Jira)]

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

Guillaume Nodet commented on MNG-8699:
--

So the problem is caused by an invalid {{root="true"}} attribute set on {{its}} 
instead of the root pom.
Given {{its}} is considered root, this means you need to install the {{mason}} 
jar in order to run its.

> Seems inferring fields is not fully covered?
> 
>
> Key: MNG-8699
> URL: https://issues.apache.org/jira/browse/MNG-8699
> Project: Maven
>  Issue Type: Bug
>Reporter: Tamas Cservenak
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-rc-4
>
>
> Strange CI output, as locally same version 4.0.0-rc-3 works.
> To reproduce: use -f its on mason
> This is on CI:
> {noformat}
> Apache Maven 4.0.0-rc-3 (3952d00ce65df6753b63a51e86b1f626c55a8df2)
> 8Maven home: /home/runner/.m2/wrapper/dists/apache-maven-4.0.0-rc-3/f94e8f8d
> 9Java version: 17.0.14, vendor: Eclipse Adoptium, runtime: 
> /usr/lib/jvm/temurin-17-jdk-amd64
> 10Default locale: en, platform encoding: UTF-8
> 11OS name: "linux", version: "6.8.0-1021-azure", arch: "amd64", family: "unix"
> 12[INFO] Error stacktraces are turned on.
> 13[INFO] Scanning for projects...
> 14Error:  Some problems were encountered while processing the POMs
> 15Error:  The build could not read 1 project -> [Help 1]
> 16org.apache.maven.project.ProjectBuildingException: Some problems were 
> encountered while processing the POMs
> 17at 
> org.apache.maven.project.DefaultProjectBuilder$BuildSession.build(DefaultProjectBuilder.java:480)
> 18at 
> org.apache.maven.project.DefaultProjectBuilder.build(DefaultProjectBuilder.java:195)
> 19at 
> org.apache.maven.project.collector.DefaultProjectsSelector.selectProjects(DefaultProjectsSelector.java:61)
> 20at 
> org.apache.maven.project.collector.RequestPomCollectionStrategy.collectProjects(RequestPomCollectionStrategy.java:49)
> 21at 
> org.apache.maven.graph.DefaultGraphBuilder.getProjectsForMavenReactor(DefaultGraphBuilder.java:367)
> 22at 
> org.apache.maven.graph.DefaultGraphBuilder.build(DefaultGraphBuilder.java:100)
> 23at org.apache.maven.DefaultMaven.buildGraph(DefaultMaven.java:629)
> 24at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:250)
> 25at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:225)
> 26at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:149)
> 27at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.doExecute(MavenInvoker.java:462)
> 28at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.execute(MavenInvoker.java:100)
> 29at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.execute(MavenInvoker.java:81)
> 30at 
> org.apache.maven.cling.invoker.LookupInvoker.doInvoke(LookupInvoker.java:165)
> 31at 
> org.apache.maven.cling.invoker.LookupInvoker.invoke(LookupInvoker.java:135)
> 32at org.apache.maven.cling.ClingSupport.run(ClingSupport.java:76)
> 33at org.apache.maven.cling.MavenCling.main(MavenCling.java:51)
> 34at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> 35at 
> jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
> 36at 
> jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> 37at java.lang.reflect.Method.invoke(Method.java:569)
> 38at 
> org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:255)
> 39at 
> org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:201)
> 40at 
> org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:361)
> 41at 
> org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:314)
> 42Error:
> 43Error:The project eu.maveniverse.maven.mason:mason-its:0.1.1-SNAPSHOT 
> (/home/runner/work/mason/mason/its/pom.xml) has 2 errors
> 44Error:  'dependencies.dependency.groupId' for null:mason:jar is 
> missing. @ line 32, column 5
> 45Error:  'dependencies.dependency.version' for null:mason:jar is 
> missing. @ line 32, column 5
> 46Error:  
> 47Error:  Re-run Maven using the '-X' switch to enable verbose output
> 48Error:  
> 49Error:  For more information about the errors and possible solutions, 
> please read the following articles:
> 50Error:  [Help 1] 
> http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
> 51Error: Process completed with exit code 1. {noformat}



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

[jira] [Closed] (MNG-8699) Seems inferring fields is not fully covered?

[Guillaume Nodet (Jira)]

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

Guillaume Nodet closed MNG-8699.

Resolution: Not A Problem

> Seems inferring fields is not fully covered?
> 
>
> Key: MNG-8699
> URL: https://issues.apache.org/jira/browse/MNG-8699
> Project: Maven
>  Issue Type: Bug
>Reporter: Tamas Cservenak
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-rc-4
>
>
> Strange CI output, as locally same version 4.0.0-rc-3 works.
> To reproduce: use -f its on mason
> This is on CI:
> {noformat}
> Apache Maven 4.0.0-rc-3 (3952d00ce65df6753b63a51e86b1f626c55a8df2)
> 8Maven home: /home/runner/.m2/wrapper/dists/apache-maven-4.0.0-rc-3/f94e8f8d
> 9Java version: 17.0.14, vendor: Eclipse Adoptium, runtime: 
> /usr/lib/jvm/temurin-17-jdk-amd64
> 10Default locale: en, platform encoding: UTF-8
> 11OS name: "linux", version: "6.8.0-1021-azure", arch: "amd64", family: "unix"
> 12[INFO] Error stacktraces are turned on.
> 13[INFO] Scanning for projects...
> 14Error:  Some problems were encountered while processing the POMs
> 15Error:  The build could not read 1 project -> [Help 1]
> 16org.apache.maven.project.ProjectBuildingException: Some problems were 
> encountered while processing the POMs
> 17at 
> org.apache.maven.project.DefaultProjectBuilder$BuildSession.build(DefaultProjectBuilder.java:480)
> 18at 
> org.apache.maven.project.DefaultProjectBuilder.build(DefaultProjectBuilder.java:195)
> 19at 
> org.apache.maven.project.collector.DefaultProjectsSelector.selectProjects(DefaultProjectsSelector.java:61)
> 20at 
> org.apache.maven.project.collector.RequestPomCollectionStrategy.collectProjects(RequestPomCollectionStrategy.java:49)
> 21at 
> org.apache.maven.graph.DefaultGraphBuilder.getProjectsForMavenReactor(DefaultGraphBuilder.java:367)
> 22at 
> org.apache.maven.graph.DefaultGraphBuilder.build(DefaultGraphBuilder.java:100)
> 23at org.apache.maven.DefaultMaven.buildGraph(DefaultMaven.java:629)
> 24at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:250)
> 25at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:225)
> 26at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:149)
> 27at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.doExecute(MavenInvoker.java:462)
> 28at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.execute(MavenInvoker.java:100)
> 29at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.execute(MavenInvoker.java:81)
> 30at 
> org.apache.maven.cling.invoker.LookupInvoker.doInvoke(LookupInvoker.java:165)
> 31at 
> org.apache.maven.cling.invoker.LookupInvoker.invoke(LookupInvoker.java:135)
> 32at org.apache.maven.cling.ClingSupport.run(ClingSupport.java:76)
> 33at org.apache.maven.cling.MavenCling.main(MavenCling.java:51)
> 34at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> 35at 
> jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
> 36at 
> jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> 37at java.lang.reflect.Method.invoke(Method.java:569)
> 38at 
> org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:255)
> 39at 
> org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:201)
> 40at 
> org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:361)
> 41at 
> org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:314)
> 42Error:
> 43Error:The project eu.maveniverse.maven.mason:mason-its:0.1.1-SNAPSHOT 
> (/home/runner/work/mason/mason/its/pom.xml) has 2 errors
> 44Error:  'dependencies.dependency.groupId' for null:mason:jar is 
> missing. @ line 32, column 5
> 45Error:  'dependencies.dependency.version' for null:mason:jar is 
> missing. @ line 32, column 5
> 46Error:  
> 47Error:  Re-run Maven using the '-X' switch to enable verbose output
> 48Error:  
> 49Error:  For more information about the errors and possible solutions, 
> please read the following articles:
> 50Error:  [Help 1] 
> http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
> 51Error: Process completed with exit code 1. {noformat}



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

Re: [PR] chore: test unused `VerificationException` in `MavenITmng5640LifecycleParticipantAfterSessionEnd` [maven]

[via GitHub]

Pankraz76 commented on PR #2387:
URL: https://github.com/apache/maven/pull/2387#issuecomment-2915559346

   might be a quick win.


-- 
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

Re: [I] Fix build failure with jdk 23 [maven-javadoc-plugin]

[via GitHub]

joakime commented on issue #373:
URL: 
https://github.com/apache/maven-javadoc-plugin/issues/373#issuecomment-2915670519

   @olamy did you notice the warning?
   
   ```
   Warning:  Javadoc Warnings
   Warning: [WARNING] warning: The -footer option is no longer supported and 
will be ignored.
   Warning:  It may be removed in a future release.
   ```


-- 
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

Re: [I] Update from 3.5.2 to 3.5.3 can break dependencies [maven-surefire]

[via GitHub]

slawekjaranowski commented on issue #850:
URL: https://github.com/apache/maven-surefire/issues/850#issuecomment-2915665734

   Using only `junit-jupiter-api/junit-jupiter` in project dependencies should 
be preferred and enough.
   
   You need not to add additional dependencies to plugin.


-- 
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

Re: [I] Update from 3.5.2 to 3.5.3 can break dependencies [maven-surefire]

[via GitHub]

feddevanderlist commented on issue #850:
URL: https://github.com/apache/maven-surefire/issues/850#issuecomment-2915677614

   > Using only `junit-jupiter-api/junit-jupiter` in project dependencies 
should be preferred and enough.
   > 
   > You need not to add additional dependencies to plugin.
   
   How would you then solve the fact that I wants to use Junit4 instead of 
junit5? (I know should not have junit4 on classpath other dependency)
   


-- 
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

Re: [I] Update from 3.5.2 to 3.5.3 can break dependencies [maven-surefire]

[via GitHub]

slawekjaranowski commented on issue #850:
URL: https://github.com/apache/maven-surefire/issues/850#issuecomment-2915716039

   > > Using only `junit-jupiter-api/junit-jupiter` in project dependencies 
should be preferred and enough.
   > > You need not to add additional dependencies to plugin.
   > 
   > How would you then solve the fact that I wants to use Junit4 instead of 
junit5? (I know should not have junit4 on classpath other dependency)
   
   If you use only JUnit4 add only dependencies to JUnit4 on project 
depndencies.
   
   if you want to use both JUnit4 nad JUnit5 add also: `junit-vintage-engine` 
to project dependencies.
   
   


-- 
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] (MNG-8746) Keep ordering of properties in Model

[Guillaume Nodet (Jira)]

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

Guillaume Nodet reassigned MNG-8746:


Assignee: Guillaume Nodet

> Keep ordering of properties in Model
> 
>
> Key: MNG-8746
> URL: https://issues.apache.org/jira/browse/MNG-8746
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Reporter: Tamas Cservenak
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-rc-4
>
>




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

[jira] [Updated] (MNG-8746) Keep ordering of properties in Model

[Guillaume Nodet (Jira)]

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

Guillaume Nodet updated MNG-8746:
-
Summary: Keep ordering of properties in Model  (was: Maven4 reorders 
properties of read POM)

> Keep ordering of properties in Model
> 
>
> Key: MNG-8746
> URL: https://issues.apache.org/jira/browse/MNG-8746
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Reporter: Tamas Cservenak
>Priority: Major
> Fix For: 4.0.0-rc-4
>
>




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

Re: [PR] chore: `ArtifactHandlerTest` add assertion to resolve `@SuppressWarnings("checkstyle:UnusedLocalVariable")` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2375:
URL: https://github.com/apache/maven/pull/2375#discussion_r2111240837


##
impl/maven-core/src/test/java/org/apache/maven/artifact/handler/ArtifactHandlerTest.java:
##
@@ -77,8 +107,15 @@ void testAptConsistency() throws Exception {
 ArtifactHandler handler =
 
container.lookup(ArtifactHandlerManager.class).getArtifactHandler(type);
 assertEquals(handler.getExtension(), extension, type + " 
extension");
-// Packaging/Directory is Maven1 remnant!!!
-// assertEquals(handler.getPackaging(), packaging, type + " 
packaging");
+assertThat(PACKAGING_TYPES).contains(handler.getPackaging(), 
packaging);
+if (handler.getPackaging().equals("test-jar")) {
+assertEquals("jar", packaging);
+
+} else if (handler.getPackaging().equals("ejb-client")) {
+assertEquals("ejb", packaging);

Review Comment:
   both look like but to me. maven-archetype",
   "maven-plugin", "java-source" are all equal so why not these two 
? So its not a hyphen thing in general, right?



-- 
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-8699) Seems inferring fields is not fully covered?

[Guillaume Nodet (Jira)]

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

Guillaume Nodet commented on MNG-8699:
--

This happens if the main jar has not been installed locally yet.
If you run a full build with install first, the problem does not happen.
However, running a `mvn package` should work, but does not.

> Seems inferring fields is not fully covered?
> 
>
> Key: MNG-8699
> URL: https://issues.apache.org/jira/browse/MNG-8699
> Project: Maven
>  Issue Type: Bug
>Reporter: Tamas Cservenak
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-rc-4
>
>
> Strange CI output, as locally same version 4.0.0-rc-3 works.
> To reproduce: use -f its on mason
> This is on CI:
> {noformat}
> Apache Maven 4.0.0-rc-3 (3952d00ce65df6753b63a51e86b1f626c55a8df2)
> 8Maven home: /home/runner/.m2/wrapper/dists/apache-maven-4.0.0-rc-3/f94e8f8d
> 9Java version: 17.0.14, vendor: Eclipse Adoptium, runtime: 
> /usr/lib/jvm/temurin-17-jdk-amd64
> 10Default locale: en, platform encoding: UTF-8
> 11OS name: "linux", version: "6.8.0-1021-azure", arch: "amd64", family: "unix"
> 12[INFO] Error stacktraces are turned on.
> 13[INFO] Scanning for projects...
> 14Error:  Some problems were encountered while processing the POMs
> 15Error:  The build could not read 1 project -> [Help 1]
> 16org.apache.maven.project.ProjectBuildingException: Some problems were 
> encountered while processing the POMs
> 17at 
> org.apache.maven.project.DefaultProjectBuilder$BuildSession.build(DefaultProjectBuilder.java:480)
> 18at 
> org.apache.maven.project.DefaultProjectBuilder.build(DefaultProjectBuilder.java:195)
> 19at 
> org.apache.maven.project.collector.DefaultProjectsSelector.selectProjects(DefaultProjectsSelector.java:61)
> 20at 
> org.apache.maven.project.collector.RequestPomCollectionStrategy.collectProjects(RequestPomCollectionStrategy.java:49)
> 21at 
> org.apache.maven.graph.DefaultGraphBuilder.getProjectsForMavenReactor(DefaultGraphBuilder.java:367)
> 22at 
> org.apache.maven.graph.DefaultGraphBuilder.build(DefaultGraphBuilder.java:100)
> 23at org.apache.maven.DefaultMaven.buildGraph(DefaultMaven.java:629)
> 24at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:250)
> 25at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:225)
> 26at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:149)
> 27at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.doExecute(MavenInvoker.java:462)
> 28at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.execute(MavenInvoker.java:100)
> 29at 
> org.apache.maven.cling.invoker.mvn.MavenInvoker.execute(MavenInvoker.java:81)
> 30at 
> org.apache.maven.cling.invoker.LookupInvoker.doInvoke(LookupInvoker.java:165)
> 31at 
> org.apache.maven.cling.invoker.LookupInvoker.invoke(LookupInvoker.java:135)
> 32at org.apache.maven.cling.ClingSupport.run(ClingSupport.java:76)
> 33at org.apache.maven.cling.MavenCling.main(MavenCling.java:51)
> 34at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> 35at 
> jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
> 36at 
> jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> 37at java.lang.reflect.Method.invoke(Method.java:569)
> 38at 
> org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:255)
> 39at 
> org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:201)
> 40at 
> org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:361)
> 41at 
> org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:314)
> 42Error:
> 43Error:The project eu.maveniverse.maven.mason:mason-its:0.1.1-SNAPSHOT 
> (/home/runner/work/mason/mason/its/pom.xml) has 2 errors
> 44Error:  'dependencies.dependency.groupId' for null:mason:jar is 
> missing. @ line 32, column 5
> 45Error:  'dependencies.dependency.version' for null:mason:jar is 
> missing. @ line 32, column 5
> 46Error:  
> 47Error:  Re-run Maven using the '-X' switch to enable verbose output
> 48Error:  
> 49Error:  For more information about the errors and possible solutions, 
> please read the following articles:
> 50Error:  [Help 1] 
> http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
> 51Error: Process completed with exit code 1. {noformat}



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

Re: [PR] chore: `ArtifactHandlerTest` add assertion to resolve `@SuppressWarnings("checkstyle:UnusedLocalVariable")` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2375:
URL: https://github.com/apache/maven/pull/2375#discussion_r2111217828


##
impl/maven-core/src/test/java/org/apache/maven/artifact/handler/ArtifactHandlerTest.java:
##
@@ -77,8 +107,8 @@ void testAptConsistency() throws Exception {
 ArtifactHandler handler =
 
container.lookup(ArtifactHandlerManager.class).getArtifactHandler(type);
 assertEquals(handler.getExtension(), extension, type + " 
extension");
-// Packaging/Directory is Maven1 remnant!!!
-// assertEquals(handler.getPackaging(), packaging, type + " 
packaging");
+assertThat(handler.getPackaging()).contains(packaging);

Review Comment:
   all hyphen needs contains. we can continue but i wont want to ask any more 
questions and would take increment.
   Whats your call?



-- 
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

Re: [PR] Bump commons-io:commons-io from 2.18.0 to 2.19.0 [maven-javadoc-plugin]

[via GitHub]

dependabot[bot] merged PR #370:
URL: https://github.com/apache/maven-javadoc-plugin/pull/370


-- 
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

Re: [PR] Bump commons-io:commons-io from 2.18.0 to 2.19.0 [maven-javadoc-plugin]

[via GitHub]

olamy commented on PR #370:
URL: 
https://github.com/apache/maven-javadoc-plugin/pull/370#issuecomment-2915241904

   @dependabot merge


-- 
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

Re: [PR] Issue #373 Fix JDK 23 build [maven-javadoc-plugin]

[via GitHub]

olamy merged PR #374:
URL: https://github.com/apache/maven-javadoc-plugin/pull/374


-- 
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

Re: [I] Fix build failure with jdk 23 [maven-javadoc-plugin]

[via GitHub]

olamy closed issue #373: Fix build failure with jdk 23
URL: https://github.com/apache/maven-javadoc-plugin/issues/373


-- 
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

Re: [I] Fix build failure with jdk 23 [maven-javadoc-plugin]

[via GitHub]

olamy commented on issue #373:
URL: 
https://github.com/apache/maven-javadoc-plugin/issues/373#issuecomment-2915251254

   fixed with #374 


-- 
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

[PR] Remove extra variable [maven]

[via GitHub]

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

   org is unclear and unneeded. Probably meant "original"? In any case, we 
don't need it. 


-- 
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

[PR] Redu [maven]

[via GitHub]

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

   (no comment)


-- 
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

[PR] Remove ignored method call [maven]

[via GitHub]

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

   (no comment)


-- 
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-8609) Dependency is not an Artifact

[Elliotte Rusty Harold (Jira)]

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

Elliotte Rusty Harold commented on MNG-8609:


I don't think it's only a documentation problem, though it is a documentation 
problem. Most docs, both Maven's own and third party's and most discussions, 
have been cavalier about the distinction between dependencies and artifacts. 
However after spending multiple years deep in the weeds of Maven dependency 
management, I am increasingly convinced that being very precise about this is 
absolutely critical to explaining and understanding how Maven works.

Currently package-info says this and it does describe the current class 
inheritance structure. However I don't think that inheritance hierarchy is 
correct, and it does lead to confusion.

```
 * {@link org.apache.maven.api.DependencyCoordinates} instances represent a 
dependency element in a POM.
 * A {@code DependencyCoordinates} extends {@code ArtifactCoordinates} with 
additional information about how
 * the artifact will be used: type, scope and obligation (whether the 
dependency is optional or mandatory).
 * The version and the obligation may not be defined precisely.
 *
 * {@link org.apache.maven.api.Dependency} instances represent artifacts in 
the repository
 * that are dependencies of the project.
 * They are created when resolving a {@code DependencyCoordinates}.
 * Resolving is the process that clarifies the obligation (optional or 
mandatory),
 * selects a particular version, and downloads the artifact into the local 
repository.
```

This is a classic example of is-a/has-a confusion in OO. It is true currently 
that {@link org.apache.maven.api.Dependency} instances represent artifacts in 
the repository. However, that shouldn't be the case. Instead something like 
"ResolvedDependency" should contain (not inherit from) an Artifact field.  


> Dependency is not an Artifact
> -
>
> Key: MNG-8609
> URL: https://issues.apache.org/jira/browse/MNG-8609
> Project: Maven
>  Issue Type: Bug
>  Components: API
>Reporter: Elliotte Rusty Harold
>Priority: Blocker
> Fix For: 4.1.0
>
>
> org.apache.maven.api.Dependency extends org.apache.maven.api.Artifact. This 
> violates the OO principal that inheritance represents an is-A relation. A 
> dependency is not an artifact. Arguably it has an artifact. However, 
> artifacts are nodes in the graph while dependencies are the arcs. These 
> aren't the same thing and mixing them up is a common source of confusion. 



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

[jira] [Commented] (MNG-8748) org.apache.maven.api.services.model.UrlNormalizer should be a concrete class

[Elliotte Rusty Harold (Jira)]

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

Elliotte Rusty Harold commented on MNG-8748:


Yes, agreed. I was noticing this one class at a time. However right now I only 
see one case where the relevant method is used in more than one class, and even 
there it's still just a single implementation function that can easily be 
static.

> org.apache.maven.api.services.model.UrlNormalizer should be a concrete class
> 
>
> Key: MNG-8748
> URL: https://issues.apache.org/jira/browse/MNG-8748
> Project: Maven
>  Issue Type: Improvement
>Reporter: Elliotte Rusty Harold
>Priority: Blocker
>
> There's only one implementation and unlikely to be others. This class should 
> likely be final. It can be made non-final later if ever a need is uncovered 
> to change the behavior.



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

Re: [I] Fix build failure with jdk 23 [maven-javadoc-plugin]

[via GitHub]

olamy commented on issue #373:
URL: 
https://github.com/apache/maven-javadoc-plugin/issues/373#issuecomment-2916147120

   > [@olamy](https://github.com/olamy) did you [notice the 
warning](https://github.com/apache/maven-javadoc-plugin/actions/runs/15291408104/job/43011781118)?
   > 
   > ```
   > Warning:  Javadoc Warnings
   > Warning: [WARNING] warning: The -footer option is no longer supported and 
will be ignored.
   > Warning:  It may be removed in a future release.
   > ```
   
   this was happening for osx using jdk 23 adoptopenjdk and has generated a 
segfault. 
   I have disable this from the build matrix.


-- 
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

[PR] Issue #369 Support --no-fonts option per default for jdk 23+ [maven-javadoc-plugin]

[via GitHub]

olamy opened a new pull request, #375:
URL: https://github.com/apache/maven-javadoc-plugin/pull/375

   Signed-off-by: Olivier Lamy 
   
   Following this checklist to help us incorporate your
   contribution quickly and easily:
   
   - [ ] Your pull request should address just one issue, without pulling in 
other changes.
   - [ ] Write a pull request description that is detailed enough to understand 
what the pull request does, how, and why.
   - [ ] Each commit in the pull request should have a meaningful subject line 
and body.
 Note that commits might be squashed by a maintainer on merge.
   - [ ] Write unit tests that match behavioral changes, where the tests fail 
if the changes to the runtime are not applied.
 This may not always be possible but is a best-practice.
   - [ ] Run `mvn verify` to make sure basic checks pass.
 A more thorough check will be performed on your pull request automatically.
   - [ ] You have run the integration tests successfully (`mvn -Prun-its 
verify`).
   
   If your pull request is about ~20 lines of code you don't need to sign an
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.pdf) if you are unsure
   please ask on the developers list.
   
   To make clear that you license your contribution under
   the [Apache License Version 2.0, January 
2004](http://www.apache.org/licenses/LICENSE-2.0)
   you have to acknowledge this by using the following check-box.
   
   - [ ] I hereby declare this contribution to be licenced under the [Apache 
License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
   - [ ] In any other case, please file an [Apache Individual Contributor 
License Agreement](https://www.apache.org/licenses/icla.pdf).
   


-- 
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

Re: [PR] Bump org.apache.maven.plugins:maven-plugins from 43 to 44 [maven-javadoc-plugin]

[via GitHub]

olamy commented on PR #361:
URL: 
https://github.com/apache/maven-javadoc-plugin/pull/361#issuecomment-2916155775

   @dependabot rebase


-- 
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

[PR] [MNG-8746] Preserve property insertion order in WrapperProperties [maven]

[via GitHub]

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

   ## Description
   
   This PR fixes MNG-8746 by implementing property insertion order preservation 
in Maven's WrapperProperties class. The issue was that properties accessed 
through `model.getProperties()` were not maintaining their insertion order, 
leading to inconsistent behavior.
   
   ## Problem
   
   The WrapperProperties class in the Maven compat layer was not preserving the 
insertion order of properties. When properties were added through the 
Properties API and then accessed via methods like `keySet()` or `entrySet()`, 
they would be returned in an arbitrary order (often alphabetical) rather than 
the order they were inserted.
   
   ## Solution
   
   Implemented a comprehensive order preservation mechanism:
   
   ### Key Changes:
   
   1. **Enhanced WrapperProperties Template** 
(`src/mdo/java/WrapperProperties.java`):
  - Added internal `OrderedProperties` class that maintains insertion order
  - Implemented caching mechanism to preserve order across read/write 
operations
  - Modified all read operations to use the ordered cache
  - Simplified write operations to work directly with the ordered properties
   
   2. **OrderedProperties Implementation**:
  - Uses `ArrayList keyOrder` to track key insertion order
  - Overrides `put()`, `setProperty()`, and `remove()` methods to maintain 
order
  - Custom `KeySet` and `EntrySet` implementations that iterate in 
insertion order
  - Removed unnecessary synchronized wrappers that interfered with ordering
   
   3. **Architecture Simplification**:
  - WrapperProperties maintains its own ordered state independently
  - Only syncs content (not order) with the v4 API model when needed
  - Eliminated complex cache invalidation logic that was causing order loss
   
   ## Testing
   
   Added comprehensive tests:
   - `PropertiesOrderTest`: Tests v4 API model order preservation
   - `WrapperPropertiesOrderTest`: Tests compat layer order preservation
   - Covers various scenarios including property modification and initial 
properties
   
   ## Verification
   
   The implementation ensures that:
   ```java
   Properties props = model.getProperties();
   props.setProperty("key3", "value3");
   props.setProperty("key1", "value1");
   props.setProperty("key2", "value2");
   
   // keySet() now returns ["key3", "key1", "key2"] in insertion order
   // instead of ["key1", "key2", "key3"] in alphabetical order
   ```
   
   ## Impact
   
   - ✅ Preserves property insertion order for consistent behavior
   - ✅ Maintains backward compatibility
   - ✅ No breaking changes to existing APIs
   - ✅ Improves reproducible builds
   
   Fixes: https://issues.apache.org/jira/browse/MNG-8746
   
   ---
   Pull Request opened by [Augment Code](https://www.augmentcode.com/) with 
guidance from the PR author


-- 
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

[PR] Bump commons-beanutils:commons-beanutils from 1.7.0 to 1.11.0 in /surefire-its/src/test/resources/webapp [maven-surefire]

[via GitHub]

dependabot[bot] opened a new pull request, #851:
URL: https://github.com/apache/maven-surefire/pull/851

   Bumps commons-beanutils:commons-beanutils from 1.7.0 to 1.11.0.
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=commons-beanutils:commons-beanutils&package-manager=maven&previous-version=1.7.0&new-version=1.11.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of 
the ignore conditions of the specified dependency
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   You can disable automated security fix PRs for this repo from the [Security 
Alerts page](https://github.com/apache/maven-surefire/network/alerts).
   
   


-- 
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

[I] minimizeJar too aggressive [maven-shade-plugin]

[via GitHub]

RefuX opened a new issue, #255:
URL: https://github.com/apache/maven-shade-plugin/issues/255

   ### Affected version
   
   3.6.0
   
   ### Bug description
   
   I'm shading the com.fasterxml.woodstox:woodstox-core:7.0.0 library.
   
   `META-INF/services/javax.xml.stream.XMLEventFactory`:
   `my.shaded.wstx.stax.WstxEventFactory`
   
   `META-INF/services/javax.xml.stream.XMLInputFactory`:
   `my.shaded.wstx.stax.WstxInputFactory`
   
   `META-INF/services/javax.xml.stream.XMLOutputFactory`:
   `my.shaded.wstx.stax.WstxOutputFactory`
   
   With minimizeJar set to true, even though we have the above references to 
these Factory classes. The Factory classes are minimized out.
   
   For now my work-around is to create a static reference to the Factories 😭


-- 
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.apache.org

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

Re: [I] Fix build failure with jdk 23 [maven-javadoc-plugin]

[via GitHub]

joakime commented on issue #373:
URL: 
https://github.com/apache/maven-javadoc-plugin/issues/373#issuecomment-2916232021

   @olamy you missed the point of that warning, it's not architecture specific, 
and it has no influence on the segfault.
   
   The `-footer` option is deprecated.
   
   https://docs.oracle.com/en/java/javase/23/docs/specs/man/javadoc.html
   
   > -footer html-code
   >This option is no longer supported and reports a warning. 
   
   It doesn't do anything right now, and may be removed in a future release.
   To me, that means the `` configuration in `maven-javadoc-plugin` is 
also deprecated, or at least deprecated based on the JDK version you are using 
with the `maven-javadoc-plugin`.


-- 
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

Re: [PR] close RandomAccessFile [maven-clean-plugin]

[via GitHub]

elharo closed pull request #257: close RandomAccessFile
URL: https://github.com/apache/maven-clean-plugin/pull/257


-- 
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

[PR] Add missing @Override to overriding and implementing methods [maven]

[via GitHub]

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

   (no comment)


-- 
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

Re: [PR] Remove ignored method call [maven]

[via GitHub]

Pankraz76 commented on code in PR #2399:
URL: https://github.com/apache/maven/pull/2399#discussion_r2111988814


##
impl/maven-impl/src/main/java/org/apache/maven/impl/standalone/ApiRunner.java:
##
@@ -402,8 +402,6 @@ static Session newSession(RepositorySystem system, Lookup 
lookup, @Nullable Loca
 mavenUserHome.resolve("settings.xml"))
 .getEffectiveSettings();
 
-settings.getProfiles();

Review Comment:
   should find such issues by tooling.
   
   https://github.com/user-attachments/assets/b39712e5-d03e-43c0-817c-2c6134f41335";
 />
   



-- 
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

Re: [PR] Remove ignored method call [maven]

[via GitHub]

Pankraz76 commented on code in PR #2399:
URL: https://github.com/apache/maven/pull/2399#discussion_r2111988814


##
impl/maven-impl/src/main/java/org/apache/maven/impl/standalone/ApiRunner.java:
##
@@ -402,8 +402,6 @@ static Session newSession(RepositorySystem system, Lookup 
lookup, @Nullable Loca
 mavenUserHome.resolve("settings.xml"))
 .getEffectiveSettings();
 
-settings.getProfiles();

Review Comment:
   should find such issues by tooling.
   
   https://github.com/user-attachments/assets/b39712e5-d03e-43c0-817c-2c6134f41335";
 />
   
   mostly IO stuff easy to rewrite: 
   https://github.com/user-attachments/assets/0e75a25e-c381-4ac7-aebb-ff64e5f4e698";
 />
   



-- 
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] (MPOM-451) Please remove repository entry from Apache parent POM

[Istvan Toth (Jira)]

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

Istvan Toth commented on MPOM-451:
--

This can also result in being blacklisted from the ASF repo.

> Please remove repository entry from Apache parent POM
> -
>
> Key: MPOM-451
> URL: https://issues.apache.org/jira/browse/MPOM-451
> Project: Maven POMs (Moved to GitHub issues) 
>  Issue Type: Bug
>Affects Versions: ASF-31
>Reporter: Lenny Primak
>Priority: Minor
> Fix For: ASF-34
>
>
> According to maven documentation, it's "bad form" to have repository entries 
> in POM files.
> Apache POM includes snapshot repository element.
> This is interfering with Apache Shiro builds, for example



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

Re: [PR] Remove extra variable [maven]

[via GitHub]

Pankraz76 commented on code in PR #2398:
URL: https://github.com/apache/maven/pull/2398#discussion_r2111997079


##
impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultInterpolator.java:
##
@@ -263,10 +263,9 @@ private static String doSubstVars(
 // Using the start and stop delimiter indices, extract
 // the first, deepest nested variable placeholder.
 String variable = val.substring(startDelim + DELIM_START.length(), 
stopDelim);
-String org = variable;
 
 String substValue = processSubstitution(
-variable, org, cycleMap, configProps, callback, postprocessor, 
defaultsToEmptyString);

Review Comment:
   yes, should be covered by 
https://pmd.github.io/pmd/pmd_rules_java_bestpractices.html#unusedassignment



-- 
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

Re: [PR] Update site descriptor [maven-clean-plugin]

[via GitHub]

slawekjaranowski merged PR #255:
URL: https://github.com/apache/maven-clean-plugin/pull/255


-- 
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

Re: [PR] [SUREFIRE-2288] Set version for maven-changes-plugin [maven-surefire]

[via GitHub]

slawekjaranowski commented on PR #846:
URL: https://github.com/apache/maven-surefire/pull/846#issuecomment-2916882442

   > Another solution is to migrate the jira issues to GH quickly and configure 
the m-changes-plugin to GH.
   
   Even we will have all issues in GitHub,I don't know what GitHub issues will 
be in such report, 
   I don't also known if GitHub PR will be included by  m-changes-plugin .
   
   I'm for preparing one release notes by one tool - we have release-drafter in 
project so should be enough.
   
   We cen link from documentation to GitHub release notes.


-- 
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

Re: [I] simplify our GH issue templates boilerplate [maven-gh-actions-shared]

[via GitHub]

Bukama commented on issue #165:
URL: 
https://github.com/apache/maven-gh-actions-shared/issues/165#issuecomment-2917464791

   Isn't it (also?) about the pull-request template?


-- 
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

Re: [PR] Bump org.codehaus.mojo:exec-maven-plugin from 3.5.0 to 3.5.1 [maven-site]

[via GitHub]

Bukama merged PR #799:
URL: https://github.com/apache/maven-site/pull/799


-- 
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

Re: [PR] Update issue-management.md with correct GH issues links [maven-site]

[via GitHub]

Bukama commented on code in PR #798:
URL: https://github.com/apache/maven-site/pull/798#discussion_r2112645834


##
content/markdown/issue-management.md:
##
@@ -37,11 +37,11 @@ issue management systems depending on which component is 
involved:
 
 * Maven Shared Components: Please refer to the [Shared 
Components](./shared/index.html) page
 
-* Doxia: 
+* Doxia: 

Review Comment:
   > copy/paste mistake :)
   
   lalala 👼



-- 
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

[I] Update issue-management.md after GitHub issues are enabled for all repositories [maven-site]

[via GitHub]

Bukama opened a new issue, #800:
URL: https://github.com/apache/maven-site/issues/800

   ### Improvement proposal
   
   The `issue-management.md` page states, that all projects use JIRA for issue 
tracking. The page needs to be updated when GH issues are enabled for 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.apache.org

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

Re: [PR] remove unused plexus-xml dependency [maven-clean-plugin]

[via GitHub]

elharo merged PR #258:
URL: https://github.com/apache/maven-clean-plugin/pull/258


-- 
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

Re: [I] Fix build failure with jdk 23 [maven-javadoc-plugin]

[via GitHub]

olamy commented on issue #373:
URL: 
https://github.com/apache/maven-javadoc-plugin/issues/373#issuecomment-2917598677

   Oh I see, good point. I was only looking at the output of `javadoc -h`, 
which doesn't mention that.



-- 
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

Re: [PR] `try` can use `automatic resource management` [maven]

[via GitHub]

Pankraz76 commented on PR #2401:
URL: https://github.com/apache/maven/pull/2401#issuecomment-2916936425

   again NPE: 
   
   ```console
   Error:  Errors: 
   Error:MavenIT0008SimplePluginTest.testit0008:45 » NullPointer Cannot 
invoke "java.io.File.exists()" because "dir" is null
   Error:MavenIT0010DependencyClosureResolutionTest.testit0010:47 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0011DefaultVersionByDependencyManagementTest.testit0011:44 
» NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0018DependencyManagementTest.testit0018:42 » NullPointer 
Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0021PomProfileTest.testit0021:42 » NullPointer Cannot 
invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0030DepPomDepMgmtInheritanceTest.testit0030:40 » 
NullPointer Cannot invoke "java.io.File.exists()" because "dir" is null
   Error:MavenIT0041ArtifactTypeFromPluginExtensionTest.testit0041:47 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0085TransitiveSystemScopeTest.testit0085:48 » NullPointer 
Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0087PluginRealmWithProjectLevelDepsTest.testit0087:53 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenIT0142DirectDependencyScopesTest.testit0142:56 » NullPointer 
Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:
MavenIT0143TransitiveDependencyScopesTest.testitCompileScope:53->run:249 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:
MavenIT0143TransitiveDependencyScopesTest.testitProvidedScope:113->run:249 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:
MavenIT0143TransitiveDependencyScopesTest.testitRuntimeScope:162->run:249 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:
MavenIT0143TransitiveDependencyScopesTest.testitTestScope:211->run:249 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:
MavenIT0146InstallerSnapshotNaming.testitNonTimestampedNameWithInstalledSNAPSHOT:102
 » NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:
MavenIT0146InstallerSnapshotNaming.testitRemoteDownloadTimestampedName:86 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenITmng0377PluginLookupFromPrefixTest.testitMNG377:49 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   Error:MavenITmng0461TolerateMissingDependencyPomTest.testit:51 » 
NullPointer Cannot invoke "java.nio.file.Path.toFile()" because "dir" is null
   [INFO] 
   Error:  Tests run: 978, Failures: 0, Errors: 18, Skipped: 49
   [INFO] 
   ```


-- 
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

Re: [PR] Update issueManagement to GitHub [maven-clean-plugin]

[via GitHub]

slawekjaranowski merged PR #256:
URL: https://github.com/apache/maven-clean-plugin/pull/256


-- 
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

Re: [PR] Remove extra variable [maven]

[via GitHub]

Pankraz76 commented on code in PR #2398:
URL: https://github.com/apache/maven/pull/2398#discussion_r2112313200


##
impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultInterpolator.java:
##
@@ -263,10 +263,9 @@ private static String doSubstVars(
 // Using the start and stop delimiter indices, extract
 // the first, deepest nested variable placeholder.
 String variable = val.substring(startDelim + DELIM_START.length(), 
stopDelim);
-String org = variable;
 
 String substValue = processSubstitution(
-variable, org, cycleMap, configProps, callback, postprocessor, 
defaultsToEmptyString);

Review Comment:
   seems its currently renaming not removal.



-- 
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

[PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

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

   - https://github.com/apache/maven/pull/2402


-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112369077


##
pom.xml:
##
@@ -816,6 +824,10 @@ under the License.
   
 
 
+  
+org.apache.maven.plugins
+maven-pmd-plugin
+  

Review Comment:
   ```suggestion
   ```
   why profile not working? its done like in quarkus:
   
https://github.com/quarkusio/quarkus/blob/d8618eccda1af323726537ba25bdbd24cdcee280/build-parent/pom.xml#L693



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112369077


##
pom.xml:
##
@@ -816,6 +824,10 @@ under the License.
   
 
 
+  
+org.apache.maven.plugins
+maven-pmd-plugin
+  

Review Comment:
   ```suggestion
   ```
   why profile `pmd` not working? its done like in quarkus:
   
https://github.com/quarkusio/quarkus/blob/d8618eccda1af323726537ba25bdbd24cdcee280/build-parent/pom.xml#L693



-- 
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

Re: [PR] avoid false `IO` catch on `mkdirs()` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2396:
URL: https://github.com/apache/maven/pull/2396#discussion_r2111960785


##
its/core-it-support/core-it-plugins/maven-it-plugin-uses-wagon/src/main/java/org/apache/maven/plugin/coreit/LookupWagonMojo.java:
##
@@ -93,8 +93,8 @@ public void execute() throws MojoExecutionException, 
MojoFailureException {
 getLog().info("[MAVEN-CORE-IT-LOG] Creating output file " + 
outputFile);
 
 OutputStream out = null;
+outputFile.getParentFile().mkdirs();

Review Comment:
   yes we can do the try with but then do it twice as there are only 4 ATM and 
with this change its a lot more, thats why i see this as enable.
   
   Lets make whats possible, doing step by step to reduce overhead. 



-- 
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

[PR] `try` can use `automatic resource management` [maven]

[via GitHub]

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

   - https://github.com/apache/maven/pull/2395
   - https://github.com/apache/maven/pull/2396


-- 
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

[PR] close RandomAccessFile [maven-clean-plugin]

[via GitHub]

elharo opened a new pull request, #257:
URL: https://github.com/apache/maven-clean-plugin/pull/257

   (no comment)


-- 
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

Re: [PR] [POC-FIX-PMD] maven-pmd-plugin: reactivate PMD - Error Prone [maven]

[via GitHub]

Pankraz76 commented on PR #2364:
URL: https://github.com/apache/maven/pull/2364#issuecomment-2916406741

   - https://github.com/apache/maven/pull/2396
   - https://github.com/apache/maven/pull/2396#pullrequestreview-2874542777
   - 
https://docs.openrewrite.org/recipes/staticanalysis/missingoverrideannotation
   - 
https://pmd.sourceforge.io/pmd-6.3.0/pmd_rules_java_bestpractices.html#missingoverride
   
   


-- 
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

Re: [PR] avoid false `IO` catch on `mkdirs()` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2396:
URL: https://github.com/apache/maven/pull/2396#discussion_r2111951540


##
its/core-it-suite/src/test/resources/mng-2135/plugin/src/main/java/coreit/ItMojo.java:
##
@@ -52,8 +52,8 @@ public class ItMojo extends AbstractMojo {
 private File outputFile;
 
 public void execute() throws MojoExecutionException {

Review Comment:
   yes will give dedicated PR for this.
   
   https://docs.openrewrite.org/recipes/staticanalysis/missingoverrideannotation
   
   another good example why we need to buckle up tooling.
   
   
https://pmd.sourceforge.io/pmd-6.3.0/pmd_rules_java_bestpractices.html#missingoverride
   
   - https://github.com/apache/maven/pull/2364



-- 
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

Re: [PR] Bump com.google.guava:guava from 33.3.0-jre to 33.4.6-jre [maven-indexer]

[via GitHub]

dependabot[bot] commented on PR #415:
URL: https://github.com/apache/maven-indexer/pull/415#issuecomment-2916751210

   Looks like this PR is already up-to-date with master! If you'd still like to 
recreate it from scratch, overwriting any edits, you can request `@dependabot 
recreate`.


-- 
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

Re: [PR] Bump com.google.guava:guava from 33.3.0-jre to 33.4.6-jre [maven-indexer]

[via GitHub]

cstamas commented on PR #415:
URL: https://github.com/apache/maven-indexer/pull/415#issuecomment-2916750955

   @dependabot rebase


-- 
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

[PR] remove unused plexus-xml dependency [maven-clean-plugin]

[via GitHub]

elharo opened a new pull request, #258:
URL: https://github.com/apache/maven-clean-plugin/pull/258

   (no comment)


-- 
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

Re: [PR] chore: `ArtifactHandlerTest` add assertion to resolve `@SuppressWarnings("checkstyle:UnusedLocalVariable")` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2375:
URL: https://github.com/apache/maven/pull/2375#discussion_r2111240837


##
impl/maven-core/src/test/java/org/apache/maven/artifact/handler/ArtifactHandlerTest.java:
##
@@ -77,8 +107,15 @@ void testAptConsistency() throws Exception {
 ArtifactHandler handler =
 
container.lookup(ArtifactHandlerManager.class).getArtifactHandler(type);
 assertEquals(handler.getExtension(), extension, type + " 
extension");
-// Packaging/Directory is Maven1 remnant!!!
-// assertEquals(handler.getPackaging(), packaging, type + " 
packaging");
+assertThat(PACKAGING_TYPES).contains(handler.getPackaging(), 
packaging);
+if (handler.getPackaging().equals("test-jar")) {
+assertEquals("jar", packaging);
+
+} else if (handler.getPackaging().equals("ejb-client")) {
+assertEquals("ejb", packaging);

Review Comment:
   both look like bug to me. maven-archetype",
   "maven-plugin", "java-source" are all equal so why not these two 
? So its not a hyphen thing in general, right?



-- 
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

Re: [PR] avoid false `IO` catch on `mkdirs()` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2396:
URL: https://github.com/apache/maven/pull/2396#discussion_r2111951540


##
its/core-it-suite/src/test/resources/mng-2135/plugin/src/main/java/coreit/ItMojo.java:
##
@@ -52,8 +52,8 @@ public class ItMojo extends AbstractMojo {
 private File outputFile;
 
 public void execute() throws MojoExecutionException {

Review Comment:
   yes will give dedicated PR for this.
   
   https://docs.openrewrite.org/recipes/staticanalysis/missingoverrideannotation
   
   another good example why we need to buckle up tooling.
   
   
https://pmd.sourceforge.io/pmd-6.3.0/pmd_rules_java_bestpractices.html#missingoverride



-- 
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

Re: [PR] avoid false `IO` catch on `mkdirs()` [maven]

[via GitHub]

Pankraz76 commented on code in PR #2396:
URL: https://github.com/apache/maven/pull/2396#discussion_r2111975276


##
its/core-it-support/core-it-plugins/maven-it-plugin-uses-wagon/src/main/java/org/apache/maven/plugin/coreit/LookupWagonMojo.java:
##
@@ -93,8 +93,8 @@ public void execute() throws MojoExecutionException, 
MojoFailureException {
 getLog().info("[MAVEN-CORE-IT-LOG] Creating output file " + 
outputFile);
 
 OutputStream out = null;
+outputFile.getParentFile().mkdirs();

Review Comment:
   https://github.com/user-attachments/assets/b701d181-1b6c-4d50-8221-ae595df79e6d";
 />
   
   - https://github.com/apache/maven/pull/2401
   
   



-- 
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

Re: [PR] Remove ignored method call [maven]

[via GitHub]

elharo merged PR #2399:
URL: https://github.com/apache/maven/pull/2399


-- 
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

Re: [I] CVE-2024-13009 [maven-site-plugin]

[via GitHub]

joakime commented on issue #237:
URL: 
https://github.com/apache/maven-site-plugin/issues/237#issuecomment-2916538531

   Note: Jetty 9 is EOL (End of Life)
   
   * https://github.com/jetty/jetty.project/issues/7958
   * https://github.com/jetty/jetty.project/issues/10485
   
   If you still need `javax.servlet` namespace support, use the `ee8` 
environment in Jetty 12.


-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112369077


##
pom.xml:
##
@@ -816,6 +824,10 @@ under the License.
   
 
 
+  
+org.apache.maven.plugins
+maven-pmd-plugin
+  

Review Comment:
   ```suggestion
   ```
   why profile not working? its done like in quarkus:
   
https://github.com/quarkusio/quarkus/blob/d8618eccda1af323726537ba25bdbd24cdcee280/build-parent/pom.xml#L693



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112396239


##
pom.xml:
##
@@ -793,13 +793,29 @@ under the License.
 
   org.apache.maven.plugins
   maven-pmd-plugin

Review Comment:
   its already there and in parent just the same. Sure we align on proper 
tooling. Ruleset will evolve with existing issues being fixed:
   - https://github.com/apache/maven/pull/2402
   - https://github.com/apache/maven/pull/2348



-- 
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

Re: [PR] UnusedAssignment [maven]

[via GitHub]

Pankraz76 commented on code in PR #2350:
URL: https://github.com/apache/maven/pull/2350#discussion_r2112399079


##
compat/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataGraphEdge.java:
##
@@ -30,9 +30,9 @@
 public class MetadataGraphEdge {
 String version;
 ArtifactScopeEnum scope;
-int depth = -1;
-int pomOrder = -1;
-boolean resolved = true;
+int depth;

Review Comment:
   thanks for advisory. Will undo and dedication to each concern.



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112396239


##
pom.xml:
##
@@ -793,13 +793,29 @@ under the License.
 
   org.apache.maven.plugins
   maven-pmd-plugin

Review Comment:
   its already there and in parent just the same. Sure we align on proper 
tooling. Ruleset will evolve with existing issues being fixed:
   - https://github.com/apache/maven/pull/2402
   - https://github.com/apache/maven/pull/2348
   - https://github.com/apache/maven/pull/2350



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112396239


##
pom.xml:
##
@@ -793,13 +793,29 @@ under the License.
 
   org.apache.maven.plugins
   maven-pmd-plugin

Review Comment:
   its already there and in parent just the same. Sure we align on proper 
tooling. Ruleset will evolve with existing issues being fixed:
   - https://github.com/apache/maven/pull/2402
   - 



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112396239


##
pom.xml:
##
@@ -793,13 +793,29 @@ under the License.
 
   org.apache.maven.plugins
   maven-pmd-plugin

Review Comment:
   its already there and in parent just the same. Sure we align on proper 
tooling. Ruleset will evolve with existing issues being fixed:
   - https://github.com/apache/maven/pull/2402
   - https://github.com/apache/maven/pull/2348
   - https://github.com/apache/maven/pull/2373
   - https://github.com/apache/maven/pull/2350



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112396239


##
pom.xml:
##
@@ -793,13 +793,29 @@ under the License.
 
   org.apache.maven.plugins
   maven-pmd-plugin

Review Comment:
   its already there, as well as in parent.
   Im sure we align on proper tooling.
   Ruleset could evolve with existing issues being fixed:
   
   - https://github.com/apache/maven/pull/2402
   - https://github.com/apache/maven/pull/2348
   - https://github.com/apache/maven/pull/2373
   - https://github.com/apache/maven/pull/2350



-- 
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

Re: [PR] PoC rule: category/java/bestpractices.xml/MissingOverride [maven]

[via GitHub]

Pankraz76 commented on code in PR #2403:
URL: https://github.com/apache/maven/pull/2403#discussion_r2112404483


##
.pmd/ruleset.xml:
##
@@ -0,0 +1,7 @@
+
+http://pmd.sourceforge.net/ruleset/2.0.0";
+ xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+ xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 
https://pmd.sourceforge.io/ruleset_2_0_0.xsd";>
+

Review Comment:
   - https://github.com/apache/maven/pull/2402



-- 
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

Re: [PR] Upgrade jetty-server dependency patch (CVE-2024-13009) [maven-site-plugin]

[via GitHub]

olamy merged PR #238:
URL: https://github.com/apache/maven-site-plugin/pull/238


-- 
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

Re: [I] CVE-2024-13009 [maven-site-plugin]

[via GitHub]

olamy closed issue #237: CVE-2024-13009
URL: https://github.com/apache/maven-site-plugin/issues/237


-- 
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

Re: [I] CVE-2024-13009 [maven-site-plugin]

[via GitHub]

olamy commented on issue #237:
URL: 
https://github.com/apache/maven-site-plugin/issues/237#issuecomment-2917897480

   @joakime can;t really upgrade to 12 as maven--site-plugin is Java8


-- 
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

Re: [I] CVE-2024-13009 [maven-site-plugin]

[via GitHub]

jpmartins commented on issue #237:
URL: 
https://github.com/apache/maven-site-plugin/issues/237#issuecomment-2918305359

   Hi everyone,
   
   Thank you for your contributions and for updating the dependency on the 
master branch. 
   
   Regarding the upgrade to Jetty 12, I understand that this would be a 
significant and potentially time-consuming breaking change. It might be more 
appropriate to plan this for the maven-site-plugin 5.x release (assuming it 
doesn't fit 4.x eventual plan)
   
   Could you please provide an update on when we can expect the next releases? 
What are the next steps in this process?
   
   Is there any possibility of backporting the changes to the maven-site-plugin 
3.x series, specifically for a 3.21.1 release?
   
   Thank you for your attention and assistance.
   
   Have a great day, and let's continue doing our best!


-- 
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

Re: [PR] Bump org.apache.groovy:groovy-bom from 4.0.26 to 4.0.27 [maven-invoker-plugin]

[via GitHub]

Bukama merged PR #287:
URL: https://github.com/apache/maven-invoker-plugin/pull/287


-- 
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

[PR] Bump jlineVersion from 3.30.3 to 3.30.4 [maven]

[via GitHub]

dependabot[bot] opened a new pull request, #2405:
URL: https://github.com/apache/maven/pull/2405

   Bumps `jlineVersion` from 3.30.3 to 3.30.4.
   Updates `org.jline:jline-reader` from 3.30.3 to 3.30.4
   
   Release notes
   Sourced from https://github.com/jline/jline3/releases";>org.jline:jline-reader's 
releases.
   
   JLine 3.30.4
   
   🐛 Bug Fixes
   
   fix: fix Jansi AnsiConsole broken color detection in uber jars (https://redirect.github.com/jline/jline3/pull/1305";>#1305) https://github.com/gnodet";>@​gnodet
   fix: Fix SyntaxHighlighter glob pattern handling for non-default file 
systems (https://redirect.github.com/jline/jline3/pull/1300";>#1300) https://github.com/gnodet";>@​gnodet
   
   📦 Dependency updates
   
   chore: Bump eu.maveniverse.maven.njord:extension from 0.5.4 to 0.6.2 (https://redirect.github.com/jline/jline3/pull/1307";>#1307) @https://github.com/apps/dependabot";>dependabot[bot]
   chore: Bump groovy.version from 4.0.26 to 4.0.27 (https://redirect.github.com/jline/jline3/pull/1306";>#1306) @https://github.com/apps/dependabot";>dependabot[bot]
   chore: Bump org.codehaus.mojo:exec-maven-plugin from 3.5.0 to 3.5.1 (https://redirect.github.com/jline/jline3/pull/1308";>#1308) @https://github.com/apps/dependabot";>dependabot[bot]
   
   📝 Documentation updates
   
   docs: Link to Pty4j on Terminal (https://redirect.github.com/jline/jline3/pull/1295";>#1295) https://github.com/vorburger";>@​vorburger
   
   👻 Maintenance
   
   chore: fix intermittent encoding test failures by handling READ_EXPIRED 
correctly (https://redirect.github.com/jline/jline3/pull/1303";>#1303) https://github.com/gnodet";>@​gnodet
   chore: improve Maven cache fallback strategy in GitHub Actions (https://redirect.github.com/jline/jline3/pull/1304";>#1304) https://github.com/gnodet";>@​gnodet
   
   
   
   
   Commits
   
   https://github.com/jline/jline3/commit/fcf6461c1b21a9875d6ba93c6a87a1806ead5091";>fcf6461
 [maven-release-plugin] prepare release jline-3.30.4
   https://github.com/jline/jline3/commit/d096f1f223141c07b75952e6387638af2c5c16c9";>d096f1f
 docs: Link to Pty4j on Terminal (https://redirect.github.com/jline/jline3/issues/1295";>#1295)
   https://github.com/jline/jline3/commit/ce2d44f46825fff2cc79938a878daffd6c3c8be8";>ce2d44f
 fix: fix Jansi AnsiConsole broken color detection in uber jars (https://redirect.github.com/jline/jline3/issues/1305";>#1305)
   https://github.com/jline/jline3/commit/ac97af1d501603dc127cf8344227d91acd1b0b7d";>ac97af1
 chore: Bump eu.maveniverse.maven.njord:extension from 0.5.4 to 0.6.2 (https://redirect.github.com/jline/jline3/issues/1307";>#1307)
   https://github.com/jline/jline3/commit/f914fa87a8e14555865c5f9696c566a77aa39f05";>f914fa8
 chore: Bump groovy.version from 4.0.26 to 4.0.27 (https://redirect.github.com/jline/jline3/issues/1306";>#1306)
   https://github.com/jline/jline3/commit/a21d9094da2f08025e10ee5bb4c6e9c8db5deb71";>a21d909
 chore: Bump org.codehaus.mojo:exec-maven-plugin from 3.5.0 to 3.5.1 (https://redirect.github.com/jline/jline3/issues/1308";>#1308)
   https://github.com/jline/jline3/commit/96c50f3027d58f667c7cde6498d4ebdf3de276d9";>96c50f3
 fix: fix SyntaxHighlighter addFiles method to handle glob patterns correctly 
...
   https://github.com/jline/jline3/commit/1bdd1ac3575427ac80bea35b1bd2e335faea106e";>1bdd1ac
 chore: fix intermittent encoding test failures by handling READ_EXPIRED 
corre...
   https://github.com/jline/jline3/commit/0cfb48c4885846624d5c9ea01286f91ce835f18e";>0cfb48c
 chore: improve Maven cache fallback strategy in GitHub Actions (https://redirect.github.com/jline/jline3/issues/1304";>#1304)
   https://github.com/jline/jline3/commit/3a1244997968592e73a31003c35ea243290b9034";>3a12449
 [maven-release-plugin] prepare for next development iteration
   See full diff in https://github.com/jline/jline3/compare/jline-3.30.3...jline-3.30.4";>compare
 view
   
   
   
   
   Updates `org.jline:jline-style` from 3.30.3 to 3.30.4
   
   Release notes
   Sourced from https://github.com/jline/jline3/releases";>org.jline:jline-style's 
releases.
   
   JLine 3.30.4
   
   🐛 Bug Fixes
   
   fix: fix Jansi AnsiConsole broken color detection in uber jars (https://redirect.github.com/jline/jline3/pull/1305";>#1305) https://github.com/gnodet";>@​gnodet
   fix: Fix SyntaxHighlighter glob pattern handling for non-default file 
systems (https://redirect.github.com/jline/jline3/pull/1300";>#1300) https://github.com/gnodet";>@​gnodet
   
   📦 Dependency updates
   
   chore: Bump eu.maveniverse.maven.njord:extension from 0.5.4 to 0.6.2 (https://redirect.github.com/jline/jline3/pull/1307";>#1307) @https://github.com/apps/dependabot";>dependabot[bot]
   chore: Bump groovy.version from 4.0.26 to 4.0.27 (https://redirect.github.com/jline/jline3/pull/1306";>#1306) @https://github.com/apps/dependabot";>dependabot[bot]
   chore: Bump org.codehaus.mojo:exec-maven-plugin from 3.5.0 to 3.5.1 (https://redirect.github.com/jline/jline3/pull/1308";>#1308) @htt

[PR] Bump org.eclipse.sisu:org.eclipse.sisu.plexus from 0.9.0.M3 to 0.9.0.M4 [maven-javadoc-plugin]

[via GitHub]

dependabot[bot] opened a new pull request, #376:
URL: https://github.com/apache/maven-javadoc-plugin/pull/376

   Bumps 
[org.eclipse.sisu:org.eclipse.sisu.plexus](https://github.com/eclipse-sisu/sisu-project)
 from 0.9.0.M3 to 0.9.0.M4.
   
   Release notes
   Sourced from https://github.com/eclipse-sisu/sisu-project/releases";>org.eclipse.sisu:org.eclipse.sisu.plexus's
 releases.
   
   0.9.0.M4
   Most important change: ASM is "demoted" to plain dependency, 
hence, consumer is able to override/update it the usual "Maven way". 
This applies to all components: inject, plexus and sisu-maven-plugin as 
well.
   Historically, Sisu shaded in ASM just like Guice did. Later Sisu started 
shipping "main" JAR with shaded ASM but also "no_asm" 
classified artifact without ASM (just like Guice did with "classes" 
classified JAR). Starting from this version, Sisu does not 
shade ASM anymore, it is "demoted" to transitive dependency. 
Sisu is built with ASM 9.8.
   Sisu is compatible with range of Guice [3,7).
   What's Changed
   
   Disable shallow clones for sonarcloud analysis by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/145";>eclipse-sisu/sisu-project#145
   Remove spurious asserts by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/147";>eclipse-sisu/sisu-project#147
   Post release cleanup by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/146";>eclipse-sisu/sisu-project#146
   Fix jacoco + code coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/151";>eclipse-sisu/sisu-project#151
   Enable code coverage again for all modules by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/153";>eclipse-sisu/sisu-project#153
   Use default property for the jacoco agent by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/154";>eclipse-sisu/sisu-project#154
   Add documentation on Plexus Configurator API by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/172";>eclipse-sisu/sisu-project#172
   Remove about.html as only relevant for Eclipse plugins by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/171";>eclipse-sisu/sisu-project#171
   Document Lifecycle support by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/175";>eclipse-sisu/sisu-project#175
   Call TypeAwareExpressionEvaluator.evaluate(String,Class) if available by 
https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/161";>eclipse-sisu/sisu-project#161
   Pass strict flag also via "discoverComponents" by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/182";>eclipse-sisu/sisu-project#182
   Embed/relocate ASM via m-shade-p by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/185";>eclipse-sisu/sisu-project#185
   Update to ASM 9.8 by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/186";>eclipse-sisu/sisu-project#186
   Increase coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/192";>eclipse-sisu/sisu-project#192
   Align subproject names (and naming) by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/191";>eclipse-sisu/sisu-project#191
   Build infra updates by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/193";>eclipse-sisu/sisu-project#193
   More coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/195";>eclipse-sisu/sisu-project#195
   More coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/196";>eclipse-sisu/sisu-project#196
   Update source header. by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/194";>eclipse-sisu/sisu-project#194
   Update headers (no code change) by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/199";>eclipse-sisu/sisu-project#199
   Reformat (no code change) by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/200";>eclipse-sisu/sisu-project#200
   Reproducible by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/202";>eclipse-sisu/sisu-project#202
   Fix sisu-maven-plugin by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/203";>eclipse-sisu/sisu-project#203
   Lax array c

[PR] Bump org.jsoup:jsoup from 1.18.1 to 1.20.1 [maven-indexer]

[via GitHub]

dependabot[bot] opened a new pull request, #417:
URL: https://github.com/apache/maven-indexer/pull/417

   Bumps [org.jsoup:jsoup](https://github.com/jhy/jsoup) from 1.18.1 to 1.20.1.
   
   Release notes
   Sourced from https://github.com/jhy/jsoup/releases";>org.jsoup:jsoup's 
releases.
   
   jsoup 1.20.1
   Changes
   
   To better follow the HTML5 spec and current browsers, the HTML parser no 
longer allows self-closing tags () to close HTML 
elements by default. Foreign content (SVG, MathML), and content parsed with the 
XML parser, still supports self-closing tags. If you need specific HTML tags to 
support self-closing, you can register a custom tag via
   the TagSet configured in Parser.tagSet(), using 
Tag#set(Tag.SelfClose). Standard void tags (such as 
, 
, etc.) continue to behave as 
usual and are not affected by this
   change. https://redirect.github.com/jhy/jsoup/issues/2300";>#2300.
   The following internal components have been deprecated. 
If you do happen to be using any of these, please take the opportunity now to 
migrate away from them, as they will be removed in jsoup 1.21.1.
   
   ChangeNotifyingArrayList, 
Document.updateMetaCharsetElement(), 
Document.updateMetaCharsetElement(boolean), 
HtmlTreeBuilder.isContentForTagData(String), 
Parser.isContentForTagData(String), 
Parser.setTreeBuilder(TreeBuilder), 
Tag.formatAsBlock(), Tag.isFormListed(), 
TokenQueue.addFirst(String), 
TokenQueue.chompTo(String), 
TokenQueue.chompToIgnoreCase(String), 
TokenQueue.consumeToIgnoreCase(String), 
TokenQueue.consumeWord(), 
TokenQueue.matchesAny(String...)
   
   
   
   Functional Improvements
   
   Rebuilt the HTML pretty-printer, to simplify and consolidate the 
implementation, improve consistency, support custom Tags, and provide a cleaner 
path for ongoing improvements. The specific HTML produced by the pretty-printer 
may be different from previous versions. https://redirect.github.com/jhy/jsoup/issues/2286";>#2286.
   Added the ability to define custom tags, and to modify properties of 
known tags, via the TagSet tag collection. Their properties can 
impact both the parse and how content is serialized (output as HTML or XML). https://redirect.github.com/jhy/jsoup/issues/2285";>#2285.
   Element.cssSelector() will prefer to return shorter 
selectors by using ancestor IDs when available and unique. E.g. #id > 
div > p instead of  html > body > div > div > 
p https://redirect.github.com/jhy/jsoup/pull/2283";>#2283.
   Added Elements.deselect(int index), 
Elements.deselect(Object o), and 
Elements.deselectAll() methods to remove elements from the 
Elements list without removing them from the underlying DOM. Also 
added Elements.asList() method to get a modifiable list of 
elements without affecting the DOM. (Individual Elements remain linked to the 
DOM.) https://redirect.github.com/jhy/jsoup/issues/2100";>#2100.
   Added support for sending a request body from an InputStream with 
Connection.requestBodyStream(InputStream stream). https://redirect.github.com/jhy/jsoup/issues/1122";>#1122.
   The XML parser now supports scoped xmlns: prefix namespace declarations, 
and applies the correct namespace to Tags and Attributes. Also, added 
Tag#prefix(), Tag#localName(), 
Attribute#prefix(), Attribute#localName(), and 
Attribute#namespace() to retrieve these. https://redirect.github.com/jhy/jsoup/issues/2299";>#2299.
   CSS identifiers are now escaped and unescaped correctly to the CSS spec. 
Element#cssSelector() will emit appropriately escaped selectors, 
and the QueryParser supports those. Added 
Selector.escapeCssIdentifier() and ` 
Selector.unescapeCssIdentifier(). https://redirect.github.com/jhy/jsoup/pull/2297";>#2297, https://redirect.github.com/jhy/jsoup/pull/2305";>#2305
   
   Structure and Performance Improvements
   
   Refactored the CSS QueryParser into a clearer recursive 
descent parser. https://redirect.github.com/jhy/jsoup/pull/2310";>#2310.
   CSS selectors with consecutive combinators (e.g. div >> 
p) will throw an explicit parse exception. https://redirect.github.com/jhy/jsoup/pull/2311";>#2311.
   Performance: reduced the shallow size of an Element from 40 to 32 bytes, 
and the NodeList from 32 to 24. https://redirect.github.com/jhy/jsoup/pull/2307";>#2307.
   Performance: reduced GC load of new StringBuilders when tokenizing input 
HTML. https://redirect.github.com/jhy/jsoup/pull/2304";>#2304.
   Made Parser instances threadsafe, so that inadvertent use 
of the same instance across threads will not lead to errors. For actual 
concurrency, use Parser#newInstance() per thread. https://redirect.github.com/jhy/jsoup/pull/2314";>#2314.
   
   Bug Fixes
   
   Element names containing characters invalid in XML are now normalized to 
valid XML names when serializing. https://redirect.github.com/jhy/jsoup/issues/1496";>#1496.
   When serializing to XML, characters that are invalid in XML 1.0 should 
be removed (not encoded). https://redirect.github.com/jhy/jsoup/i

Re: [PR] Bump org.jsoup:jsoup from 1.18.1 to 1.19.1 [maven-indexer]

[via GitHub]

dependabot[bot] commented on PR #412:
URL: https://github.com/apache/maven-indexer/pull/412#issuecomment-2918233894

   Superseded by #417.


-- 
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

Re: [PR] Bump org.jsoup:jsoup from 1.18.1 to 1.19.1 [maven-indexer]

[via GitHub]

dependabot[bot] closed pull request #412: Bump org.jsoup:jsoup from 1.18.1 to 
1.19.1
URL: https://github.com/apache/maven-indexer/pull/412


-- 
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

Re: [PR] Bump lucene.version from 9.11.1 to 10.1.0 [maven-indexer]

[via GitHub]

dependabot[bot] closed pull request #404: Bump lucene.version from 9.11.1 to 
10.1.0
URL: https://github.com/apache/maven-indexer/pull/404


-- 
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

Re: [PR] Bump lucene.version from 9.11.1 to 10.1.0 [maven-indexer]

[via GitHub]

dependabot[bot] commented on PR #404:
URL: https://github.com/apache/maven-indexer/pull/404#issuecomment-2918233678

   Superseded by #416.


-- 
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

[PR] Bump lucene.version from 9.11.1 to 10.2.1 [maven-indexer]

[via GitHub]

dependabot[bot] opened a new pull request, #416:
URL: https://github.com/apache/maven-indexer/pull/416

   Bumps `lucene.version` from 9.11.1 to 10.2.1.
   Updates `org.apache.lucene:lucene-core` from 9.11.1 to 10.2.1
   
   Updates `org.apache.lucene:lucene-queryparser` from 9.11.1 to 10.2.1
   
   Updates `org.apache.lucene:lucene-analysis-common` from 9.11.1 to 10.2.1
   
   Updates `org.apache.lucene:lucene-highlighter` from 9.11.1 to 10.2.1
   
   Updates `org.apache.lucene:lucene-backward-codecs` from 9.11.1 to 10.2.1
   
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of 
the ignore conditions of the specified dependency
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   
   
   


-- 
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

[PR] Bump org.eclipse.sisu:org.eclipse.sisu.inject from 0.9.0.M3 to 0.9.0.M4 [maven-indexer]

[via GitHub]

dependabot[bot] opened a new pull request, #418:
URL: https://github.com/apache/maven-indexer/pull/418

   Bumps 
[org.eclipse.sisu:org.eclipse.sisu.inject](https://github.com/eclipse-sisu/sisu-project)
 from 0.9.0.M3 to 0.9.0.M4.
   
   Release notes
   Sourced from https://github.com/eclipse-sisu/sisu-project/releases";>org.eclipse.sisu:org.eclipse.sisu.inject's
 releases.
   
   0.9.0.M4
   Most important change: ASM is "demoted" to plain dependency, 
hence, consumer is able to override/update it the usual "Maven way". 
This applies to all components: inject, plexus and sisu-maven-plugin as 
well.
   Historically, Sisu shaded in ASM just like Guice did. Later Sisu started 
shipping "main" JAR with shaded ASM but also "no_asm" 
classified artifact without ASM (just like Guice did with "classes" 
classified JAR). Starting from this version, Sisu does not 
shade ASM anymore, it is "demoted" to transitive dependency. 
Sisu is built with ASM 9.8.
   Sisu is compatible with range of Guice [3,7).
   What's Changed
   
   Disable shallow clones for sonarcloud analysis by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/145";>eclipse-sisu/sisu-project#145
   Remove spurious asserts by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/147";>eclipse-sisu/sisu-project#147
   Post release cleanup by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/146";>eclipse-sisu/sisu-project#146
   Fix jacoco + code coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/151";>eclipse-sisu/sisu-project#151
   Enable code coverage again for all modules by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/153";>eclipse-sisu/sisu-project#153
   Use default property for the jacoco agent by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/154";>eclipse-sisu/sisu-project#154
   Add documentation on Plexus Configurator API by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/172";>eclipse-sisu/sisu-project#172
   Remove about.html as only relevant for Eclipse plugins by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/171";>eclipse-sisu/sisu-project#171
   Document Lifecycle support by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/175";>eclipse-sisu/sisu-project#175
   Call TypeAwareExpressionEvaluator.evaluate(String,Class) if available by 
https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/161";>eclipse-sisu/sisu-project#161
   Pass strict flag also via "discoverComponents" by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/182";>eclipse-sisu/sisu-project#182
   Embed/relocate ASM via m-shade-p by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/185";>eclipse-sisu/sisu-project#185
   Update to ASM 9.8 by https://github.com/kwin";>@​kwin in https://redirect.github.com/eclipse-sisu/sisu-project/pull/186";>eclipse-sisu/sisu-project#186
   Increase coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/192";>eclipse-sisu/sisu-project#192
   Align subproject names (and naming) by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/191";>eclipse-sisu/sisu-project#191
   Build infra updates by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/193";>eclipse-sisu/sisu-project#193
   More coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/195";>eclipse-sisu/sisu-project#195
   More coverage by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/196";>eclipse-sisu/sisu-project#196
   Update source header. by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/194";>eclipse-sisu/sisu-project#194
   Update headers (no code change) by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/199";>eclipse-sisu/sisu-project#199
   Reformat (no code change) by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/200";>eclipse-sisu/sisu-project#200
   Reproducible by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/202";>eclipse-sisu/sisu-project#202
   Fix sisu-maven-plugin by https://github.com/cstamas";>@​cstamas in https://redirect.github.com/eclipse-sisu/sisu-project/pull/203";>eclipse-sisu/sisu-project#203
   Lax array converte

Re: [PR] Bump com.google.guava:guava from 33.3.0-jre to 33.4.6-jre [maven-indexer]

[via GitHub]

dependabot[bot] closed pull request #415: Bump com.google.guava:guava from 
33.3.0-jre to 33.4.6-jre
URL: https://github.com/apache/maven-indexer/pull/415


-- 
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

Re: [PR] Bump com.google.guava:guava from 33.3.0-jre to 33.4.6-jre [maven-indexer]

[via GitHub]

dependabot[bot] commented on PR #415:
URL: https://github.com/apache/maven-indexer/pull/415#issuecomment-2918234084

   Superseded by #419.


-- 
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

[PR] Bump com.google.code.gson:gson from 2.11.0 to 2.13.1 [maven-indexer]

[via GitHub]

dependabot[bot] opened a new pull request, #420:
URL: https://github.com/apache/maven-indexer/pull/420

   Bumps [com.google.code.gson:gson](https://github.com/google/gson) from 
2.11.0 to 2.13.1.
   
   Release notes
   Sourced from https://github.com/google/gson/releases";>com.google.code.gson:gson's 
releases.
   
   Gson 2.13.1
   What's Changed
   
   Give FieldNamingStrategy the ability to return multiple String names by 
https://github.com/mfriesen";>@​mfriesen in https://redirect.github.com/google/gson/pull/2776";>google/gson#2776
   Remove outdated android-proguard-example by https://github.com/Goooler";>@​Goooler in https://redirect.github.com/google/gson/pull/2843";>google/gson#2843
   Adjust Troubleshooting Guide ProGuard / R8 section by https://github.com/Marcono1234";>@​Marcono1234 in https://redirect.github.com/google/gson/pull/2844";>google/gson#2844
   Update dependencies, including the problematic 
com.google.errorprone:error_prone_annotations:2.37.0.
   
   New Contributors
   
   https://github.com/mfriesen";>@​mfriesen made 
their first contribution in https://redirect.github.com/google/gson/pull/2776";>google/gson#2776
   https://github.com/Goooler";>@​Goooler made 
their first contribution in https://redirect.github.com/google/gson/pull/2843";>google/gson#2843
   
   Full Changelog: https://github.com/google/gson/compare/gson-parent-2.13.0...gson-parent-2.13.1";>https://github.com/google/gson/compare/gson-parent-2.13.0...gson-parent-2.13.1
   Gson 2.13.0
   What's Changed
   
   
   A bug in deserializing collections has been fixed. Previously, if you did 
something like this:
   gson.fromJson(jsonString, new 
TypeToken>() {})
   
   then the inferred type would be ImmutableList, 
but Gson actually gave you an ArrayList. Usually 
that would lead to an immediate ClassCastException, but in some 
circumstances the code might sometimes succeed despite the wrong type. Now you 
will see an exception like this:
   com.google.gson.JsonIOException: Abstract classes can't be 
instantiated!
   Adjust the R8 configuration or register an InstanceCreator or a TypeAdapter 
for this type.
   Class name: com.google.common.collect.ImmutableList
   
   because Gson now really is trying to create an ImmutableList 
through its constructor, but that isn't possible.
   Either change the requested type (in the TypeToken) to 
List, or register a TypeAdapter or 
JsonDeserializer for ImmutableList.
   
   
   The internal classes $Gson$Types and 
$Gson$Preconditions have been renamed to remove the $ 
characters. Since these are internal classes (as signaled not only by the 
package name but by the $ characters), client code should not be 
affected. If your code was depending on these classes then we suggest 
making a copy of the class (subject to the https://www.apache.org/licenses/LICENSE-2.0";>license) rather than 
depending on the new names.
   
   
   Full Changelog: https://github.com/google/gson/compare/gson-parent-2.12.1...gson-parent-2.13.0";>https://github.com/google/gson/compare/gson-parent-2.12.1...gson-parent-2.13.0
   Gson 2.12.1
   The only difference between this release and 2.12.0 is that OSGi 
declarations in the Gson jar now specify that 
com.google.errorprone.annotations  is an optional dependency, not 
a required one. If you do not use OSGi then there is no effective change.
   Gson 2.12.0
   What's Changed
   The biggest change is that we no longer support Java 7. People who still 
need to run on Java 7 will need to use an earlier version of Gson.
   Other changes:
   
   Allow registering adapters for JsonElement again by https://github.com/Marcono1234";>@​Marcono1234 in https://redirect.github.com/google/gson/pull/2789";>google/gson#2789
   Add nesting limit for JsonReader by https://github.com/Marcono1234";>@​Marcono1234 in https://redirect.github.com/google/gson/pull/2588";>google/gson#2588
   Add @CheckReturnValue to our packages. by https://github.com/cpovirk";>@​cpovirk in https://redirect.github.com/google/gson/pull/2693";>google/gson#2693
   Add NullSafeTypeAdapter to prevent TypeAdapter.nullSafe() from returning 
nested null-safe type adapters (https://redirect.github.com/google/gson/issues/2729";>#2729) by https://github.com/lyubomyr-shaydariv";>@​lyubomyr-shaydariv
 in https://redirect.github.com/google/gson/pull/2731";>google/gson#2731
   Support Properties subclasses in GsonTypes.getMapKeyAndValueTypes by https://github.com/panic08";>@​panic08 in https://redirect.github.com/google/gson/pull/2758";>google/gson#2758
   Enforce rawType to be a Class in ParameterizedTypeImpl by https://github.com/panic08";>@​panic08 in https://redirect.github.com/google/gson/pull/2759";>google/gson#2759
   Remove AccessController usage for enum adapter by https://github.com/Marcono1234";>@​Marcono1234 in https://redirect.github.com/google/gson/pull/2704";>google/gson#2704
   Fix typeArguments array not being cloned when resolving 
ParameterizedType 

[PR] Bump org.apache.maven.archetype:archetype-catalog from 3.2.1 to 3.4.0 [maven-indexer]

[via GitHub]

dependabot[bot] opened a new pull request, #421:
URL: https://github.com/apache/maven-indexer/pull/421

   Bumps org.apache.maven.archetype:archetype-catalog from 3.2.1 to 3.4.0.
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.maven.archetype:archetype-catalog&package-manager=maven&previous-version=3.2.1&new-version=3.4.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of 
the ignore conditions of the specified dependency
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   
   
   


-- 
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

Re: [PR] Bump com.google.code.gson:gson from 2.11.0 to 2.12.1 [maven-indexer]

[via GitHub]

dependabot[bot] commented on PR #409:
URL: https://github.com/apache/maven-indexer/pull/409#issuecomment-2918235357

   Superseded by #420.


-- 
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

[PR] Bump com.google.guava:guava from 33.3.0-jre to 33.4.8-jre [maven-indexer]

[via GitHub]

dependabot[bot] opened a new pull request, #419:
URL: https://github.com/apache/maven-indexer/pull/419

   Bumps [com.google.guava:guava](https://github.com/google/guava) from 
33.3.0-jre to 33.4.8-jre.
   
   Release notes
   Sourced from https://github.com/google/guava/releases";>com.google.guava:guava's 
releases.
   
   33.4.8
   Guava 33.4.8 fixes a problem that we introduced while starting to migrate 
guava-android off Unsafe in https://github.com/google/guava/releases/tag/v33.4.7";>33.4.7.
   Even if you're not upgrading from Guava 33.4.0 or earlier, still read https://github.com/google/guava/releases/tag/v33.4.1";>the release notes 
for Guava 33.4.1. Those release notes contain information about the effects 
of Guava 33.4.5 and higher on the module system.
   Maven
   
 com.google.guava
 guava
 33.4.8-jre
 
 33.4.8-android
   
   
   Jar files
   
   https://repo1.maven.org/maven2/com/google/guava/guava/33.4.8-jre/guava-33.4.8-jre.jar";>33.4.8-jre.jar
   https://repo1.maven.org/maven2/com/google/guava/guava/33.4.8-android/guava-33.4.8-android.jar";>33.4.8-android.jar
   
   Guava requires https://github.com/google/guava/wiki/UseGuavaInYourBuild#what-about-guavas-own-dependencies";>one
 runtime dependency, which you can download here:
   
   https://repo1.maven.org/maven2/com/google/guava/failureaccess/1.0.3/failureaccess-1.0.3.jar";>failureaccess-1.0.3.jar
   
   Javadoc
   
   https://guava.dev/releases/33.4.8-jre/api/docs/";>33.4.8-jre
   https://guava.dev/releases/33.4.8-android/api/docs/";>33.4.8-android
   
   JDiff
   
   https://guava.dev/releases/33.4.8-jre/api/diffs/";>33.4.8-jre 
vs. 33.4.7-jre
   https://guava.dev/releases/33.4.8-android/api/diffs/";>33.4.8-android vs. 
33.4.7-android
   https://guava.dev/releases/33.4.8-android/api/androiddiffs/";>33.4.8-android
 vs. 33.4.8-jre
   
   Changelog
   
   util.concurrent: Removed our VarHandle code 
from guava-android. While the code was never used at runtime under 
Android, it was causing https://redirect.github.com/google/guava/issues/7769";>problems under the 
Android Gradle Plugin with a minSdkVersion below 26. To 
continue to avoid sun.misc.Unsafe under the JVM, 
guava-android will now always use 
AtomicReferenceFieldUpdater when run there. (75da92419a)
   
   33.4.7
   Prefer to upgrade straight to https://github.com/google/guava/releases/tag/v33.4.8";>33.4.8:
 33.4.7 https://redirect.github.com/google/guava/issues/7769";>breaks 
the build of Android apps with a minSdkVersion below 26. We will publish a 
fixed version soon. This problem is fixed in 33.4.8.
   Guava 33.4.7, like https://github.com/google/guava/releases/tag/v33.4.6";>33.4.6, fixes 
two problems that we introduced while modularizing Guava and migrating off 
Unsafe in https://github.com/google/guava/releases/tag/v33.4.5";>33.4.5.
   Even if you're not upgrading from Guava 33.4.0 or earlier, still read https://github.com/google/guava/releases/tag/v33.4.1";>the release notes 
for Guava 33.4.1. Those release notes contain information about the effects 
of Guava 33.4.5 and higher on the module system.
   Maven
   
   
   ... (truncated)
   
   
   Commits
   
   See full diff in https://github.com/google/guava/commits";>compare view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.google.guava:guava&package-manager=maven&previous-version=33.3.0-jre&new-version=33.4.8-jre)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of 
the ignore conditions of the specified dependency
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or