Re[2]: [VOTE] Release Apache Maven Javadoc Plugin version 2.10.4

2016-07-08 Thread Richard Sand 
Hi Robert - I attached a patch to MJAVADOC-452 for the onlySaveIfChanged 
fix following your suggestion - I pass a change-detection boolean into 
the rewriting methods so we know if a change has been made during 
processing.


Best regards,

Richard

-- Original Message --
From: "Robert Scholte" 
To: "Maven Developers List" 
Sent: 6/29/2016 1:34:19 PM
Subject: Re: [VOTE] Release Apache Maven Javadoc Plugin version 2.10.4

Let's start with a simple segment: don't update the file if there are 
no  changes. That's an isolated issue.
You suggestion is to do a String compare, which seems to me quite  
expensive with large files. Instead I would introduce a changed-flag.  
AFAIK all rewriting methods are now void-methods. Return true if 
changed;  if changed then rewrite file.

After that we can go for the next part.

Robert

On Wed, 29 Jun 2016 06:30:34 +0200, Richard Sand  
 wrote:




Hi Maven Developers - now that Javadoc plugin 2.10.4 is released and
work has commenced on the 3.0 version, can we revisit the patch I
submitted for MJAVADOC-452? Its difficult to break the patch into
smaller patches because most of the fixes are interrelated, but I'd be
happy to recreate the fixes on the 2.10.4 tag or the latest trunk. How
can I help?

-Richard

Richard Sand 
June 8, 2016 at 2:11 PM
Hi Robert,

For the skip parameter, the test is very simple. Usage of the fix 
goal
without disabling CLIRR will result in a loop where fix invokes 
clirr,

and clirr runs fix.

I understand your reticence about this skip option - but in scenarios
where one wants to have the plugin execute automatically with 
"force",

having an override is useful. So from my perspective it is useful and
it follows the patterns that other plugins use, including the javadoc
plugin - just this goal is missing a skip option.

I didn't break the patch up into smaller patches, but I did annotate
all of the changes that the patch performs line-by-line in the Jira
case. I hope that helps "demystify" the fixes.

Best regards,

Richard


Robert Scholte 
June 8, 2016 at 2:01 PM
Hi Richard,

this release is to push the final 2.x version before moving to a pure
Maven 3 implementation (there were quite some fixed issues waiting to
be released).
Once released I will continue, also including some Java9 related
improvements. That will give me enough time to go through your
patches. Since you weren't able to split it into smaller pieces I 
need

more time to verify and apply it.

thanks,
Robert

ps. In case you are still convinced the skip parameter is required,
you need a complete testcase to show it to me. In general *I* won't
apply any requests to add a skip parameter, there are often better
solutions.

On Wed, 08 Jun 2016 00:53:33 +0200, Richard Sand
 wrote:


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org




-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Re: Support for Java 9 builds?

2016-07-08 Thread Mark Derricutt 
On 9 Jul 2016, at 10:29, Robert Scholte wrote:

> However, looking at the stacktrace this doesn't seem to be the issue for you.
> It probably has to do with the calculation of the java version, which is 
> already fixed.
> So please add a newer version of plexus-archiver to the maven-javadoc-plugin 
> and verify if this works.

Thats works!  Also had to bump the javadoc plugin to .4 ( I was still using .3 
) - excellent.


-- 
Mark Derricutt
http://www.theoryinpractice.net
http://www.chaliceofblood.net
http://plus.google.com/+MarkDerricutt
http://twitter.com/talios
http://facebook.com/mderricutt


signature.asc
Description: OpenPGP digital signature

[GitHub] maven-surefire issue #44: Fixed NullPointerException on SmartStackTraceParse...

2016-07-08 Thread digantchauhan 
Github user digantchauhan commented on the issue:

https://github.com/apache/maven-surefire/pull/44
  
We have found out that scenario name was having parentheses in scenario 
name, so surefire was failing in parsing the scenario name. So no action is 
required.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

[GitHub] maven-surefire issue #44: Fixed NullPointerException on SmartStackTraceParse...

2016-07-08 Thread digantchauhan 
Github user digantchauhan commented on the issue:

https://github.com/apache/maven-surefire/pull/44
  
Hi, I am seeing the similar issue in latest surefire:
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test (default-test) on 
project accounts-webservice: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test failed: There was an 
error in the forked process
[ERROR] org.apache.maven.surefire.testset.TestSetFailedException: 
java.lang.NullPointerException
[ERROR] at 
org.apache.maven.surefire.common.junit4.JUnit4RunListener.rethrowAnyTestMechanismFailures(JUnit4RunListener.java:209)
[ERROR] at 
org.apache.maven.surefire.junitcore.JUnitCoreWrapper.createRequestAndRun(JUnitCoreWrapper.java:138)
[ERROR] at 
org.apache.maven.surefire.junitcore.JUnitCoreWrapper.executeEager(JUnitCoreWrapper.java:107)
[ERROR] at 
org.apache.maven.surefire.junitcore.JUnitCoreWrapper.execute(JUnitCoreWrapper.java:83)
[ERROR] at 
org.apache.maven.surefire.junitcore.JUnitCoreWrapper.execute(JUnitCoreWrapper.java:75)
[ERROR] at 
org.apache.maven.surefire.junitcore.JUnitCoreProvider.invoke(JUnitCoreProvider.java:161)
[ERROR] at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:290)
[ERROR] at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:242)
[ERROR] at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:121)
[ERROR] Caused by: java.lang.NullPointerException
[ERROR] at 
org.apache.maven.surefire.junitcore.ConcurrentRunListener.testStarting(ConcurrentRunListener.java:135)
[ERROR] at 
org.apache.maven.surefire.common.junit4.JUnit4RunListener.testStarted(JUnit4RunListener.java:91)
[ERROR] at 
org.junit.runner.notification.RunNotifier$3.notifyListener(RunNotifier.java:115)
[ERROR] at 
org.junit.runner.notification.RunNotifier$SafeNotifier.run(RunNotifier.java:61)
[ERROR] at 
org.junit.runner.notification.RunNotifier.fireTestStarted(RunNotifier.java:112)
[ERROR] at 
org.apache.maven.surefire.common.junit4.Notifier.fireTestStarted(Notifier.java:100)
[ERROR] at 
org.junit.internal.runners.model.EachTestNotifier.fireTestStarted(EachTestNotifier.java:43)
[ERROR] at 
cucumber.runtime.junit.JUnitReporter.startExecutionUnit(JUnitReporter.java:50)
[ERROR] at 
cucumber.runtime.junit.ExecutionUnitRunner.run(ExecutionUnitRunner.java:89)
[ERROR] at 
cucumber.runtime.junit.FeatureRunner.runChild(FeatureRunner.java:63)
[ERROR] at 
cucumber.runtime.junit.FeatureRunner.runChild(FeatureRunner.java:18)
[ERROR] at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
[ERROR] at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
[ERROR] at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
[ERROR] at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
[ERROR] at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
[ERROR] at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
[ERROR] at cucumber.runtime.junit.FeatureRunner.run(FeatureRunner.java:70)
[ERROR] at cucumber.api.junit.Cucumber.runChild(Cucumber.java:93)
[ERROR] at cucumber.api.junit.Cucumber.runChild(Cucumber.java:37)
[ERROR] at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
[ERROR] at 
org.apache.maven.surefire.junitcore.pc.Scheduler$1.run(Scheduler.java:393)
[ERROR] at 
org.apache.maven.surefire.junitcore.pc.InvokerStrategy.schedule(InvokerStrategy.java:54)
[ERROR] at 
org.apache.maven.surefire.junitcore.pc.Scheduler.schedule(Scheduler.java:352)
[ERROR] at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
[ERROR] at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
[ERROR] at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
[ERROR] at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
[ERROR] at 
org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
[ERROR] at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
[ERROR] at cucumber.api.junit.Cucumber.run(Cucumber.java:98)
[ERROR] at org.junit.runners.Suite.runChild(Suite.java:127)
[ERROR] at org.junit.runners.Suite.runChild(Suite.java:26)
[ERROR] at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
[ERROR] at 
org.apache.maven.surefire.junitcore.pc.Scheduler$1.run(Scheduler.java:393)
[ERROR] at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
[ERROR] at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[ERROR] at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
[ERROR] at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
[ERROR] at java.lang.Thread.run(Thread.java:745)


---
If your