[GitHub] [maven-artifact-plugin] bmarwell commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


bmarwell commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-763384330


   I'd say `-1` to this one. Either create a `maven-lang` lib for the same 
reason as guava, commons-lang, etc. exist, or just wait for Java 8. @elharo no 
offense intended, but I honestly think (while I fully agree with what you 
mentioned earlier!) pulling in another dependency replacing a not-removable 
dependency is not worth it. Even if more changes like this are to come. 
   It is well-tested enough as this code existed for years and did not break.
   
   So, Maven 4 + Java 8 is not far away. Lets wait a few months and do a *real* 
java8 clean up then, wdyt?



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.

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




[jira] [Commented] (MNG-7074) Environment Variable defaulting

2021-01-19 Thread Jira


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

Loïc Hermann commented on MNG-7074:
---

Environnement variables can be accessed as properties: 
https://github.com/apache/maven/blob/39641ac803e17360df40288aaeb40ea0c5ccd77d/maven-core/src/main/java/org/apache/maven/properties/internal/EnvironmentUtils.java

> Environment Variable defaulting
> ---
>
> Key: MNG-7074
> URL: https://issues.apache.org/jira/browse/MNG-7074
> Project: Maven
>  Issue Type: New Feature
>Reporter: Loïc Hermann
>Priority: Major
> Fix For: waiting-for-feedback
>
>
> Hi all,
> maven build in CI environment often rely on environnement variables.
> If those environment variables are not defined on the developper environment 
> the build will fail.
> {{For now defaulting those environment variable can only be done by defining 
> one profile per variable which is really verbose.}}
> {{it would be easier if we can either}}
> {{add a default value where the variable is used:}}
> ```
> {{$\{env.VARIABLE_NAME:defaultValue}}}
> ```
> or have a section to define default values in the pom:
> ```
> 
>    <{{VARIABLE_NAME}}>defaultValue
> 
> ```
> I would love to work on a PR to implements one of those solution if this make 
> sense to you.
>  
> Regards



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (WAGON-603) Connection reset error from Azure pipelines

2021-01-19 Thread Arul Elvis J (Jira)


[ 
https://issues.apache.org/jira/browse/WAGON-603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268338#comment-17268338
 ] 

Arul Elvis J commented on WAGON-603:


[~michael-o] Managed to get some extra logs throwing 403 forbidden error before 
failing.

 
{noformat}
2021-01-19T15:18:23.4816376Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - 
2021-01-19T15:18:23.5872831Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - < 
au.com.bat:bata-s-sfdc-deferredpromo >
2021-01-19T15:18:23.5875769Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - Building 
bata-s-sfdc-deferredpromo 1.0.2
2021-01-19T15:18:23.5878345Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - --[ 
mule-application ]--
2021-01-19T15:18:29.6441716Z [main] [WARNING] 
org.codehaus.plexus.PlexusContainer - Could not transfer metadata 
com.fasterxml.jackson.core:jackson-core/maven-metadata.xml from/to 
microsoft-releases 
(https://mvnrepository.com/artifact/com.microsoft.sqlserver/mssql-jdbc/): 
Forbidden (403)
2021-01-19T15:19:35.6448025Z [main] [WARNING] 
org.codehaus.plexus.PlexusContainer - Could not transfer metadata 
com.fasterxml.jackson.core:jackson-annotations/maven-metadata.xml from/to 
microsoft-releases 
(https://mvnrepository.com/artifact/com.microsoft.sqlserver/mssql-jdbc/): 
Forbidden (403)
2021-01-19T15:19:56.1497288Z [main] [WARNING] 
org.codehaus.plexus.PlexusContainer - Could not transfer metadata 
com.fasterxml.jackson.core:jackson-databind/maven-metadata.xml from/to 
microsoft-releases 
(https://mvnrepository.com/artifact/com.microsoft.sqlserver/mssql-jdbc/): 
Forbidden (403)
2021-01-19T15:26:17.0569935Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - 
2021-01-19T15:26:17.0571308Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - --- 
maven-clean-plugin:2.5:clean (default-clean-1) @ bata-s-sfdc-deferredpromo ---
2021-01-19T15:26:42.4112274Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - 

2021-01-19T15:26:42.4112942Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - BUILD FAILURE
2021-01-19T15:26:42.4113552Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - 

2021-01-19T15:26:42.4114570Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - Total time:  08:53 min
2021-01-19T15:26:42.4115073Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - Finished at: 
2021-01-19T15:26:42Z
2021-01-19T15:26:42.4115665Z [main] [INFO] 
org.apache.maven.cli.event.ExecutionEventLogger - 

2021-01-19T15:26:42.4438501Z [main] [ERROR] org.apache.maven.cli.MavenCli - 
Failed to execute goal org.apache.maven.plugins:maven-clean-plugin:2.5:clean 
(default-clean-1) on project bata-s-sfdc-deferredpromo: Execution 
default-clean-1 of goal org.apache.maven.plugins:maven-clean-plugin:2.5:clean 
failed: Plugin org.apache.maven.plugins:maven-clean-plugin:2.5 or one of its 
dependencies could not be resolved: Failed to collect dependencies at 
org.apache.maven.plugins:maven-clean-plugin:jar:2.5 -> 
org.apache.maven:maven-plugin-api:jar:2.0.6: Failed to read artifact descriptor 
for org.apache.maven:maven-plugin-api:jar:2.0.6: Could not transfer artifact 
org.apache.maven:maven-plugin-api:pom:2.0.6 from/to central 
(https://repo.maven.apache.org/maven2): Connection reset -> [Help 1]{noformat}
Check if this helps. Attaching the full logs as well : 
[^Failed_build_debug_logJan20.txt]

> Connection reset error from Azure pipelines
> ---
>
> Key: WAGON-603
> URL: https://issues.apache.org/jira/browse/WAGON-603
> Project: Maven Wagon
>  Issue Type: Bug
> Environment: In both Ubuntu and Windows agent specification
>Reporter: Arul Elvis J
>Priority: Blocker
> Fix For: waiting-for-feedback
>
> Attachments: Failed_build_debug_logJan20.txt, Maven Goal - Deploy - 
> Debug Logs.txt, failed_logs-batch-wired.txt, 
> failed_logs-batch-wired_jan15.txt, failed_logs-batch-wired_jan15.txt, 
> failed_logs-batch-wired_latest.txt, logs_143613.zip
>
>   Original Estimate: 408h
>  Remaining Estimate: 408h
>
> When the pipelines are executed in Azure pipelines, we are facing connection 
> reset error. Azure pipelines team claims that connection is closed from Maven 
> central repo end. Below is the error message. Have attached the complete log 
> zip file for your reference. Can you please prioritize as soon as possible, 
> as it is impacting all the pipelines including the production builds ?
>  
> 

[jira] [Updated] (WAGON-603) Connection reset error from Azure pipelines

2021-01-19 Thread Arul Elvis J (Jira)


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

Arul Elvis J updated WAGON-603:
---
Attachment: Failed_build_debug_logJan20.txt

> Connection reset error from Azure pipelines
> ---
>
> Key: WAGON-603
> URL: https://issues.apache.org/jira/browse/WAGON-603
> Project: Maven Wagon
>  Issue Type: Bug
> Environment: In both Ubuntu and Windows agent specification
>Reporter: Arul Elvis J
>Priority: Blocker
> Fix For: waiting-for-feedback
>
> Attachments: Failed_build_debug_logJan20.txt, Maven Goal - Deploy - 
> Debug Logs.txt, failed_logs-batch-wired.txt, 
> failed_logs-batch-wired_jan15.txt, failed_logs-batch-wired_jan15.txt, 
> failed_logs-batch-wired_latest.txt, logs_143613.zip
>
>   Original Estimate: 408h
>  Remaining Estimate: 408h
>
> When the pipelines are executed in Azure pipelines, we are facing connection 
> reset error. Azure pipelines team claims that connection is closed from Maven 
> central repo end. Below is the error message. Have attached the complete log 
> zip file for your reference. Can you please prioritize as soon as possible, 
> as it is impacting all the pipelines including the production builds ?
>  
> {{ }}
> [ERROR] Failed to execute goal 
> org.apache.maven.plugins:maven-resources-plugin:3.0.2:resources 
> (default-resources) on project bata-s-sfdc-deferredpromo: Execution 
> default-resources of goal 
> org.apache.maven.plugins:maven-resources-plugin:3.0.2:resources failed: 
> Plugin org.apache.maven.plugins:maven-resources-plugin:3.0.2 or one of its 
> dependencies could not be resolved: Failed to collect dependencies at 
> org.apache.maven.plugins:maven-resources-plugin:jar:3.0.2 -> 
> org.apache.maven:maven-plugin-api:jar:3.0 -> 
> org.sonatype.sisu:sisu-inject-plexus:jar:1.4.2 -> 
> org.codehaus.plexus:plexus-component-annotations:jar:1.6: Failed to read 
> artifact descriptor for 
> org.codehaus.plexus:plexus-component-annotations:jar:1.6: Could not transfer 
> artifact org.codehaus.plexus:plexus-component-annotations:pom:1.6 from/to 
> central ([https://repo.maven.apache.org/maven2)]: Connection reset -> [Help 
> 1] 
> [ERROR] 
> [ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
> switch. 
> [ERROR] Re-run Maven using the -X switch to enable full debug logging. 
> [ERROR] 
> [ERROR] For more information about the errors and possible solutions, please 
> read the following articles: 
> [ERROR] [Help 1] 
> [http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException] 
> The process 
> 'C:\ProgramData\chocolatey\lib\maven\apache-maven-3.6.3\bin\mvn.cmd' failed 
> with exit code 1 
> Could not retrieve code analysis results - Maven run failed. 
> {{##[error]Build failed. }}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Updated] (SUREFIRE-1879) M5 swallows exception output from shutdown hooks

2021-01-19 Thread Falko Modler (Jira)


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

Falko Modler updated SUREFIRE-1879:
---
Description: 
With 3.0.0-M5, an exception from a shutdown hook is not printed to the console.

This works as expected with 3.0.0-M4.

To reproduce, run the project from the attached zipfile:

{noformat}
mvn test
{noformat}

{noformat}
[INFO] Running org.apache.maven.surefire.HookOutputTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.029 s 
- in org.apache.maven.surefire.HookOutputTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
{noformat}

{noformat}
mvn test -Dsurefire.version=3.0.0-M4
{noformat}

{noformat}
[INFO] Running org.apache.maven.surefire.HookOutputTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.028 s 
- in org.apache.maven.surefire.HookOutputTest
Exception in thread "Thread-3" java.lang.IllegalStateException: test
at 
org.apache.maven.surefire.HookOutputTest.lambda$testHookOutput$0(HookOutputTest.java:10)
at 
org.apache.maven.surefire.HookOutputTest$$Lambda$314/00.run(Unknown
 Source)
at java.base/java.lang.Thread.run(Thread.java:836)
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
{noformat}

PS: I originally ran into this here: 
https://github.com/quarkusio/quarkus/pull/14311#pullrequestreview-569179938

  was:
With 3.0.0-M5, an exception from a shutdown hook is not printed to the console.

This works as expected with 3.0.0-M4.

To reproduce, run the project from the attached zipfile:

{noformat}
mvn test
{noformat}

{noformat}
[INFO] Running org.apache.maven.surefire.HookOutputTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.029 s 
- in org.apache.maven.surefire.HookOutputTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
{noformat}

{noformat}
mvn test -Dsurefire.version=3.0.0-M4
{noformat}

{noformat}
[INFO] Running org.apache.maven.surefire.AppTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.012 s 
- in org.apache.maven.surefire.AppTest
Exception in thread "Thread-3" java.lang.IllegalStateException: test
at 
org.apache.maven.surefire.AppTest.lambda$testHookOutput$0(AppTest.java:42)
at 
org.apache.maven.surefire.AppTest$$Lambda$29/00.run(Unknown 
Source)
at java.base/java.lang.Thread.run(Thread.java:836)
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0
{noformat}

PS: I originally ran into this here: 
https://github.com/quarkusio/quarkus/pull/14311#pullrequestreview-569179938


> M5 swallows exception output from shutdown hooks
> 
>
> Key: SUREFIRE-1879
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1879
> Project: Maven Surefire
>  Issue Type: Bug
>Affects Versions: 3.0.0-M5
> Environment: Apache Maven 3.6.3 
> (cecedd343002696d0abb50b32b541b8a6ba2883f)
> Maven home: /home/moldowan/.sdkman/candidates/maven/current
> Java version: 11.0.9, vendor: AdoptOpenJDK, runtime: 
> /home/moldowan/.sdkman/candidates/java/11.0.9.j9-adpt
> Default locale: c.u_US, platform encoding: UTF-8
> OS name: "linux", version: "4.19.128-microsoft-standard", arch: "amd64", 
> family: "unix"
>Reporter: Falko Modler
>Priority: Major
> Attachments: surefire-bug-hook-output.zip
>
>
> With 3.0.0-M5, an exception from a shutdown hook is not printed to the 
> console.
> This works as expected with 3.0.0-M4.
> To reproduce, run the project from the attached zipfile:
> {noformat}
> mvn test
> {noformat}
> {noformat}
> [INFO] Running org.apache.maven.surefire.HookOutputTest
> [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.029 
> s - in org.apache.maven.surefire.HookOutputTest
> [INFO]
> [INFO] Results:
> [INFO]
> [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
> {noformat}
> {noformat}
> mvn test -Dsurefire.version=3.0.0-M4
> {noformat}
> {noformat}
> [INFO] Running org.apache.maven.surefire.HookOutputTest
> [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.028 
> s - in org.apache.maven.surefire.HookOutputTest
> Exception in thread "Thread-3" java.lang.IllegalStateException: test
> at 
> org.apache.maven.surefire.HookOutputTest.lambda$testHookOutput$0(HookOutputTest.java:10)
> at 
> org.apache.maven.surefire.HookOutputTest$$Lambda$314/00.run(Unknown
>  Source)
> at java.base/java.lang.Thread.run(Thread.java:836)
> [INFO]
> [INFO] Results:
> [INFO]
> [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
> {noformat}
> PS: I originally ran into this here: 
> 

[jira] [Created] (SUREFIRE-1879) M5 swallows exception output from shutdown hooks

2021-01-19 Thread Falko Modler (Jira)
Falko Modler created SUREFIRE-1879:
--

 Summary: M5 swallows exception output from shutdown hooks
 Key: SUREFIRE-1879
 URL: https://issues.apache.org/jira/browse/SUREFIRE-1879
 Project: Maven Surefire
  Issue Type: Bug
Affects Versions: 3.0.0-M5
 Environment: Apache Maven 3.6.3 
(cecedd343002696d0abb50b32b541b8a6ba2883f)
Maven home: /home/moldowan/.sdkman/candidates/maven/current
Java version: 11.0.9, vendor: AdoptOpenJDK, runtime: 
/home/moldowan/.sdkman/candidates/java/11.0.9.j9-adpt
Default locale: c.u_US, platform encoding: UTF-8
OS name: "linux", version: "4.19.128-microsoft-standard", arch: "amd64", 
family: "unix"
Reporter: Falko Modler
 Attachments: surefire-bug-hook-output.zip

With 3.0.0-M5, an exception from a shutdown hook is not printed to the console.

This works as expected with 3.0.0-M4.

To reproduce, run the project from the attached zipfile:

{noformat}
mvn test
{noformat}

{noformat}
[INFO] Running org.apache.maven.surefire.HookOutputTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.029 s 
- in org.apache.maven.surefire.HookOutputTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
{noformat}

{noformat}
mvn test -Dsurefire.version=3.0.0-M4
{noformat}

{noformat}
[INFO] Running org.apache.maven.surefire.AppTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.012 s 
- in org.apache.maven.surefire.AppTest
Exception in thread "Thread-3" java.lang.IllegalStateException: test
at 
org.apache.maven.surefire.AppTest.lambda$testHookOutput$0(AppTest.java:42)
at 
org.apache.maven.surefire.AppTest$$Lambda$29/00.run(Unknown 
Source)
at java.base/java.lang.Thread.run(Thread.java:836)
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0
{noformat}

PS: I originally ran into this here: 
https://github.com/quarkusio/quarkus/pull/14311#pullrequestreview-569179938



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (SUREFIRE-1801) New TCP communication mode defers "Listening for transport" debug message

2021-01-19 Thread Falko Modler (Jira)


[ 
https://issues.apache.org/jira/browse/SUREFIRE-1801?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268270#comment-17268270
 ] 

Falko Modler commented on SUREFIRE-1801:


[~tibordigana] I had a look at the code back in December and I was totally lost.

Unfortunately, I cannot spend multiple evenings on this but I could help here 
and there, just let me know.

> New TCP communication mode defers "Listening for transport" debug message
> -
>
> Key: SUREFIRE-1801
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1801
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: process forking
>Affects Versions: 3.0.0-M5
> Environment: $ mvn -version
> Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
> Maven home: C:\Develop\some-project\middleware\_develop\maven\default
> Java version: 11.0.7, vendor: AdoptOpenJDK, runtime: 
> C:\Develop\some-project\middleware\_develop\java\default
> Default locale: de_DE, platform encoding: Cp1252
> OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
>Reporter: Falko Modler
>Priority: Minor
> Fix For: 3.0.0-M6
>
>
> When using
> {code:xml}
>  implementation="org.apache.maven.plugin.surefire.extensions.SurefireForkNodeFactory"/>
> {code}
> and {{-Dmaven.surefire.debug}} you see the message
> {noformat}
> Listening for transport dt_socket at address: 5005
> {noformat}
> _only after_ you have connected the debugger, which defeats the whole point 
> of this message and leaves the user under the impression that test execution 
> is stuck before opening the debug port.
> In "legacy" mode the message is displayed as expected (before connecting the 
> debugger).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MRELEASE-1053) scm element removed during release:prepare when leveraging custom inheritance rules

2021-01-19 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/MRELEASE-1053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268266#comment-17268266
 ] 

Herve Boutemy commented on MRELEASE-1053:
-

why is it defined in every POM and not only in parent?
https://github.com/apache/jackrabbit-filevault/commit/f8387a5a1cb46e90350dfdf62da1b194c6ae9f0f#diff-1f25614fa9ce293f1e5a645b1ff32692b242000903dbd29d43ead3ef8e48093e

> scm element removed during release:prepare when leveraging custom inheritance 
> rules
> ---
>
> Key: MRELEASE-1053
> URL: https://issues.apache.org/jira/browse/MRELEASE-1053
> Project: Maven Release Plugin
>  Issue Type: Bug
>  Components: prepare
>Affects Versions: 3.0.0-M1
>Reporter: Konrad Windszus
>Priority: Major
>
> When leveraging the custom inheritance rules being introduced with MNG-6059 
> the m-release-p removes the {{scm}} element from the pom.xml during 
> {{release:prepare}}.
> You find an example for that behaviour at 
> https://github.com/apache/jackrabbit-filevault/commit/3a175724d8d3608bc96e92ed7fd9154fd56d1e30#diff-04dc5d029560773ac79faaf3da0fbb22L68



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (SUREFIRE-1878) Add failOnFlake option

2021-01-19 Thread Stefan Oehme (Jira)


[ 
https://issues.apache.org/jira/browse/SUREFIRE-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268188#comment-17268188
 ] 

Stefan Oehme commented on SUREFIRE-1878:


Or maybe even better `skipAfterFlakeCount` to match `skipAfterFailureCount`.

> Add failOnFlake option
> --
>
> Key: SUREFIRE-1878
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1878
> Project: Maven Surefire
>  Issue Type: New Feature
>  Components: Maven Surefire Plugin
>Reporter: Stefan Oehme
>Priority: Minor
>
> The Surefire plugin currently covers two use cases with its 
> `rerunFailingTestCount` option:
> 1. rerunFailingTestCount=0: The build will fail immediately, but I don't know 
> if it was a real failure or a flake. This option is best for healthy code 
> bases with very little or no flakiness.
> 2. rerunFailingTestCount>0: I will know when a test is flaky, but the build 
> will be successful, so there is no pressure to fix the flakes. This helps 
> when the situation is really bad and I just want to get some green build 
> again to boost team morale :) 
> I'd like to support a third use case:
> 3. rerunFailingTestCount>0 and a new option failOnFlake=true: The build will 
> fail (so there is actually pressure to deal with flakiness) and I can tell 
> the difference between real failures and flakes. I think this would be the 
> best option for projects with a medium amount of flaky tests and a team that 
> wants to take care of them.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MNG-5760) Add `-r/--resume` to automatically resume from the last failure point

2021-01-19 Thread Martin Kanters (Jira)


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

Martin Kanters commented on MNG-5760:
-

[~max.ander...@jboss.com] thanks for your interest in this issue and I agree 
that the Hudson notifications are annoying, I'll see how/if we can avoid that.
Anyhow, it's a feature that will definitely be part of Maven 4. So keep your 
eye on that release :) 

> Add `-r/--resume` to automatically resume from the last failure point
> -
>
> Key: MNG-5760
> URL: https://issues.apache.org/jira/browse/MNG-5760
> Project: Maven
>  Issue Type: Improvement
>  Components: Command Line
>Affects Versions: 3.2.5
>Reporter: Phillip Webb
>Assignee: Robert Scholte
>Priority: Trivial
> Fix For: 4.0.0, 4.0.0-alpha-1
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Currently when a multi-module build fails the {{mvn}} command line prints the 
> following error:
> {noformat}
> [ERROR] After correcting the problems, you can resume the build with the 
> command
> [ERROR]   mvn  -rf :some-module-name
> {noformat}
> Since I almost always want to use this flag with the next build it would be 
> very useful if you could type {{mvn  -rf}} and have the project name 
> inferred from the last failure rather than needing to copy/paste from the 
> terminal.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


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

2021-01-19 Thread GitBox


gnodet commented on pull request #402:
URL: https://github.com/apache/maven/pull/402#issuecomment-763061798


   @rfscholte I've rebased and fix the integration tests.



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.

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




[GitHub] [maven-compiler-plugin] elharo commented on a change in pull request #34: [MCOMPILER-428] improve incremental compilation documentation

2021-01-19 Thread GitBox


elharo commented on a change in pull request #34:
URL: 
https://github.com/apache/maven-compiler-plugin/pull/34#discussion_r560415436



##
File path: 
src/main/java/org/apache/maven/plugin/compiler/AbstractCompilerMojo.java
##
@@ -483,7 +483,19 @@
 private List fileExtensions;
 
 /**
- * to enable/disable incrementation compilation feature
+ * to enable/disable incrementation compilation feature.

Review comment:
   incrementation --> incremental





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.

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




[jira] [Commented] (MRELEASE-1053) scm element removed during release:prepare when leveraging custom inheritance rules

2021-01-19 Thread Konrad Windszus (Jira)


[ 
https://issues.apache.org/jira/browse/MRELEASE-1053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268138#comment-17268138
 ] 

Konrad Windszus commented on MRELEASE-1053:
---

IMHO the prepare release commit looks good. The tag scm element is correctly 
adjusted. Why do you think the tag element is wrong?

> scm element removed during release:prepare when leveraging custom inheritance 
> rules
> ---
>
> Key: MRELEASE-1053
> URL: https://issues.apache.org/jira/browse/MRELEASE-1053
> Project: Maven Release Plugin
>  Issue Type: Bug
>  Components: prepare
>Affects Versions: 3.0.0-M1
>Reporter: Konrad Windszus
>Priority: Major
>
> When leveraging the custom inheritance rules being introduced with MNG-6059 
> the m-release-p removes the {{scm}} element from the pom.xml during 
> {{release:prepare}}.
> You find an example for that behaviour at 
> https://github.com/apache/jackrabbit-filevault/commit/3a175724d8d3608bc96e92ed7fd9154fd56d1e30#diff-04dc5d029560773ac79faaf3da0fbb22L68



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [maven-artifact-plugin] hboutemy commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


hboutemy commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-763046022


   I don't have time to play on that: working on real plugin enhancements is 
more useful than personal taste changes
   
   if such changes is the only feedback I get, I don't need feedback, thank you



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.

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




[jira] [Commented] (MRELEASE-1053) scm element removed during release:prepare when leveraging custom inheritance rules

2021-01-19 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/MRELEASE-1053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268130#comment-17268130
 ] 

Herve Boutemy commented on MRELEASE-1053:
-

uh, strange
IMHO, it's interesting to look at previous "prepare release" commit, because it 
adds unwanted scm/tag elements: 
https://github.com/apache/jackrabbit-filevault/commit/f8387a5a1cb46e90350dfdf62da1b194c6ae9f0f

> scm element removed during release:prepare when leveraging custom inheritance 
> rules
> ---
>
> Key: MRELEASE-1053
> URL: https://issues.apache.org/jira/browse/MRELEASE-1053
> Project: Maven Release Plugin
>  Issue Type: Bug
>  Components: prepare
>Affects Versions: 3.0.0-M1
>Reporter: Konrad Windszus
>Priority: Major
>
> When leveraging the custom inheritance rules being introduced with MNG-6059 
> the m-release-p removes the {{scm}} element from the pom.xml during 
> {{release:prepare}}.
> You find an example for that behaviour at 
> https://github.com/apache/jackrabbit-filevault/commit/3a175724d8d3608bc96e92ed7fd9154fd56d1e30#diff-04dc5d029560773ac79faaf3da0fbb22L68



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [maven-compiler-plugin] rfscholte commented on pull request #34: [MCOMPILER-428] improve incremental compilation documentation

2021-01-19 Thread GitBox


rfscholte commented on pull request #34:
URL: 
https://github.com/apache/maven-compiler-plugin/pull/34#issuecomment-763004082


   @elharo is the native english speaker here



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.

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




[jira] [Commented] (MCOMPILER-428) Documentation regarding useIncrementalCompilation not very useful

2021-01-19 Thread Thomas Bracher (Jira)


[ 
https://issues.apache.org/jira/browse/MCOMPILER-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17268035#comment-17268035
 ] 

Thomas Bracher commented on MCOMPILER-428:
--

I feel like this PR is under the radar. If anyone has time to check it again, 
that'll be much appreciated!

> Documentation regarding useIncrementalCompilation not very useful
> -
>
> Key: MCOMPILER-428
> URL: https://issues.apache.org/jira/browse/MCOMPILER-428
> Project: Maven Compiler Plugin
>  Issue Type: Improvement
>Reporter: Thomas Bracher
>Priority: Minor
>  Labels: documentation
>
> As of today, maven-compiler-plugin documentation for 
> useIncrementalCompilation says: "to enable/disable incrementation compilation 
> feature". Since there is no documentation of said feature, users will have to 
> either:
> 1) take a guess at how the feature behaves
> 2) painfully read the plugin code to understand what it does
> This is less than optimal and lead to misunderstanding, like this comment 
> https://issues.apache.org/jira/browse/MCOMPILER-205?focusedCommentId=14428972=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14428972
>  
> So far the most explicit documentation I have found is in answer of a 
> stackoverflow issue: https://stackoverflow.com/a/49700942
> I think it would benefit a lot of users to understand the implication of both 
> options of this flag. I am trying to submit a PR providing an example of text.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [maven-compiler-plugin] sadraskol commented on pull request #34: [MCOMPILER-428] improve incremental compilation documentation

2021-01-19 Thread GitBox


sadraskol commented on pull request #34:
URL: 
https://github.com/apache/maven-compiler-plugin/pull/34#issuecomment-762978139


   I feel like this PR is under the radar. If anyone has time to check it 
again, that'll be much appreciated!



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.

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




[GitHub] [maven-artifact-plugin] rmannibucau commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


rmannibucau commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-762973071


   @michael-o most of these utilities are in java 8 or are trivial (like 1 or 2 
calls) so the need to argument about a 3rd party disappear for all modules.



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.

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




[jira] [Assigned] (MNG-7078) The integration tests use the default maven settings

2021-01-19 Thread Michael Osipov (Jira)


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

Michael Osipov reassigned MNG-7078:
---

Assignee: Michael Osipov

> The integration tests use the default maven settings
> 
>
> Key: MNG-7078
> URL: https://issues.apache.org/jira/browse/MNG-7078
> Project: Maven
>  Issue Type: Improvement
>  Components: Integration Tests
>Reporter: Guillaume Nodet
>Assignee: Michael Osipov
>Priority: Major
>
> I have a profile that caused one integration test to fail, but it took me 
> some time to realise the problem came from my {{~/.m2/settings.xml}} file.  
> It would be better to have the tests use an empty settings file.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MNG-7078) The integration tests use the default maven settings

2021-01-19 Thread Michael Osipov (Jira)


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

Michael Osipov commented on MNG-7078:
-

MNG-6772? This will be rewritten, removed soon.

> The integration tests use the default maven settings
> 
>
> Key: MNG-7078
> URL: https://issues.apache.org/jira/browse/MNG-7078
> Project: Maven
>  Issue Type: Improvement
>  Components: Integration Tests
>Reporter: Guillaume Nodet
>Priority: Major
>
> I have a profile that caused one integration test to fail, but it took me 
> some time to realise the problem came from my {{~/.m2/settings.xml}} file.  
> It would be better to have the tests use an empty settings file.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Created] (MNG-7078) The integration tests use the default maven settings

2021-01-19 Thread Guillaume Nodet (Jira)
Guillaume Nodet created MNG-7078:


 Summary: The integration tests use the default maven settings
 Key: MNG-7078
 URL: https://issues.apache.org/jira/browse/MNG-7078
 Project: Maven
  Issue Type: Improvement
  Components: Integration Tests
Reporter: Guillaume Nodet


I have a profile that caused one integration test to fail, but it took me some 
time to realise the problem came from my {{~/.m2/settings.xml}} file.  It would 
be better to have the tests use an empty settings file.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Updated] (SUREFIRE-1878) Add failOnFlake option

2021-01-19 Thread Stefan Oehme (Jira)


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

Stefan Oehme updated SUREFIRE-1878:
---
Description: 
The Surefire plugin currently covers two use cases with its 
`rerunFailingTestCount` option:

1. rerunFailingTestCount=0: The build will fail immediately, but I don't know 
if it was a real failure or a flake. This option is best for healthy code bases 
with very little or no flakiness.
2. rerunFailingTestCount>0: I will know when a test is flaky, but the build 
will be successful, so there is no pressure to fix the flakes. This helps when 
the situation is really bad and I just want to get some green build again to 
boost team morale :) 

I'd like to support a third use case:

3. rerunFailingTestCount>0 and a new option failOnFlake=true: The build will 
fail (so there is actually pressure to deal with flakiness) and I can tell the 
difference between real failures and flakes. I think this would be the best 
option for projects with a medium amount of flaky tests and a team that wants 
to take care of them.

  was:
The Surefire plugin currently covers two use cases with its 
`rerunFailingTestCount` option:

1. rerunFailingTestCount=0: The build will fail on failed tests, but I don't 
know if it was a real failure or a flake. This option is best for healthy code 
bases with very little or no flakiness.
2. rerunFailingTestCount>0: I will know when a test is flaky, but the build 
will be successful, so there is no pressure to fix the flakes. This helps when 
the situation is really bad and I just want to get some green build again to 
boost team morale :) 

I'd like to support a third use case:

3. rerunFailingTestCount>0 and a new option failOnFlake=true: The build will 
fail (so there is actually pressure to deal with flakiness) and I can tell the 
difference between real failures and flakes. I think this would be the best 
option for projects with a medium amount of flaky tests and a team that wants 
to take care of them.


> Add failOnFlake option
> --
>
> Key: SUREFIRE-1878
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1878
> Project: Maven Surefire
>  Issue Type: New Feature
>  Components: Maven Surefire Plugin
>Reporter: Stefan Oehme
>Priority: Minor
>
> The Surefire plugin currently covers two use cases with its 
> `rerunFailingTestCount` option:
> 1. rerunFailingTestCount=0: The build will fail immediately, but I don't know 
> if it was a real failure or a flake. This option is best for healthy code 
> bases with very little or no flakiness.
> 2. rerunFailingTestCount>0: I will know when a test is flaky, but the build 
> will be successful, so there is no pressure to fix the flakes. This helps 
> when the situation is really bad and I just want to get some green build 
> again to boost team morale :) 
> I'd like to support a third use case:
> 3. rerunFailingTestCount>0 and a new option failOnFlake=true: The build will 
> fail (so there is actually pressure to deal with flakiness) and I can tell 
> the difference between real failures and flakes. I think this would be the 
> best option for projects with a medium amount of flaky tests and a team that 
> wants to take care of them.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Created] (SUREFIRE-1878) Add failOnFlake option

2021-01-19 Thread Stefan Oehme (Jira)
Stefan Oehme created SUREFIRE-1878:
--

 Summary: Add failOnFlake option
 Key: SUREFIRE-1878
 URL: https://issues.apache.org/jira/browse/SUREFIRE-1878
 Project: Maven Surefire
  Issue Type: New Feature
  Components: Maven Surefire Plugin
Reporter: Stefan Oehme


The Surefire plugin currently covers two use cases with its 
`rerunFailingTestCount` option:

1. rerunFailingTestCount=0: The build will fail on failed tests, but I don't 
know if it was a real failure or a flake. This option is best for healthy code 
bases with very little or no flakiness.
2. rerunFailingTestCount>0: I will know when a test is flaky, but the build 
will be successful, so there is no pressure to fix the flakes. This helps when 
the situation is really bad and I just want to get some green build again to 
boost team morale :) 

I'd like to support a third use case:

3. rerunFailingTestCount>0 and a new option failOnFlake=true: The build will 
fail (so there is actually pressure to deal with flakiness) and I can tell the 
difference between real failures and flakes. I think this would be the best 
option for projects with a medium amount of flaky tests and a team that wants 
to take care of them.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MNG-5760) Add `-r/--resume` to automatically resume from the last failure point

2021-01-19 Thread Max Rydahl Andersen (Jira)


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

Max Rydahl Andersen commented on MNG-5760:
--

would love to keep following this issue but with hudson being massive noise 
generator I'll have to unsubscribe.

ping me if something need to test/verify/feedback on.

> Add `-r/--resume` to automatically resume from the last failure point
> -
>
> Key: MNG-5760
> URL: https://issues.apache.org/jira/browse/MNG-5760
> Project: Maven
>  Issue Type: Improvement
>  Components: Command Line
>Affects Versions: 3.2.5
>Reporter: Phillip Webb
>Assignee: Robert Scholte
>Priority: Trivial
> Fix For: 4.0.0, 4.0.0-alpha-1
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Currently when a multi-module build fails the {{mvn}} command line prints the 
> following error:
> {noformat}
> [ERROR] After correcting the problems, you can resume the build with the 
> command
> [ERROR]   mvn  -rf :some-module-name
> {noformat}
> Since I almost always want to use this flag with the next build it would be 
> very useful if you could type {{mvn  -rf}} and have the project name 
> inferred from the last failure rather than needing to copy/paste from the 
> terminal.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [maven-artifact-plugin] michael-o commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


michael-o commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-762865807


   > 
   > 
   > Lets validate we move to j8 for that too, then all these utilities are 
useless.
   
   What do you mean by that?



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

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




[GitHub] [maven-artifact-plugin] rmannibucau commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


rmannibucau commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-762854674


   Lets validate we move to j8 for that too, then all these utilities are 
useless.



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.

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




[GitHub] [maven-artifact-plugin] michael-o commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


michael-o commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-762829347


   I fully agree with @elharo 



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.

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




[GitHub] [maven-surefire] Tibor17 commented on pull request #331: [SUREFIRE-1876] Added support for skipping unit test execution not affecting integration test execution

2021-01-19 Thread GitBox


Tibor17 commented on pull request #331:
URL: https://github.com/apache/maven-surefire/pull/331#issuecomment-762823393


   We are aiming for splitting both plugins, so that each of them would be 
controlled by unique property. But i think we do not have to create a new 
property.



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.

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




[GitHub] [maven-artifact-plugin] elharo commented on pull request #7: prefer Apache Commons StringUtils

2021-01-19 Thread GitBox


elharo commented on pull request #7:
URL: 
https://github.com/apache/maven-artifact-plugin/pull/7#issuecomment-762820234


   Yes, that's why maven-shared-utils was made. And plexus-utils. And Apache 
commons lang. And Guava. Programmers seem really fond of building utility 
libraries to do the same hundred or so not-especially-difficult operations. We 
should standardize on one, and apache commons lang is by far the best 
maintained such library already in our tree. (Guava's good too but is generally 
not common in Apache projects. It also has some pretty bad version conflicts.)
   
   maven-shared-utils should be used for methods that are maven specific in 
some way, not general string and file manipulation utilities. Ultimately I want 
to deprecate and remove all such general purpose methods from 
maven-shared-utils so we have less generic utility code to maintain and can 
focus on Maven itself. 
   
   



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.

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




[jira] [Created] (MDEPLOY-283) Improved configuration syntax for deploy-file

2021-01-19 Thread Gintas Grigelionis (Jira)
Gintas Grigelionis created MDEPLOY-283:
--

 Summary: Improved configuration syntax for deploy-file
 Key: MDEPLOY-283
 URL: https://issues.apache.org/jira/browse/MDEPLOY-283
 Project: Maven Deploy Plugin
  Issue Type: Improvement
Reporter: Gintas Grigelionis


The configuration syntax for deployment of multiple files is ugly: , 
 and  as unrelated comma-separated and repetitive (for 
types) lists. I would like to propose

{{}}
 {{  }}
 {{    ...}}
 {{    ...}}
 {{    ...}}
 {{  }}
 {{}}

as a complement (and maybe a replacement). 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Comment Edited] (MRELEASE-1053) scm element removed during release:prepare when leveraging custom inheritance rules

2021-01-19 Thread Konrad Windszus (Jira)


[ 
https://issues.apache.org/jira/browse/MRELEASE-1053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17267820#comment-17267820
 ] 

Konrad Windszus edited comment on MRELEASE-1053 at 1/19/21, 11:16 AM:
--

It happened again in 
https://github.com/apache/jackrabbit-filevault/commit/00ec91784b52f3b87ff890535228d45e37a4630f.
 
[~hboutemy] Do you have any pointers what may go wrong here, as you implemented 
https://issues.apache.org/jira/browse/MNG-6059?

Probably something in 
https://github.com/apache/maven-release/blob/44ff401499713c981732f3379de2cf6693e0fdef/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhase.java#L48
 goes wrong.


was (Author: kwin):
It happened again in 
https://github.com/apache/jackrabbit-filevault/commit/00ec91784b52f3b87ff890535228d45e37a4630f.
 
[~hboutemy] Do you have any pointers what may go wrong here, as you implemented 
https://issues.apache.org/jira/browse/MNG-6059?

> scm element removed during release:prepare when leveraging custom inheritance 
> rules
> ---
>
> Key: MRELEASE-1053
> URL: https://issues.apache.org/jira/browse/MRELEASE-1053
> Project: Maven Release Plugin
>  Issue Type: Bug
>  Components: prepare
>Affects Versions: 3.0.0-M1
>Reporter: Konrad Windszus
>Priority: Major
>
> When leveraging the custom inheritance rules being introduced with MNG-6059 
> the m-release-p removes the {{scm}} element from the pom.xml during 
> {{release:prepare}}.
> You find an example for that behaviour at 
> https://github.com/apache/jackrabbit-filevault/commit/3a175724d8d3608bc96e92ed7fd9154fd56d1e30#diff-04dc5d029560773ac79faaf3da0fbb22L68



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MRELEASE-1053) scm element removed during release:prepare when leveraging custom inheritance rules

2021-01-19 Thread Konrad Windszus (Jira)


[ 
https://issues.apache.org/jira/browse/MRELEASE-1053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17267820#comment-17267820
 ] 

Konrad Windszus commented on MRELEASE-1053:
---

It happened again in 
https://github.com/apache/jackrabbit-filevault/commit/00ec91784b52f3b87ff890535228d45e37a4630f.
 
[~hboutemy] Do you have any pointers what may go wrong here, as you implemented 
https://issues.apache.org/jira/browse/MNG-6059?

> scm element removed during release:prepare when leveraging custom inheritance 
> rules
> ---
>
> Key: MRELEASE-1053
> URL: https://issues.apache.org/jira/browse/MRELEASE-1053
> Project: Maven Release Plugin
>  Issue Type: Bug
>  Components: prepare
>Affects Versions: 3.0.0-M1
>Reporter: Konrad Windszus
>Priority: Major
>
> When leveraging the custom inheritance rules being introduced with MNG-6059 
> the m-release-p removes the {{scm}} element from the pom.xml during 
> {{release:prepare}}.
> You find an example for that behaviour at 
> https://github.com/apache/jackrabbit-filevault/commit/3a175724d8d3608bc96e92ed7fd9154fd56d1e30#diff-04dc5d029560773ac79faaf3da0fbb22L68



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MDEPLOY-272) Add skip capability to deploy-file goal

2021-01-19 Thread Gintas Grigelionis (Jira)


[ 
https://issues.apache.org/jira/browse/MDEPLOY-272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17267819#comment-17267819
 ] 

Gintas Grigelionis commented on MDEPLOY-272:


I assume that this would prevent the deploy plugin from complaining about the 
configuration being incomplete, too?

> Add skip capability to deploy-file goal
> ---
>
> Key: MDEPLOY-272
> URL: https://issues.apache.org/jira/browse/MDEPLOY-272
> Project: Maven Deploy Plugin
>  Issue Type: New Feature
>  Components: deploy:deploy-file
>Affects Versions: 3.0.0-M1
>Reporter: Tom Esh
>Priority: Major
>
> Currently the deploy-file goal doesn't support the  
> configuration parameter.
> I have a case where that would be very useful, as I would like to 
> conditionally deploy a file based on a maven property.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)