Re: [PR] [MPH-183] Effective POM path to source [maven-help-plugin]

2024-04-12 Thread via GitHub


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562100440


##
src/main/java/org/apache/maven/plugins/help/DefaultInputLocationFormatter.java:
##
@@ -0,0 +1,42 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Maven 3.x-based implementation of {@link InputLocation.StringFormatter}.

Review Comment:
   Agreed, we should probably change the name of the class and edit the JDocs. 
We would like to backport it to Maven 3, but initially want to make it work for 
Maven 4 first without breaking other versions.



-- 
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] (MPH-183) Effective-pom + verbose should show import path to BOM dependencyManagement

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPH-183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836433#comment-17836433
 ] 

ASF GitHub Bot commented on MPH-183:


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562100440


##
src/main/java/org/apache/maven/plugins/help/DefaultInputLocationFormatter.java:
##
@@ -0,0 +1,42 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Maven 3.x-based implementation of {@link InputLocation.StringFormatter}.

Review Comment:
   Agreed, we should probably change the name of the class and edit the JDocs. 
We would like to backport it to Maven 3, but initially want to make it work for 
Maven 4 first without breaking other versions.





> Effective-pom + verbose should show import path to BOM dependencyManagement
> ---
>
> Key: MPH-183
> URL: https://issues.apache.org/jira/browse/MPH-183
> Project: Maven Help Plugin
>  Issue Type: Improvement
>Affects Versions: 3.2.0
>Reporter: Robert Scholte
>Assignee: Maarten Mulders
>Priority: Major
> Attachments: mph-183-it.zip
>
>
> The popular spring-boot makes a lot of use of BOMs. Using BOMs is a good 
> practice, but right now it is very hard to determine where 
> dependencyManagement dependencies and especially their versions are coming 
> from.
> Instead of only showing only the final location (from the BOM POM), it should 
> also show the import path from the current project to that specific pom 
> (where is the BOM POM imported?).
> This way it will be easier to figure out which dependency in which POM needs 
> to be upgraded: it's the version in the POM declaring the import of the BOM 
> POM, not the version in the imported BOM POM.



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


[jira] [Assigned] (MPLUGIN-517) GoalRenderer renderParameterDetails() renders in wrong order

2024-04-12 Thread Michael Osipov (Jira)


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

Michael Osipov reassigned MPLUGIN-517:
--

Assignee: Michael Osipov

> GoalRenderer renderParameterDetails() renders in wrong order
> 
>
> Key: MPLUGIN-517
> URL: https://issues.apache.org/jira/browse/MPLUGIN-517
> Project: Maven Plugin Tools
>  Issue Type: Bug
>  Components: Plugin Reporting Plugin
>Affects Versions: 3.11.0, 3.12.0
>Reporter: Max Philipp Wriedt
>Assignee: Michael Osipov
>Priority: Major
>
> GoalRenderer.java:360
> {code:java}
> sink.anchor(parameter.getName());
> startSection(format("parameter.name", parameter.getName()));
> sink.anchor_();
> {code}
> The closing {{anchor_();}} should be before starting the section.



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


Re: [PR] [MRESOLVER-518] Version selector improvements [maven-resolver]

2024-04-12 Thread via GitHub


cstamas merged PR #450:
URL: https://github.com/apache/maven-resolver/pull/450


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

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

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



[jira] [Closed] (MRESOLVER-518) Improvements for version selector

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak closed MRESOLVER-518.
-
Resolution: Fixed

> Improvements for version selector
> -
>
> Key: MRESOLVER-518
> URL: https://issues.apache.org/jira/browse/MRESOLVER-518
> Project: Maven Resolver
>  Issue Type: New Feature
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 2.0.0, 2.0.0-alpha-11
>
>
> Currently Resolver "silently" resolves conflict based on "nearest" strategy, 
> without any assumption about selected version (simply the fact it is "nearer 
> to root" makes it win).
> Extend this logic by ability to fail collection in case of version conflict, 
> version divergence and "version incompatibility" for start. As it may come 
> handy, and user may want to fix the build differently than Maven would.



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


[jira] [Updated] (MRESOLVER-535) DependenctGraphDumper should be configurable

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak updated MRESOLVER-535:
--
Summary: DependenctGraphDumper should be configurable  (was: 
DependenctGraphDumper decorator)

> DependenctGraphDumper should be configurable
> 
>
> Key: MRESOLVER-535
> URL: https://issues.apache.org/jira/browse/MRESOLVER-535
> Project: Maven Resolver
>  Issue Type: Improvement
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 2.0.0, 2.0.0-alpha-11
>
>
> Allow passing in custom decorator to pull decoration from.



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


[jira] [Commented] (MRESOLVER-518) Improvements for version selector

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MRESOLVER-518:
--

cstamas merged PR #450:
URL: https://github.com/apache/maven-resolver/pull/450




> Improvements for version selector
> -
>
> Key: MRESOLVER-518
> URL: https://issues.apache.org/jira/browse/MRESOLVER-518
> Project: Maven Resolver
>  Issue Type: New Feature
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 2.0.0, 2.0.0-alpha-11
>
>
> Currently Resolver "silently" resolves conflict based on "nearest" strategy, 
> without any assumption about selected version (simply the fact it is "nearer 
> to root" makes it win).
> Extend this logic by ability to fail collection in case of version conflict, 
> version divergence and "version incompatibility" for start. As it may come 
> handy, and user may want to fix the build differently than Maven would.



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


[jira] [Commented] (MPLUGIN-517) GoalRenderer renderParameterDetails() renders in wrong order

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPLUGIN-517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836437#comment-17836437
 ] 

ASF GitHub Bot commented on MPLUGIN-517:


michael-o opened a new pull request, #277:
URL: https://github.com/apache/maven-plugin-tools/pull/277

   …order




> GoalRenderer renderParameterDetails() renders in wrong order
> 
>
> Key: MPLUGIN-517
> URL: https://issues.apache.org/jira/browse/MPLUGIN-517
> Project: Maven Plugin Tools
>  Issue Type: Bug
>  Components: Plugin Reporting Plugin
>Affects Versions: 3.11.0, 3.12.0
>Reporter: Max Philipp Wriedt
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.1
>
>
> GoalRenderer.java:360
> {code:java}
> sink.anchor(parameter.getName());
> startSection(format("parameter.name", parameter.getName()));
> sink.anchor_();
> {code}
> The closing {{anchor_();}} should be before starting the section.



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


[jira] [Commented] (MPH-183) Effective-pom + verbose should show import path to BOM dependencyManagement

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPH-183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836443#comment-17836443
 ] 

ASF GitHub Bot commented on MPH-183:


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562151626


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {
+private final Method getImportedFromMethod;
+private final MavenProject project;
+
+public ImportedFromLocationFormatter(final Method getImportedFromMethod, 
final MavenProject project) {
+this.getImportedFromMethod = getImportedFromMethod;
+this.project = project;
+}
+
+@Override
+public String toString(InputLocation location) {
+InputSource source = location.getSource();
+
+String s = source.getModelId(); // by default, display modelId
+
+if (StringUtils.isBlank(s) || s.contains("[unknown-version]")) {
+// unless it is blank or does not provide version information
+s = source.toString();
+}
+
+InputLocation importedFrom = getImportedFrom(location);
+
+StringBuilder p = new StringBuilder();
+
+while (importedFrom != null
+&& 
!source.toString().equals(importedFrom.getSource().toString())) {
+p.append(" from ").append(importedFrom.getSource().getModelId());

Review Comment:
   If I understand your comment correctly, we print the line in the return 
statement (line 67).
   If we have a project like:
   ```txt
   bom1 -> dependency x (line 12)
 ^
 |
   bom2 -> import bom1
 ^
 |
   project -> import bom2
   ```
   
   The output of the effective pom of this project's dependency shows `bom1, 
line 12 from bom2`





> Effective-pom + verbose should show import path to BOM dependencyManagement
> ---
>
> Key: MPH-183
> URL: https://issues.apache.org/jira/browse/MPH-183
> Project: Maven Help Plugin
>  Issue Type: Improvement
>Affects Versions: 3.2.0
>Reporter: Robert Scholte
>Assignee: Maarten Mulders
>Priority: Major
> Attachments: mph-183-it.zip
>
>
> The popular spring-boot makes a lot of use of BOMs. Using BOMs is a good 
> practice, but right now it is very hard to determine where 
> dependencyManagement dependencies and especially their versions are coming 
> from.
> Instead of only showing only the final location (from the BOM POM), it should 
> also show the import path from the current project to that specific pom 
> (where is the BOM POM imported?).
> This way it will be easier to figure out which dependency in which POM needs 
> to be upgraded: it's the version in the POM declaring the import of the BOM 
> POM, not the version in the imported BOM POM.



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


[jira] [Commented] (MPH-183) Effective-pom + verbose should show import path to BOM dependencyManagement

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPH-183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836447#comment-17836447
 ] 

ASF GitHub Bot commented on MPH-183:


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562157766


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {

Review Comment:
   We thought it would make sense to separate the actual display logic from the 
logic on whether or not to use the correct formatter. We could change that but 
it would bundle more logic in to 1 bigger class?





> Effective-pom + verbose should show import path to BOM dependencyManagement
> ---
>
> Key: MPH-183
> URL: https://issues.apache.org/jira/browse/MPH-183
> Project: Maven Help Plugin
>  Issue Type: Improvement
>Affects Versions: 3.2.0
>Reporter: Robert Scholte
>Assignee: Maarten Mulders
>Priority: Major
> Attachments: mph-183-it.zip
>
>
> The popular spring-boot makes a lot of use of BOMs. Using BOMs is a good 
> practice, but right now it is very hard to determine where 
> dependencyManagement dependencies and especially their versions are coming 
> from.
> Instead of only showing only the final location (from the BOM POM), it should 
> also show the import path from the current project to that specific pom 
> (where is the BOM POM imported?).
> This way it will be easier to figure out which dependency in which POM needs 
> to be upgraded: it's the version in the POM declaring the import of the BOM 
> POM, not the version in the imported BOM POM.



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


Re: [PR] [MPH-183] Effective POM path to source [maven-help-plugin]

2024-04-12 Thread via GitHub


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562157766


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {

Review Comment:
   We thought it would make sense to separate the actual display logic from the 
logic on whether or not to use the correct formatter. We could change that but 
it would bundle more logic in to 1 bigger class?



-- 
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] [MNG-7344] track dependencyManagement import location in effective Model for MPH-183 [maven]

2024-04-12 Thread via GitHub


Giovds commented on code in PR #603:
URL: https://github.com/apache/maven/pull/603#discussion_r1562165582


##
api/maven-api-model/src/main/java/org/apache/maven/api/model/InputSource.java:
##
@@ -64,6 +74,14 @@ public String getModelId() {
 return this.modelId;
 }
 
+public InputLocation getImportedFrom() {
+return importedFrom;
+}
+
+public InputSource importedFrom(InputLocation importedFrom) {
+return new InputSource(modelId, location, importedFrom);

Review Comment:
   This is due to the immutable model, but I think we can simply remove this 
and make the new constructor public. We don't seem to use it (anymore) upon 
further inspection...



-- 
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-7344) Effective pom should contain more finegrained details regarding its content origin: track dependencyManagement import

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-7344:
-

Giovds commented on code in PR #603:
URL: https://github.com/apache/maven/pull/603#discussion_r1562165582


##
api/maven-api-model/src/main/java/org/apache/maven/api/model/InputSource.java:
##
@@ -64,6 +74,14 @@ public String getModelId() {
 return this.modelId;
 }
 
+public InputLocation getImportedFrom() {
+return importedFrom;
+}
+
+public InputSource importedFrom(InputLocation importedFrom) {
+return new InputSource(modelId, location, importedFrom);

Review Comment:
   This is due to the immutable model, but I think we can simply remove this 
and make the new constructor public. We don't seem to use it (anymore) upon 
further inspection...





> Effective pom should contain more finegrained details regarding its content 
> origin: track dependencyManagement import
> -
>
> Key: MNG-7344
> URL: https://issues.apache.org/jira/browse/MNG-7344
> Project: Maven
>  Issue Type: Improvement
>  Components: Inheritance and Interpolation, POM
>Reporter: Robert Scholte
>Priority: Major
> Fix For: 3.9.x-candidate, 4.0.x-candidate
>
> Attachments: MicrosoftTeams-image.png
>
>
> To support MPH-183 some changes needs to be done in Maven Core.
> For every element that is not part of the raw model, it must be possible to 
> get the "resolution path" to that element. 
> Until now, only the usual pure inheritance is tracked though InputLocation, 
> as done in MNG-1803, later displayed by verbose help effective-pom with 
> MPH-160
> The following are known to add elements to the effective pom:
> - BOMs dependencyManagement import
> - [Tiles Maven Plugin|https://github.com/repaint-io/maven-tiles] by [~talios]
> Without this feature, it is very hard to detect where these extra elements 
> are coming from.



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


[jira] [Commented] (MPLUGIN-517) GoalRenderer renderParameterDetails() renders in wrong order

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPLUGIN-517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836441#comment-17836441
 ] 

ASF GitHub Bot commented on MPLUGIN-517:


asfgit closed pull request #277: [MPLUGIN-517] GoalRenderer 
renderParameterDetails() renders in wrong …
URL: https://github.com/apache/maven-plugin-tools/pull/277




> GoalRenderer renderParameterDetails() renders in wrong order
> 
>
> Key: MPLUGIN-517
> URL: https://issues.apache.org/jira/browse/MPLUGIN-517
> Project: Maven Plugin Tools
>  Issue Type: Bug
>  Components: Plugin Reporting Plugin
>Affects Versions: 3.11.0, 3.12.0
>Reporter: Max Philipp Wriedt
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.1
>
>
> GoalRenderer.java:360
> {code:java}
> sink.anchor(parameter.getName());
> startSection(format("parameter.name", parameter.getName()));
> sink.anchor_();
> {code}
> The closing {{anchor_();}} should be before starting the section.



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


[jira] [Closed] (MPLUGIN-517) GoalRenderer renderParameterDetails() renders in wrong order

2024-04-12 Thread Michael Osipov (Jira)


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

Michael Osipov closed MPLUGIN-517.
--
Resolution: Fixed

Fixed with 
[e746d9cc35cb03c03c61101d2b11f8247e6dcc82|https://gitbox.apache.org/repos/asf?p=maven-plugin-tools.git=commit=e746d9cc35cb03c03c61101d2b11f8247e6dcc82].

> GoalRenderer renderParameterDetails() renders in wrong order
> 
>
> Key: MPLUGIN-517
> URL: https://issues.apache.org/jira/browse/MPLUGIN-517
> Project: Maven Plugin Tools
>  Issue Type: Bug
>  Components: Plugin Reporting Plugin
>Affects Versions: 3.11.0, 3.12.0
>Reporter: Max Philipp Wriedt
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.1
>
>
> GoalRenderer.java:360
> {code:java}
> sink.anchor(parameter.getName());
> startSection(format("parameter.name", parameter.getName()));
> sink.anchor_();
> {code}
> The closing {{anchor_();}} should be before starting the section.



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


Re: [PR] [MPLUGIN-517] GoalRenderer renderParameterDetails() renders in wrong … [maven-plugin-tools]

2024-04-12 Thread via GitHub


asfgit closed pull request #277: [MPLUGIN-517] GoalRenderer 
renderParameterDetails() renders in wrong …
URL: https://github.com/apache/maven-plugin-tools/pull/277


-- 
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] [MPH-183] Effective POM path to source [maven-help-plugin]

2024-04-12 Thread via GitHub


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562151626


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {
+private final Method getImportedFromMethod;
+private final MavenProject project;
+
+public ImportedFromLocationFormatter(final Method getImportedFromMethod, 
final MavenProject project) {
+this.getImportedFromMethod = getImportedFromMethod;
+this.project = project;
+}
+
+@Override
+public String toString(InputLocation location) {
+InputSource source = location.getSource();
+
+String s = source.getModelId(); // by default, display modelId
+
+if (StringUtils.isBlank(s) || s.contains("[unknown-version]")) {
+// unless it is blank or does not provide version information
+s = source.toString();
+}
+
+InputLocation importedFrom = getImportedFrom(location);
+
+StringBuilder p = new StringBuilder();
+
+while (importedFrom != null
+&& 
!source.toString().equals(importedFrom.getSource().toString())) {
+p.append(" from ").append(importedFrom.getSource().getModelId());

Review Comment:
   If I understand your comment correctly, we print the line in the return 
statement (line 67).
   If we have a project like:
   ```txt
   bom1 -> dependency x (line 12)
 ^
 |
   bom2 -> import bom1
 ^
 |
   project -> import bom2
   ```
   
   The output of the effective pom of this project's dependency shows `bom1, 
line 12 from bom2`



-- 
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] [MPOM-472] Bump org.apache.maven.plugins:maven-gpg-plugin from 3.2.2 to 3.2.3 [maven-apache-parent]

2024-04-12 Thread via GitHub


slawekjaranowski merged PR #208:
URL: https://github.com/apache/maven-apache-parent/pull/208


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

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

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



[jira] [Updated] (MPOM-472) Bump org.apache.maven.plugins:maven-gpg-plugin from 3.1.0 to 3.2.3

2024-04-12 Thread Slawomir Jaranowski (Jira)


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

Slawomir Jaranowski updated MPOM-472:
-
Summary: Bump org.apache.maven.plugins:maven-gpg-plugin from 3.1.0 to 3.2.3 
 (was: Bump org.apache.maven.plugins:maven-gpg-plugin from 3.1.0 to 3.2.2)

> Bump org.apache.maven.plugins:maven-gpg-plugin from 3.1.0 to 3.2.3
> --
>
> Key: MPOM-472
> URL: https://issues.apache.org/jira/browse/MPOM-472
> Project: Maven POMs
>  Issue Type: Dependency upgrade
>  Components: asf
>Reporter: Slawomir Jaranowski
>Assignee: Slawomir Jaranowski
>Priority: Major
> Fix For: ASF-32
>
>




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


[jira] [Updated] (MPLUGIN-517) GoalRenderer renderParameterDetails() renders in wrong order

2024-04-12 Thread Michael Osipov (Jira)


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

Michael Osipov updated MPLUGIN-517:
---
Fix Version/s: 3.12.1

> GoalRenderer renderParameterDetails() renders in wrong order
> 
>
> Key: MPLUGIN-517
> URL: https://issues.apache.org/jira/browse/MPLUGIN-517
> Project: Maven Plugin Tools
>  Issue Type: Bug
>  Components: Plugin Reporting Plugin
>Affects Versions: 3.11.0, 3.12.0
>Reporter: Max Philipp Wriedt
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.1
>
>
> GoalRenderer.java:360
> {code:java}
> sink.anchor(parameter.getName());
> startSection(format("parameter.name", parameter.getName()));
> sink.anchor_();
> {code}
> The closing {{anchor_();}} should be before starting the section.



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


Re: [PR] [MPH-183] Effective POM path to source [maven-help-plugin]

2024-04-12 Thread via GitHub


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562132972


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {
+private final Method getImportedFromMethod;
+private final MavenProject project;
+
+public ImportedFromLocationFormatter(final Method getImportedFromMethod, 
final MavenProject project) {
+this.getImportedFromMethod = getImportedFromMethod;
+this.project = project;
+}
+
+@Override
+public String toString(InputLocation location) {
+InputSource source = location.getSource();
+
+String s = source.getModelId(); // by default, display modelId
+
+if (StringUtils.isBlank(s) || s.contains("[unknown-version]")) {
+// unless it is blank or does not provide version information
+s = source.toString();
+}
+
+InputLocation importedFrom = getImportedFrom(location);
+
+StringBuilder p = new StringBuilder();
+
+while (importedFrom != null
+&& 
!source.toString().equals(importedFrom.getSource().toString())) {
+p.append(" from ").append(importedFrom.getSource().getModelId());
+importedFrom = getImportedFrom(importedFrom);
+}
+
+return '}' + s + ((location.getLineNumber() >= 0) ? ", line " + 
location.getLineNumber() : "") + p;
+}
+
+protected InputLocation getImportedFrom(final InputLocation location) {
+try {
+InputLocation result = (InputLocation) 
getImportedFromMethod.invoke(location);
+
+if (result == null && project != null) {

Review Comment:
   When there is an importedFrom on the current dependency we can simply return 
the result, if there is not an importedFrom on the current dependency we try to 
find a match in the dependencies of dependencyManagement



-- 
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] (MPH-183) Effective-pom + verbose should show import path to BOM dependencyManagement

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPH-183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836438#comment-17836438
 ] 

ASF GitHub Bot commented on MPH-183:


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562132972


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {
+private final Method getImportedFromMethod;
+private final MavenProject project;
+
+public ImportedFromLocationFormatter(final Method getImportedFromMethod, 
final MavenProject project) {
+this.getImportedFromMethod = getImportedFromMethod;
+this.project = project;
+}
+
+@Override
+public String toString(InputLocation location) {
+InputSource source = location.getSource();
+
+String s = source.getModelId(); // by default, display modelId
+
+if (StringUtils.isBlank(s) || s.contains("[unknown-version]")) {
+// unless it is blank or does not provide version information
+s = source.toString();
+}
+
+InputLocation importedFrom = getImportedFrom(location);
+
+StringBuilder p = new StringBuilder();
+
+while (importedFrom != null
+&& 
!source.toString().equals(importedFrom.getSource().toString())) {
+p.append(" from ").append(importedFrom.getSource().getModelId());
+importedFrom = getImportedFrom(importedFrom);
+}
+
+return '}' + s + ((location.getLineNumber() >= 0) ? ", line " + 
location.getLineNumber() : "") + p;
+}
+
+protected InputLocation getImportedFrom(final InputLocation location) {
+try {
+InputLocation result = (InputLocation) 
getImportedFromMethod.invoke(location);
+
+if (result == null && project != null) {

Review Comment:
   When there is an importedFrom on the current dependency we can simply return 
the result, if there is not an importedFrom on the current dependency we try to 
find a match in the dependencies of dependencyManagement





> Effective-pom + verbose should show import path to BOM dependencyManagement
> ---
>
> Key: MPH-183
> URL: https://issues.apache.org/jira/browse/MPH-183
> Project: Maven Help Plugin
>  Issue Type: Improvement
>Affects Versions: 3.2.0
>Reporter: Robert Scholte
>Assignee: Maarten Mulders
>Priority: Major
> Attachments: mph-183-it.zip
>
>
> The popular spring-boot makes a lot of use of BOMs. Using BOMs is a good 
> practice, but right now it is very hard to determine where 
> dependencyManagement dependencies and especially their versions are coming 
> from.
> Instead of only showing only the final location (from the BOM POM), it should 
> also show the import path from the current project to that specific pom 
> (where is the BOM POM imported?).
> This way it will be easier to figure out which dependency in which POM needs 
> to be upgraded: it's the version in the POM declaring the import of the BOM 
> POM, not the version in the imported BOM POM.



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


Re: [PR] [MPH-183] Effective POM path to source [maven-help-plugin]

2024-04-12 Thread via GitHub


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562155117


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {
+private final Method getImportedFromMethod;
+private final MavenProject project;
+
+public ImportedFromLocationFormatter(final Method getImportedFromMethod, 
final MavenProject project) {
+this.getImportedFromMethod = getImportedFromMethod;
+this.project = project;
+}
+
+@Override
+public String toString(InputLocation location) {
+InputSource source = location.getSource();
+
+String s = source.getModelId(); // by default, display modelId
+
+if (StringUtils.isBlank(s) || s.contains("[unknown-version]")) {
+// unless it is blank or does not provide version information
+s = source.toString();
+}
+
+InputLocation importedFrom = getImportedFrom(location);
+
+StringBuilder p = new StringBuilder();
+
+while (importedFrom != null
+&& 
!source.toString().equals(importedFrom.getSource().toString())) {
+p.append(" from ").append(importedFrom.getSource().getModelId());
+importedFrom = getImportedFrom(importedFrom);
+}
+
+return '}' + s + ((location.getLineNumber() >= 0) ? ", line " + 
location.getLineNumber() : "") + p;
+}
+
+protected InputLocation getImportedFrom(final InputLocation location) {
+try {
+InputLocation result = (InputLocation) 
getImportedFromMethod.invoke(location);
+
+if (result == null && project != null) {
+for (Dependency dependency : 
project.getDependencyManagement().getDependencies()) {

Review Comment:
   I've no experience with the maven tiles plugin. If this results in the same 
model (which I assume it does) it probably would still work, we would have to 
test this though.



-- 
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] (MPH-183) Effective-pom + verbose should show import path to BOM dependencyManagement

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MPH-183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836446#comment-17836446
 ] 

ASF GitHub Bot commented on MPH-183:


Giovds commented on code in PR #37:
URL: https://github.com/apache/maven-help-plugin/pull/37#discussion_r1562155117


##
src/main/java/org/apache/maven/plugins/help/ImportedFromLocationFormatter.java:
##
@@ -0,0 +1,125 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.maven.plugins.help;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.maven.model.Dependency;
+import org.apache.maven.model.InputLocation;
+import org.apache.maven.model.InputSource;
+import org.apache.maven.project.MavenProject;
+import org.codehaus.plexus.util.StringUtils;
+
+/**
+ * Implementation of {@link InputLocation.StringFormatter}. Enhances the 
default implementation with support for
+ * following "references" (caused by e.g. dependency management imports).
+ */
+public class ImportedFromLocationFormatter extends 
InputLocation.StringFormatter {
+private final Method getImportedFromMethod;
+private final MavenProject project;
+
+public ImportedFromLocationFormatter(final Method getImportedFromMethod, 
final MavenProject project) {
+this.getImportedFromMethod = getImportedFromMethod;
+this.project = project;
+}
+
+@Override
+public String toString(InputLocation location) {
+InputSource source = location.getSource();
+
+String s = source.getModelId(); // by default, display modelId
+
+if (StringUtils.isBlank(s) || s.contains("[unknown-version]")) {
+// unless it is blank or does not provide version information
+s = source.toString();
+}
+
+InputLocation importedFrom = getImportedFrom(location);
+
+StringBuilder p = new StringBuilder();
+
+while (importedFrom != null
+&& 
!source.toString().equals(importedFrom.getSource().toString())) {
+p.append(" from ").append(importedFrom.getSource().getModelId());
+importedFrom = getImportedFrom(importedFrom);
+}
+
+return '}' + s + ((location.getLineNumber() >= 0) ? ", line " + 
location.getLineNumber() : "") + p;
+}
+
+protected InputLocation getImportedFrom(final InputLocation location) {
+try {
+InputLocation result = (InputLocation) 
getImportedFromMethod.invoke(location);
+
+if (result == null && project != null) {
+for (Dependency dependency : 
project.getDependencyManagement().getDependencies()) {

Review Comment:
   I've no experience with the maven tiles plugin. If this results in the same 
model (which I assume it does) it probably would still work, we would have to 
test this though.





> Effective-pom + verbose should show import path to BOM dependencyManagement
> ---
>
> Key: MPH-183
> URL: https://issues.apache.org/jira/browse/MPH-183
> Project: Maven Help Plugin
>  Issue Type: Improvement
>Affects Versions: 3.2.0
>Reporter: Robert Scholte
>Assignee: Maarten Mulders
>Priority: Major
> Attachments: mph-183-it.zip
>
>
> The popular spring-boot makes a lot of use of BOMs. Using BOMs is a good 
> practice, but right now it is very hard to determine where 
> dependencyManagement dependencies and especially their versions are coming 
> from.
> Instead of only showing only the final location (from the BOM POM), it should 
> also show the import path from the current project to that specific pom 
> (where is the BOM POM imported?).
> This way it will be easier to figure out which dependency in which POM needs 
> to be upgraded: it's the version in the POM declaring the import of the BOM 
> POM, not the version in the imported BOM POM.



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


[jira] [Commented] (MPLUGIN-517) GoalRenderer renderParameterDetails() renders in wrong order

2024-04-12 Thread Michael Osipov (Jira)


[ 
https://issues.apache.org/jira/browse/MPLUGIN-517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836435#comment-17836435
 ] 

Michael Osipov commented on MPLUGIN-517:


That one slipped through...will fix.

> GoalRenderer renderParameterDetails() renders in wrong order
> 
>
> Key: MPLUGIN-517
> URL: https://issues.apache.org/jira/browse/MPLUGIN-517
> Project: Maven Plugin Tools
>  Issue Type: Bug
>  Components: Plugin Reporting Plugin
>Affects Versions: 3.11.0, 3.12.0
>Reporter: Max Philipp Wriedt
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.1
>
>
> GoalRenderer.java:360
> {code:java}
> sink.anchor(parameter.getName());
> startSection(format("parameter.name", parameter.getName()));
> sink.anchor_();
> {code}
> The closing {{anchor_();}} should be before starting the section.



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


Re: [PR] [MRESOLVER-535] Add decorator ability to graph dumper [maven-resolver]

2024-04-12 Thread via GitHub


cstamas commented on PR #464:
URL: https://github.com/apache/maven-resolver/pull/464#issuecomment-2051242596

   This PR as is is not gonna cut it, it need more. Updated 
https://issues.apache.org/jira/browse/MRESOLVER-535 with some context.


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

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

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



[jira] [Updated] (MRESOLVER-535) DependenctGraphDumper should be configurable

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak updated MRESOLVER-535:
--
Description: 
In a sense, we just accumulate what all it prints out (see class history), but 
this should be in fact somehow configurable.

Maybe refactor all the output into "decorator" (appender)? And the let ctor 
just accept a collection of them (with some predefined collections for some 
defaults)? As then, user/caller is fully in charge what the dumper is writing 
out, plus, is extensible with original "decorator" idea.

Original decorator idea was that I had a lookup table of some "extra info", and 
decorator was basically appending info from that lookup table.

  was:Allow passing in custom decorator to pull decoration from.


> DependenctGraphDumper should be configurable
> 
>
> Key: MRESOLVER-535
> URL: https://issues.apache.org/jira/browse/MRESOLVER-535
> Project: Maven Resolver
>  Issue Type: Improvement
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 2.0.0, 2.0.0-alpha-11
>
>
> In a sense, we just accumulate what all it prints out (see class history), 
> but this should be in fact somehow configurable.
> Maybe refactor all the output into "decorator" (appender)? And the let ctor 
> just accept a collection of them (with some predefined collections for some 
> defaults)? As then, user/caller is fully in charge what the dumper is writing 
> out, plus, is extensible with original "decorator" idea.
> Original decorator idea was that I had a lookup table of some "extra info", 
> and decorator was basically appending info from that lookup table.



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


[jira] [Commented] (MRESOLVER-535) DependenctGraphDumper should be configurable

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MRESOLVER-535:
--

cstamas commented on PR #464:
URL: https://github.com/apache/maven-resolver/pull/464#issuecomment-2051242596

   This PR as is is not gonna cut it, it need more. Updated 
https://issues.apache.org/jira/browse/MRESOLVER-535 with some context.




> DependenctGraphDumper should be configurable
> 
>
> Key: MRESOLVER-535
> URL: https://issues.apache.org/jira/browse/MRESOLVER-535
> Project: Maven Resolver
>  Issue Type: Improvement
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 2.0.0, 2.0.0-alpha-11
>
>
> In a sense, we just accumulate what all it prints out (see class history), 
> but this should be in fact somehow configurable.
> Maybe refactor all the output into "decorator" (appender)? And the let ctor 
> just accept a collection of them (with some predefined collections for some 
> defaults)? As then, user/caller is fully in charge what the dumper is writing 
> out, plus, is extensible with original "decorator" idea.
> Original decorator idea was that I had a lookup table of some "extra info", 
> and decorator was basically appending info from that lookup table.



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


[jira] [Commented] (MGPG-125) Due to default value of gpg.passphraseServerId, bestPractices=true will always fail

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MGPG-125:
-

slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562201582


##
src/it/sign-release-best-practices/verify.groovy:
##
@@ -0,0 +1,38 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+var artifactDir = new File(localRepositoryPath, 
"org/apache/maven/its/gpg/sr/test/1.0")
+
+var expectedFiles = [
+"test-1.0.pom",
+"test-1.0.pom.asc",
+"test-1.0.jar",
+"test-1.0.jar.asc",
+"test-1.0-sources.jar",
+"test-1.0-sources.jar.asc"
+]
+
+for (String expectedFile : expectedFiles) {
+var file = new File(artifactDir, expectedFile)
+
+println "Checking for existence of $file"
+
+if (!file.isFile()) {

Review Comment:
   `assert file.isFile()`





> Due to default value of gpg.passphraseServerId, bestPractices=true will 
> always fail
> ---
>
> Key: MGPG-125
> URL: https://issues.apache.org/jira/browse/MGPG-125
> Project: Maven GPG Plugin
>  Issue Type: Bug
>Affects Versions: 3.2.3
>Reporter: Benjamin Marwell
>Assignee: Tamas Cservenak
>Priority: Critical
> Fix For: 3.2.4
>
>
> Running gpg-plugin with best practices from the CLI, it will always fail 
> since gpg.passphraseServerId does have a default value set.
> See:
> https://github.com/apache/maven-gpg-plugin/blob/master/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java#L140
> CC [~cstamas]



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


Re: [PR] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562201582


##
src/it/sign-release-best-practices/verify.groovy:
##
@@ -0,0 +1,38 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+var artifactDir = new File(localRepositoryPath, 
"org/apache/maven/its/gpg/sr/test/1.0")
+
+var expectedFiles = [
+"test-1.0.pom",
+"test-1.0.pom.asc",
+"test-1.0.jar",
+"test-1.0.jar.asc",
+"test-1.0-sources.jar",
+"test-1.0-sources.jar.asc"
+]
+
+for (String expectedFile : expectedFiles) {
+var file = new File(artifactDir, expectedFile)
+
+println "Checking for existence of $file"
+
+if (!file.isFile()) {

Review Comment:
   `assert file.isFile()`



-- 
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-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-8084:
-

gnodet commented on PR #1461:
URL: https://github.com/apache/maven/pull/1461#issuecomment-2051604730

   ![maven-4 0 
0-alpha-14-SNAPSHOT](https://github.com/apache/maven/assets/84022/021f8313-b34b-4b9b-95e1-ca6697745521)
   
   It's really readable unfortunately, but it started from generated graph, so 
it's correct with current master.




> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-alpha-14
>
>




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


Re: [PR] [MNG-8084] add di & api-impl [maven]

2024-04-12 Thread via GitHub


gnodet commented on PR #1461:
URL: https://github.com/apache/maven/pull/1461#issuecomment-2051624139

   > ![maven-4 0 
0-alpha-14-SNAPSHOT](https://private-user-images.githubusercontent.com/84022/321970671-021f8313-b34b-4b9b-95e1-ca6697745521.jpg?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTI5MjI3MDgsIm5iZiI6MTcxMjkyMjQwOCwicGF0aCI6Ii84NDAyMi8zMjE5NzA2NzEtMDIxZjgzMTMtYjM0Yi00YjliLTk1ZTEtY2E2Njk3NzQ1NTIxLmpwZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA0MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNDEyVDExNDY0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTMxYzQwOGU4OTllNTZjYWNlMTMzMTFlMDc0Zjk2OTAwM2M5Mzc2NzE0NDBmNzRjYzhkMTU0YTg2YmZkMWYwMzYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0._SfX7Kr1noVKChnkU4nTEzkCp5I0vzeNivFk45WT03g)
   > 
   > It's not readable unfortunately, but it started from generated graph, so 
it's correct with current master.
   
   Note that the ultimate goal would be, imho, to have a maven-compat3 with all 
the things from maven 3, and have `maven-core` not depend on the v3 artifacts / 
api anymore.


-- 
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] [MPOM-480] Remove maven-site-plugin:attach-descriptor [maven-apache-parent]

2024-04-12 Thread via GitHub


slawekjaranowski merged PR #207:
URL: https://github.com/apache/maven-apache-parent/pull/207


-- 
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] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562199284


##
src/it/sign-release-best-practices-fail/pom.xml:
##
@@ -0,0 +1,103 @@
+
+
+
+
+http://maven.apache.org/POM/4.0.0; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; 
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xsd/maven-4.0.0.xsd;>
+  4.0.0
+
+  org.apache.maven.its.gpg.sr
+  test
+  1.0
+  jar
+
+  
+Tests the installation of a simple release JAR with an attached artifact 
and its signatures.
+  
+
+  
+true
+  
+
+  
+
+  
+org.apache.maven.plugins
+maven-compiler-plugin
+2.0.2
+  
+  
+org.apache.maven.plugins
+maven-gpg-plugin
+@project.version@
+
+  
+sign-artifacts
+
+  sign
+
+  
+
+
+  true
+  
+  
+  
gpg.passphrase-sign-with-passphase-from-maven-settings
+
+  
+  
+org.apache.maven.plugins
+maven-install-plugin
+2.2

Review 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] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562198479


##
src/it/sign-release-best-practices-fail/pom.xml:
##
@@ -0,0 +1,103 @@
+
+
+
+
+http://maven.apache.org/POM/4.0.0; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; 
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xsd/maven-4.0.0.xsd;>
+  4.0.0
+
+  org.apache.maven.its.gpg.sr
+  test
+  1.0
+  jar
+
+  
+Tests the installation of a simple release JAR with an attached artifact 
and its signatures.
+  
+
+  
+true
+  
+
+  
+
+  
+org.apache.maven.plugins
+maven-compiler-plugin
+2.0.2

Review Comment:
   ?
   we can use `@version.maven-compiler-plugin@` properties for plugins versions 
are defined in parent



-- 
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] (MPOM-481) Improve description of maven-gpg-plugin settings

2024-04-12 Thread Slawomir Jaranowski (Jira)
Slawomir Jaranowski created MPOM-481:


 Summary: Improve description of maven-gpg-plugin settings
 Key: MPOM-481
 URL: https://issues.apache.org/jira/browse/MPOM-481
 Project: Maven POMs
  Issue Type: Improvement
  Components: asf
Reporter: Slawomir Jaranowski
Assignee: Slawomir Jaranowski
 Fix For: ASF-32






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


[jira] [Closed] (MPOM-480) Remove maven-site-plugin:attach-descriptor from ASF parent

2024-04-12 Thread Slawomir Jaranowski (Jira)


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

Slawomir Jaranowski closed MPOM-480.

Resolution: Fixed

> Remove maven-site-plugin:attach-descriptor from ASF parent
> --
>
> Key: MPOM-480
> URL: https://issues.apache.org/jira/browse/MPOM-480
> Project: Maven POMs
>  Issue Type: Improvement
>Reporter: Slawomir Jaranowski
>Assignee: Slawomir Jaranowski
>Priority: Major
> Fix For: ASF-32
>
>
> We have defined {{maven-site-plugin:attach-descriptor}} in {{build/plugins}}
> It cause to *every* ASF projects have execute this goals - but it is only 
> needed for parents pom and only in case when parent want to publish site 
> descriptor.
> In child projects such task is not needed.
> Each ASF project should manage it and add this only in place where is needed, 
> with *inherited* set to {*}false{*}, as example:
> {code:xml}
> 
>   org.apache.maven.plugins
>   maven-site-plugin
>   false
>   
> 
>   attach-descriptor
>   
> attach-descriptor
>   
> 
>   
> 
> {code}
> After removing from ASF parent we can simplify next step of releasing ASF 
> parent - special manually maintained documentation layout.



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


[jira] [Assigned] (MNG-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread Guillaume Nodet (Jira)


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

Guillaume Nodet reassigned MNG-8084:


Assignee: Guillaume Nodet

> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Guillaume Nodet
>Priority: Major
>




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


[jira] [Closed] (MNG-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread Guillaume Nodet (Jira)


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

Guillaume Nodet closed MNG-8084.

Resolution: Fixed

> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-alpha-14
>
>




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


[jira] [Commented] (MNG-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-8084:
-

gnodet merged PR #1457:
URL: https://github.com/apache/maven/pull/1457




> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Priority: Major
>




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


[jira] [Updated] (MNG-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread Guillaume Nodet (Jira)


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

Guillaume Nodet updated MNG-8084:
-
Fix Version/s: 4.0.0-alpha-14

> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-alpha-14
>
>




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


Re: [PR] [MNG-8084] Move ModelBuilder and resolver provider to v4 api [maven]

2024-04-12 Thread via GitHub


gnodet merged PR #1457:
URL: https://github.com/apache/maven/pull/1457


-- 
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] [MNG-8084] add di & api-impl [maven]

2024-04-12 Thread via GitHub


gnodet commented on PR #1461:
URL: https://github.com/apache/maven/pull/1461#issuecomment-2051576594

   > Fwiw, this will be outdated with #1465 and later #1457
   
   Also maven-compat is not used anymore (but only included in the distribution 
for compatibility).


-- 
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-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-8084:
-

gnodet commented on PR #1461:
URL: https://github.com/apache/maven/pull/1461#issuecomment-2051576594

   > Fwiw, this will be outdated with #1465 and later #1457
   
   Also maven-compat is not used anymore (but only included in the distribution 
for compatibility).




> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-alpha-14
>
>




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


[jira] [Commented] (MNG-8084) Make the v4 api usable outside the Maven runtime

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-8084:
-

gnodet commented on PR #1461:
URL: https://github.com/apache/maven/pull/1461#issuecomment-2051624139

   > ![maven-4 0 
0-alpha-14-SNAPSHOT](https://private-user-images.githubusercontent.com/84022/321970671-021f8313-b34b-4b9b-95e1-ca6697745521.jpg?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTI5MjI3MDgsIm5iZiI6MTcxMjkyMjQwOCwicGF0aCI6Ii84NDAyMi8zMjE5NzA2NzEtMDIxZjgzMTMtYjM0Yi00YjliLTk1ZTEtY2E2Njk3NzQ1NTIxLmpwZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA0MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNDEyVDExNDY0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTMxYzQwOGU4OTllNTZjYWNlMTMzMTFlMDc0Zjk2OTAwM2M5Mzc2NzE0NDBmNzRjYzhkMTU0YTg2YmZkMWYwMzYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0._SfX7Kr1noVKChnkU4nTEzkCp5I0vzeNivFk45WT03g)
   > 
   > It's not readable unfortunately, but it started from generated graph, so 
it's correct with current master.
   
   Note that the ultimate goal would be, imho, to have a maven-compat3 with all 
the things from maven 3, and have `maven-core` not depend on the v3 artifacts / 
api anymore.




> Make the v4 api usable outside the Maven runtime
> 
>
> Key: MNG-8084
> URL: https://issues.apache.org/jira/browse/MNG-8084
> Project: Maven
>  Issue Type: New Feature
>Reporter: Guillaume Nodet
>Assignee: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.0-alpha-14
>
>




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


Re: [PR] [MNG-8084] add di & api-impl [maven]

2024-04-12 Thread via GitHub


gnodet commented on PR #1461:
URL: https://github.com/apache/maven/pull/1461#issuecomment-2051604730

   ![maven-4 0 
0-alpha-14-SNAPSHOT](https://github.com/apache/maven/assets/84022/021f8313-b34b-4b9b-95e1-ca6697745521)
   
   It's really readable unfortunately, but it started from generated graph, so 
it's correct with current master.


-- 
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] (MGPG-125) Due to default value of gpg.passphraseServerId, bestPractices=true will always fail

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MGPG-125:
-

slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562198479


##
src/it/sign-release-best-practices-fail/pom.xml:
##
@@ -0,0 +1,103 @@
+
+
+
+
+http://maven.apache.org/POM/4.0.0; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; 
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xsd/maven-4.0.0.xsd;>
+  4.0.0
+
+  org.apache.maven.its.gpg.sr
+  test
+  1.0
+  jar
+
+  
+Tests the installation of a simple release JAR with an attached artifact 
and its signatures.
+  
+
+  
+true
+  
+
+  
+
+  
+org.apache.maven.plugins
+maven-compiler-plugin
+2.0.2

Review Comment:
   ?
   we can use `@version.maven-compiler-plugin@` properties for plugins versions 
are defined in parent





> Due to default value of gpg.passphraseServerId, bestPractices=true will 
> always fail
> ---
>
> Key: MGPG-125
> URL: https://issues.apache.org/jira/browse/MGPG-125
> Project: Maven GPG Plugin
>  Issue Type: Bug
>Affects Versions: 3.2.3
>Reporter: Benjamin Marwell
>Assignee: Tamas Cservenak
>Priority: Critical
> Fix For: 3.2.4
>
>
> Running gpg-plugin with best practices from the CLI, it will always fail 
> since gpg.passphraseServerId does have a default value set.
> See:
> https://github.com/apache/maven-gpg-plugin/blob/master/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java#L140
> CC [~cstamas]



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


[jira] [Commented] (MGPG-125) Due to default value of gpg.passphraseServerId, bestPractices=true will always fail

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MGPG-125:
-

slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562199284


##
src/it/sign-release-best-practices-fail/pom.xml:
##
@@ -0,0 +1,103 @@
+
+
+
+
+http://maven.apache.org/POM/4.0.0; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; 
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/xsd/maven-4.0.0.xsd;>
+  4.0.0
+
+  org.apache.maven.its.gpg.sr
+  test
+  1.0
+  jar
+
+  
+Tests the installation of a simple release JAR with an attached artifact 
and its signatures.
+  
+
+  
+true
+  
+
+  
+
+  
+org.apache.maven.plugins
+maven-compiler-plugin
+2.0.2
+  
+  
+org.apache.maven.plugins
+maven-gpg-plugin
+@project.version@
+
+  
+sign-artifacts
+
+  sign
+
+  
+
+
+  true
+   Due to default value of gpg.passphraseServerId, bestPractices=true will 
> always fail
> ---
>
> Key: MGPG-125
> URL: https://issues.apache.org/jira/browse/MGPG-125
> Project: Maven GPG Plugin
>  Issue Type: Bug
>Affects Versions: 3.2.3
>Reporter: Benjamin Marwell
>Assignee: Tamas Cservenak
>Priority: Critical
> Fix For: 3.2.4
>
>
> Running gpg-plugin with best practices from the CLI, it will always fail 
> since gpg.passphraseServerId does have a default value set.
> See:
> https://github.com/apache/maven-gpg-plugin/blob/master/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java#L140
> CC [~cstamas]



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


Re: [PR] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562205236


##
src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java:
##
@@ -131,13 +131,14 @@ public abstract class AbstractGpgMojo extends 
AbstractMojo {
  * Server id to lookup the passphrase under Maven settings. Do not use 
this parameter, it leaks
  * sensitive data. Passphrase should be provided only via gpg-agent or via 
env variable.
  * If parameter {@link #bestPractices} set to {@code true}, plugin fails 
when this parameter is configured.
+ * Is programatically defaulted to {@link #GPG_PASSPHRASE}.
  *
  * @since 1.6
  * @deprecated Do not use this configuration, it may leak sensitive 
information. Rely on gpg-agent or env
  * variables instead.
  **/
 @Deprecated
-@Parameter(property = "gpg.passphraseServerId", defaultValue = 
GPG_PASSPHRASE)
+@Parameter(property = "gpg.passphraseServerId")

Review Comment:
   I would like to not remove default values  can be assumed as present by 
many configurations



-- 
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] [MNG-7344] track dependencyManagement import location in effective Model for MPH-183 [maven]

2024-04-12 Thread via GitHub


juulhobert commented on code in PR #603:
URL: https://github.com/apache/maven/pull/603#discussion_r1562205753


##
maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java:
##
@@ -1683,13 +1684,20 @@ private void importDependencyManagement(
 
 it.remove();
 
+// Model v3
 DependencyManagement importMgmt = loadDependencyManagement(model, 
request, problems, dependency, importIds);
+if (importMgmt == null) {
+continue;
+}
 
-if (importMgmt != null) {
-if (importMgmts == null) {
-importMgmts = new ArrayList<>();
-}
-
+if (request.isLocationTracking()) {
+// Keep track of why this DependencyManagement was imported.
+// And map model v3 to model v4 -> 
importMgmt(v3).getDelegate() returns a v4 object
+importMgmts.add(
+
org.apache.maven.api.model.DependencyManagement.newBuilder(importMgmt.getDelegate(),
 true)
+
.importedFrom(dependency.getDelegate().getLocation(""))

Review Comment:
   @hboutemy you're right. It should only map it from model v3 to model v4. 
We're going to remove the line 
`.importedFrom(dependency.getDelegate().getLocation(""))`



-- 
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] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


slawekjaranowski commented on code in PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#discussion_r1562205236


##
src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java:
##
@@ -131,13 +131,14 @@ public abstract class AbstractGpgMojo extends 
AbstractMojo {
  * Server id to lookup the passphrase under Maven settings. Do not use 
this parameter, it leaks
  * sensitive data. Passphrase should be provided only via gpg-agent or via 
env variable.
  * If parameter {@link #bestPractices} set to {@code true}, plugin fails 
when this parameter is configured.
+ * Is programatically defaulted to {@link #GPG_PASSPHRASE}.
  *
  * @since 1.6
  * @deprecated Do not use this configuration, it may leak sensitive 
information. Rely on gpg-agent or env
  * variables instead.
  **/
 @Deprecated
-@Parameter(property = "gpg.passphraseServerId", defaultValue = 
GPG_PASSPHRASE)
+@Parameter(property = "gpg.passphraseServerId")

Review Comment:
   I would like to not remove default values  can be assumed as present by 
many configurations



-- 
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-7344) Effective pom should contain more finegrained details regarding its content origin: track dependencyManagement import

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-7344:
-

juulhobert commented on code in PR #603:
URL: https://github.com/apache/maven/pull/603#discussion_r1562205753


##
maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java:
##
@@ -1683,13 +1684,20 @@ private void importDependencyManagement(
 
 it.remove();
 
+// Model v3
 DependencyManagement importMgmt = loadDependencyManagement(model, 
request, problems, dependency, importIds);
+if (importMgmt == null) {
+continue;
+}
 
-if (importMgmt != null) {
-if (importMgmts == null) {
-importMgmts = new ArrayList<>();
-}
-
+if (request.isLocationTracking()) {
+// Keep track of why this DependencyManagement was imported.
+// And map model v3 to model v4 -> 
importMgmt(v3).getDelegate() returns a v4 object
+importMgmts.add(
+
org.apache.maven.api.model.DependencyManagement.newBuilder(importMgmt.getDelegate(),
 true)
+
.importedFrom(dependency.getDelegate().getLocation(""))

Review Comment:
   @hboutemy you're right. It should only map it from model v3 to model v4. 
We're going to remove the line 
`.importedFrom(dependency.getDelegate().getLocation(""))`





> Effective pom should contain more finegrained details regarding its content 
> origin: track dependencyManagement import
> -
>
> Key: MNG-7344
> URL: https://issues.apache.org/jira/browse/MNG-7344
> Project: Maven
>  Issue Type: Improvement
>  Components: Inheritance and Interpolation, POM
>Reporter: Robert Scholte
>Priority: Major
> Fix For: 3.9.x-candidate, 4.0.x-candidate
>
> Attachments: MicrosoftTeams-image.png
>
>
> To support MPH-183 some changes needs to be done in Maven Core.
> For every element that is not part of the raw model, it must be possible to 
> get the "resolution path" to that element. 
> Until now, only the usual pure inheritance is tracked though InputLocation, 
> as done in MNG-1803, later displayed by verbose help effective-pom with 
> MPH-160
> The following are known to add elements to the effective pom:
> - BOMs dependencyManagement import
> - [Tiles Maven Plugin|https://github.com/repaint-io/maven-tiles] by [~talios]
> Without this feature, it is very hard to detect where these extra elements 
> are coming from.



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


[jira] [Updated] (MRESOLVER-535) DependencyGraphDumper should be configurable

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak updated MRESOLVER-535:
--
Summary: DependencyGraphDumper should be configurable  (was: 
DependenctGraphDumper should be configurable)

> DependencyGraphDumper should be configurable
> 
>
> Key: MRESOLVER-535
> URL: https://issues.apache.org/jira/browse/MRESOLVER-535
> Project: Maven Resolver
>  Issue Type: Improvement
>  Components: Resolver
>Reporter: Tamas Cservenak
>Assignee: Tamas Cservenak
>Priority: Major
> Fix For: 2.0.0, 2.0.0-alpha-11
>
>
> In a sense, we just accumulate what all it prints out (see class history), 
> but this should be in fact somehow configurable.
> Maybe refactor all the output into "decorator" (appender)? And the let ctor 
> just accept a collection of them (with some predefined collections for some 
> defaults)? As then, user/caller is fully in charge what the dumper is writing 
> out, plus, is extensible with original "decorator" idea.
> Original decorator idea was that I had a lookup table of some "extra info", 
> and decorator was basically appending info from that lookup table.



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


[jira] [Commented] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MNG-8095:
-

??? I am genuinly confused. Those exist with Maven 4 only. How are they 
supposed to work with 3?

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.0, 3.9.1, 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


Re: [I] TerminalOutput.pathToMaven throws StringIndexOutOfBoundsException [maven-mvnd]

2024-04-12 Thread via GitHub


ppalaga commented on issue #929:
URL: https://github.com/apache/maven-mvnd/issues/929#issuecomment-2051843233

   
[daemon-167cc94d.log](https://github.com/apache/maven-mvnd/files/14959771/daemon-167cc94d.log)
   Attaching the whole log after applying the linked patch in case somebody 
would like to investigate.


-- 
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-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MNG-8095:
-

I see now. I am certain that it is implemented differently than in Maven 4 and 
this is the result now.

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[jira] [Commented] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise commented on MNG-8095:
--

Yes the implementation between Maven 4 and Maven 3.X is different... 

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[jira] [Comment Edited] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise edited comment on MNG-8095 at 4/12/24 2:16 PM:
---

The release notes of Maven 3.9.2 on 
https://maven.apache.org/docs/3.9.2/release-notes.html#notable-new-features 
saying something different..


was (Author: khmarbaise):
The release notes of Maven 3.9.2 on 
https://maven.apache.org/docs/3.9.2/release-notes.html saying something 
different..

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[jira] [Commented] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise commented on MNG-8095:
--

The release notes of Maven 3.9.2 on 
https://maven.apache.org/docs/3.9.2/release-notes.html saying something 
different..

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[jira] [Comment Edited] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak edited comment on MNG-8095 at 4/12/24 3:06 PM:
---

Rel notes says specifically "implemented interpolation for configuration and 
command line".

So, in 3.9.x this is NOT a property, hence cannot be refd from POM/plugin 
config etc, they are interpolated ONLY from command line and 
{{{}.mvn/maven.config{}}}.

In 4.0 this IS fully fledged property, and can be referenced from POM as well.


was (Author: cstamas):
Rel notes says specifically "implemented interpolation for configuration and 
command line".

So, in 3.9.x this is NOT a property (so cannot be refd from POM etc), ONLY from 
command line and .mvn/maven.config.

In 4.0 this IS fully fledged property, and can be referenced from POM as well.

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[jira] [Comment Edited] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak edited comment on MNG-8095 at 4/12/24 3:07 PM:
---

Rel notes says specifically "implemented interpolation for configuration and 
command line".

So, in 3.9.x this is NOT a property, hence cannot be refd from POM/plugin 
config etc, they are interpolated ONLY from command line and 
{{{}.mvn/maven.config{}}}.

In 4.0 this IS fully fledged property, and can be referenced from POM as well.

So Maven 3.9.x and 4.x are _fully aligned_ only for (or "up to") configuration 
and command line, while 4.0 supports these in POM as well. 


was (Author: cstamas):
Rel notes says specifically "implemented interpolation for configuration and 
command line".

So, in 3.9.x this is NOT a property, hence cannot be refd from POM/plugin 
config etc, they are interpolated ONLY from command line and 
{{{}.mvn/maven.config{}}}.

In 4.0 this IS fully fledged property, and can be referenced from POM as well.

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[I] TerminalOutput.pathToMaven throws StringIndexOutOfBoundsException [maven-mvnd]

2024-04-12 Thread via GitHub


ppalaga opened a new issue, #929:
URL: https://github.com/apache/maven-mvnd/issues/929

   Steps to reproduce:
   
   ```
   git clone g...@github.com:quarkiverse/quarkus-langchain4j.git
   cd quarkus-langchain4j
   git reset --hard 177efc97b49823ad747a87766ea1e1d0c2b71ef6
   
   mvnd --stop
   mvnd -v 
   Apache Maven Daemon (mvnd) 1.0-m8 linux-amd64 native client 
(0f4bdb6df5e74453d8d558d292789da4e66a7933)Terminal: 
org.jline.terminal.impl.PosixSysTerminal with pty 
org.jline.terminal.impl.jansi.linux.LinuxNativePty
   Apache Maven 3.9.5 (57804ffe001d7215b5e7bcb531cf83df38f93546)
   Maven home: /home/ppalaga/.sdkman/candidates/mvnd/1.0-m8-m39/mvn
   Java version: 17.0.9, vendor: Eclipse Adoptium, runtime: 
/home/ppalaga/.sdkman/candidates/java/17.0.9-tem
   Default locale: en_IE, platform encoding: UTF-8
   OS name: "linux", version: "6.4.11-200.fc38.x86_64", arch: "amd64", family: 
"unix"
   
   rm -Rf /tmp/empty-local-maven-repo
   mvnd clean install -DskipTests -Dquarkus.build.skip 
-Dmaven.repo.local=/tmp/empty-local-maven-repo
   ...
   Exception in thread "main" java.lang.StringIndexOutOfBoundsException: String 
index out of range: 17
   at 
java.base@17.0.6/java.lang.StringLatin1.charAt(StringLatin1.java:48)
   at java.base@17.0.6/java.lang.String.charAt(String.java:1513)
   at 
org.mvndaemon.mvnd.common.logging.TerminalOutput.pathToMaven(TerminalOutput.java:724)
   at 
org.mvndaemon.mvnd.common.logging.TerminalOutput.formatTransfers(TerminalOutput.java:697)
   at 
org.mvndaemon.mvnd.common.logging.TerminalOutput.addProjectLine(TerminalOutput.java:810)
   at 
org.mvndaemon.mvnd.common.logging.TerminalOutput.update(TerminalOutput.java:615)
   at 
org.mvndaemon.mvnd.common.logging.TerminalOutput.accept(TerminalOutput.java:217)
   at 
org.mvndaemon.mvnd.client.DefaultClient.execute(DefaultClient.java:366)
   at 
org.mvndaemon.mvnd.client.DefaultClient.main(DefaultClient.java:161)
   ```
   
   The daemon log contains the following lines that cause the issue in the 
client:
   
   ```
   15:23:09.559 I Dispatch message: 
TransferInitiated{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-jackson/3.8.2/quarkus-jackson-3.8.2', 
contentLength=-1, transferredBytes=0, exception='null'}
   15:23:09.559 I Dispatch message: 
ProjectLogMessage{projectId='quarkus-langchain4j-parent', 
message='[INFO] Downloading from central: 
https://repo.maven.apache.org/maven2/io/quarkus/quarkus-jackson/3.8.2/quarkus-jackson-3.8.2'}
   ```
   
   Normally the artifact URL has an extension, such as `.pom` or `.jar`. There 
is no such in 
`https://repo.maven.apache.org/maven2/io/quarkus/quarkus-jackson/3.8.2/quarkus-jackson-3.8.2`
 above. 
   I have no idea where it is coming from. 
   
   In any case, we should make the `TerminalOutput.pathToMaven()` method more 
robust so that it does not fail on this kind of input.
   


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



[jira] [Commented] (MNG-8025) API incompatibility with m-remote-resources-p

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise commented on MNG-8025:
--

Is this still relevant? Or can it simply being "solved" by using a more recent 
version of maven-resources-plugin (3.3.1)?

> API incompatibility with m-remote-resources-p
> -
>
> Key: MNG-8025
> URL: https://issues.apache.org/jira/browse/MNG-8025
> Project: Maven
>  Issue Type: Bug
>Affects Versions: 4.0.0-alpha-12
>Reporter: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.x-candidate
>
>
> {code:java}
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0:process 
> (default) on project sshd: Execution default of goal 
> org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0:process failed: 
> An API incompatibility was encountered while executing 
> org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0:process: 
> java.lang.NoSuchMethodError: 'void 
> org.apache.maven.model.Plugin.unsetInheritanceApplied()'
> [ERROR] -
> [ERROR] realm =    
> plugin>org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0
> [ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy
>  {code}



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


[jira] [Updated] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise updated MNG-8095:
-
Description: 
I have setup a project which uses the following parts 
(https://maven.apache.org/docs/3.9.2/release-notes.html)
* {{$session.topDirectory}} 
* {{$session.rootDirectory}} 
* {{$project.rootDirectory}} 
but unfortately they are not being replaced during the run of the project... A 
small excerpt from the project... using the following simple configuration:
{code:xml}
  
com.soebes.maven.plugins
echo-maven-plugin
0.5.0

  
initialize

  echo


  

project.rootDirectory: '${project.rootDirectory}'
session.rootDirectory: '${session.rootDirectory}'
session.topDirectory: '${session.topDirectory}'
project.basedir: '${project.basedir}'
project.build.directory: 
'${project.build.directory}'

  

  

  
{code}

My expectation is that during the build the values should be replaced with the 
appropriate values which works with Maven 4 (Tested with Maven 4-alpha-13) like 
this:
{code}
[INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
[INFO] =
[INFO] project.rootDirectory: '/projects/props'
[INFO] session.rootDirectory: '/projects/props'
[INFO] session.topDirectory: '/projects/props'
[INFO] project.basedir: '/projects/props/sub-mod-2'
[INFO] project.build.directory: '/projects/props/sub-mod-2/target'
[INFO] =
{code}

But the run with Maven 3.9.X it looks like this:
{code}
[INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
[INFO] =
[INFO] project.rootDirectory: '${project.rootDirectory}'
[INFO] session.rootDirectory: '${session.rootDirectory}'
[INFO] session.topDirectory: '${session.topDirectory}'
[INFO] project.basedir: '/projects/props/sub-mod-2'
[INFO] project.build.directory: '/projects/props/sub-mod-2/target'
[INFO] =
{code}

I have setup a full working example here:
https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095

  was:
I have setup a project which uses the following parts 
(https://maven.apache.org/docs/3.9.2/release-notes.html)
* {{$session.topDirectory}} 
* {{$session.rootDirectory}} 
* {{$project.rootDirectory}} 
but unfortately they are not being replaced during the run of the project... A 
small excerpt from the project... using the following simple configuration:
{code:xml}
  
com.soebes.maven.plugins
echo-maven-plugin
0.5.0

  
initialize

  echo


  

project.rootDirectory: '${project.rootDirectory}'
session.rootDirectory: '${session.rootDirectory}'
session.topDirectory: '${session.topDirectory}'
project.basedir: '${project.basedir}'
project.build.directory: 
'${project.build.directory}'

  

  

  
{code}

My expectation is that during the build the values should be replaced with the 
appropriate values which works with Maven 4 (Tested with Maven 4-alpha-13) like 
this:
{code}
[INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
[INFO] =
[INFO] project.rootDirectory: '/projects/props'
[INFO] session.rootDirectory: '/projects/props'
[INFO] session.topDirectory: '/projects/props'
[INFO] project.basedir: '/projects/props/sub-mod-2'
[INFO] project.build.directory: '/projects/props/sub-mod-2/target'
[INFO] =
{code}

But the run with Maven 3.9.X it looks like this:
{code}
[INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
[INFO] =
[INFO] project.rootDirectory: '${project.rootDirectory}'
[INFO] session.rootDirectory: '${session.rootDirectory}'
[INFO] session.topDirectory: '${session.topDirectory}'
[INFO] project.basedir: '/projects/props/sub-mod-2'
[INFO] project.build.directory: '/projects/props/sub-mod-2/target'
[INFO] =
{code}


> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.0, 3.9.1, 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz 

[jira] [Commented] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak commented on MNG-8095:
--

Agreed, most probably release notes needs some better (or clearer) wording, 
that we mean 'maven configuration' and not ANY 'configuration' like plugin 
configuration is...

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


Re: [PR] [MPOM-481] Improve description of maven-gpg-plugin settings [maven-apache-parent]

2024-04-12 Thread via GitHub


slawekjaranowski merged PR #209:
URL: https://github.com/apache/maven-apache-parent/pull/209


-- 
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] TerminalOutput.pathToMaven throws StringIndexOutOfBoundsException [maven-mvnd]

2024-04-12 Thread via GitHub


ppalaga commented on issue #929:
URL: https://github.com/apache/maven-mvnd/issues/929#issuecomment-2051825980

   When I make `TerminalOutput.pathToMaven()` more robust, I see many such 
extension-less URLs
   
   ```
   $ grep "\-3.8.2'" daemon-167cc94d.log 
   15:43:45.779 I Dispatch message: 
TransferInitiated{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-junit5-internal/3.8.2/quarkus-junit5-internal-3.8.2',
 contentLength=-1, transferredBytes=0, exception='null'}
   15:43:45.780 I Dispatch message: 
ProjectLogMessage{projectId='quarkus-langchain4j-parent', message='[INFO] 
Downloading from central: 
https://repo.maven.apache.org/maven2/io/quarkus/quarkus-junit5-internal/3.8.2/quarkus-junit5-internal-3.8.2'}
   15:43:45.878 I Dispatch message: 
TransferFailed{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-junit5-internal/3.8.2/quarkus-junit5-internal-3.8.2',
 contentLength=-1, transferredBytes=0, 
exception='org.eclipse.aether.transfer.ArtifactNotFoundException: Could not 
find artifact io.quarkus:quarkus-junit5-internal::3.8.2 in central 
(https://repo.maven.apache.org/maven2)'}
   15:43:51.300 I Dispatch message: 
TransferInitiated{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-vertx-http-dev-ui-tests/3.8.2/quarkus-vertx-http-dev-ui-tests-3.8.2',
 contentLength=-1, transferredBytes=0, exception='null'}
   15:43:51.300 I Dispatch message: 
ProjectLogMessage{projectId='quarkus-langchain4j-parent', message='[INFO] 
Downloading from central: 
https://repo.maven.apache.org/maven2/io/quarkus/quarkus-vertx-http-dev-ui-tests/3.8.2/quarkus-vertx-http-dev-ui-tests-3.8.2'}
   15:43:51.392 I Dispatch message: 
TransferFailed{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-vertx-http-dev-ui-tests/3.8.2/quarkus-vertx-http-dev-ui-tests-3.8.2',
 contentLength=-1, transferredBytes=0, 
exception='org.eclipse.aether.transfer.ArtifactNotFoundException: Could not 
find artifact io.quarkus:quarkus-vertx-http-dev-ui-tests::3.8.2 in central 
(https://repo.maven.apache.org/maven2)'}
   15:43:53.943 I Dispatch message: 
TransferInitiated{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-smallrye-fault-tolerance/3.8.2/quarkus-smallrye-fault-tolerance-3.8.2',
 contentLength=-1, transferredBytes=0, exception='null'}
   15:43:53.943 I Dispatch message: 
ProjectLogMessage{projectId='quarkus-langchain4j-parent', message='[INFO] 
Downloading from central: 
https://repo.maven.apache.org/maven2/io/quarkus/quarkus-smallrye-fault-tolerance/3.8.2/quarkus-smallrye-fault-tolerance-3.8.2'}
   15:43:54.035 I Dispatch message: 
TransferFailed{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-smallrye-fault-tolerance/3.8.2/quarkus-smallrye-fault-tolerance-3.8.2',
 contentLength=-1, transferredBytes=0, 
exception='org.eclipse.aether.transfer.ArtifactNotFoundException: Could not 
find artifact io.quarkus:quarkus-smallrye-fault-tolerance::3.8.2 in central 
(https://repo.maven.apache.org/maven2)'}
   15:43:54.425 I Dispatch message: 
TransferInitiated{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-junit5/3.8.2/quarkus-junit5-3.8.2', 
contentLength=-1, transferredBytes=0, exception='null'}
   15:43:54.425 I Dispatch message: 
ProjectLogMessage{projectId='quarkus-langchain4j-parent', message='[INFO] 
Downloading from central: 
https://repo.maven.apache.org/maven2/io/quarkus/quarkus-junit5/3.8.2/quarkus-junit5-3.8.2'}
   15:43:54.518 I Dispatch message: 
TransferFailed{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-junit5/3.8.2/quarkus-junit5-3.8.2', 
contentLength=-1, transferredBytes=0, 
exception='org.eclipse.aether.transfer.ArtifactNotFoundException: Could not 
find artifact io.quarkus:quarkus-junit5::3.8.2 in central 
(https://repo.maven.apache.org/maven2)'}
   15:44:06.374 I Dispatch message: 
TransferInitiated{projectId=quarkus-langchain4j-parent, requestType=0, 
repositoryId='central', repositoryUrl='https://repo.maven.apache.org/maven2/', 
resourceName='io/quarkus/quarkus-devtools-testing/3.8.2/quarkus-devtools-testing-3.8.2',
 contentLength=-1, transferredBytes=0, exception='null'}
   15:44:06.374 I 

[jira] [Created] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)
Karl Heinz Marbaise created MNG-8095:


 Summary: Usage of session.rootDirectory / session.topDirectory 
does not work
 Key: MNG-8095
 URL: https://issues.apache.org/jira/browse/MNG-8095
 Project: Maven
  Issue Type: Bug
  Components: Core
Affects Versions: 3.9.6, 3.9.5, 3.9.4, 3.9.3, 3.9.2, 3.9.1, 3.9.0
Reporter: Karl Heinz Marbaise


I have setup a project which uses the following parts 
(https://maven.apache.org/docs/3.9.2/release-notes.html)
* {{$session.topDirectory}} 
* {{$session.rootDirectory}} 
* {{$project.rootDirectory}} 
but unfortately they are not being replaced during the run of the project... A 
small excerpt from the project... using the following simple configuration:
{code:xml}
  
com.soebes.maven.plugins
echo-maven-plugin
0.5.0

  
initialize

  echo


  

project.rootDirectory: '${project.rootDirectory}'
session.rootDirectory: '${session.rootDirectory}'
session.topDirectory: '${session.topDirectory}'
project.basedir: '${project.basedir}'
project.build.directory: 
'${project.build.directory}'

  

  

  
{code}

My expectation is that during the build the values should be replaced with the 
appropriate values which works with Maven 4 (Tested with Maven 4-alpha-13) like 
this:
{code}
[INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
[INFO] =
[INFO] project.rootDirectory: '/projects/props'
[INFO] session.rootDirectory: '/projects/props'
[INFO] session.topDirectory: '/projects/props'
[INFO] project.basedir: '/projects/props/sub-mod-2'
[INFO] project.build.directory: '/projects/props/sub-mod-2/target'
[INFO] =
{code}

But the run with Maven 3.9.X it looks like this:
{code}
[INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
[INFO] =
[INFO] project.rootDirectory: '${project.rootDirectory}'
[INFO] session.rootDirectory: '${session.rootDirectory}'
[INFO] session.topDirectory: '${session.topDirectory}'
[INFO] project.basedir: '/projects/props/sub-mod-2'
[INFO] project.build.directory: '/projects/props/sub-mod-2/target'
[INFO] =
{code}



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


[jira] [Updated] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise updated MNG-8095:
-
Affects Version/s: (was: 3.9.0)
   (was: 3.9.1)

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


Re: [I] TerminalOutput.pathToMaven throws StringIndexOutOfBoundsException [maven-mvnd]

2024-04-12 Thread via GitHub


cstamas commented on issue #929:
URL: https://github.com/apache/maven-mvnd/issues/929#issuecomment-2051952768

   Stunts like these ("reverse engineering URL into artifact" IIUC) should be 
avoided. Please do not do this :smile: Would logging just bare file name not be 
enough?


-- 
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-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise commented on MNG-8095:
--

Ok I have feared such a thing. That means we have to make the release notes 
more clear on that part... If I read "properties" and "configuration" which 
could be mistaken with the configuration of the plugins... Ok... 

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


[jira] [Closed] (MPOM-481) Improve description of maven-gpg-plugin settings

2024-04-12 Thread Slawomir Jaranowski (Jira)


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

Slawomir Jaranowski closed MPOM-481.

Resolution: Fixed

> Improve description of maven-gpg-plugin settings
> 
>
> Key: MPOM-481
> URL: https://issues.apache.org/jira/browse/MPOM-481
> Project: Maven POMs
>  Issue Type: Improvement
>  Components: asf
>Reporter: Slawomir Jaranowski
>Assignee: Slawomir Jaranowski
>Priority: Major
> Fix For: ASF-32
>
>




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


[jira] [Commented] (MNG-8025) API incompatibility with m-remote-resources-p

2024-04-12 Thread Guillaume Nodet (Jira)


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

Guillaume Nodet commented on MNG-8025:
--

I think it is.  The current code still uses {{unsetInheritanceApplied}} and 
it's not present in the current {{master}}.

> API incompatibility with m-remote-resources-p
> -
>
> Key: MNG-8025
> URL: https://issues.apache.org/jira/browse/MNG-8025
> Project: Maven
>  Issue Type: Bug
>Affects Versions: 4.0.0-alpha-12
>Reporter: Guillaume Nodet
>Priority: Major
> Fix For: 4.0.x-candidate
>
>
> {code:java}
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0:process 
> (default) on project sshd: Execution default of goal 
> org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0:process failed: 
> An API incompatibility was encountered while executing 
> org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0:process: 
> java.lang.NoSuchMethodError: 'void 
> org.apache.maven.model.Plugin.unsetInheritanceApplied()'
> [ERROR] -
> [ERROR] realm =    
> plugin>org.apache.maven.plugins:maven-remote-resources-plugin:3.1.0
> [ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy
>  {code}



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


[jira] [Commented] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise commented on MNG-8095:
--

Apart from that the branch on 3.9.X contains refererences like this:
{code:java}
@Test
public void testPropertiesInterpolation() throws Exception {
// Arrange
CliRequest request = new CliRequest(
new String[] {
"-Dfoo=bar",
"-DvalFound=s${foo}i",
"-DvalNotFound=s${foz}i",
"-DvalRootDirectory=${session.rootDirectory}/.mvn/foo",
"-DvalTopDirectory=${session.topDirectory}/pom.xml",
"-f",
"${session.rootDirectory}/my-child",
"prefix:3.0.0:${foo}",
"validate"
},
null);
request.rootDirectory = Paths.get("myRootDirectory");
request.topDirectory = Paths.get("myTopDirectory");
{code}
and also the code in {{MavenCli.java}} contains 
((https://github.com/apache/maven/blob/maven-3.9.x/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java#L1563):
{code:java}
protected static StringSearchInterpolator createInterpolator(CliRequest 
cliRequest, Properties... properties) {
StringSearchInterpolator interpolator = new StringSearchInterpolator();
interpolator.addValueSource(new AbstractValueSource(false) {
@Override
public Object getValue(String expression) {
if ("session.topDirectory".equals(expression)) {
Path topDirectory = cliRequest.topDirectory;
if (topDirectory != null) {
return topDirectory.toString();
} else {
throw new IllegalUseOfUndefinedProperty(expression);
}
} else if ("session.rootDirectory".equals(expression)) {
Path rootDirectory = cliRequest.rootDirectory;
if (rootDirectory != null) {
return rootDirectory.toString();
} else {
throw new IllegalUseOfUndefinedProperty(expression);
}
}
return null;
}
});
... 
{code}


> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] 

[jira] [Commented] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Tamas Cservenak (Jira)


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

Tamas Cservenak commented on MNG-8095:
--

Rel notes says specifically "implemented interpolation for configuration and 
command line".

So, in 3.9.x this is NOT a property (so cannot be refd from POM etc), ONLY from 
command line and .mvn/maven.config.

In 4.0 this IS fully fledged property, and can be referenced from POM as well.

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


Re: [PR] Bump apache/maven-gh-actions-shared from 3 to 4 [maven-parent]

2024-04-12 Thread via GitHub


CrazyHZM merged PR #172:
URL: https://github.com/apache/maven-parent/pull/172


-- 
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] (MBUILDCACHE-45) Compiled classes are not copied/restored to target, causing higher phase builds to fail

2024-04-12 Thread Jira


[ 
https://issues.apache.org/jira/browse/MBUILDCACHE-45?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836656#comment-17836656
 ] 

Réda Housni Alaoui commented on MBUILDCACHE-45:
---

I have the same issue with the sonar-maven-plugin. The latter expects .class 
files in target folders.

> Compiled classes are not copied/restored to target, causing higher phase 
> builds to fail
> ---
>
> Key: MBUILDCACHE-45
> URL: https://issues.apache.org/jira/browse/MBUILDCACHE-45
> Project: Maven Build Cache Extension
>  Issue Type: Bug
> Environment: windows
>Reporter: Miguel Ortega
>Priority: Major
> Attachments: maven-cache.zip
>
>
> +*Step to reproduce*+
> 1. Execute a first command on a pristine projet( NO CACHE has ever been 
> stored)
> {code:java}
> mvn test{code}
> Results are cached (contrary to documentation that states that "cache will 
> kick-in automatically on every lifecycle build of phase {{package}} or 
> higher.") This could also be a feature since tests can be skipped if nothing 
> changes.
> 2 . But if I run next: 
> {code:java}
> mvn clean verify
> #OR
> mvn clean install{code}
> Then the build fails
> {code:java}
> [ERROR] Failed to execute goal 
> org.springframework.boot:spring-boot-maven-plugin:3.0.2:repackage (repackage) 
> on project demo: Execution repackage of goal 
> org.springframework.boot:spring-boot-maven-plugin:3.0.2:repackage failed: 
> Unable to find main class -> [Help 1]
> [ERROR]
>  {code}
> The root cause seems to be that after "clean", the maven target directory is 
> cleaned and even if the cache is detected, classes are not restored to the 
> target folder anymore
> {code:java}
> [INFO] Attempting to restore project com.example:demo from build cache
> [INFO] Remote cache is incomplete or missing, trying local build for 
> com.example:demo
> [INFO] Local build found by checksum 596f60b3f5056d7d
> [INFO] Found cached build, restoring com.example:demo from cache by checksum 
> 596f60b3f5056d7d
> [INFO] Project com.example:demo restored partially. Highest cached goal: 
> test, requested: install
> [INFO] Skipping plugin execution (cached): resources:resources
> [INFO] Skipping plugin execution (cached): compiler:compile
> [INFO] Skipping plugin execution (cached): resources:testResources
> [INFO] Skipping plugin execution (cached): compiler:testCompile
> [INFO] Skipping plugin execution (cached): surefire:test {code}
> *+Workaround+*
> Manually remove cache data or disable cache via command line argument
> *+Environment+*
> {code:java}
> Apache Maven 3.9.0 (9b58d2bad23a66be161c4664ef21ce219c2c8584)
> Maven home: 
> Java version: 17.0.1, vendor: Eclipse Adoptium, runtime: 
> Default locale: fr_FR, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"{code}



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


[jira] [Created] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread Karl Heinz Marbaise (Jira)
Karl Heinz Marbaise created MNGSITE-535:
---

 Summary: Improve the Release Notes about 3.9.2 - related to 
topDirectory/rootDirectory
 Key: MNGSITE-535
 URL: https://issues.apache.org/jira/browse/MNGSITE-535
 Project: Maven Project Web Site
  Issue Type: Improvement
Reporter: Karl Heinz Marbaise
Assignee: Karl Heinz Marbaise


Related to this: MNG-8095 there should be hint where the following:

* {{$session.topDirectory}} 
* {{$session.rootDirectory}} 

can be used an can not being used.



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


[jira] [Closed] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise closed MNGSITE-535.
---
Resolution: Done

> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


[jira] [Commented] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread Karl Heinz Marbaise (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836661#comment-17836661
 ] 

Karl Heinz Marbaise commented on MNGSITE-535:
-

Done in [7d8d3c187215da8137e6ff7a174a741d59c8a3e7|.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
http://localhost:8081/nexus/content/groups
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
(/Users/khm/ws-git-maven/maven-site
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
http://localhost:8081/nexus/content/groups
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.?p=1.0:
The
following
artifacts
could
not
be
resolved:
org.apache.maven:maven-parent:pom:42-SNAPSHOT
(absent):
org.apache.maven:maven-parent:pom:42-SNAPSHOT
was
not
found
in
public
during
a
previous
attempt.
This
failure
was
cached
in
the
local
repository
and
resolution
is
not
reattempted
until
the
update
interval
of
nexus
has
elapsed
or
updates
are
forced
and
'parent.relativePath'
points
at
no
local
POM
@
line
23,
column
11
@
[ERROR]
The
build
could
not
read
1
project
->
[Help
1]
[ERROR]
1.0
pom.xml)
has
1
error
1.0:
The
following
artifacts
could
not
be
resolved:
org.apache.maven:maven-parent:pom:42-SNAPSHOT
(absent):
org.apache.maven:maven-parent:pom:42-SNAPSHOT
was
not
found
in
public
during
a
previous
attempt.
This
failure
was
cached
in
the
local
repository
and
resolution
is
not
reattempted
until
the
update
interval
of
nexus
has
elapsed
or
updates
are
forced
and
'parent.relativePath'
points
at
no
local
POM
@
line
23,
column
11
->
[Help
2]
[ERROR]
[ERROR]
To
see
the
full
stack
trace
of
the
errors,
re-run
Maven
with
the
-e
switch.
[ERROR]
Re-run
Maven
using
the
-X
switch
to
enable
full
debug
logging.
[ERROR];a=commitdiff;h=7d8d3c187215da8137e6ff7a174a741d59c8a3e7]

> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


[jira] [Comment Edited] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread Karl Heinz Marbaise (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836661#comment-17836661
 ] 

Karl Heinz Marbaise edited comment on MNGSITE-535 at 4/12/24 3:48 PM:
--

Done in [7d8d3c187215da8137e6ff7a174a741d59c8a3e7|


was (Author: khmarbaise):
Done in [7d8d3c187215da8137e6ff7a174a741d59c8a3e7|.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
http://localhost:8081/nexus/content/groups
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
(/Users/khm/ws-git-maven/maven-site
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
http://localhost:8081/nexus/content/groups
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.?p=1.0:
The
following
artifacts
could
not
be
resolved:
org.apache.maven:maven-parent:pom:42-SNAPSHOT
(absent):
org.apache.maven:maven-parent:pom:42-SNAPSHOT
was
not
found
in
public
during
a
previous
attempt.
This
failure
was
cached
in
the
local
repository
and
resolution
is
not
reattempted
until
the
update
interval
of
nexus
has
elapsed
or
updates
are
forced
and
'parent.relativePath'
points
at
no
local
POM
@
line
23,
column
11
@
[ERROR]
The
build
could
not
read
1
project
->
[Help
1]
[ERROR]
1.0
pom.xml)
has
1
error
1.0:
The
following
artifacts
could
not
be
resolved:
org.apache.maven:maven-parent:pom:42-SNAPSHOT
(absent):
org.apache.maven:maven-parent:pom:42-SNAPSHOT
was
not
found
in
public
during
a
previous
attempt.
This
failure
was
cached
in
the
local
repository
and
resolution
is
not
reattempted
until
the
update
interval
of
nexus
has
elapsed
or
updates
are
forced
and
'parent.relativePath'
points
at
no
local
POM
@
line
23,
column
11
->
[Help
2]
[ERROR]
[ERROR]
To
see
the
full
stack
trace
of
the
errors,
re-run
Maven
with
the
-e
switch.
[ERROR]
Re-run
Maven
using
the
-X
switch
to
enable
full
debug
logging.
[ERROR];a=commitdiff;h=7d8d3c187215da8137e6ff7a174a741d59c8a3e7]

> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


Re: [PR] [MNGSITE-535] - Improve the Release Notes about 3.9.2 - related to to… [maven-site]

2024-04-12 Thread via GitHub


asfgit merged PR #510:
URL: https://github.com/apache/maven-site/pull/510


-- 
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] (MNGSITE-532) Move plugin-report to Reporting Plugin table

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise reassigned MNGSITE-532:
---

Assignee: Karl Heinz Marbaise

> Move plugin-report to Reporting Plugin table
> 
>
> Key: MNGSITE-532
> URL: https://issues.apache.org/jira/browse/MNGSITE-532
> Project: Maven Project Web Site
>  Issue Type: Task
>Reporter: Michael Osipov
>Assignee: Karl Heinz Marbaise
>Priority: Major
>
> In https://maven.apache.org/plugins/index.html {{plugin-report}} is a pure 
> reporting plugin, like {{surefire-report}}, it should be in reporting...



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


Re: [PR] [MNG-8081] interpolate available properties during default profile selection (Maven 4.x) [maven]

2024-04-12 Thread via GitHub


mbenson commented on PR #1446:
URL: https://github.com/apache/maven/pull/1446#issuecomment-2052059016

   @gnodet do you have any thoughts on the best way to backport this feature to 
Maven 3? https://github.com/apache/maven/pull/1447 is my original PR, but a 
wildly divergent implementation may cause headaches in future. I wonder if it 
would make sense [from a future merge perspective] to stabilize the v3 impl and 
include the same commit in this branch prior to being replaced by the actual v4 
implementation. 樂 


-- 
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] [MNGSITE-532] - Move plugin-report to Reporting Plugin table [maven-site]

2024-04-12 Thread via GitHub


asfgit merged PR #511:
URL: https://github.com/apache/maven-site/pull/511


-- 
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] (MNGSITE-536) Link to Reference "Maven Plugin Tools and annotations" wrong

2024-04-12 Thread Karl Heinz Marbaise (Jira)
Karl Heinz Marbaise created MNGSITE-536:
---

 Summary: Link to Reference "Maven Plugin Tools and annotations" 
wrong
 Key: MNGSITE-536
 URL: https://issues.apache.org/jira/browse/MNGSITE-536
 Project: Maven Project Web Site
  Issue Type: Bug
Reporter: Karl Heinz Marbaise
Assignee: Karl Heinz Marbaise


The link on the page https://maven.apache.org/plugin-developers/index.html to 
the annotation {{Maven Plugin Tools and annotations}} is wrong.



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


Re: [PR] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


cstamas commented on PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#issuecomment-2052127538

   For last build this PR:
   ```
   [INFO] Building: sign-release-best-practices/pom.xml
   [INFO] run post-build script verify.groovy
   [INFO]   sign-release-best-practices/pom.xml .. SUCCESS 
(4.002 s)
   ```
   2nd build/commit:
   ```
   [INFO] Building: sign-release-best-practices/pom.xml
   [INFO] run post-build script verify.groovy
   [INFO]   sign-release-best-practices/pom.xml .. SUCCESS 
(3.877 s)
   ```
   
   Hm, minimal... unsure what I "feel" locally. Maybe that these two ITs use 
_new_ plugins (hence downoads them) while all the rest use old/ancient plugins 
but some 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] [MNGSITE-536] - Link to Reference "Maven Plugin Tools and annotations… [maven-site]

2024-04-12 Thread via GitHub


asfgit merged PR #512:
URL: https://github.com/apache/maven-site/pull/512


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

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

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



[jira] [Updated] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise updated MNGSITE-535:

Description: 
Related to this: MNG-8095 there should more clear wording when the following 
properties can be used and when not:

* {{$session.topDirectory}} 
* {{$session.rootDirectory}} 


  was:
Related to this: MNG-8095 there should be hint where the following:

* {{$session.topDirectory}} 
* {{$session.rootDirectory}} 

can be used an can not being used.


> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


[jira] [Commented] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread Tamas Cservenak (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836658#comment-17836658
 ] 

Tamas Cservenak commented on MNGSITE-535:
-

In Maven 3.9.2+ (since introduction): only on command line (user properties) 
and {{{}.mvn/maven.config{}}}.

In Maven 4 everywhere.

> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


[jira] [Commented] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836659#comment-17836659
 ] 

ASF GitHub Bot commented on MNGSITE-535:


khmarbaise opened a new pull request, #510:
URL: https://github.com/apache/maven-site/pull/510

   …pDirectory/rootDirectory




> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


[jira] [Closed] (MNGSITE-532) Move plugin-report to Reporting Plugin table

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise closed MNGSITE-532.
---
Resolution: Fixed

> Move plugin-report to Reporting Plugin table
> 
>
> Key: MNGSITE-532
> URL: https://issues.apache.org/jira/browse/MNGSITE-532
> Project: Maven Project Web Site
>  Issue Type: Task
>Reporter: Michael Osipov
>Assignee: Karl Heinz Marbaise
>Priority: Major
>
> In https://maven.apache.org/plugins/index.html {{plugin-report}} is a pure 
> reporting plugin, like {{surefire-report}}, it should be in reporting...



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


[jira] [Commented] (MNGSITE-532) Move plugin-report to Reporting Plugin table

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-532?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836683#comment-17836683
 ] 

ASF GitHub Bot commented on MNGSITE-532:


asfgit merged PR #511:
URL: https://github.com/apache/maven-site/pull/511




> Move plugin-report to Reporting Plugin table
> 
>
> Key: MNGSITE-532
> URL: https://issues.apache.org/jira/browse/MNGSITE-532
> Project: Maven Project Web Site
>  Issue Type: Task
>Reporter: Michael Osipov
>Assignee: Karl Heinz Marbaise
>Priority: Major
>
> In https://maven.apache.org/plugins/index.html {{plugin-report}} is a pure 
> reporting plugin, like {{surefire-report}}, it should be in reporting...



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


[jira] [Closed] (MNGSITE-536) Link to Reference "Maven Plugin Tools and annotations" wrong

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise closed MNGSITE-536.
---
Resolution: Fixed

> Link to Reference "Maven Plugin Tools and annotations" wrong
> 
>
> Key: MNGSITE-536
> URL: https://issues.apache.org/jira/browse/MNGSITE-536
> Project: Maven Project Web Site
>  Issue Type: Bug
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> The link on the page https://maven.apache.org/plugin-developers/index.html to 
> the annotation {{Maven Plugin Tools and annotations}} is wrong.



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


Re: [PR] [MGPG-125] Fix "bestPractices" [maven-gpg-plugin]

2024-04-12 Thread via GitHub


slawekjaranowski commented on PR #95:
URL: https://github.com/apache/maven-gpg-plugin/pull/95#issuecomment-2052186688

   It can be a time needed for downloading new plugins version into 
tarted/local-repo
   Too small difference to more investigating
   
   build with `mvn clean ...`
   ```
   gpg
   [INFO]   sign-release-best-practices/pom.xml .. SUCCESS 
(8.135 s)
   
   bc
   [INFO]   sign-release-best-practices/pom.xml .. SUCCESS 
(2.189 s)
   ```
   
   next without `clean`
   
   ```
   gpg
   [INFO]   sign-release-best-practices/pom.xml .. SUCCESS 
(2.653 s)
   
   bc
   [INFO]   sign-release-best-practices/pom.xml .. SUCCESS 
(2.237 s)
   
   ```
   


-- 
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] (MNGSITE-535) Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836660#comment-17836660
 ] 

ASF GitHub Bot commented on MNGSITE-535:


asfgit merged PR #510:
URL: https://github.com/apache/maven-site/pull/510




> Improve the Release Notes about 3.9.2 - related to topDirectory/rootDirectory
> -
>
> Key: MNGSITE-535
> URL: https://issues.apache.org/jira/browse/MNGSITE-535
> Project: Maven Project Web Site
>  Issue Type: Improvement
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> Related to this: MNG-8095 there should more clear wording when the following 
> properties can be used and when not:
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 



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


[jira] [Commented] (MNG-8081) default profile activation should consider available system and user properties

2024-04-12 Thread ASF GitHub Bot (Jira)


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

ASF GitHub Bot commented on MNG-8081:
-

mbenson commented on PR #1446:
URL: https://github.com/apache/maven/pull/1446#issuecomment-2052059016

   @gnodet do you have any thoughts on the best way to backport this feature to 
Maven 3? https://github.com/apache/maven/pull/1447 is my original PR, but a 
wildly divergent implementation may cause headaches in future. I wonder if it 
would make sense [from a future merge perspective] to stabilize the v3 impl and 
include the same commit in this branch prior to being replaced by the actual v4 
implementation. 樂 




> default profile activation should consider available system and user 
> properties
> ---
>
> Key: MNG-8081
> URL: https://issues.apache.org/jira/browse/MNG-8081
> Project: Maven
>  Issue Type: Improvement
>  Components: Profiles
>Affects Versions: 3.9.6, 4.0.0
>Reporter: Matthew Jason Benson
>Priority: Minor
> Fix For: 3.9.7, 4.0.0, 4.0.0-alpha-14
>
>
> As discussed in my open PR, my use case is to compare between environment 
> variables e.g.:
> {code:java}
> 
>   
> env.FOO
> ${env.BAR}
>   
> {code}
> Limiting the interpolation to user/system properties means that there is no 
> mindf*ck resulting from profile activation order, etc., and keeps this 
> request nonthreatening.



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


[jira] [Commented] (MNGSITE-532) Move plugin-report to Reporting Plugin table

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-532?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836672#comment-17836672
 ] 

ASF GitHub Bot commented on MNGSITE-532:


khmarbaise opened a new pull request, #511:
URL: https://github.com/apache/maven-site/pull/511

   (no comment)




> Move plugin-report to Reporting Plugin table
> 
>
> Key: MNGSITE-532
> URL: https://issues.apache.org/jira/browse/MNGSITE-532
> Project: Maven Project Web Site
>  Issue Type: Task
>Reporter: Michael Osipov
>Assignee: Karl Heinz Marbaise
>Priority: Major
>
> In https://maven.apache.org/plugins/index.html {{plugin-report}} is a pure 
> reporting plugin, like {{surefire-report}}, it should be in reporting...



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


[jira] [Commented] (MNGSITE-536) Link to Reference "Maven Plugin Tools and annotations" wrong

2024-04-12 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNGSITE-536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17836694#comment-17836694
 ] 

ASF GitHub Bot commented on MNGSITE-536:


khmarbaise opened a new pull request, #512:
URL: https://github.com/apache/maven-site/pull/512

   …" wrong




> Link to Reference "Maven Plugin Tools and annotations" wrong
> 
>
> Key: MNGSITE-536
> URL: https://issues.apache.org/jira/browse/MNGSITE-536
> Project: Maven Project Web Site
>  Issue Type: Bug
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> The link on the page https://maven.apache.org/plugin-developers/index.html to 
> the annotation {{Maven Plugin Tools and annotations}} is wrong.



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


[jira] [Closed] (MNG-8095) Usage of session.rootDirectory / session.topDirectory does not work

2024-04-12 Thread Karl Heinz Marbaise (Jira)


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

Karl Heinz Marbaise closed MNG-8095.

  Assignee: Karl Heinz Marbaise
Resolution: Not A Problem

> Usage of session.rootDirectory / session.topDirectory does not work
> ---
>
> Key: MNG-8095
> URL: https://issues.apache.org/jira/browse/MNG-8095
> Project: Maven
>  Issue Type: Bug
>  Components: Core
>Affects Versions: 3.9.2, 3.9.3, 3.9.4, 3.9.5, 3.9.6
>Reporter: Karl Heinz Marbaise
>Assignee: Karl Heinz Marbaise
>Priority: Minor
>
> I have setup a project which uses the following parts 
> (https://maven.apache.org/docs/3.9.2/release-notes.html)
> * {{$session.topDirectory}} 
> * {{$session.rootDirectory}} 
> * {{$project.rootDirectory}} 
> but unfortately they are not being replaced during the run of the project... 
> A small excerpt from the project... using the following simple configuration:
> {code:xml}
>   
> com.soebes.maven.plugins
> echo-maven-plugin
> 0.5.0
> 
>   
> initialize
> 
>   echo
> 
> 
>   
> 
> project.rootDirectory: '${project.rootDirectory}'
> session.rootDirectory: '${session.rootDirectory}'
> session.topDirectory: '${session.topDirectory}'
> project.basedir: '${project.basedir}'
> project.build.directory: 
> '${project.build.directory}'
> 
>   
> 
>   
> 
>   
> {code}
> My expectation is that during the build the values should be replaced with 
> the appropriate values which works with Maven 4 (Tested with Maven 
> 4-alpha-13) like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '/projects/props'
> [INFO] session.rootDirectory: '/projects/props'
> [INFO] session.topDirectory: '/projects/props'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> But the run with Maven 3.9.X it looks like this:
> {code}
> [INFO] --- echo:0.5.0:echo (default) @ sub-mod-2 ---
> [INFO] =
> [INFO] project.rootDirectory: '${project.rootDirectory}'
> [INFO] session.rootDirectory: '${session.rootDirectory}'
> [INFO] session.topDirectory: '${session.topDirectory}'
> [INFO] project.basedir: '/projects/props/sub-mod-2'
> [INFO] project.build.directory: '/projects/props/sub-mod-2/target'
> [INFO] =
> {code}
> I have setup a full working example here:
> https://github.com/khmarbaise/maven-bugs/tree/master/MNG-8095



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


  1   2   >