[jira] [Created] (MJAVADOC-671) Generated javadoc.bat fails to execute in Windows10

2021-02-12 Thread Jesus Salvo (Jira)
Jesus Salvo created MJAVADOC-671: Summary: Generated javadoc.bat fails to execute in Windows10 Key: MJAVADOC-671 URL: https://issues.apache.org/jira/browse/MJAVADOC-671 Project: Maven Javadoc Plugin

[jira] [Commented] (MNGSITE-430) Clarify remote repository resolution order

2021-02-12 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/MNGSITE-430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17284031#comment-17284031 ] Hudson commented on MNGSITE-430: Build succeeded in Jenkins: Maven » Maven TLP » maven-s

[jira] [Commented] (MWRAPPER-11) Could not find artifact org.apache.maven:apache-maven-wrapper:zip:script

2021-02-12 Thread Anthony Whitford (Jira)
[ https://issues.apache.org/jira/browse/MWRAPPER-11?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283984#comment-17283984 ] Anthony Whitford commented on MWRAPPER-11: -- Is this a duplicate of MWRAPPER-6?

[GitHub] [maven-integration-testing] MartinKanters commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
MartinKanters commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778442266 > > > I've fixed the comments and change the tests to rely on the `verify` goal rather than `install`. > > > > > > Great. They are still running a

[GitHub] [maven-integration-testing] gnodet commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
gnodet commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778440870 > > I've fixed the comments and change the tests to rely on the `verify` goal rather than `install`. > > Great. They are still running atm, but the results l

[GitHub] [maven] MartinKanters commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
MartinKanters commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778436709 Cool, your ITs are looking good right now, so the PR can be merged in my opinion. @mthmulders do you want to take another look? Otherwise I can merge somewhere this weekend.

[GitHub] [maven-integration-testing] MartinKanters commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
MartinKanters commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778436035 > I've fixed the comments and change the tests to rely on the `verify` goal rather than `install`. Great. They are still running atm, but the results

[GitHub] [maven-integration-testing] gnodet commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
gnodet commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778426174 I've fixed the comments and change the tests to rely on the `verify` goal rather than `install`.

[GitHub] [maven-integration-testing] MartinKanters commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
MartinKanters commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778420212 > I've rerun the GitHub Actions build on your maven PR, this should take these new ITs into account as well (as both branches are named the same): > https

[jira] [Assigned] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Martin Kanters (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Kanters reassigned MNG-7099: --- Assignee: Martin Kanters > Can not resume build without installing the artifacts > ---

[jira] [Closed] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Martin Kanters (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Kanters closed MNG-7099. --- Resolution: Duplicate > Can not resume build without installing the artifacts > ---

[jira] [Commented] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Martin Kanters (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283945#comment-17283945 ] Martin Kanters commented on MNG-7099: - Thanks anyway for the PR and the bug report!  >

[GitHub] [maven-integration-testing] gnodet closed pull request #99: [MNG-7099] Add integration test

2021-02-12 Thread GitBox
gnodet closed pull request #99: URL: https://github.com/apache/maven-integration-testing/pull/99 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [maven-integration-testing] gnodet commented on pull request #99: [MNG-7099] Add integration test

2021-02-12 Thread GitBox
gnodet commented on pull request #99: URL: https://github.com/apache/maven-integration-testing/pull/99#issuecomment-778414851 Duplicate, already fixed in maven 4 This is an automated message from the Apache Git Service. To r

[jira] [Commented] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Guillaume Nodet (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283943#comment-17283943 ] Guillaume Nodet commented on MNG-7099: -- Yes, this is clearly a duplicate. Feel free t

[GitHub] [maven-integration-testing] MartinKanters commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
MartinKanters commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778413713 Also I don't mind it too much, but perhaps we could avoid `install` now, if MNG-4660 indeed has fixed the issue you described in MNG-7099.

[GitHub] [maven] MartinKanters edited a comment on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
MartinKanters edited a comment on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778412923 > > > @mthmulders @MartinKanters quick question. > > > I'm working on improving the integration test and have the following scenario: a parent, 3 modules named `A`, `

[GitHub] [maven] MartinKanters commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
MartinKanters commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778412923 > > > @mthmulders @MartinKanters quick question. > > > I'm working on improving the integration test and have the following scenario: a parent, 3 modules named `A`, `B`, `C`

[GitHub] [maven-integration-testing] MartinKanters commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
MartinKanters commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778411008 I've rerun the GitHub Actions build on your maven PR, this should take these new ITs into account as well (as both branches are named the same): https://g

[GitHub] [maven-integration-testing] MartinKanters commented on a change in pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
MartinKanters commented on a change in pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#discussion_r57547 ## File path: core-it-suite/src/test/java/org/apache/maven/it/MavenITmng5760ResumeFeatureTest.java ## @@ -175,6 +179,112 @@ public v

[jira] [Commented] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Martin Kanters (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283935#comment-17283935 ] Martin Kanters commented on MNG-7099: - Hey [~gnodet], I totally understand you, this ha

[GitHub] [maven] MartinKanters commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
MartinKanters commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575186110 ## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionAnalyzer.java ## @@ -47,132 +41,49 @@ @Override publi

[GitHub] [maven] gnodet commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778388548 I've raised MNG-7099 and provided a PR with an integration test. This is an automated message from the Apache Git Ser

[jira] [Commented] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Guillaume Nodet (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283918#comment-17283918 ] Guillaume Nodet commented on MNG-7099: -- Fwiw, I think that's the reason why I've never

[jira] [Commented] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Guillaume Nodet (Jira)
[ https://issues.apache.org/jira/browse/MNG-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283917#comment-17283917 ] Guillaume Nodet commented on MNG-7099: -- A PR with an integration test is available at

[GitHub] [maven-integration-testing] gnodet opened a new pull request #99: [MNG-7099] Add integration test

2021-02-12 Thread GitBox
gnodet opened a new pull request #99: URL: https://github.com/apache/maven-integration-testing/pull/99 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[jira] [Created] (MNG-7099) Can not resume build without installing the artifacts

2021-02-12 Thread Guillaume Nodet (Jira)
Guillaume Nodet created MNG-7099: Summary: Can not resume build without installing the artifacts Key: MNG-7099 URL: https://issues.apache.org/jira/browse/MNG-7099 Project: Maven Issue Type: T

[jira] [Created] (MSHADE-382) Add an option to skip execution

2021-02-12 Thread Andres Almiray (Jira)
Andres Almiray created MSHADE-382: - Summary: Add an option to skip execution Key: MSHADE-382 URL: https://issues.apache.org/jira/browse/MSHADE-382 Project: Maven Shade Plugin Issue Type: Impr

[GitHub] [maven] gnodet commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778360471 > > @mthmulders @MartinKanters quick question. > > I'm working on improving the integration test and have the following scenario: a parent, 3 modules named `A`, `B`, `C` with `C` d

[GitHub] [maven-integration-testing] gnodet commented on pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
gnodet commented on pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98#issuecomment-778325826 I'll add another test This is an automated message from the Apache Git Service. To respond to the

[jira] [Commented] (MNG-7077) Use JUnit 5 extension

2021-02-12 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/MNG-7077?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283833#comment-17283833 ] Hudson commented on MNG-7077: - Build succeeded in Jenkins: Maven » Maven TLP » maven » master #

[GitHub] [maven] mthmulders commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
mthmulders commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778301747 > @mthmulders @MartinKanters quick question. > I'm working on improving the integration test and have the following scenario: a parent, 3 modules named `A`, `B`, `C` with `C` d

[GitHub] [maven] rfscholte closed pull request #432: [MNG-7077] Use JUnit 5 extension

2021-02-12 Thread GitBox
rfscholte closed pull request #432: URL: https://github.com/apache/maven/pull/432 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 th

[GitHub] [maven] rfscholte commented on pull request #432: [MNG-7077] Use JUnit 5 extension

2021-02-12 Thread GitBox
rfscholte commented on pull request #432: URL: https://github.com/apache/maven/pull/432#issuecomment-778294714 Merged with https://github.com/apache/maven/commit/48fc10889ef89d12ebe2258029937010d81aebb7 This is an automated

[jira] [Closed] (MNG-7077) Use JUnit 5 extension

2021-02-12 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7077?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Scholte closed MNG-7077. --- Fix Version/s: 4.0.0-alpha-1 4.0.0 Assignee: Robert Scholte Resolutio

[GitHub] [maven] gnodet commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778273066 I've added an integration test that tries to make sure the resume won't rebuild already built projects, see https://github.com/apache/maven-integration-testing/pull/98 Event

[GitHub] [maven-integration-testing] gnodet opened a new pull request #98: [MNG-7095] Support for parallel builds when resuming

2021-02-12 Thread GitBox
gnodet opened a new pull request #98: URL: https://github.com/apache/maven-integration-testing/pull/98 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [maven] gnodet commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778266531 > @mthmulders @MartinKanters quick question. > I'm working on improving the integration test and have the following scenario: a parent, 3 modules named `A`, `B`, `C` with `C` depen

[GitHub] [maven] gnodet commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778264311 @mthmulders @MartinKanters quick question. I'm working on improving the integration test and have the following scenario: a parent, 3 modules named `A`, `B`, `C` with `C` depending

[GitHub] [maven] rfscholte commented on pull request #402: DefaultProjectBuilder enhancements

2021-02-12 Thread GitBox
rfscholte commented on pull request #402: URL: https://github.com/apache/maven/pull/402#issuecomment-778258696 ok, let me check that one first This is an automated message from the Apache Git Service. To respond to the messag

[jira] [Commented] (MNG-7063) Infinite loop using Shade plugin and JUnit 5 dependency

2021-02-12 Thread Michael Osipov (Jira)
[ https://issues.apache.org/jira/browse/MNG-7063?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283768#comment-17283768 ] Michael Osipov commented on MNG-7063: - [~cstamas], this is likely the same as your issu

[GitHub] [maven] gnodet commented on pull request #402: DefaultProjectBuilder enhancements

2021-02-12 Thread GitBox
gnodet commented on pull request #402: URL: https://github.com/apache/maven/pull/402#issuecomment-778257427 > I just had another look at this. It looks to me that this commit has become too big. We should split it into 2 pieces: > > * rewriting JUnit tests using maven-test-support.

[GitHub] [maven] rfscholte commented on pull request #402: DefaultProjectBuilder enhancements

2021-02-12 Thread GitBox
rfscholte commented on pull request #402: URL: https://github.com/apache/maven/pull/402#issuecomment-778249119 I just had another look at this. It looks to me that this commit has become too big. We should split it into 2 pieces: * rewriting JUnit tests using maven-test-support. * R

[GitHub] [maven] gnodet commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575250093 ## File path: maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java ## @@ -1032,20 +1031,6 @@ private int execute( CliRequest cliRequest )

[GitHub] [maven] MartinKanters commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
MartinKanters commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575239494 ## File path: maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java ## @@ -1032,20 +1031,6 @@ private int execute( CliRequest cliRequest )

[jira] [Commented] (MNG-6763) Restrict repositories to specific groupIds

2021-02-12 Thread Michael Osipov (Jira)
[ https://issues.apache.org/jira/browse/MNG-6763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283700#comment-17283700 ] Michael Osipov commented on MNG-6763: - I believe that this can be down with {{maven-com

[GitHub] [maven] mthmulders commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
mthmulders commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575231766 ## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionDataRepository.java ## @@ -133,22 +126,14 @@ private Properties lo

[GitHub] [maven] mthmulders commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
mthmulders commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575231427 ## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionDataRepository.java ## @@ -133,22 +126,14 @@ private Properties lo

[GitHub] [maven] mthmulders commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
mthmulders commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575230619 ## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionAnalyzer.java ## @@ -47,132 +41,49 @@ @Override public O

[GitHub] [maven] gnodet commented on pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on pull request #444: URL: https://github.com/apache/maven/pull/444#issuecomment-778165900 > I like the fact that the solution becomes much simpler. Could you please add a unit test or an [integration test](https://github.com/apache/maven-integration-testing/) that proves

[GitHub] [maven] MartinKanters commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
MartinKanters commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575186110 ## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionAnalyzer.java ## @@ -47,132 +41,49 @@ @Override publi

[GitHub] [maven] gnodet commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575184291 ## File path: maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java ## @@ -1032,20 +1031,6 @@ private int execute( CliRequest cliRequest )

[GitHub] [maven] gnodet commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575182290 ## File path: maven-core/src/test/java/org/apache/maven/execution/DefaultBuildResumptionDataRepositoryTest.java ## @@ -54,38 +55,38 @@ public void resumeFrom

[GitHub] [maven] gnodet commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575178829 ## File path: maven-core/src/main/java/org/apache/maven/execution/BuildResumptionData.java ## @@ -32,36 +29,20 @@ /** * The project where the next

[GitHub] [maven] gnodet commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
gnodet commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575178653 ## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionAnalyzer.java ## @@ -47,132 +41,49 @@ @Override public Optio

[jira] [Created] (MNG-7098) Project counter should be cumulative when using resume

2021-02-12 Thread Robert Scholte (Jira)
Robert Scholte created MNG-7098: --- Summary: Project counter should be cumulative when using resume Key: MNG-7098 URL: https://issues.apache.org/jira/browse/MNG-7098 Project: Maven Issue Type: Im

[GitHub] [maven] mthmulders commented on a change in pull request #444: [MNG-7095] Fix resume for parallel builds

2021-02-12 Thread GitBox
mthmulders commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575106537 ## File path: maven-core/src/main/java/org/apache/maven/execution/BuildResumptionData.java ## @@ -32,36 +29,20 @@ /** * The project where the n