[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17262113#comment-17262113 ] Pavel_K commented on MNG-7037: -- [~rfscholte] Important note - I don't know maven internal work

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17262110#comment-17262110 ] Pavel_K commented on MNG-7037: -- [~rfscholte] Yes, I understand that to modularize the project

[GitHub] [maven] mkarg commented on pull request #421: Artifact.getPath() and .setPath()

2021-01-09 Thread GitBox
mkarg commented on pull request #421: URL: https://github.com/apache/maven/pull/421#issuecomment-756912897 So if there is nobody voting -1 I would kindly ask someone to merge this, as I am not a committer. :-) This is an aut

[GitHub] [maven] MartinKanters commented on a change in pull request #429: [MNG-7051] - Optionally skip non-existing profiles and break on missing required profiles

2021-01-09 Thread GitBox
MartinKanters commented on a change in pull request #429: URL: https://github.com/apache/maven/pull/429#discussion_r554435075 ## File path: maven-core/src/main/java/org/apache/maven/execution/ProfileActivationRequest.java ## @@ -0,0 +1,180 @@ +package org.apache.maven.executio

[GitHub] [maven-apache-parent] hboutemy merged pull request #34: [MPOM-260] simplify m-javadoc-p configuration

2021-01-09 Thread GitBox
hboutemy merged pull request #34: URL: https://github.com/apache/maven-apache-parent/pull/34 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

[GitHub] [maven-apache-parent] kwin edited a comment on pull request #31: MPOM-260 configure javadoc for reproducible builds

2021-01-09 Thread GitBox
kwin edited a comment on pull request #31: URL: https://github.com/apache/maven-apache-parent/pull/31#issuecomment-757183847 It would be good to explain the relation to https://maven.apache.org/ref/3.6.3/maven-model-builder/, especially the role of https://maven.apache.org/ref/3.6.3/maven

[GitHub] [maven] rmannibucau commented on pull request #421: Artifact.getPath() and .setPath()

2021-01-09 Thread GitBox
rmannibucau commented on pull request #421: URL: https://github.com/apache/maven/pull/421#issuecomment-756929903 Ok for me when all *our* impl will have implemented it and do not rely on the default method. This is an automa

[GitHub] [maven] rfscholte commented on a change in pull request #429: [MNG-7051] - Optionally skip non-existing profiles and break on missing required profiles

2021-01-09 Thread GitBox
rfscholte commented on a change in pull request #429: URL: https://github.com/apache/maven/pull/429#discussion_r554032511 ## File path: maven-core/src/main/java/org/apache/maven/execution/ProfileActivationRequest.java ## @@ -0,0 +1,180 @@ +package org.apache.maven.execution; +

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

2021-01-09 Thread GitBox
rfscholte commented on pull request #402: URL: https://github.com/apache/maven/pull/402#issuecomment-756893732 Probably related is MNG-7063: if trying to read the same file multiple times (in this case {{dependency-reduced-pom.xml}}, it'll be picked up from cache. In this case the model sh

[GitHub] [maven-release] K-J-Henken commented on a change in pull request #62: [MRELEASE-899] Feature/lineseparator

2021-01-09 Thread GitBox
K-J-Henken commented on a change in pull request #62: URL: https://github.com/apache/maven-release/pull/62#discussion_r553916385 ## File path: maven-release-plugin/src/main/java/org/apache/maven/plugins/release/PrepareReleaseMojo.java ## @@ -387,4 +403,27 @@ protected void pre

[GitHub] [maven] michael-o commented on pull request #429: [MNG-7051] - Optionally skip non-existing profiles and break on missing required profiles

2021-01-09 Thread GitBox
michael-o commented on pull request #429: URL: https://github.com/apache/maven/pull/429#issuecomment-756748127 I will happily review this next week. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [maven] rmannibucau commented on pull request #400: [MNG-7035] Migrate all tests to JUnit 4

2021-01-09 Thread GitBox
rmannibucau commented on pull request #400: URL: https://github.com/apache/maven/pull/400#issuecomment-756641275 Guess the question is do we switch to junit4 ot junit5 directly today. I think in both cases we need to write the BaseTest as an extension (junit5) or rule (junit4) so the effor

[GitHub] [maven-apache-parent] kwin commented on pull request #31: MPOM-260 configure javadoc for reproducible builds

2021-01-09 Thread GitBox
kwin commented on pull request #31: URL: https://github.com/apache/maven-apache-parent/pull/31#issuecomment-756635706 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[jira] [Commented] (DOXIA-618) doxia-module-markdown: Reincorporate maven-site-plugin's Doxia-specific integration tests

2021-01-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/DOXIA-618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17262015#comment-17262015 ] ASF GitHub Bot commented on DOXIA-618: -- elharo commented on pull request #52: URL: ht

[GitHub] [maven-doxia] elharo commented on pull request #52: [DOXIA-618] import tests from maven-site-plugin

2021-01-09 Thread GitBox
elharo commented on pull request #52: URL: https://github.com/apache/maven-doxia/pull/52#issuecomment-756729841 The branches are a bit confusing but I'm running it through now This is an automated message from the Apache Git

[GitHub] [maven-surefire] Tibor17 commented on pull request #329: ci: JDK 8, JDK 11, JDK 15

2021-01-09 Thread GitBox
Tibor17 commented on pull request #329: URL: https://github.com/apache/maven-surefire/pull/329#issuecomment-756703691 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [maven-surefire] Tibor17 merged pull request #329: ci: JDK 8, JDK 11, JDK 15

2021-01-09 Thread GitBox
Tibor17 merged pull request #329: URL: https://github.com/apache/maven-surefire/pull/329 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 g

[GitHub] [maven] MartinKanters commented on pull request #429: [MNG-7051] - Optionally skip non-existing profiles and break on missing required profiles

2021-01-09 Thread GitBox
MartinKanters commented on pull request #429: URL: https://github.com/apache/maven/pull/429#issuecomment-756779444 > I will happily review this next week. Great! Thanks in advance This is an automated message from the

[GitHub] [maven-resolver] cstamas commented on a change in pull request #87: [EXPERIMENT] Proper remote repository scoping during dependency resolution

2021-01-09 Thread GitBox
cstamas commented on a change in pull request #87: URL: https://github.com/apache/maven-resolver/pull/87#discussion_r553925217 ## File path: maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultRemoteRepositoryManager.java ## @@ -201,7 +210,14 @@ public De

[GitHub] [maven-apache-parent] hboutemy commented on pull request #31: MPOM-260 configure javadoc for reproducible builds

2021-01-09 Thread GitBox
hboutemy commented on pull request #31: URL: https://github.com/apache/maven-apache-parent/pull/31#issuecomment-756610411 This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [maven-pmd-plugin] adangel merged pull request #41: [MPMD-314] PMD report extension not set correctly for custom renderer

2021-01-09 Thread GitBox
adangel merged pull request #41: URL: https://github.com/apache/maven-pmd-plugin/pull/41 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 g

[GitHub] [maven] gnodet commented on pull request #400: [MNG-7035] Migrate all tests to JUnit 4

2021-01-09 Thread GitBox
gnodet commented on pull request #400: URL: https://github.com/apache/maven/pull/400#issuecomment-756611580 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [maven-resolver] michael-o commented on a change in pull request #87: [EXPERIMENT] Proper remote repository scoping during dependency resolution

2021-01-09 Thread GitBox
michael-o commented on a change in pull request #87: URL: https://github.com/apache/maven-resolver/pull/87#discussion_r553906483 ## File path: maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultRemoteRepositoryManager.java ## @@ -201,7 +210,14 @@ public

[GitHub] [maven-pmd-plugin] adangel commented on a change in pull request #41: [MPMD-314] Report extension not set correctly when custom format

2021-01-09 Thread GitBox
adangel commented on a change in pull request #41: URL: https://github.com/apache/maven-pmd-plugin/pull/41#discussion_r553814436 ## File path: src/main/java/org/apache/maven/plugins/pmd/exec/PmdExecutor.java ## @@ -429,7 +430,7 @@ private void writeFormattedReport( Report repor

[GitHub] [maven] cstamas commented on pull request #400: [MNG-7035] Migrate all tests to JUnit 4

2021-01-09 Thread GitBox
cstamas commented on pull request #400: URL: https://github.com/apache/maven/pull/400#issuecomment-756658412 I'd prefer "baby steps", so this PR as is is okay for me, and let have later go with other changes... This is an au

[GitHub] [maven] slachiewicz commented on pull request #427: [MNG-6727] Changed expression check to project.version and project.parent.version

2021-01-09 Thread GitBox
slachiewicz commented on pull request #427: URL: https://github.com/apache/maven/pull/427#issuecomment-756642164 FYI - I fixed ITS (by default compiler.target was set to Java 6. CI run in progress: https://ci-builds.apache.org/job/Maven/job/maven-box/job/maven/job/MNG-6727/ ---

[GitHub] [maven] slachiewicz commented on a change in pull request #400: [MNG-7035] Migrate all tests to JUnit 4

2021-01-09 Thread GitBox
slachiewicz commented on a change in pull request #400: URL: https://github.com/apache/maven/pull/400#discussion_r553820298 ## File path: maven-compat/src/test/java/org/apache/maven/PlexusTestCase.java ## @@ -0,0 +1,317 @@ +package org.apache.maven; + +/* + * Licensed to the Ap

[GitHub] [maven-surefire] Tibor17 commented on a change in pull request #330: modified the document inclusion-exclusion.apt.vm;

2021-01-09 Thread GitBox
Tibor17 commented on a change in pull request #330: URL: https://github.com/apache/maven-surefire/pull/330#discussion_r553887117 ## File path: maven-surefire-plugin/src/site/apt/examples/inclusion-exclusion.apt.vm ## @@ -61,7 +61,9 @@ Inclusions and Exclusions of Tests []

[GitHub] [maven] slachiewicz commented on pull request #400: [MNG-7035] Migrate all tests to JUnit 4

2021-01-09 Thread GitBox
slachiewicz commented on pull request #400: URL: https://github.com/apache/maven/pull/400#issuecomment-756631816 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [maven] cstamas commented on pull request #428: [EXPERIMENT] Update resolver and use newly introduced method

2021-01-09 Thread GitBox
cstamas commented on pull request #428: URL: https://github.com/apache/maven/pull/428#issuecomment-756645983 Was m-a-t initial goal to "hide" org.sonatype vs org.eclipse resolver diffs? So to say maven 3.0 vs maven 3.1. Also, it is sitting between maven (core) and resolver (core), but m-a-

[GitHub] [maven] slachiewicz commented on pull request #428: [EXPERIMENT] Update resolver and use newly introduced method

2021-01-09 Thread GitBox
slachiewicz commented on pull request #428: URL: https://github.com/apache/maven/pull/428#issuecomment-756633507 I'm really interested in how this will work (ie use the newer resolver method) in plugins - that run in older maven 3.6.3 (as a replacement to m-artifact-transfer. ---

[GitHub] [maven] gnodet commented on a change in pull request #400: [MNG-7035] Migrate all tests to JUnit 4

2021-01-09 Thread GitBox
gnodet commented on a change in pull request #400: URL: https://github.com/apache/maven/pull/400#discussion_r553839266 ## File path: maven-compat/src/test/java/org/apache/maven/PlexusTestCase.java ## @@ -0,0 +1,317 @@ +package org.apache.maven; + +/* + * Licensed to the Apache

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261960#comment-17261960 ] Robert Scholte commented on MNG-7037: - One idea I had is to start with a complete new s

[jira] [Comment Edited] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261950#comment-17261950 ] Pavel_K edited comment on MNG-7037 at 1/9/21, 9:11 PM: --- [~rfscholte]

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261950#comment-17261950 ] Pavel_K commented on MNG-7037: -- [~rfscholte] From here https://stackoverflow.com/a/46504438/50

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261949#comment-17261949 ] Pavel_K commented on MNG-7037: -- [~rfscholte] Ok. Can you say, if this issue will be resolved a

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261937#comment-17261937 ] Robert Scholte commented on MNG-7037: - One way to solve this is to bring back all class

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261934#comment-17261934 ] Robert Scholte commented on MNG-7037: - [~Pavel_K] several fellow member of the [JSR 376

[GitHub] [maven] rfscholte commented on a change in pull request #429: [MNG-7051] - Optionally skip non-existing profiles and break on missing required profiles

2021-01-09 Thread GitBox
rfscholte commented on a change in pull request #429: URL: https://github.com/apache/maven/pull/429#discussion_r554451003 ## File path: maven-core/src/main/java/org/apache/maven/execution/ProfileActivationRequest.java ## @@ -0,0 +1,180 @@ +package org.apache.maven.execution; +

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261883#comment-17261883 ] Pavel_K commented on MNG-7037: -- [~rfscholte] By the way, could you say, what experts agree tha

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Pavel_K (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261879#comment-17261879 ] Pavel_K commented on MNG-7037: -- [~rfscholte] There are different module types (https://github

[GitHub] [maven] MartinKanters commented on a change in pull request #429: [MNG-7051] - Optionally skip non-existing profiles and break on missing required profiles

2021-01-09 Thread GitBox
MartinKanters commented on a change in pull request #429: URL: https://github.com/apache/maven/pull/429#discussion_r554435075 ## File path: maven-core/src/main/java/org/apache/maven/execution/ProfileActivationRequest.java ## @@ -0,0 +1,180 @@ +package org.apache.maven.executio

[jira] [Closed] (MNG-6980) Add opposite to --resume-from

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-6980?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Scholte closed MNG-6980. --- Assignee: Robert Scholte Resolution: Won't Fix To me this doesn't make sense. You want to build a

[jira] [Updated] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Scholte updated MNG-7037: Priority: Minor (was: Major) > Add JPMS support -> solve split packages problem > --

[jira] [Commented] (MNG-7037) Add JPMS support -> solve split packages problem

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7037?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261873#comment-17261873 ] Robert Scholte commented on MNG-7037: - bq. No, as I need to use org.apache.maven.repos

[GitHub] [maven-apache-parent] kwin edited a comment on pull request #31: MPOM-260 configure javadoc for reproducible builds

2021-01-09 Thread GitBox
kwin edited a comment on pull request #31: URL: https://github.com/apache/maven-apache-parent/pull/31#issuecomment-757183847 It would be good to explain the relation to https://maven.apache.org/ref/3.6.3/maven-model-builder/, especially the role of https://maven.apache.org/ref/3.6.3/maven

[GitHub] [maven-apache-parent] kwin commented on pull request #31: MPOM-260 configure javadoc for reproducible builds

2021-01-09 Thread GitBox
kwin commented on pull request #31: URL: https://github.com/apache/maven-apache-parent/pull/31#issuecomment-757183847 It would be good to explain the relation to https://maven.apache.org/ref/3.6.3/maven-model-builder/. This

[jira] [Commented] (MNG-7051) Optionally skip non-existing profiles

2021-01-09 Thread Robert Scholte (Jira)
[ https://issues.apache.org/jira/browse/MNG-7051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261855#comment-17261855 ] Robert Scholte commented on MNG-7051: - [~slawekjaranowski] we need to draw a line somew