[GitHub] [maven-surefire] sbabcoc commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


sbabcoc commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851694849


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   If all supported versions of TestNG provide a default for this setting, it 
may be best to not inject a default here. Just let TestNG provide its own 
default value. That's how the new code for TestNG 7.4+ behaves. I don't know 
the history of the original code to understand why this was coded to inject a 
default value of 1. (What's the benefit of specifying parallel execution if the 
thread count is 1?)
   If the default implementation is changed so that it behaves like the new 
override, this override can be removed.



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

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

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



[GitHub] [maven-surefire] sbabcoc commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


sbabcoc commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851697867


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   Why would we override the default value provided by TestNG? What's the 
benefit of specifying parallel execution if the thread count is 1?



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

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

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



[GitHub] [maven-surefire] sbabcoc commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


sbabcoc commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851694849


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   If all supported versions of TestNG provide a default for this setting, it 
may be best to not inject a default here. Just let TestNG provide its own 
default value. That's how the new code for TestNG 7.4+ behaves. I don't know 
the history of the original code to understand why this was coded to inject a 
default value of 1.
   If the default implementation is changed so that it behaves like the new 
override, this override can be removed.



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

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

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



[GitHub] [maven-surefire] sbabcoc commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


sbabcoc commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851694849


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   If all supported versions of TestNG provide a default for this setting, it 
may be best to not inject a default here. Just let TestNG provide its own 
default value. That's how the new code for TestNG 7.4+ behaves. I don't know 
the history of the original code to understand why this was coded to inject a 
default value of 1.



-- 
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] (MJAVADOC-714) Upgrade to Maven 3.2.5

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MJAVADOC-714:
-

Build unstable in Jenkins: Maven » Maven TLP » maven-javadoc-plugin » master #16

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-javadoc-plugin/job/master/16/

> Upgrade to Maven 3.2.5
> --
>
> Key: MJAVADOC-714
> URL: https://issues.apache.org/jira/browse/MJAVADOC-714
> Project: Maven Javadoc Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.4.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MJAVADOC-714) Upgrade to Maven 3.2.5

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MJAVADOC-714.
---
Resolution: Fixed

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

> Upgrade to Maven 3.2.5
> --
>
> Key: MJAVADOC-714
> URL: https://issues.apache.org/jira/browse/MJAVADOC-714
> Project: Maven Javadoc Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: next-release
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-javadoc-plugin] asfgit closed pull request #134: [MJAVADOC-714] Upgrade to Maven 3.2.5

2022-04-16 Thread GitBox


asfgit closed pull request #134: [MJAVADOC-714] Upgrade to Maven 3.2.5
URL: https://github.com/apache/maven-javadoc-plugin/pull/134


-- 
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] (MJAVADOC-510) Investigate JavadocUtil.invokeMaven

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MJAVADOC-510:

Fix Version/s: (was: next-release)

> Investigate JavadocUtil.invokeMaven
> ---
>
> Key: MJAVADOC-510
> URL: https://issues.apache.org/jira/browse/MJAVADOC-510
> Project: Maven Javadoc Plugin
>  Issue Type: Task
>  Components: fix
>Reporter: Stephen Connolly
>Priority: Major
>
> Spotted while fine-tuning the Jenkinsfile builds. Unclear why Javadoc plugin 
> needs to fork Maven and whether it is doing it the correct way if the 
> requirement is actually valid.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-surefire] Tibor17 commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


Tibor17 commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851683061


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   I found the default value in TestNG:
   `private int m_threadCount = 5;`
   We should use the same line as before, means `suite.setThreadCount( 
threadCount == null ? 1 : Integer.parseInt( threadCount ) )`



##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   I found the default value in TestNG:
   `private int m_threadCount = 5;`
   We should use the same line as before, means `suite.setThreadCount( 
threadCount == null ? 1 : parseInt( threadCount ) )`



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

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

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



[GitHub] [maven-surefire] Tibor17 commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


Tibor17 commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851683061


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   I found the default value in TestNG:
   `private int m_threadCount = 5;`
   We should use the same line as before, means `suite.setThreadCount( 
threadCountAsString == null ? 1 : Integer.parseInt( threadCount ) )`



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

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

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



[GitHub] [maven-surefire] Tibor17 commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


Tibor17 commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851683061


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   I found the default value in TestNG:
   `private int m_threadCount = 5;`
   We should use the same as before, means `suite.setThreadCount( 
threadCountAsString == null ? 1 : Integer.parseInt( threadCount ) )`



-- 
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] (MJAVADOC-714) Upgrade to Maven 3.2.5

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MJAVADOC-714:
---

 Summary: Upgrade to Maven 3.2.5
 Key: MJAVADOC-714
 URL: https://issues.apache.org/jira/browse/MJAVADOC-714
 Project: Maven Javadoc Plugin
  Issue Type: Dependency upgrade
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: next-release






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-surefire] Tibor17 commented on a diff in pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


Tibor17 commented on code in PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#discussion_r851681413


##
surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/conf/TestNG740Configurator.java:
##
@@ -37,33 +36,77 @@
  *
  * @since 3.0.0-M6
  */
-public class TestNG740Configurator extends TestNG60Configurator
+public class TestNG740Configurator
+extends TestNG60Configurator
 {
+/**
+ * Convert and apply the value of the [threadcount] option.
+ * 
+ * @param suite TestNG {@link XmlSuite} object
+ * @param options Surefire plugin configuration options
+ * @throws TestSetFailedException if unable to convert specified 
[threadcount] setting
+ */
 @Override
-public void configure( XmlSuite suite, Map options )
+protected void configureThreadCount( XmlSuite suite, Map 
options )
 throws TestSetFailedException
 {
-String threadCountAsString = options.get( THREADCOUNT_PROP );
-int threadCount = threadCountAsString == null ? 1 : parseInt( 
threadCountAsString );
-suite.setThreadCount( threadCount );
-
-String parallel = options.get( PARALLEL_PROP );
-if ( parallel != null )
+String threadCount = options.get( THREADCOUNT_PROP );
+// if [threadcount] spec'd
+if ( threadCount != null )
 {
-if ( !"methods".equalsIgnoreCase( parallel ) && 
!"classes".equalsIgnoreCase( parallel ) )
+try
 {
-throw new TestSetFailedException( "Unsupported TestNG parallel 
setting: "
-+ parallel + " ( only METHODS or CLASSES supported )" );
+// convert and apply [threadcount] setting
+suite.setThreadCount( Integer.parseInt( threadCount ) );

Review Comment:
   Previously it was `threadCountAsString == null ? 1 : ...`. What would happen 
if parallel=classes but the ThreadCount is not set? How is TestNG implemented? 
It has default value 1?



-- 
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] (MSITE-890) Upgrade Jetty to 9.4.46.v20220331

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-890:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #14

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/14/

> Upgrade Jetty to 9.4.46.v20220331
> -
>
> Key: MSITE-890
> URL: https://issues.apache.org/jira/browse/MSITE-890
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MSITE-891) Upgrade plugins in ITs

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-891:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #14

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/14/

> Upgrade plugins in ITs
> --
>
> Key: MSITE-891
> URL: https://issues.apache.org/jira/browse/MSITE-891
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>
> * Maven Checkstyle Plugin to 3.1.2
> * Maven PMD Plugin to 3.16.0
> * Maven JXR Plugin to 3.2.0
> * Maven Project Info Reports Plugin to 3.2.2



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MSITE-857) Jetty engine fails to resolve web.xml DTD behind corporate proxy

2022-04-16 Thread Hudson (Jira)
r.java:347)
> {code}
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-javadoc-plugin] michael-o opened a new pull request, #134: start

2022-04-16 Thread GitBox


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

   Following this checklist to help us incorporate your 
   contribution quickly and easily:
   
- [ ] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/browse/MJAVADOC) filed 
  for the change (usually before you start working on it).  Trivial 
changes like typos do not 
  require a JIRA issue.  Your pull request should address just this 
issue, without 
  pulling in other changes.
- [ ] Each commit in the pull request should have a meaningful subject line 
and body.
- [ ] Format the pull request title like `[MJAVADOC-XXX] - Fixes bug in 
ApproximateQuantiles`,
  where you replace `MJAVADOC-XXX` with the appropriate JIRA issue. 
Best practice
  is to use the JIRA issue title in the pull request title and in the 
first line of the 
  commit message.
- [ ] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
- [ ] Run `mvn clean verify -Prun-its` to make sure basic checks pass. A 
more thorough check will 
  be performed on your pull request automatically.
   
   If your pull request is about ~20 lines of code you don't need to sign an
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.pdf) if you are unsure
   please ask on the developers list.
   
   To make clear that you license your contribution under 
   the [Apache License Version 2.0, January 
2004](http://www.apache.org/licenses/LICENSE-2.0)
   you have to acknowledge this by using the following check-box.
   
- [ ] I hereby declare this contribution to be licenced under the [Apache 
License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
   
- [ ] In any other case, please file an [Apache Individual Contributor 
License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   


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

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

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



[GitHub] [maven-site-plugin] dependabot[bot] closed pull request #79: Bump mavenVersion from 3.0.5 to 3.8.5

2022-04-16 Thread GitBox


dependabot[bot] closed pull request #79: Bump mavenVersion from 3.0.5 to 3.8.5
URL: https://github.com/apache/maven-site-plugin/pull/79


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

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

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



[GitHub] [maven-site-plugin] dependabot[bot] commented on pull request #79: Bump mavenVersion from 3.0.5 to 3.8.5

2022-04-16 Thread GitBox


dependabot[bot] commented on PR #79:
URL: https://github.com/apache/maven-site-plugin/pull/79#issuecomment-1100754693

   Looks like these dependencies are no longer updatable, so this is no longer 
needed.


-- 
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] (MSITE-888) Upgrade to Maven 3.2.5

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-888:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #15

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/15/

> Upgrade to Maven 3.2.5
> --
>
> Key: MSITE-888
> URL: https://issues.apache.org/jira/browse/MSITE-888
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Affects Versions: 3.11.0
>Reporter: Michael Osipov
>Assignee: Tamás Cservenák
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Updated] (MSITE-874) Skip site:deploy instead of fail if distributionManagement.site.url is scm:*

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MSITE-874:
-
Fix Version/s: (was: 3.12.0)

> Skip site:deploy instead of fail if distributionManagement.site.url is scm:*
> 
>
> Key: MSITE-874
> URL: https://issues.apache.org/jira/browse/MSITE-874
> Project: Maven Site Plugin
>  Issue Type: Wish
>  Components: site:deploy
>Affects Versions: 3.9.1
>Reporter: Herve Boutemy
>Priority: Major
>
> when deploying to SCM, 
> [maven-scm-publish-plugin|https://maven.apache.org/plugins/maven-scm-publish-plugin/]
>  is the way to go
> instead of failing, site:deploy could simply skip deploy because it is known: 
> this would permit simpler config for 
> https://maven.apache.org/plugins/maven-scm-publish-plugin/examples/one-module-configuration.html
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-site-plugin] michael-o commented on pull request #74: Example of passing ITs for PR #73

2022-04-16 Thread GitBox


michael-o commented on PR #74:
URL: https://github.com/apache/maven-site-plugin/pull/74#issuecomment-1100753468

   There was a bug in Plexus Utils which is now fixed with: 
https://github.com/codehaus-plexus/plexus-utils/commit/b99f7c0e46d8b08d63afa4c2c7d274ebf5ebcc94


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

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

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



[GitHub] [maven-site-plugin] dependabot[bot] commented on pull request #78: Bump jettyVersion from 9.4.44.v20210927 to 11.0.9

2022-04-16 Thread GitBox


dependabot[bot] commented on PR #78:
URL: https://github.com/apache/maven-site-plugin/pull/78#issuecomment-1100752746

   OK, I won't notify you again about this release, but will get in touch when 
a new version is available. You can also ignore all major, minor, or patch 
releases for a dependency by adding an [`ignore` 
condition](https://docs.github.com/en/code-security/supply-chain-security/configuration-options-for-dependency-updates#ignore)
 with the desired `update_types` to your config file.
   
   If you change your mind, just re-open this PR and I'll resolve any conflicts 
on it.


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

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

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



[GitHub] [maven-site-plugin] michael-o closed pull request #78: Bump jettyVersion from 9.4.44.v20210927 to 11.0.9

2022-04-16 Thread GitBox


michael-o closed pull request #78: Bump jettyVersion from 9.4.44.v20210927 to 
11.0.9
URL: https://github.com/apache/maven-site-plugin/pull/78


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

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

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



[jira] [Closed] (MSITE-888) Upgrade to Maven 3.2.5

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-888.

Resolution: Fixed

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

> Upgrade to Maven 3.2.5
> --
>
> Key: MSITE-888
> URL: https://issues.apache.org/jira/browse/MSITE-888
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Affects Versions: 3.11.0
>Reporter: Michael Osipov
>Assignee: Tamás Cservenák
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-site-plugin] asfgit merged pull request #73: Upgrade to maven 3.2.5 + Java8

2022-04-16 Thread GitBox


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


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

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

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



[jira] [Closed] (MSITE-857) Jetty engine fails to resolve web.xml DTD behind corporate proxy

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-857.

Resolution: Fixed

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

> Jetty engine fails to resolve web.xml DTD behind corporate proxy
> 
>
> Key: MSITE-857
> URL: https://issues.apache.org/jira/browse/MSITE-857
> Project: Maven Site Plugin
>  Issue Type: Bug
>  Components: site:run
>Affects Versions: 3.9.0
> Environment: Windows 10, Intel 4 cores, 16GB RAM
>Reporter: Valentino Pinna
>Assignee: Michael Osipov
>Priority: Critical
> Fix For: 3.12.0
>
>
> To reproduce, ensure to be behind corporate proxy (with settings.xml 
> correctly configured) and execute:
> {code:java}
> mvn -X org.apache.maven.plugins:maven-site-plugin:3.9.0:run
> {code}
> Extract from log:
> {code:java}
> [DEBUG] resolveEntity(-//Sun Microsystems, Inc.//DTD Web Application 2.3//EN, 
> http://java.sun.com/dtd/web-app_2_3.dtd)
> [DEBUG] Can't exact match entity in redirect map, trying web-app_2_3.dtd
> [WARNING] Failed startup of context 
> o.e.j.w.WebAppContext@523a7801{/,file:///C:/Users/m027907/eclipse-workspace/vtl/target/site-stage/,UNAVAILABLE}
> java.net.ConnectException: Connection timed out: connect
> at java.net.PlainSocketImpl.connect0 (Native Method)
> at java.net.PlainSocketImpl.socketConnect (PlainSocketImpl.java:101)
> at java.net.AbstractPlainSocketImpl.doConnect 
> (AbstractPlainSocketImpl.java:399)
> at java.net.AbstractPlainSocketImpl.connectToAddress 
> (AbstractPlainSocketImpl.java:242)
> at java.net.AbstractPlainSocketImpl.connect 
> (AbstractPlainSocketImpl.java:224)
> at java.net.Socket.connect (Socket.java:609)
> at java.net.Socket.connect (Socket.java:558)
> at sun.net.NetworkClient.doConnect (NetworkClient.java:182)
> at sun.net.www.http.HttpClient.openServer (HttpClient.java:474)
> at sun.net.www.http.HttpClient.openServer (HttpClient.java:569)
> at sun.net.www.http.HttpClient. (HttpClient.java:242)
> at sun.net.www.http.HttpClient.New (HttpClient.java:341)
> at sun.net.www.http.HttpClient.New (HttpClient.java:362)
> at sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient 
> (HttpURLConnection.java:1248)
> at sun.net.www.protocol.http.HttpURLConnection.plainConnect0 
> (HttpURLConnection.java:1187)
> at sun.net.www.protocol.http.HttpURLConnection.plainConnect 
> (HttpURLConnection.java:1081)
> at sun.net.www.protocol.http.HttpURLConnection.connect 
> (HttpURLConnection.java:1015)
> at sun.net.www.protocol.http.HttpURLConnection.getInputStream0 
> (HttpURLConnection.java:1587)
> at sun.net.www.protocol.http.HttpURLConnection.getInputStream 
> (HttpURLConnection.java:1515)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLEntityManager.setupCurrentEntity 
> (XMLEntityManager.java:676)
> at com.sun.org.apache.xerces.internal.impl.XMLEntityManager.startEntity 
> (XMLEntityManager.java:1398)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLEntityManager.startDTDEntity 
> (XMLEntityManager.java:1364)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDTDScannerImpl.setInputSource 
> (XMLDTDScannerImpl.java:257)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$DTDDriver.dispatch
>  (XMLDocumentScannerImpl.java:1152)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$DTDDriver.next 
> (XMLDocumentScannerImpl.java:1040)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$PrologDriver.next
>  (XMLDocumentScannerImpl.java:943)
> at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next 
> (XMLDocumentScannerImpl.java:605)
> at com.sun.org.apache.xerces.internal.impl.XMLNSDocumentScannerImpl.next 
> (XMLNSDocumentScannerImpl.java:112)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument
>  (XMLDocumentFragmentScannerImpl.java:534)
> at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse 
> (XML11Configuration.java:888)
> at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse 
> (XML11Configuration.java:824)
> at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse 
> (XMLParser.java:141)
> at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse 
> (AbstractSAXParser.java:1216)
> at 
> com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse 
> (SAXParserImpl.java:635)
> at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.parse 
> (SAXParserImpl.java:324)
> at org.eclipse.jetty.xml.XmlParser.par

[jira] [Closed] (MSITE-891) Upgrade plugins in ITs

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-891.

Resolution: Fixed

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

> Upgrade plugins in ITs
> --
>
> Key: MSITE-891
> URL: https://issues.apache.org/jira/browse/MSITE-891
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>
> * Maven Checkstyle Plugin to 3.1.2
> * Maven PMD Plugin to 3.16.0
> * Maven JXR Plugin to 3.2.0
> * Maven Project Info Reports Plugin to 3.2.2



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-890) Upgrade Jetty to 9.4.46.v20220331

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-890.

Resolution: Fixed

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

> Upgrade Jetty to 9.4.46.v20220331
> -
>
> Key: MSITE-890
> URL: https://issues.apache.org/jira/browse/MSITE-890
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-surefire] Tibor17 commented on pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


Tibor17 commented on PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#issuecomment-1100747935

   @slawekjaranowski Can you pls help me making a review as well? Pls 
participate.
   @sbabcoc Feel free to work on the Parser, pls don't stop. We can introduce 
it in any PR or here if we are fast.


-- 
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] (MSITE-891) Upgrade plugins in ITs

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-891:


 Summary: Upgrade plugins in ITs
 Key: MSITE-891
 URL: https://issues.apache.org/jira/browse/MSITE-891
 Project: Maven Site Plugin
  Issue Type: Dependency upgrade
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0


* Maven Checkstyle Plugin to 3.1.2
* Maven PMD Plugin to 3.16.0
* Maven JXR Plugin to 3.2.0
* Maven Project Info Reports Plugin to 3.2.2



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-890) Upgrade Jetty to 9.4.46.v20220331

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-890:


 Summary: Upgrade Jetty to 9.4.46.v20220331
 Key: MSITE-890
 URL: https://issues.apache.org/jira/browse/MSITE-890
 Project: Maven Site Plugin
  Issue Type: Dependency upgrade
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Assigned] (MSITE-857) Jetty engine fails to resolve web.xml DTD behind corporate proxy

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov reassigned MSITE-857:


Assignee: Michael Osipov

> Jetty engine fails to resolve web.xml DTD behind corporate proxy
> 
>
> Key: MSITE-857
> URL: https://issues.apache.org/jira/browse/MSITE-857
> Project: Maven Site Plugin
>  Issue Type: Bug
>  Components: site:run
>Affects Versions: 3.9.0
> Environment: Windows 10, Intel 4 cores, 16GB RAM
>Reporter: Valentino Pinna
>Assignee: Michael Osipov
>Priority: Critical
>
> To reproduce, ensure to be behind corporate proxy (with settings.xml 
> correctly configured) and execute:
> {code:java}
> mvn -X org.apache.maven.plugins:maven-site-plugin:3.9.0:run
> {code}
> Extract from log:
> {code:java}
> [DEBUG] resolveEntity(-//Sun Microsystems, Inc.//DTD Web Application 2.3//EN, 
> http://java.sun.com/dtd/web-app_2_3.dtd)
> [DEBUG] Can't exact match entity in redirect map, trying web-app_2_3.dtd
> [WARNING] Failed startup of context 
> o.e.j.w.WebAppContext@523a7801{/,file:///C:/Users/m027907/eclipse-workspace/vtl/target/site-stage/,UNAVAILABLE}
> java.net.ConnectException: Connection timed out: connect
> at java.net.PlainSocketImpl.connect0 (Native Method)
> at java.net.PlainSocketImpl.socketConnect (PlainSocketImpl.java:101)
> at java.net.AbstractPlainSocketImpl.doConnect 
> (AbstractPlainSocketImpl.java:399)
> at java.net.AbstractPlainSocketImpl.connectToAddress 
> (AbstractPlainSocketImpl.java:242)
> at java.net.AbstractPlainSocketImpl.connect 
> (AbstractPlainSocketImpl.java:224)
> at java.net.Socket.connect (Socket.java:609)
> at java.net.Socket.connect (Socket.java:558)
> at sun.net.NetworkClient.doConnect (NetworkClient.java:182)
> at sun.net.www.http.HttpClient.openServer (HttpClient.java:474)
> at sun.net.www.http.HttpClient.openServer (HttpClient.java:569)
> at sun.net.www.http.HttpClient. (HttpClient.java:242)
> at sun.net.www.http.HttpClient.New (HttpClient.java:341)
> at sun.net.www.http.HttpClient.New (HttpClient.java:362)
> at sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient 
> (HttpURLConnection.java:1248)
> at sun.net.www.protocol.http.HttpURLConnection.plainConnect0 
> (HttpURLConnection.java:1187)
> at sun.net.www.protocol.http.HttpURLConnection.plainConnect 
> (HttpURLConnection.java:1081)
> at sun.net.www.protocol.http.HttpURLConnection.connect 
> (HttpURLConnection.java:1015)
> at sun.net.www.protocol.http.HttpURLConnection.getInputStream0 
> (HttpURLConnection.java:1587)
> at sun.net.www.protocol.http.HttpURLConnection.getInputStream 
> (HttpURLConnection.java:1515)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLEntityManager.setupCurrentEntity 
> (XMLEntityManager.java:676)
> at com.sun.org.apache.xerces.internal.impl.XMLEntityManager.startEntity 
> (XMLEntityManager.java:1398)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLEntityManager.startDTDEntity 
> (XMLEntityManager.java:1364)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDTDScannerImpl.setInputSource 
> (XMLDTDScannerImpl.java:257)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$DTDDriver.dispatch
>  (XMLDocumentScannerImpl.java:1152)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$DTDDriver.next 
> (XMLDocumentScannerImpl.java:1040)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$PrologDriver.next
>  (XMLDocumentScannerImpl.java:943)
> at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next 
> (XMLDocumentScannerImpl.java:605)
> at com.sun.org.apache.xerces.internal.impl.XMLNSDocumentScannerImpl.next 
> (XMLNSDocumentScannerImpl.java:112)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument
>  (XMLDocumentFragmentScannerImpl.java:534)
> at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse 
> (XML11Configuration.java:888)
> at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse 
> (XML11Configuration.java:824)
> at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse 
> (XMLParser.java:141)
> at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse 
> (AbstractSAXParser.java:1216)
> at 
> com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse 
> (SAXParserImpl.java:635)
> at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.parse 
> (SAXParserImpl.java:324)
> at org.eclipse.jetty.xml.XmlParser.parse (XmlParser.java:244)
> at org.eclipse.jetty.webapp.Descriptor.parse (Descriptor.java:53)
> at org.eclipse.jetty.webapp.WebDescriptor.parse (WebDescriptor.java:213)
> at o

[jira] [Updated] (MSITE-857) Jetty engine fails to resolve web.xml DTD behind corporate proxy

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MSITE-857:
-
Fix Version/s: 3.12.0

> Jetty engine fails to resolve web.xml DTD behind corporate proxy
> 
>
> Key: MSITE-857
> URL: https://issues.apache.org/jira/browse/MSITE-857
> Project: Maven Site Plugin
>  Issue Type: Bug
>  Components: site:run
>Affects Versions: 3.9.0
> Environment: Windows 10, Intel 4 cores, 16GB RAM
>Reporter: Valentino Pinna
>Assignee: Michael Osipov
>Priority: Critical
> Fix For: 3.12.0
>
>
> To reproduce, ensure to be behind corporate proxy (with settings.xml 
> correctly configured) and execute:
> {code:java}
> mvn -X org.apache.maven.plugins:maven-site-plugin:3.9.0:run
> {code}
> Extract from log:
> {code:java}
> [DEBUG] resolveEntity(-//Sun Microsystems, Inc.//DTD Web Application 2.3//EN, 
> http://java.sun.com/dtd/web-app_2_3.dtd)
> [DEBUG] Can't exact match entity in redirect map, trying web-app_2_3.dtd
> [WARNING] Failed startup of context 
> o.e.j.w.WebAppContext@523a7801{/,file:///C:/Users/m027907/eclipse-workspace/vtl/target/site-stage/,UNAVAILABLE}
> java.net.ConnectException: Connection timed out: connect
> at java.net.PlainSocketImpl.connect0 (Native Method)
> at java.net.PlainSocketImpl.socketConnect (PlainSocketImpl.java:101)
> at java.net.AbstractPlainSocketImpl.doConnect 
> (AbstractPlainSocketImpl.java:399)
> at java.net.AbstractPlainSocketImpl.connectToAddress 
> (AbstractPlainSocketImpl.java:242)
> at java.net.AbstractPlainSocketImpl.connect 
> (AbstractPlainSocketImpl.java:224)
> at java.net.Socket.connect (Socket.java:609)
> at java.net.Socket.connect (Socket.java:558)
> at sun.net.NetworkClient.doConnect (NetworkClient.java:182)
> at sun.net.www.http.HttpClient.openServer (HttpClient.java:474)
> at sun.net.www.http.HttpClient.openServer (HttpClient.java:569)
> at sun.net.www.http.HttpClient. (HttpClient.java:242)
> at sun.net.www.http.HttpClient.New (HttpClient.java:341)
> at sun.net.www.http.HttpClient.New (HttpClient.java:362)
> at sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient 
> (HttpURLConnection.java:1248)
> at sun.net.www.protocol.http.HttpURLConnection.plainConnect0 
> (HttpURLConnection.java:1187)
> at sun.net.www.protocol.http.HttpURLConnection.plainConnect 
> (HttpURLConnection.java:1081)
> at sun.net.www.protocol.http.HttpURLConnection.connect 
> (HttpURLConnection.java:1015)
> at sun.net.www.protocol.http.HttpURLConnection.getInputStream0 
> (HttpURLConnection.java:1587)
> at sun.net.www.protocol.http.HttpURLConnection.getInputStream 
> (HttpURLConnection.java:1515)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLEntityManager.setupCurrentEntity 
> (XMLEntityManager.java:676)
> at com.sun.org.apache.xerces.internal.impl.XMLEntityManager.startEntity 
> (XMLEntityManager.java:1398)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLEntityManager.startDTDEntity 
> (XMLEntityManager.java:1364)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDTDScannerImpl.setInputSource 
> (XMLDTDScannerImpl.java:257)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$DTDDriver.dispatch
>  (XMLDocumentScannerImpl.java:1152)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$DTDDriver.next 
> (XMLDocumentScannerImpl.java:1040)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$PrologDriver.next
>  (XMLDocumentScannerImpl.java:943)
> at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next 
> (XMLDocumentScannerImpl.java:605)
> at com.sun.org.apache.xerces.internal.impl.XMLNSDocumentScannerImpl.next 
> (XMLNSDocumentScannerImpl.java:112)
> at 
> com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument
>  (XMLDocumentFragmentScannerImpl.java:534)
> at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse 
> (XML11Configuration.java:888)
> at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse 
> (XML11Configuration.java:824)
> at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse 
> (XMLParser.java:141)
> at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse 
> (AbstractSAXParser.java:1216)
> at 
> com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse 
> (SAXParserImpl.java:635)
> at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.parse 
> (SAXParserImpl.java:324)
> at org.eclipse.jetty.xml.XmlParser.parse (XmlParser.java:244)
> at org.eclipse.jetty.webapp.Descriptor.parse (Descriptor.java:53)
> at org.eclipse.jetty.webapp.WebDescriptor.parse (WebDescripto

[jira] [Commented] (MSITE-889) Upgrade Plexus Utils to 3.3.1

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-889:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #13

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/13/

> Upgrade Plexus Utils to 3.3.1
> -
>
> Key: MSITE-889
> URL: https://issues.apache.org/jira/browse/MSITE-889
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Affects Versions: 3.11.0
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-889) Upgrade Plexus Utils to 3.3.1

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-889.

Resolution: Fixed

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

> Upgrade Plexus Utils to 3.3.1
> -
>
> Key: MSITE-889
> URL: https://issues.apache.org/jira/browse/MSITE-889
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Affects Versions: 3.11.0
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MSITE-886) Upgrade Maven Wagon to 3.5.1

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-886:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #12

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/12/

> Upgrade Maven Wagon to 3.5.1
> 
>
> Key: MSITE-886
> URL: https://issues.apache.org/jira/browse/MSITE-886
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MSITE-887) Deprecate templateFile parameter

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-887:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #12

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/12/

> Deprecate templateFile parameter
> 
>
> Key: MSITE-887
> URL: https://issues.apache.org/jira/browse/MSITE-887
> Project: Maven Site Plugin
>  Issue Type: Task
>  Components: doxia integration
>Affects Versions: 3.11.0
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>
> Doxia 2 will remove support for templates files and will only support skins 
> from JARs, no template files. We are going to deprecate this parameter for 
> removal.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MSITE-884) Drop/replace usage of Commons Lang 3

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-884:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #12

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/12/

> Drop/replace usage of Commons Lang 3
> 
>
> Key: MSITE-884
> URL: https://issues.apache.org/jira/browse/MSITE-884
> Project: Maven Site Plugin
>  Issue Type: Task
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MSITE-885) Upgrade Commons IO to 2.11.0

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MSITE-885:
--

Build failed in Jenkins: Maven » Maven TLP » maven-site-plugin » master #12

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/master/12/

> Upgrade Commons IO to 2.11.0
> 
>
> Key: MSITE-885
> URL: https://issues.apache.org/jira/browse/MSITE-885
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-889) Upgrade Plexus Utils to 3.3.1

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-889:


 Summary: Upgrade Plexus Utils to 3.3.1
 Key: MSITE-889
 URL: https://issues.apache.org/jira/browse/MSITE-889
 Project: Maven Site Plugin
  Issue Type: Dependency upgrade
Affects Versions: 3.11.0
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-888) Upgrade to Maven 3.2.5

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-888:


 Summary: Upgrade to Maven 3.2.5
 Key: MSITE-888
 URL: https://issues.apache.org/jira/browse/MSITE-888
 Project: Maven Site Plugin
  Issue Type: Dependency upgrade
Affects Versions: 3.11.0
Reporter: Michael Osipov
Assignee: Tamás Cservenák
 Fix For: 3.12.0






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-887) Deprecate templateFile parameter

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-887.

Resolution: Fixed

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

> Deprecate templateFile parameter
> 
>
> Key: MSITE-887
> URL: https://issues.apache.org/jira/browse/MSITE-887
> Project: Maven Site Plugin
>  Issue Type: Task
>  Components: doxia integration
>Affects Versions: 3.11.0
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>
> Doxia 2 will remove support for templates files and will only support skins 
> from JARs, no template files. We are going to deprecate this parameter for 
> removal.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-885) Upgrade Commons IO to 2.11.0

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-885.

Resolution: Fixed

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

> Upgrade Commons IO to 2.11.0
> 
>
> Key: MSITE-885
> URL: https://issues.apache.org/jira/browse/MSITE-885
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-884) Drop/replace usage of Commons Lang 3

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-884.

Resolution: Fixed

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

> Drop/replace usage of Commons Lang 3
> 
>
> Key: MSITE-884
> URL: https://issues.apache.org/jira/browse/MSITE-884
> Project: Maven Site Plugin
>  Issue Type: Task
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-886) Upgrade Maven Wagon to 3.5.1

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-886.

Resolution: Fixed

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

> Upgrade Maven Wagon to 3.5.1
> 
>
> Key: MSITE-886
> URL: https://issues.apache.org/jira/browse/MSITE-886
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-887) Deprecate templateFile parameter

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-887:


 Summary: Deprecate templateFile parameter
 Key: MSITE-887
 URL: https://issues.apache.org/jira/browse/MSITE-887
 Project: Maven Site Plugin
  Issue Type: Task
  Components: doxia integration
Affects Versions: 3.11.0
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0


Doxia 2 will remove support for templates files and will only support skins 
from JARs, no template files. We are going to deprecate this parameter for 
removal.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-surefire] sbabcoc commented on pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


sbabcoc commented on PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#issuecomment-1100729707

   Resolves #517 


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

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

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



[GitHub] [maven-surefire] sbabcoc commented on pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


sbabcoc commented on PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#issuecomment-1100727903

   @Tibor17: The revisions in this PR have been squashed to a single commit. 
Thanks for all your efforts on this project!


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

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

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



[jira] [Closed] (MSITE-829) Upgrade Jetty to 9.4.x

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-829.


> Upgrade Jetty to 9.4.x
> --
>
> Key: MSITE-829
> URL: https://issues.apache.org/jira/browse/MSITE-829
> Project: Maven Site Plugin
>  Issue Type: Dependency upgrade
>Reporter: Olivier Lamy
>Assignee: Olivier Lamy
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MSITE-828) Upgrade to Java 8

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MSITE-828.


> Upgrade to Java 8
> -
>
> Key: MSITE-828
> URL: https://issues.apache.org/jira/browse/MSITE-828
> Project: Maven Site Plugin
>  Issue Type: Task
>Reporter: Olivier Lamy
>Assignee: Olivier Lamy
>Priority: Major
> Fix For: 3.12.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-886) Upgrade Maven Wagon to 3.5.1

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-886:


 Summary: Upgrade Maven Wagon to 3.5.1
 Key: MSITE-886
 URL: https://issues.apache.org/jira/browse/MSITE-886
 Project: Maven Site Plugin
  Issue Type: Dependency upgrade
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-885) Upgrade Commons IO to 2.11.0

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-885:


 Summary: Upgrade Commons IO to 2.11.0
 Key: MSITE-885
 URL: https://issues.apache.org/jira/browse/MSITE-885
 Project: Maven Site Plugin
  Issue Type: Dependency upgrade
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (MSITE-884) Drop/replace usage of Commons Lang 3

2022-04-16 Thread Michael Osipov (Jira)
Michael Osipov created MSITE-884:


 Summary: Drop/replace usage of Commons Lang 3
 Key: MSITE-884
 URL: https://issues.apache.org/jira/browse/MSITE-884
 Project: Maven Site Plugin
  Issue Type: Task
Reporter: Michael Osipov
Assignee: Michael Osipov
 Fix For: 3.12.0






--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7454) Include resolver-transport-http in Maven

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7454:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven » master #32

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/master/32/

> Include resolver-transport-http in Maven
> 
>
> Key: MNG-7454
> URL: https://issues.apache.org/jira/browse/MNG-7454
> Project: Maven
>  Issue Type: New Feature
>  Components: Dependencies
>Reporter: Tamás Cservenák
>Assignee: Tamás Cservenák
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> We should include maven-resolver-transport-http (along with existing 
> maven-resolver-transport-wagon) but retain Wagon as default transport, while 
> offer ability for users to use http transport to utilize new resolver 
> features like "smart checksums".
> On positive side, this will finally get rid of shaded httpClient and 
> wagon-http-shaded as well.
> To switch transport use {{-Dmaven.resolver.transport}} with following values:
>  * {{wagon}} - to use wagon (is default as well)
>  * {{native}} - to use resolver own highly optimized HTTP and FILE transport 
> (supports "smart checksums")
>  * {{auto}} - to rely on resolver own discovery, or if another transport 
> extension used
> The selected transport can be seen in debug (-X) output on line `[DEBUG] 
> Using transporter XXX...`



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7454) Include resolver-transport-http in Maven

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7454:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven » maven-3.9.x #18

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/maven-3.9.x/18/

> Include resolver-transport-http in Maven
> 
>
> Key: MNG-7454
> URL: https://issues.apache.org/jira/browse/MNG-7454
> Project: Maven
>  Issue Type: New Feature
>  Components: Dependencies
>Reporter: Tamás Cservenák
>Assignee: Tamás Cservenák
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> We should include maven-resolver-transport-http (along with existing 
> maven-resolver-transport-wagon) but retain Wagon as default transport, while 
> offer ability for users to use http transport to utilize new resolver 
> features like "smart checksums".
> On positive side, this will finally get rid of shaded httpClient and 
> wagon-http-shaded as well.
> To switch transport use {{-Dmaven.resolver.transport}} with following values:
>  * {{wagon}} - to use wagon (is default as well)
>  * {{native}} - to use resolver own highly optimized HTTP and FILE transport 
> (supports "smart checksums")
>  * {{auto}} - to rely on resolver own discovery, or if another transport 
> extension used
> The selected transport can be seen in debug (-X) output on line `[DEBUG] 
> Using transporter XXX...`



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-700) Plugin duplicates classes in Java 8 all-classes lists

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-700:
-

My fix does not work. Windows fails.

> Plugin duplicates classes in Java 8 all-classes  lists
> --
>
> Key: MJAVADOC-700
> URL: https://issues.apache.org/jira/browse/MJAVADOC-700
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>Affects Versions: 3.3.0, 3.3.1
>Reporter: Rob Spoor
>Priority: Major
>
> Versions 3.3.0 and 3.3.1 of the Javadoc Plugin create allclasses-frame.html 
> and allclasses-noframe.html files that duplicate all classes, interfaces, etc.
> To reproduce:
> * Clone https://github.com/robtimus/data-url (a simple and small project that 
> can be used to showcase the issue)
> * Run {{mvn javadoc:javadoc}}, which uses version 3.2.0
> * Check target/site/apidocs/allclasses-frame.html - no duplicates
> * Run {{mvn javadoc:javadoc -Dversion.plugin.javadoc=3.3.0}} to build the 
> same with version 3.3.0
> * Check target/site/apidocs/allclasses-frame.html - all classes are duplicated
> The same issue occurs with {{javadoc:aggregate}}, possibly others as well.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MPOM-316) Upgrade ASF Parent to 26

2022-04-16 Thread Slawomir Jaranowski (Jira)


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

Slawomir Jaranowski closed MPOM-316.

Resolution: Fixed

> Upgrade ASF Parent to 26
> 
>
> Key: MPOM-316
> URL: https://issues.apache.org/jira/browse/MPOM-316
> Project: Maven POMs
>  Issue Type: Dependency upgrade
>  Components: maven
>Reporter: Slawomir Jaranowski
>Assignee: Slawomir Jaranowski
>Priority: Major
> Fix For: MAVEN-36
>
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-parent] slawekjaranowski merged pull request #62: Bump apache from 25 to 26

2022-04-16 Thread GitBox


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


-- 
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] (MNG-7454) Include resolver-transport-http in Maven

2022-04-16 Thread Jira


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

Tamás Cservenák updated MNG-7454:
-
Description: 
We should include maven-resolver-transport-http (along with existing 
maven-resolver-transport-wagon) but retain Wagon as default transport, while 
offer ability for users to use http transport to utilize new resolver features 
like "smart checksums".
On positive side, this will finally get rid of shaded httpClient and 
wagon-http-shaded as well.

To switch transport use {{-Dmaven.resolver.transport}} with following values:
 * {{wagon}} - to use wagon (is default as well)
 * {{native}} - to use resolver own highly optimized HTTP and FILE transport 
(supports "smart checksums")
 * {{auto}} - to rely on resolver own discovery, or if another transport 
extension used

The selected transport can be seen in debug (-X) output on line `[DEBUG] Using 
transporter XXX...`

  was:
We should include maven-resolver-transport-http (along with existing 
maven-resolver-transport-wagon) but retain Wagon as default transport, while 
offer ability for users to use http transport to utilize new resolver features 
like "smart checksums".
On positive side, this will finally get rid of shaded httpClient and 
wagon-http-shaded as well.


> Include resolver-transport-http in Maven
> 
>
> Key: MNG-7454
> URL: https://issues.apache.org/jira/browse/MNG-7454
> Project: Maven
>  Issue Type: New Feature
>  Components: Dependencies
>Reporter: Tamás Cservenák
>Assignee: Tamás Cservenák
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> We should include maven-resolver-transport-http (along with existing 
> maven-resolver-transport-wagon) but retain Wagon as default transport, while 
> offer ability for users to use http transport to utilize new resolver 
> features like "smart checksums".
> On positive side, this will finally get rid of shaded httpClient and 
> wagon-http-shaded as well.
> To switch transport use {{-Dmaven.resolver.transport}} with following values:
>  * {{wagon}} - to use wagon (is default as well)
>  * {{native}} - to use resolver own highly optimized HTTP and FILE transport 
> (supports "smart checksums")
>  * {{auto}} - to rely on resolver own discovery, or if another transport 
> extension used
> The selected transport can be seen in debug (-X) output on line `[DEBUG] 
> Using transporter XXX...`



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Comment Edited] (MJAVADOC-700) Plugin duplicates classes in Java 8 all-classes lists

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov edited comment on MJAVADOC-700 at 4/16/22 4:24 PM:
--

I can confirm this:
{noformat}
commit 55982df63f8de5bfbb48aee353bc29b35b1c6a2c
Author: rfscholte 
Date:   Mon May 17 22:50:04 2021 +0200

[MJAVADOC-584] excludePackageNames is not working as documented anymore
{noformat}

[~rfscholte], WDYT?

I am currently evaluating

{noformat}
diff --git a/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java 
b/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
index 0e5eda01..3f9cb2b5 100644
--- a/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
+++ b/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
@@ -379,7 +379,7 @@ public class JavadocUtil

 if ( !excluded )
 {
-files.add( file.replace( '\\', '/' ) );
+files.add( file.replace( '\\', File.separatorChar ) );
 }
 }
{noformat}


was (Author: michael-o):
I can confirm this:
{noformat}
commit 55982df63f8de5bfbb48aee353bc29b35b1c6a2c
Author: rfscholte 
Date:   Mon May 17 22:50:04 2021 +0200

[MJAVADOC-584] excludePackageNames is not working as documented anymore
{noformat}

[~rfscholte], WDYT?

I am currently evaluating

{noformat}
diff --git a/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java 
b/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
index 0e5eda01..3f9cb2b5 100644
--- a/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
+++ b/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
@@ -379,7 +379,7 @@ public class JavadocUtil

 if ( !excluded )
 {
-files.add( file.replace( '\\', '/' ) );
+files.add( file.replace( '\\', File.separatorChar ) );^M
 }
 }
{noformat}

> Plugin duplicates classes in Java 8 all-classes  lists
> --
>
> Key: MJAVADOC-700
> URL: https://issues.apache.org/jira/browse/MJAVADOC-700
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>Affects Versions: 3.3.0, 3.3.1
>Reporter: Rob Spoor
>Priority: Major
>
> Versions 3.3.0 and 3.3.1 of the Javadoc Plugin create allclasses-frame.html 
> and allclasses-noframe.html files that duplicate all classes, interfaces, etc.
> To reproduce:
> * Clone https://github.com/robtimus/data-url (a simple and small project that 
> can be used to showcase the issue)
> * Run {{mvn javadoc:javadoc}}, which uses version 3.2.0
> * Check target/site/apidocs/allclasses-frame.html - no duplicates
> * Run {{mvn javadoc:javadoc -Dversion.plugin.javadoc=3.3.0}} to build the 
> same with version 3.3.0
> * Check target/site/apidocs/allclasses-frame.html - all classes are duplicated
> The same issue occurs with {{javadoc:aggregate}}, possibly others as well.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-700) Plugin duplicates classes in Java 8 all-classes lists

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-700:
-

I can confirm this:
{noformat}
commit 55982df63f8de5bfbb48aee353bc29b35b1c6a2c
Author: rfscholte 
Date:   Mon May 17 22:50:04 2021 +0200

[MJAVADOC-584] excludePackageNames is not working as documented anymore
{noformat}

[~rfscholte], WDYT?

I am currently evaluating

{noformat}
diff --git a/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java 
b/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
index 0e5eda01..3f9cb2b5 100644
--- a/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
+++ b/src/main/java/org/apache/maven/plugins/javadoc/JavadocUtil.java
@@ -379,7 +379,7 @@ public class JavadocUtil

 if ( !excluded )
 {
-files.add( file.replace( '\\', '/' ) );
+files.add( file.replace( '\\', File.separatorChar ) );^M
 }
 }
{noformat}

> Plugin duplicates classes in Java 8 all-classes  lists
> --
>
> Key: MJAVADOC-700
> URL: https://issues.apache.org/jira/browse/MJAVADOC-700
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>Affects Versions: 3.3.0, 3.3.1
>Reporter: Rob Spoor
>Priority: Major
>
> Versions 3.3.0 and 3.3.1 of the Javadoc Plugin create allclasses-frame.html 
> and allclasses-noframe.html files that duplicate all classes, interfaces, etc.
> To reproduce:
> * Clone https://github.com/robtimus/data-url (a simple and small project that 
> can be used to showcase the issue)
> * Run {{mvn javadoc:javadoc}}, which uses version 3.2.0
> * Check target/site/apidocs/allclasses-frame.html - no duplicates
> * Run {{mvn javadoc:javadoc -Dversion.plugin.javadoc=3.3.0}} to build the 
> same with version 3.3.0
> * Check target/site/apidocs/allclasses-frame.html - all classes are duplicated
> The same issue occurs with {{javadoc:aggregate}}, possibly others as well.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven] cstamas merged pull request #720: [MNG-7454] Include resolver-transport-http in Maven

2022-04-16 Thread GitBox


cstamas merged PR #720:
URL: https://github.com/apache/maven/pull/720


-- 
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] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-696:
-

I understood that. Whatever is  {{@Parameter}} is mostly private and linking to 
private is going into nirvana.

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Olivier Cailloux (Jira)


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

Olivier Cailloux commented on MJAVADOC-696:
---

I see (and thanks for the update, anyway). I guess that the title of this issue 
was a poor choice of mine, then. What I meant with this issue really was that 
documentation is currently missing about which javadoc links are used when 
using detectJavaApiLink.

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-696:
-

I have only updated the source code, generation will happen at release only. 
Removing them is the only way to solve this because Javadoc does not show 
anything below {{protected}}, this making the links dead. Where is the point in 
having them if they are not accessible?

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Olivier Cailloux (Jira)


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

Olivier Cailloux commented on MJAVADOC-696:
---

It does not seem to me that the proposed fix solves the stated problem, in 
particular, the first part of the description. The documentation of 
detectJavaApiLink still mentions: See Javadoc for the default values, linking 
to 
https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS.
 This anchor still does not exist.

Please note that the intent of this request is for the reader to be able to 
effectively find which values are used. Solving the dead links by removing the 
links is not really a solution…

Please tell me if I missed something.

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MJAVADOC-696:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven-javadoc-plugin » master 
#15

See 
https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven-javadoc-plugin/job/master/15/

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-685) Stylesheet "maven" does look ugly

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-685:
-

Not only the CSS would go, but also the {{stylesheet}} parameter.

> Stylesheet "maven" does look ugly
> -
>
> Key: MJAVADOC-685
> URL: https://issues.apache.org/jira/browse/MJAVADOC-685
> Project: Maven Javadoc Plugin
>  Issue Type: Improvement
>Affects Versions: 3.3.0
>Reporter: Konrad Windszus
>Priority: Major
> Attachments: Screenshot 2021-07-25 at 10.55.44.png
>
>
> When configuring m-javadoc-p 3.3.0 with stylesheet {{maven}} (according to 
> https://maven.apache.org/plugins/maven-javadoc-plugin/examples/stylesheet-configuration.html)
>  it only loads the simple CSS from 
> https://github.com/apache/maven-javadoc-plugin/blob/master/src/main/resources/org/apache/maven/plugins/javadoc/css/stylesheet.css
>  which has only very few styling options and is not suited for styling the 
> markup emitted by javadoc correctly. This affects both JDK8 and JDK11.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MJAVADOC-696.
---
Resolution: Fixed

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

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7438) add execution id to "Configuring mojo xxx with basic configurator" debug message

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7438:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven » PR-565 #14

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-565/14/

> add execution id to "Configuring mojo xxx with basic configurator" debug 
> message
> 
>
> Key: MNG-7438
> URL: https://issues.apache.org/jira/browse/MNG-7438
> Project: Maven
>  Issue Type: Improvement
>  Components: Core
>Affects Versions: 3.8.5
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> currently, when running goals/mojos, we have in debug logs
> {noformat}
> [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ 
> org.example.apache.maven.execution ---
> [DEBUG] Configuring mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator -->
> [DEBUG]   (s) additionalClasspathElements = []
> [DEBUG]   (s) basedir = /home/herve/tmp/org.example.apache.maven.execution
> {noformat}
> on the "{{Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator}}" line, it completely hides the fact that configuration is 
> valid for an execution id, that has a default value when nobody defined any 
> (see https://maven.apache.org/guides/mini/guide-default-execution-ids.html )
> Just adding "{{execution 'default-test'}}" would make things more clear so 
> people learn about execution ids
> {noformat}
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' execution 
> 'default-test' with basic configurator -->
> {noformat}
> h2. final debug messages
> {noformat}
> [DEBUG] Loading mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo execution 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' 
> with basic configurator -->
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7438) add execution id to "Configuring mojo xxx with basic configurator" debug message

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7438:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven » PR-691 #9

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-691/9/

> add execution id to "Configuring mojo xxx with basic configurator" debug 
> message
> 
>
> Key: MNG-7438
> URL: https://issues.apache.org/jira/browse/MNG-7438
> Project: Maven
>  Issue Type: Improvement
>  Components: Core
>Affects Versions: 3.8.5
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> currently, when running goals/mojos, we have in debug logs
> {noformat}
> [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ 
> org.example.apache.maven.execution ---
> [DEBUG] Configuring mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator -->
> [DEBUG]   (s) additionalClasspathElements = []
> [DEBUG]   (s) basedir = /home/herve/tmp/org.example.apache.maven.execution
> {noformat}
> on the "{{Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator}}" line, it completely hides the fact that configuration is 
> valid for an execution id, that has a default value when nobody defined any 
> (see https://maven.apache.org/guides/mini/guide-default-execution-ids.html )
> Just adding "{{execution 'default-test'}}" would make things more clear so 
> people learn about execution ids
> {noformat}
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' execution 
> 'default-test' with basic configurator -->
> {noformat}
> h2. final debug messages
> {noformat}
> [DEBUG] Loading mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo execution 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' 
> with basic configurator -->
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7438) add execution id to "Configuring mojo xxx with basic configurator" debug message

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7438:
-

Build unstable in Jenkins: Maven » Maven TLP » maven » PR-509 #14

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-509/14/

> add execution id to "Configuring mojo xxx with basic configurator" debug 
> message
> 
>
> Key: MNG-7438
> URL: https://issues.apache.org/jira/browse/MNG-7438
> Project: Maven
>  Issue Type: Improvement
>  Components: Core
>Affects Versions: 3.8.5
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> currently, when running goals/mojos, we have in debug logs
> {noformat}
> [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ 
> org.example.apache.maven.execution ---
> [DEBUG] Configuring mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator -->
> [DEBUG]   (s) additionalClasspathElements = []
> [DEBUG]   (s) basedir = /home/herve/tmp/org.example.apache.maven.execution
> {noformat}
> on the "{{Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator}}" line, it completely hides the fact that configuration is 
> valid for an execution id, that has a default value when nobody defined any 
> (see https://maven.apache.org/guides/mini/guide-default-execution-ids.html )
> Just adding "{{execution 'default-test'}}" would make things more clear so 
> people learn about execution ids
> {noformat}
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' execution 
> 'default-test' with basic configurator -->
> {noformat}
> h2. final debug messages
> {noformat}
> [DEBUG] Loading mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo execution 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' 
> with basic configurator -->
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7438) add execution id to "Configuring mojo xxx with basic configurator" debug message

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7438:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven » PR-600 #14

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-600/14/

> add execution id to "Configuring mojo xxx with basic configurator" debug 
> message
> 
>
> Key: MNG-7438
> URL: https://issues.apache.org/jira/browse/MNG-7438
> Project: Maven
>  Issue Type: Improvement
>  Components: Core
>Affects Versions: 3.8.5
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> currently, when running goals/mojos, we have in debug logs
> {noformat}
> [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ 
> org.example.apache.maven.execution ---
> [DEBUG] Configuring mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator -->
> [DEBUG]   (s) additionalClasspathElements = []
> [DEBUG]   (s) basedir = /home/herve/tmp/org.example.apache.maven.execution
> {noformat}
> on the "{{Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator}}" line, it completely hides the fact that configuration is 
> valid for an execution id, that has a default value when nobody defined any 
> (see https://maven.apache.org/guides/mini/guide-default-execution-ids.html )
> Just adding "{{execution 'default-test'}}" would make things more clear so 
> people learn about execution ids
> {noformat}
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' execution 
> 'default-test' with basic configurator -->
> {noformat}
> h2. final debug messages
> {noformat}
> [DEBUG] Loading mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo execution 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' 
> with basic configurator -->
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7438) add execution id to "Configuring mojo xxx with basic configurator" debug message

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7438:
-

Build succeeded in Jenkins: Maven » Maven TLP » maven » PR-721 #2

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-721/2/

> add execution id to "Configuring mojo xxx with basic configurator" debug 
> message
> 
>
> Key: MNG-7438
> URL: https://issues.apache.org/jira/browse/MNG-7438
> Project: Maven
>  Issue Type: Improvement
>  Components: Core
>Affects Versions: 3.8.5
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> currently, when running goals/mojos, we have in debug logs
> {noformat}
> [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ 
> org.example.apache.maven.execution ---
> [DEBUG] Configuring mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator -->
> [DEBUG]   (s) additionalClasspathElements = []
> [DEBUG]   (s) basedir = /home/herve/tmp/org.example.apache.maven.execution
> {noformat}
> on the "{{Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator}}" line, it completely hides the fact that configuration is 
> valid for an execution id, that has a default value when nobody defined any 
> (see https://maven.apache.org/guides/mini/guide-default-execution-ids.html )
> Just adding "{{execution 'default-test'}}" would make things more clear so 
> people learn about execution ids
> {noformat}
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' execution 
> 'default-test' with basic configurator -->
> {noformat}
> h2. final debug messages
> {noformat}
> [DEBUG] Loading mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo execution 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' 
> with basic configurator -->
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MNG-7438) add execution id to "Configuring mojo xxx with basic configurator" debug message

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-7438:
-

Build unstable in Jenkins: Maven » Maven TLP » maven » PR-394 #15

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-394/15/

> add execution id to "Configuring mojo xxx with basic configurator" debug 
> message
> 
>
> Key: MNG-7438
> URL: https://issues.apache.org/jira/browse/MNG-7438
> Project: Maven
>  Issue Type: Improvement
>  Components: Core
>Affects Versions: 3.8.5
>Reporter: Herve Boutemy
>Assignee: Herve Boutemy
>Priority: Major
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>
> currently, when running goals/mojos, we have in debug logs
> {noformat}
> [INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ 
> org.example.apache.maven.execution ---
> [DEBUG] Configuring mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator -->
> [DEBUG]   (s) additionalClasspathElements = []
> [DEBUG]   (s) basedir = /home/herve/tmp/org.example.apache.maven.execution
> {noformat}
> on the "{{Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' with basic 
> configurator}}" line, it completely hides the fact that configuration is 
> valid for an execution id, that has a default value when nobody defined any 
> (see https://maven.apache.org/guides/mini/guide-default-execution-ids.html )
> Just adding "{{execution 'default-test'}}" would make things more clear so 
> people learn about execution ids
> {noformat}
> [DEBUG] Configuring mojo 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test' execution 
> 'default-test' with basic configurator -->
> {noformat}
> h2. final debug messages
> {noformat}
> [DEBUG] Loading mojo 
> org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin 
> realm 
> ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, 
> parent: sun.misc.Launcher$AppClassLoader@7852e922]
> [DEBUG] Configuring mojo execution 
> 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' 
> with basic configurator -->
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Updated] (MJAVADOC-696) Invalid anchors in Javadoc and plugin mojo

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MJAVADOC-696:

Summary: Invalid anchors in Javadoc and plugin mojo  (was: Default values 
for detectJavaApiLink)

> Invalid anchors in Javadoc and plugin mojo
> --
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven] slawekjaranowski commented on pull request #722: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


slawekjaranowski commented on PR #722:
URL: https://github.com/apache/maven/pull/722#issuecomment-1100660353

   PR description updated.


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

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

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



[GitHub] [maven-integration-testing] michael-o commented on a diff in pull request #151: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


michael-o commented on code in PR #151:
URL: 
https://github.com/apache/maven-integration-testing/pull/151#discussion_r851625439


##
core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/DeprecatedConfigMojo.java:
##
@@ -47,6 +48,18 @@
 @Parameter( defaultValue = "${basedir}", readonly = true )
 private File basedir;
 
+/**
+ * @deprecated bean read only
+ */
+@Parameter(  defaultValue = "${project.artifact}", readonly = true )
+private Artifact deprecatedBeanReadOnly;
+
+/**
+ * @deprecated bean
+ */
+@Parameter(  defaultValue = "${project.artifact}" )
+private Artifact deprecatedBean;

Review Comment:
   I see, there is one space too much



-- 
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] (MJAVADOC-696) Default values for detectJavaApiLink

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov reassigned MJAVADOC-696:
---

Assignee: Michael Osipov

> Default values for detectJavaApiLink
> 
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Updated] (MJAVADOC-696) Default values for detectJavaApiLink

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MJAVADOC-696:

Fix Version/s: next-release

> Default values for detectJavaApiLink
> 
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-696) Default values for detectJavaApiLink

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-696:
-

193ff34935bd9ebb60613dfc37c42e33b8017169 has removed that property/anchor.

> Default values for detectJavaApiLink
> 
>
> Key: MJAVADOC-696
> URL: https://issues.apache.org/jira/browse/MJAVADOC-696
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Priority: Minor
>
> The 
> [documentation|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink]
>  of {{detectJavaApiLink}} mentions: {{See Javadoc for the default values}}, 
> linking to 
> {{https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html#DEFAULT_JAVA_API_LINKS}}.
>  This anchor does not exist currently.
> Relatedly, several references in the [AbstractJavadocMojo javadoc 
> page|https://maven.apache.org/plugins/maven-javadoc-plugin/apidocs/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.html]
>  do not exist, for example, the references called {{detectJavaApiLink}} or 
> {{javaApiLinks}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-integration-testing] slawekjaranowski commented on a diff in pull request #151: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


slawekjaranowski commented on code in PR #151:
URL: 
https://github.com/apache/maven-integration-testing/pull/151#discussion_r851624698


##
core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/DeprecatedConfigMojo.java:
##
@@ -47,6 +48,18 @@
 @Parameter( defaultValue = "${basedir}", readonly = true )
 private File basedir;
 
+/**
+ * @deprecated bean read only
+ */
+@Parameter(  defaultValue = "${project.artifact}", readonly = true )
+private Artifact deprecatedBeanReadOnly;
+
+/**
+ * @deprecated bean
+ */
+@Parameter(  defaultValue = "${project.artifact}" )
+private Artifact deprecatedBean;

Review Comment:
   It is corner case when internal value, project object is used as property 
without read only flags.
   Such parameter should not generated warning.



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

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

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



[GitHub] [maven-integration-testing] slawekjaranowski commented on a diff in pull request #151: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


slawekjaranowski commented on code in PR #151:
URL: 
https://github.com/apache/maven-integration-testing/pull/151#discussion_r851624623


##
core-it-suite/src/test/java/org/apache/maven/it/MavenITmng5222MojoDeprecatedParamsTest.java:
##
@@ -35,7 +35,7 @@
 {
 public MavenITmng5222MojoDeprecatedParamsTest()
 {
-super( "[3.9.0,)" );
+super( "[4.0.0-alpha-1,)" );

Review Comment:
   When this change will be merged with current fix to Maven master test on 3.9 
will fail. 
   I will restore it with together cherry pick/port change to 3.9.x branch of 
Maven core.



-- 
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] (MJAVADOC-713) Skipping Javadoc reportset leaves empty Javadoc link in site

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MJAVADOC-713:

Fix Version/s: waiting-for-feedback

> Skipping Javadoc reportset leaves empty Javadoc link in site
> 
>
> Key: MJAVADOC-713
> URL: https://issues.apache.org/jira/browse/MJAVADOC-713
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Joshua
>Priority: Major
> Fix For: waiting-for-feedback
>
> Attachments: image-2022-04-15-17-47-49-771.png
>
>
> I am trying to skip a reportset that is inherited from a parent pom
> This is what the report plugin looks like in the parent pom
>             
>                 org.apache.maven.plugins
>                 maven-javadoc-plugin
>                 
>                     
>                         javadoc-no-fork
>                         
>                             javadoc-no-fork
>                         
>                     
>                     
>                         test-javadoc-no-fork
>                         
>                             test-javadoc-no-fork
>                         
>                         
>                             package
>                         
>                     
>                 
>             
> I don't want the test-javadoc-no-fork report in my project
>  
> If I do this in my pom
> 
>     org.apache.maven.plugins
>     maven-javadoc-plugin
>     
>              
>                      test-javadoc-no-fork
>                       
>                            true
>                       
>              
>     
> 
>  
> It will skip the Javadoc generation but still adds a link to site and when 
> you click on the link you get a 404 since the test Javadoc wasn't inherited.
> !image-2022-04-15-17-47-49-771.png!
> I logged this as a defect because I would think skip would prevent the 
> Javadoc generation and NOT add a link to the site for a report set that 
> wasn't generated.
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (MJAVADOC-713) Skipping Javadoc reportset leaves empty Javadoc link in site

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MJAVADOC-713:
-

Provide a sample project.

> Skipping Javadoc reportset leaves empty Javadoc link in site
> 
>
> Key: MJAVADOC-713
> URL: https://issues.apache.org/jira/browse/MJAVADOC-713
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>Affects Versions: 3.3.0
>Reporter: Joshua
>Priority: Major
> Attachments: image-2022-04-15-17-47-49-771.png
>
>
> I am trying to skip a reportset that is inherited from a parent pom
> This is what the report plugin looks like in the parent pom
>             
>                 org.apache.maven.plugins
>                 maven-javadoc-plugin
>                 
>                     
>                         javadoc-no-fork
>                         
>                             javadoc-no-fork
>                         
>                     
>                     
>                         test-javadoc-no-fork
>                         
>                             test-javadoc-no-fork
>                         
>                         
>                             package
>                         
>                     
>                 
>             
> I don't want the test-javadoc-no-fork report in my project
>  
> If I do this in my pom
> 
>     org.apache.maven.plugins
>     maven-javadoc-plugin
>     
>              
>                      test-javadoc-no-fork
>                       
>                            true
>                       
>              
>     
> 
>  
> It will skip the Javadoc generation but still adds a link to site and when 
> you click on the link you get a 404 since the test Javadoc wasn't inherited.
> !image-2022-04-15-17-47-49-771.png!
> I logged this as a defect because I would think skip would prevent the 
> Javadoc generation and NOT add a link to the site for a report set that 
> wasn't generated.
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MJAVADOC-669) Generated javadoc JARs contain jQuery and other MIT-licenced works without reproducing a copy of the MIT licence, same for GPL-licenced works

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MJAVADOC-669.
---
Resolution: Not A Bug

Basically, this has been partially addressed by the OpenJDK team, there is 
nothing we can do about. Closing.

> Generated javadoc JARs contain jQuery and other MIT-licenced works without 
> reproducing a copy of the MIT licence, same for GPL-licenced works
> -
>
> Key: MJAVADOC-669
> URL: https://issues.apache.org/jira/browse/MJAVADOC-669
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.2.0
>Reporter: Thorsten Glaser
>Priority: Blocker
>  Labels: legal, licensing
>
> A javadoc JAR generated by the Maven Javadoc Plugin 3.2.0 contains multiple 
> components under the MIT licence:
>  * jQuery 3.5.1
>  ** {{jquery/external/jquery/jquery.js}}
>  ** {{jquery/jquery-3.5.1.js}} (duplicate of the above, blowing up the PKZIP 
> archive size of the JAR, why is it included like this?)
>  * JSZip 3.2.1
>  ** {{jquery/jszip/dist/jszip.js}}
>  ** {{jquery/jszip-utils/dist/jszip-utils-ie.js}}
>  ** {{jquery/jszip-utils/dist/jszip-utils.js}}
>  * jQuery UI 1.12.1
>  ** {{jquery/jquery-ui.css}}
>  ** {{jquery/jquery-ui.js}}
>  ** {{jquery/jquery-ui.structure.css}}
>  * and their respective minified versions
> It also contains {{script.js}} and {{search.js}} which are 
> GPLv2-with-Classpath-exception-licenced and refer to “as provided by Oracle 
> in the LICENSE file that accompanied this code” but no such file accompanies 
> said code.
> There are also multiple static {{resources}} and {{jquery/images}} whose 
> licence is not documented.
> The MIT licence specifically *requires* that “The […] copyright notice and 
> this permission notice [the licence body] shall be included in all copies or 
> substantial portions of the Software.” The distribution PKZIP archives (JAR 
> files) created by the Maven Javadoc Plugin violate this licence, making them 
> not redistributable.
> Similarily, the GPLv2 used by the Oracle-provided files *requires* that 
> redistributors “give any other recipients of the Program a copy of this 
> License along with the Program.” The “if not, write to the Free Software 
> Foundation” comment is specifically *not sufficient* for this and only 
> provided as fallback should distributors violate this clause, as Maven 
> Javadoc Plugin-generated PKZIP archives do. To be effective, the Classpath 
> exception must also be provided.
> h2. Suggested fix
> Include the following new files:
>  * {{jquery/LICENCE}} containing the MIT licence and all respective copyright 
> notices for the various jQuery-related projects (including those _they_ 
> include, i.e. Sizzle, widget.js, position.js, keycode.js, unique-id.js, 
> widgets/autocomplete.js, widgets/menu.js, pako, and possibly others)
>  * {{js/LICENSE}} (creating a new subdirectory) containing the Classpath 
> exception as provided by Oracle
>  * {{COPYING}} or {{js/COPYING}} (this being the customary name for this 
> file) containing the verbatim text of the GNU GPL version 2
>  * Ideally, add a top-level {{LICENCE}} file pointing out those three and 
> briefly documenting the licence of all other non-generated files and state 
> all other files are generated from the original project and share its licence



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MJAVADOC-708) How to get javadoc "locale" configuration from a Taglet in jdk9

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MJAVADOC-708.
---
Resolution: Information Provided

You should rather ask the OpenJDK javadoc mailing list whether this is possible 
at all. If not, nothing can be exposed through this plugin.

> How to get javadoc "locale" configuration from a Taglet in jdk9
> ---
>
> Key: MJAVADOC-708
> URL: https://issues.apache.org/jira/browse/MJAVADOC-708
> Project: Maven Javadoc Plugin
>  Issue Type: Wish
>  Components: javadoc
>Affects Versions: 3.3.2
>Reporter: DavidHsing
>Priority: Trivial
>  Labels: locale
>
> I'm writting a target class which extends from jdk.javadoc.doclet.Taglet, 
> this taglet needs to display multilingual “” in javadoc.
> Is there any way to detect the "locale" configuration of 
> maven-javadoc-plugin,?
> Thanks very much!



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MJAVADOC-695) Default link to Java 17 API is dead

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MJAVADOC-695.
---
Fix Version/s: (was: next-release)
   Resolution: Not A Problem

>From Java 16, this is done automatically: 
>https://github.com/openjdk/jdk/commit/1e8e543b#diff-42c90a8eded4f2fafe6386521e56956312a33016f23ce8a8316ff10f8bc358fe

> Default link to Java 17 API is dead
> ---
>
> Key: MJAVADOC-695
> URL: https://issues.apache.org/jira/browse/MJAVADOC-695
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
>
> The default base url to the Java 17 API, used by 
> [detectJavaApiLink|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink],
>  seems to be {{https://download.java.net/java/early_access/jdk17/docs/api/}}. 
> This url is now dead and should be replaced by 
> {{https://docs.oracle.com/en/java/javase/17/docs/api/}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Updated] (MJAVADOC-695) Default link to Java 17 API is dead

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov updated MJAVADOC-695:

Fix Version/s: next-release

> Default link to Java 17 API is dead
> ---
>
> Key: MJAVADOC-695
> URL: https://issues.apache.org/jira/browse/MJAVADOC-695
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
> Fix For: next-release
>
>
> The default base url to the Java 17 API, used by 
> [detectJavaApiLink|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink],
>  seems to be {{https://download.java.net/java/early_access/jdk17/docs/api/}}. 
> This url is now dead and should be replaced by 
> {{https://docs.oracle.com/en/java/javase/17/docs/api/}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Assigned] (MJAVADOC-695) Default link to Java 17 API is dead

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov reassigned MJAVADOC-695:
---

Assignee: Michael Osipov

> Default link to Java 17 API is dead
> ---
>
> Key: MJAVADOC-695
> URL: https://issues.apache.org/jira/browse/MJAVADOC-695
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>  Components: javadoc
>Affects Versions: 3.3.1
>Reporter: Olivier Cailloux
>Assignee: Michael Osipov
>Priority: Minor
>
> The default base url to the Java 17 API, used by 
> [detectJavaApiLink|https://maven.apache.org/plugins/maven-javadoc-plugin/javadoc-mojo.html#detectJavaApiLink],
>  seems to be {{https://download.java.net/java/early_access/jdk17/docs/api/}}. 
> This url is now dead and should be replaced by 
> {{https://docs.oracle.com/en/java/javase/17/docs/api/}}.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Closed] (MJAVADOC-703) maven-javadoc-plugin > 3.0.1 fail on JDK 8

2022-04-16 Thread Michael Osipov (Jira)


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

Michael Osipov closed MJAVADOC-703.
---
Resolution: Not A Problem

> maven-javadoc-plugin > 3.0.1 fail on JDK 8
> --
>
> Key: MJAVADOC-703
> URL: https://issues.apache.org/jira/browse/MJAVADOC-703
> Project: Maven Javadoc Plugin
>  Issue Type: Bug
>Affects Versions: 3.1.0, 3.3.1
>Reporter: Thorsten Glaser
>Priority: Major
> Attachments: verbose1.txt
>
>
> Trying to use 3.1.0 or 3.3.1 with JDK 8 results in {{--release}} being passed 
> to the {{javadoc}} command, which does not know it in JDK 8:
> {{[ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-javadoc-plugin:3.3.1:jar (attach-javadocs) on 
> project maven-parent-lib: MavenReportException: Error while generating 
> Javadoc: }}
> {{[ERROR] Exit code: 1 - javadoc: error - invalid flag: --release}}
> {{[ERROR]}}
> {{[ERROR] Command line was: 
> /usr/lib/jvm/java-8-openjdk-i386/jre/../bin/javadoc @options @packages}}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-integration-testing] michael-o commented on a diff in pull request #151: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


michael-o commented on code in PR #151:
URL: 
https://github.com/apache/maven-integration-testing/pull/151#discussion_r851620420


##
core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/DeprecatedConfigMojo.java:
##
@@ -47,6 +48,18 @@
 @Parameter( defaultValue = "${basedir}", readonly = true )
 private File basedir;
 
+/**
+ * @deprecated bean read only
+ */
+@Parameter(  defaultValue = "${project.artifact}", readonly = true )
+private Artifact deprecatedBeanReadOnly;
+
+/**
+ * @deprecated bean
+ */
+@Parameter(  defaultValue = "${project.artifact}" )
+private Artifact deprecatedBean;

Review Comment:
   I don't see the corresponding test...



##
core-it-suite/src/test/java/org/apache/maven/it/MavenITmng5222MojoDeprecatedParamsTest.java:
##
@@ -35,7 +35,7 @@
 {
 public MavenITmng5222MojoDeprecatedParamsTest()
 {
-super( "[3.9.0,)" );
+super( "[4.0.0-alpha-1,)" );

Review Comment:
   Why back to 4 and not with 3.9?



-- 
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] (SUREFIRE-2074) Junit 5 @Nested Tests executes when excluded

2022-04-16 Thread Tibor Digana (Jira)


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

Tibor Digana updated SUREFIRE-2074:
---
Fix Version/s: waiting-for-apache-feedback

> Junit 5 @Nested Tests executes when excluded
> 
>
> Key: SUREFIRE-2074
> URL: https://issues.apache.org/jira/browse/SUREFIRE-2074
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: JUnit 5.x support
>Affects Versions: 3.0.0-M6
>Reporter: Henrik Josefsson
>Priority: Minor
> Fix For: waiting-for-apache-feedback
>
>
> Test classes with the Junit 5 `@Nested` annotation is executed even when 
> explicitly excluded with the `` tag.
> h2. Minimal reproducing example
> [https://github.com/Steinstark/surefire-issue] see the IssueTest.
> h2. Surefire configuration
> {code:java}
> 
>     org.apache.maven.plugins
>     maven-surefire-plugin
>     3.0.0-M6
>     
>     
>         **/IssueTest$NestedTests
>         
>     
> {code}
> h2. Expected behavior
> The tests inside the `@Nested` excluded class is not run.
> h2. Actual behavior
> The tests inside the `@Nested` excluded class is run.
> h2. Additional info
> I believe the problem is caused by only setting the selectors based on the 
> exact classes surefire is expected to run for the 
> LauncherDiscoveryRequestBuilder in JunitPlatformProvider but as far as I can 
> tell Junit will by default discover `@Nested` classes aswell from the 
> selectors. 
> The solution is probably to provide a filter in addition to the selector. Not 
> sure if it is a good solution but locally I changed the newFilters method to 
> always add the TestMethodFilter and it appear to have worked. Might be 
> something worth looking further into at least.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Created] (SUREFIRE-2074) Junit 5 @Nested Tests executes when excluded

2022-04-16 Thread Henrik Josefsson (Jira)
Henrik Josefsson created SUREFIRE-2074:
--

 Summary: Junit 5 @Nested Tests executes when excluded
 Key: SUREFIRE-2074
 URL: https://issues.apache.org/jira/browse/SUREFIRE-2074
 Project: Maven Surefire
  Issue Type: Bug
  Components: JUnit 5.x support
Affects Versions: 3.0.0-M6
Reporter: Henrik Josefsson


Test classes with the Junit 5 `@Nested` annotation is executed even when 
explicitly excluded with the `` tag.
h2. Minimal reproducing example

[https://github.com/Steinstark/surefire-issue] see the IssueTest.
h2. Surefire configuration
{code:java}

    org.apache.maven.plugins
    maven-surefire-plugin
    3.0.0-M6
    
    
        **/IssueTest$NestedTests
        
    
{code}
h2. Expected behavior

The tests inside the `@Nested` excluded class is not run.
h2. Actual behavior

The tests inside the `@Nested` excluded class is run.
h2. Additional info

I believe the problem is caused by only setting the selectors based on the 
exact classes surefire is expected to run for the 
LauncherDiscoveryRequestBuilder in JunitPlatformProvider but as far as I can 
tell Junit will by default discover `@Nested` classes aswell from the 
selectors. 

The solution is probably to provide a filter in addition to the selector. Not 
sure if it is a good solution but locally I changed the newFilters method to 
always add the TestMethodFilter and it appear to have worked. Might be 
something worth looking further into at least.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven-surefire] Tibor17 commented on pull request #517: [SUREFIRE-2064] Allow all supported values of [parallel] option

2022-04-16 Thread GitBox


Tibor17 commented on PR #517:
URL: https://github.com/apache/maven-surefire/pull/517#issuecomment-1100627414

   @sbabcoc 
   Pls squash these commits to one and we wil make a final review.
   
   The situation is as follows.
   We are in a little progress with our internal wishes and we have to fix 
broken M6.
   Additionally, I have backported cca 20 bug fixes from M6 to 2.22.3 but also 
the TestNG 7.4 bug fix and this one is missing.
   I guess @slawekjaranowski would continue on fixing `Configuration` 
implementation and removal of 5.x. The fix of impl of `Configuration` interface 
is important for M7 and it would be nice in 2.22.3.


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

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

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



[GitHub] [maven] slawekjaranowski commented on pull request #722: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


slawekjaranowski commented on PR #722:
URL: https://github.com/apache/maven/pull/722#issuecomment-1100623733

   build on Jenkins fails due to using master branch from integration-test


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

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

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



[jira] [Commented] (MNG-5222) Maven 3 no longer logs warnings about deprecated plugin parameters.

2022-04-16 Thread Hudson (Jira)


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

Hudson commented on MNG-5222:
-

Build unstable in Jenkins: Maven » Maven TLP » maven » PR-722 #2

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/PR-722/2/

> Maven 3 no longer logs warnings about deprecated plugin parameters.
> ---
>
> Key: MNG-5222
> URL: https://issues.apache.org/jira/browse/MNG-5222
> Project: Maven
>  Issue Type: Bug
>  Components: Plugins and Lifecycle
>Affects Versions: 3.0.3
>Reporter: Christian Schulte
>Assignee: Slawomir Jaranowski
>Priority: Minor
> Fix For: 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Providing a value for a deprecated plugin parameter, Maven 2.2.1 used to log 
> a warning. Currently Maven 3 does not.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[GitHub] [maven] Aylinee-RH-Arletee commented on pull request #720: [MNG-7454] Include resolver-transport-http in Maven

2022-04-16 Thread GitBox


Aylinee-RH-Arletee commented on PR #720:
URL: https://github.com/apache/maven/pull/720#issuecomment-1100612599

   d826b7c78ccf3edcb9ea365480c2cfff2332c335


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

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

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



[GitHub] [maven-integration-testing] slawekjaranowski opened a new pull request, #151: [MNG-5222] Improvement in deprecated params detection

2022-04-16 Thread GitBox


slawekjaranowski opened a new pull request, #151:
URL: https://github.com/apache/maven-integration-testing/pull/151

   for https://github.com/apache/maven/pull/722


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



  1   2   >