[jira] [Commented] (SUREFIRE-1227) The Description argument RunListener#testRunStarted(Description) doesn't include test methods

2016-02-09 Thread Victor Sidochenko (JIRA)

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

Victor Sidochenko commented on SUREFIRE-1227:
-

[~tibor17]
It is important for me. I`m using (in 
[allure-cucumber-jvm-adaptor|https://github.com/allure-framework/allure-cucumber-jvm-adaptor])
 description with all its childern in testRunStarted to find out parents for 
descriptions passed from testStarted. 

And one question. Junit (since 4.12) has uniqueId parameter as argument to 
create description and cucumber-jvm uses it to find out needed objects.
In surefire-plugin 2.18.1 Description wich comes as parameter to testRunStarted 
has different uniqueIds than in testStarted/testIfnored. This behaviour happens 
only if RunListener passed from pom.xml as surefire-plugin configuration 
parameter. If it set from junit test - all fine and uniqueIds are valid.
So if you will decide to return back child descriptions, could you look at this 
problem? [Details|https://github.com/cucumber/cucumber-jvm/issues/959].

> The Description argument RunListener#testRunStarted(Description) doesn't 
> include test methods
> -
>
> Key: SUREFIRE-1227
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1227
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19.1
>Reporter: Jiri Locker
> Attachments: maven-surefire-run-listener.zip
>
>
> In Surefire 2.19 the Description passed to testRunStarted() is a test run 
> description and has 2 levels of children (test classes > test methods). The 
> top level Description's testCount property is equal to the number of test 
> methods in all test classes that are going to be run. When dumping the 
> description tree it may look like this:
> {noformat}
> Description: ~ null (6)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (2)
> Description: ~ ~ ~ 
> testMethod1(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ ~ 
> testMethod2(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (4)
> Description: ~ ~ ~ 
> testMethodA(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodB(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodC(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodD(io.github.yurloc.example.runlistener.MySecondTest) (1)
> {noformat}
> However in Surefire 2.19.1 the leaf descriptions of test methods are missing:
> {noformat}
> Description: ~ null (2)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (1)
> {noformat}
> Because of this I am unable to access the total test count, which I use to 
> calculate ETA of the test run finish. I seem that this change of behavior was 
> introduced by fixing SUREFIRE-1187.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (SUREFIRE-1227) The Description argument RunListener#testRunStarted(Description) doesn't include test methods

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana edited comment on SUREFIRE-1227 at 2/10/16 2:41 AM:
-

[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children. That's the reason we did not use 
JUnit to ask for children. On the other side we did pass only description 
without having the children. In order to substitute JUnit it would never be 
successful without using real runner. Some runners are not so transparent 
(Parameterized, Cucumber) like pure non-parameterized JUnit runner and better 
to prevent from null description however with missing children.


was (Author: tibor17):
[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children. That's the reason we did not use 
JUnit to ask for children. On the other side we did pass only description 
without having the children. In order to substitute JUnit it would never be 
successful without using real runner. Some runners are not so transparent 
(Parameterized, Cucumber) like JUnit runners and better to prevent from null 
description however missing children.

> The Description argument RunListener#testRunStarted(Description) doesn't 
> include test methods
> -
>
> Key: SUREFIRE-1227
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1227
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19.1
>Reporter: Jiri Locker
> Attachments: maven-surefire-run-listener.zip
>
>
> In Surefire 2.19 the Description passed to testRunStarted() is a test run 
> description and has 2 levels of children (test classes > test methods). The 
> top level Description's testCount property is equal to the number of test 
> methods in all test classes that are going to be run. When dumping the 
> description tree it may look like this:
> {noformat}
> Description: ~ null (6)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (2)
> Description: ~ ~ ~ 
> testMethod1(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ ~ 
> testMethod2(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (4)
> Description: ~ ~ ~ 
> testMethodA(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodB(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodC(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodD(io.github.yurloc.example.runlistener.MySecondTest) (1)
> {noformat}
> However in Surefire 2.19.1 the leaf descriptions of test methods are missing:
> {noformat}
> Description: ~ null (2)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (1)
> {noformat}
> Because of this I am unable to access the total test count, which I use to 
> calculate ETA of the test run finish. I seem that this change of behavior was 
> introduced by fixing SUREFIRE-1187.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (SUREFIRE-1227) The Description argument RunListener#testRunStarted(Description) doesn't include test methods

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana edited comment on SUREFIRE-1227 at 2/10/16 2:39 AM:
-

[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children. That's the reason we did not use 
JUnit to ask for children. On the other side we did pass only empty description 
without discoverying the children. In order to substitute JUnit it would never 
be successful without using real runner. Some runners are not so transparent 
(Parameterized, Cucumber) like JUnit runners and better to prevent from null 
description however missing children.


was (Author: tibor17):
[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children.
That's the reason we did not use JUnit to ask for children. On the other side 
we did pass only empty description without discoverying the children. In order 
to substitute JUnit it would never be successful without using real runner. 
Some runners are not so transparent (Cucumber) like JUnit runners and better to 
prevent from null description however missing children.

> The Description argument RunListener#testRunStarted(Description) doesn't 
> include test methods
> -
>
> Key: SUREFIRE-1227
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1227
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19.1
>Reporter: Jiri Locker
> Attachments: maven-surefire-run-listener.zip
>
>
> In Surefire 2.19 the Description passed to testRunStarted() is a test run 
> description and has 2 levels of children (test classes > test methods). The 
> top level Description's testCount property is equal to the number of test 
> methods in all test classes that are going to be run. When dumping the 
> description tree it may look like this:
> {noformat}
> Description: ~ null (6)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (2)
> Description: ~ ~ ~ 
> testMethod1(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ ~ 
> testMethod2(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (4)
> Description: ~ ~ ~ 
> testMethodA(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodB(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodC(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodD(io.github.yurloc.example.runlistener.MySecondTest) (1)
> {noformat}
> However in Surefire 2.19.1 the leaf descriptions of test methods are missing:
> {noformat}
> Description: ~ null (2)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (1)
> {noformat}
> Because of this I am unable to access the total test count, which I use to 
> calculate ETA of the test run finish. I seem that this change of behavior was 
> introduced by fixing SUREFIRE-1187.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (SUREFIRE-1227) The Description argument RunListener#testRunStarted(Description) doesn't include test methods

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana edited comment on SUREFIRE-1227 at 2/10/16 2:40 AM:
-

[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children. That's the reason we did not use 
JUnit to ask for children. On the other side we did pass only description 
without having the children. In order to substitute JUnit it would never be 
successful without using real runner. Some runners are not so transparent 
(Parameterized, Cucumber) like JUnit runners and better to prevent from null 
description however missing children.


was (Author: tibor17):
[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children. That's the reason we did not use 
JUnit to ask for children. On the other side we did pass only empty description 
without discoverying the children. In order to substitute JUnit it would never 
be successful without using real runner. Some runners are not so transparent 
(Parameterized, Cucumber) like JUnit runners and better to prevent from null 
description however missing children.

> The Description argument RunListener#testRunStarted(Description) doesn't 
> include test methods
> -
>
> Key: SUREFIRE-1227
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1227
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19.1
>Reporter: Jiri Locker
> Attachments: maven-surefire-run-listener.zip
>
>
> In Surefire 2.19 the Description passed to testRunStarted() is a test run 
> description and has 2 levels of children (test classes > test methods). The 
> top level Description's testCount property is equal to the number of test 
> methods in all test classes that are going to be run. When dumping the 
> description tree it may look like this:
> {noformat}
> Description: ~ null (6)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (2)
> Description: ~ ~ ~ 
> testMethod1(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ ~ 
> testMethod2(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (4)
> Description: ~ ~ ~ 
> testMethodA(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodB(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodC(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodD(io.github.yurloc.example.runlistener.MySecondTest) (1)
> {noformat}
> However in Surefire 2.19.1 the leaf descriptions of test methods are missing:
> {noformat}
> Description: ~ null (2)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (1)
> {noformat}
> Because of this I am unable to access the total test count, which I use to 
> calculate ETA of the test run finish. I seem that this change of behavior was 
> introduced by fixing SUREFIRE-1187.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MNG-5626) Avoid negative durations or handle them correctly

2016-02-09 Thread Christian Schulte (JIRA)

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

Christian Schulte commented on MNG-5626:


Ok. Are you planning to work on this?

> Avoid negative durations or handle them correctly
> -
>
> Key: MNG-5626
> URL: https://issues.apache.org/jira/browse/MNG-5626
> Project: Maven
>  Issue Type: Bug
>  Components: Logging
>Affects Versions: 3.2.1
>Reporter: Christian Jung
>Priority: Minor
>
> In issue MNG-5623 we reported an exception when printing the reactor summary 
> if one of the times was negative.
> I saw in one case, that the overall maven build time, as measured from 
> outside (i.e. by our QuickBuild system) was -10.8 seconds. The corresponding 
> reactor summary was:
> {code}
> 13:55:25,184 INFO  - Reactor Summary:
> 13:55:25,184 INFO  -
> 13:55:25,184 INFO  - module1 ... 
> SUCCESS [ 5.911 s]
> 13:55:25,184 INFO  - module2 ... 
> SUCCESS [ 0.255 s]
> 13:55:25,184 INFO  - gpPlaygroundBase-lnx-x64-gcc4 . 
> SUCCESS [-27.-64 s]
> 13:55:25,185 INFO  - 
> 
> 13:55:25,185 INFO  - BUILD SUCCESS
> 13:55:25,185 INFO  - 
> 
> 13:55:25,185 INFO  - Total time: -20.-73 s
> 13:55:25,185 INFO  - Finished at: 2014-04-28T13:55:25+01:00
> 13:55:25,572 INFO  - Final Memory: 32M/439M
> 13:55:25,572 INFO  - 
> 
> {code}
> The thing is quite hard to reproduce, the machines were virtual machines that 
> have been running for quite a long time.
> Our administrators suspected that just at this point, the local clock was 
> synchronized with some outer source.
> We should check if such negative durations can be avoided, and if not, they 
> should be handled correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SUREFIRE-1227) The Description argument RunListener#testRunStarted(Description) doesn't include test methods

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana commented on SUREFIRE-1227:


[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children.
That's the reason we did not use JUnit to ask for children. On the other side 
we did passed only empty description without discoverying the children. In 
order to substitute JUnit it would never be successful without using real 
runner. Some runners are not so transparent (Cucumber) like JUnit runners and 
better to prevent from null description however missing children.

> The Description argument RunListener#testRunStarted(Description) doesn't 
> include test methods
> -
>
> Key: SUREFIRE-1227
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1227
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19.1
>Reporter: Jiri Locker
> Attachments: maven-surefire-run-listener.zip
>
>
> In Surefire 2.19 the Description passed to testRunStarted() is a test run 
> description and has 2 levels of children (test classes > test methods). The 
> top level Description's testCount property is equal to the number of test 
> methods in all test classes that are going to be run. When dumping the 
> description tree it may look like this:
> {noformat}
> Description: ~ null (6)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (2)
> Description: ~ ~ ~ 
> testMethod1(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ ~ 
> testMethod2(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (4)
> Description: ~ ~ ~ 
> testMethodA(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodB(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodC(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodD(io.github.yurloc.example.runlistener.MySecondTest) (1)
> {noformat}
> However in Surefire 2.19.1 the leaf descriptions of test methods are missing:
> {noformat}
> Description: ~ null (2)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (1)
> {noformat}
> Because of this I am unable to access the total test count, which I use to 
> calculate ETA of the test run finish. I seem that this change of behavior was 
> introduced by fixing SUREFIRE-1187.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (SUREFIRE-1227) The Description argument RunListener#testRunStarted(Description) doesn't include test methods

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana edited comment on SUREFIRE-1227 at 2/10/16 2:32 AM:
-

[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children.
That's the reason we did not use JUnit to ask for children. On the other side 
we did pass only empty description without discoverying the children. In order 
to substitute JUnit it would never be successful without using real runner. 
Some runners are not so transparent (Cucumber) like JUnit runners and better to 
prevent from null description however missing children.


was (Author: tibor17):
[~jloc...@redhat.com]
Is this fix very important for you?
We had to do this because otherwise you would reach NPE since of null in the 
method argument. Other users argued that Arquillian runner was started twice, 
it's because of discoverying the children.
That's the reason we did not use JUnit to ask for children. On the other side 
we did passed only empty description without discoverying the children. In 
order to substitute JUnit it would never be successful without using real 
runner. Some runners are not so transparent (Cucumber) like JUnit runners and 
better to prevent from null description however missing children.

> The Description argument RunListener#testRunStarted(Description) doesn't 
> include test methods
> -
>
> Key: SUREFIRE-1227
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1227
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19.1
>Reporter: Jiri Locker
> Attachments: maven-surefire-run-listener.zip
>
>
> In Surefire 2.19 the Description passed to testRunStarted() is a test run 
> description and has 2 levels of children (test classes > test methods). The 
> top level Description's testCount property is equal to the number of test 
> methods in all test classes that are going to be run. When dumping the 
> description tree it may look like this:
> {noformat}
> Description: ~ null (6)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (2)
> Description: ~ ~ ~ 
> testMethod1(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ ~ 
> testMethod2(io.github.yurloc.example.runlistener.MyFirstTest) (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (4)
> Description: ~ ~ ~ 
> testMethodA(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodB(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodC(io.github.yurloc.example.runlistener.MySecondTest) (1)
> Description: ~ ~ ~ 
> testMethodD(io.github.yurloc.example.runlistener.MySecondTest) (1)
> {noformat}
> However in Surefire 2.19.1 the leaf descriptions of test methods are missing:
> {noformat}
> Description: ~ null (2)
> Description: ~ ~ io.github.yurloc.example.runlistener.MyFirstTest (1)
> Description: ~ ~ io.github.yurloc.example.runlistener.MySecondTest (1)
> {noformat}
> Because of this I am unable to access the total test count, which I use to 
> calculate ETA of the test run finish. I seem that this change of behavior was 
> introduced by fixing SUREFIRE-1187.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SUREFIRE-1225) Can't run a single parameterized test

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana commented on SUREFIRE-1225:


Surefire did exactly as you said. Sometimes the developers rely on wrong 
behavior of previous version of Surefire. The pattern internally does not make 
any assumptions and does not need to add wildcard after #testWithParameters. 
That's maybe what the user should do and then Surefire algorithms are more 
strict in finding the matching method names.
The problem is that the method name is not the one you can see in sources. To 
understand this the user needs to have a little knowhow of JUnit internals 
because the test method is always {{described}} with different name if using 
{{Parameterized}}. The reason is simple in JUnit so that every test must have a 
unique {{Description}} and because parameterized tests could break this rule if 
using always the same method name, the JUnit runner added brackets behind your 
test.
Please read our documentation. You have two option, use wildcard or brackets 
after method name in the pattern.
https://maven.apache.org/surefire/maven-surefire-plugin/examples/single-test.html#Multiple_Formats_in_One

> Can't run a single parameterized test
> -
>
> Key: SUREFIRE-1225
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1225
> Project: Maven Surefire
>  Issue Type: Bug
>Affects Versions: 2.19.1
>Reporter: Dmitry Baev
>
> The sample test to reproduce the problem:
> {code:java|title=com/github/baev/MyTest.java}
> @RunWith(Parameterized.class)
> public class MyTest {
> @Parameterized.Parameters
> public static Collection data() {
> return Arrays.asList(
> new Object[]{},
> new Object[]{}
> );
> }
> @Test
> public void testWithParameters() throws Exception {
> }
> }
> {code}
> and then run 
> {{clean test -Dtest=com.github.baev.MyTest#testWithParameters}}
> The output is
> {{Failed to execute goal 
> org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test (default-test) on 
> project surefire-base-test-bug: No tests were executed!}}
> Is it expected?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SUREFIRE-1226) Surefire trims all stack trace elements except those in the test that failed

2016-02-09 Thread Tibor Digana (JIRA)

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

Tibor Digana commented on SUREFIRE-1226:


[~jloc...@redhat.com]
This was introduced with a fix in 2.19. The parameter {{trimStackTrace}} was 
always set to {{true}}. We only covered the description behind this parameter 
and thus we removed lines which do not list the location of the test.
https://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#trimStackTrace
If you set the parameter to {{false}} you will have your expected result. I 
think the fix was ok from my PoV and we can close this issue.

> Surefire trims all stack trace elements except those in the test that failed
> 
>
> Key: SUREFIRE-1226
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1226
> Project: Maven Surefire
>  Issue Type: Bug
>  Components: Maven Surefire Plugin
>Affects Versions: 2.19
>Reporter: Jiri Locker
>Priority: Minor
> Attachments: mvn_stacktrace.tar.gz
>
>
> This looks like a re-occurrence of SUREFIRE-992 but now the stack trace is 
> even more trimmed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-153) Deprecate direct template usage in favor of skin only

2016-02-09 Thread Hudson (JIRA)

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

Hudson commented on DOXIASITETOOLS-153:
---

SUCCESS: Integrated in doxia-all #250 (See 
[https://builds.apache.org/job/doxia-all/250/])
[DOXIASITETOOLS-153] Deprecate direct template usage in favor of skin only 
(michaelo: [http://svn.apache.org/viewvc/?view=rev&rev=1729502])
* 
./doxia-sitetools/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java
* 
./doxia-sitetools/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/Renderer.java


> Deprecate direct template usage in favor of skin only
> -
>
> Key: DOXIASITETOOLS-153
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-153
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
> Fix For: 1.7
>
>
> The usage of sole Velocity templates disables resources copying and other 
> processes. Using skins is recommended. Let's deprecate this and support skins 
> only.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (DOXIASITETOOLS-134) Remove special handling of date format in DefaultSiteRenderer

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov updated DOXIASITETOOLS-134:
--
Component/s: Site renderer

> Remove special handling of date format in DefaultSiteRenderer
> -
>
> Key: DOXIASITETOOLS-134
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-134
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
> Fix For: 1.7
>
>
> {{dateFormat}} is double-checked from validity and existance in decoration 
> model and set with a fallback when everything fails. We should solely rely on 
> the decoration model because it has a fixed value. This would save code and 
> duplicate fixed values. Every deviation from it is undefined behavior.
> Redudant code for example:
> {code:java}
> DateFormat dateFormat = DateFormat.getDateInstance( DateFormat.DEFAULT, 
> locale );
> PublishDate publishDate = 
> siteRenderingContext.getDecoration().getPublishDate();
> if ( publishDate != null && StringUtils.isNotBlank( publishDate.getFormat() ) 
> )
> {
> dateFormat = new SimpleDateFormat( publishDate.getFormat(), locale );
> }
> context.put( "dateFormat", dateFormat );
> {code}
> or in {{site.vm}}:
> {code}
>   #if ( $decorationPublishDate && $decorationPublishDate.format )
> #set ( $format = $decorationPublishDate.format )
>   #else
> #set ( $format = "-MM-dd" )
>   #end
> ##
>   $dateFormat.applyPattern( $format )
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Closed] (DOXIASITETOOLS-153) Deprecate direct template usage in favor of skin only

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov closed DOXIASITETOOLS-153.
-
Resolution: Fixed

Fixed with [r1729502|http://svn.apache.org/r1729502].

> Deprecate direct template usage in favor of skin only
> -
>
> Key: DOXIASITETOOLS-153
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-153
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
> Fix For: 1.7
>
>
> The usage of sole Velocity templates disables resources copying and other 
> processes. Using skins is recommended. Let's deprecate this and support skins 
> only.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-154) Add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on DOXIASITETOOLS-154:
---

True!

> Add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (DOXIASITETOOLS-153) Deprecate direct template usage in favor of skin only

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov updated DOXIASITETOOLS-153:
--
Fix Version/s: 1.7

> Deprecate direct template usage in favor of skin only
> -
>
> Key: DOXIASITETOOLS-153
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-153
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
> Fix For: 1.7
>
>
> The usage of sole Velocity templates disables resources copying and other 
> processes. Using skins is recommended. Let's deprecate this and support skins 
> only.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-154) Add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread JIRA

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

Hervé Boutemy commented on DOXIASITETOOLS-154:
--

IIRC, it was about newlines
we don't have many issues left, it's now easy to find DOXIASITETOOLS-88 :)

> Add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-154) Add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on DOXIASITETOOLS-154:
---

I remember we had this discussion already but do not remember the ticket...

> Add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MNG-5968) Default plugin version updates.

2016-02-09 Thread Hudson (JIRA)

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

Hudson commented on MNG-5968:
-

SUCCESS: Integrated in maven-3.x #1214 (See 
[https://builds.apache.org/job/maven-3.x/1214/])
[MNG-5968] Default plugin version updates. (schulte: rev 
c37ea427337df28bb260a92ee016a0d464567bb7)
* maven-core/src/main/resources/META-INF/plexus/default-bindings.xml


> Default plugin version updates.
> ---
>
> Key: MNG-5968
> URL: https://issues.apache.org/jira/browse/MNG-5968
> Project: Maven
>  Issue Type: Improvement
>Reporter: Christian Schulte
>Assignee: Christian Schulte
>Priority: Minor
> Fix For: 3.4.0
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-154) Add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread JIRA

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

Hervé Boutemy commented on DOXIASITETOOLS-154:
--

yes, why not
this has to be used when copying text resources, but not binary resources: this 
will be more complex to code
perhaps another Jira issue to use this encoding for resources

> Add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (DOXIASITETOOLS-153) Deprecate direct template usage in favor of skin only

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov reassigned DOXIASITETOOLS-153:
-

Assignee: Michael Osipov

> Deprecate direct template usage in favor of skin only
> -
>
> Key: DOXIASITETOOLS-153
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-153
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>
> The usage of sole Velocity templates disables resources copying and other 
> processes. Using skins is recommended. Let's deprecate this and support skins 
> only.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Closed] (DOXIASITETOOLS-149) Create a skin descriptor to contain metadata about the skin

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov closed DOXIASITETOOLS-149.
-
Resolution: Fixed

Version upated in [r1729489|http://svn.apache.org/r1729489].

> Create a skin descriptor to contain metadata about the skin
> ---
>
> Key: DOXIASITETOOLS-149
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-149
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.6
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, a skin is a jar containing at least {{META-INF/maven/site.vm}}, 
> which is the Velocity template file
> adding a skin descriptor (as XML with Modello, to be consistent with Maven 
> style) would permit meta-data definition.
> The first case I see is Doxia Site Tools prerequisites, to have a skin make 
> clear that it requires a newer version of Doxia Site Tools (which is a 
> dependency of maven-site-plugin = what end-users see): this would permit 
> newer Doxia Site Tools versions to provide new features for skins (like 
> DOXIASITETOOLS-150) and skins using these features without fearing failing 
> because being used in older maven-site-plugin versions
> this would also create a new "skin-model" module that would give us a natural 
> place to document practices around skins



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-149) Create a skin descriptor to contain metadata about the skin

2016-02-09 Thread Hudson (JIRA)

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

Hudson commented on DOXIASITETOOLS-149:
---

SUCCESS: Integrated in doxia-all #249 (See 
[https://builds.apache.org/job/doxia-all/249/])
[DOXIASITETOOLS-149] first version is 1.7.0 (hboutemy: 
[http://svn.apache.org/viewvc/?view=rev&rev=1729489])
* ./doxia-sitetools/doxia-skin-model/pom.xml


> Create a skin descriptor to contain metadata about the skin
> ---
>
> Key: DOXIASITETOOLS-149
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-149
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.6
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, a skin is a jar containing at least {{META-INF/maven/site.vm}}, 
> which is the Velocity template file
> adding a skin descriptor (as XML with Modello, to be consistent with Maven 
> style) would permit meta-data definition.
> The first case I see is Doxia Site Tools prerequisites, to have a skin make 
> clear that it requires a newer version of Doxia Site Tools (which is a 
> dependency of maven-site-plugin = what end-users see): this would permit 
> newer Doxia Site Tools versions to provide new features for skins (like 
> DOXIASITETOOLS-150) and skins using these features without fearing failing 
> because being used in older maven-site-plugin versions
> this would also create a new "skin-model" module that would give us a natural 
> place to document practices around skins



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-154) Add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on DOXIASITETOOLS-154:
---

I would even extend the title, we could even apply the encoding to all text 
files which we would read in with a reader, not only {{site.vm}}. WDYT?

> Add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (DOXIASITETOOLS-154) Add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov updated DOXIASITETOOLS-154:
--
Summary: Add encoding to skin descriptor to define the encoding of site.vm  
(was: add encoding to skin descriptor to define the encoding of site.vm)

> Add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-148) Remove SiteRenderingContext#publishDate (undo DOXIASITETOOLS-63)

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on DOXIASITETOOLS-148:
---

I just don't want this to be forgotten for years in JIRA...

> Remove SiteRenderingContext#publishDate (undo DOXIASITETOOLS-63)
> 
>
> Key: DOXIASITETOOLS-148
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-148
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>
> The field {{publishDate}} has been introduced DOXIATESITETOOLS-63 but cannot 
> be set anywhere. Either provide a sane setting option including proper 
> documentation or drop it completely.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MNG-5626) Avoid negative durations or handle them correctly

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on MNG-5626:
-

No one's talking about a replacement for a timestamp but merely not to 
calculate elapsed time based on {{System.currentTimeMillis()}} but only on 
{{System.nanoTime()}}.

> Avoid negative durations or handle them correctly
> -
>
> Key: MNG-5626
> URL: https://issues.apache.org/jira/browse/MNG-5626
> Project: Maven
>  Issue Type: Bug
>  Components: Logging
>Affects Versions: 3.2.1
>Reporter: Christian Jung
>Priority: Minor
>
> In issue MNG-5623 we reported an exception when printing the reactor summary 
> if one of the times was negative.
> I saw in one case, that the overall maven build time, as measured from 
> outside (i.e. by our QuickBuild system) was -10.8 seconds. The corresponding 
> reactor summary was:
> {code}
> 13:55:25,184 INFO  - Reactor Summary:
> 13:55:25,184 INFO  -
> 13:55:25,184 INFO  - module1 ... 
> SUCCESS [ 5.911 s]
> 13:55:25,184 INFO  - module2 ... 
> SUCCESS [ 0.255 s]
> 13:55:25,184 INFO  - gpPlaygroundBase-lnx-x64-gcc4 . 
> SUCCESS [-27.-64 s]
> 13:55:25,185 INFO  - 
> 
> 13:55:25,185 INFO  - BUILD SUCCESS
> 13:55:25,185 INFO  - 
> 
> 13:55:25,185 INFO  - Total time: -20.-73 s
> 13:55:25,185 INFO  - Finished at: 2014-04-28T13:55:25+01:00
> 13:55:25,572 INFO  - Final Memory: 32M/439M
> 13:55:25,572 INFO  - 
> 
> {code}
> The thing is quite hard to reproduce, the machines were virtual machines that 
> have been running for quite a long time.
> Our administrators suspected that just at this point, the local clock was 
> synchronized with some outer source.
> We should check if such negative durations can be avoided, and if not, they 
> should be handled correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MSITE-765) upgrade Doxia from 1.6 to 1.7

2016-02-09 Thread JIRA

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

Hervé Boutemy updated MSITE-765:

Description: 
Release Notes - Maven Doxia - Version 1.7

Bug
* [DOXIA-482] - add an API to avoid translating APT source comments into 
output comments
* [DOXIA-485] - Outdated links in xdoc format description
* [DOXIA-508] - Format characters are interpreted in monospaced text
* [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
* [DOXIA-537] - NullPointerException parsing table cell with single space

Improvement
* [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
for sections
* [DOXIA-492] - Add support for doxia macros in markdown documents.
* [DOXIA-505] - Upgrade to Commons Configuration 1.10
* [DOXIA-527] - Allow multiple extensions for given format
* [DOXIA-529] - Add macro support to XHTML 

New Feature
* [DOXIA-224] - Add source name in parser
* [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
comments starting with #)
* [DOXIA-535] - Support .markdown file extension (in addition to current 
.md)

Task
* [DOXIA-511] - remove deprecated o.a.m.doxia.module.site package
* [DOXIA-526] - remove xerces:xercesImpl dependency
* [DOXIA-531] - improve MacroRequest API with parser and sourceContent 
parameters requirements

Wish
* [DOXIA-506] - move classes from doxia-core in sink package (same as 
doxia-sink-api) to dedicated sink.impl package


  was:
Release Notes - Maven Doxia - Version 1.7

Bug
* [DOXIA-482] - add an API to avoid translating APT source comments into 
output comments
* [DOXIA-485] - Outdated links in xdoc format description
* [DOXIA-508] - Format characters are interpreted in monospaced text
* [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
* [DOXIA-537] - NullPointerException parsing table cell with single space

Improvement
* [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
for sections
* [DOXIA-492] - Add support for doxia macros in markdown documents.
* [DOXIA-505] - Upgrade to Commons Configuration 1.10
* [DOXIA-527] - Allow multiple extensions for given format
* [DOXIA-529] - Add macro support to XHTML 

New Feature
* [DOXIA-224] - Add source name in parser
* [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
comments starting with #)
* [DOXIA-535] - Support .markdown file extension (in addition to current 
.md)



> upgrade Doxia from 1.6 to 1.7
> -
>
> Key: MSITE-765
> URL: https://issues.apache.org/jira/browse/MSITE-765
> Project: Maven Site Plugin
>  Issue Type: Improvement
>  Components: doxia integration
>Affects Versions: 3.4
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 3.5
>
>
> Release Notes - Maven Doxia - Version 1.7
> Bug
> * [DOXIA-482] - add an API to avoid translating APT source comments into 
> output comments
> * [DOXIA-485] - Outdated links in xdoc format description
> * [DOXIA-508] - Format characters are interpreted in monospaced text
> * [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
> * [DOXIA-537] - NullPointerException parsing table cell with single space
> Improvement
> * [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
> for sections
> * [DOXIA-492] - Add support for doxia macros in markdown documents.
> * [DOXIA-505] - Upgrade to Commons Configuration 1.10
> * [DOXIA-527] - Allow multiple extensions for given format
> * [DOXIA-529] - Add macro support to XHTML 
> New Feature
> * [DOXIA-224] - Add source name in parser
> * [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
> comments starting with #)
> * [DOXIA-535] - Support .markdown file extension (in addition to current 
> .md)
> Task
> * [DOXIA-511] - remove deprecated o.a.m.doxia.module.site package
> * [DOXIA-526] - remove xerces:xercesImpl dependency
> * [DOXIA-531] - improve MacroRequest API with parser and sourceContent 
> parameters requirements
> Wish
> * [DOXIA-506] - move classes from doxia-core in sink package (same as 
> doxia-sink-api) to dedicated sink.impl package



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Closed] (MSITE-765) upgrade Doxia from 1.6 to 1.7

2016-02-09 Thread JIRA

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

Hervé Boutemy closed MSITE-765.
---
Resolution: Fixed

> upgrade Doxia from 1.6 to 1.7
> -
>
> Key: MSITE-765
> URL: https://issues.apache.org/jira/browse/MSITE-765
> Project: Maven Site Plugin
>  Issue Type: Improvement
>  Components: doxia integration
>Affects Versions: 3.4
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 3.5
>
>
> Release Notes - Maven Doxia - Version 1.7
> Bug
> * [DOXIA-482] - add an API to avoid translating APT source comments into 
> output comments
> * [DOXIA-485] - Outdated links in xdoc format description
> * [DOXIA-508] - Format characters are interpreted in monospaced text
> * [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
> * [DOXIA-537] - NullPointerException parsing table cell with single space
> Improvement
> * [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
> for sections
> * [DOXIA-492] - Add support for doxia macros in markdown documents.
> * [DOXIA-505] - Upgrade to Commons Configuration 1.10
> * [DOXIA-527] - Allow multiple extensions for given format
> * [DOXIA-529] - Add macro support to XHTML 
> New Feature
> * [DOXIA-224] - Add source name in parser
> * [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
> comments starting with #)
> * [DOXIA-535] - Support .markdown file extension (in addition to current 
> .md)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MSITE-765) upgrade Doxia from 1.6 to 1.7

2016-02-09 Thread Hudson (JIRA)

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

Hudson commented on MSITE-765:
--

UNSTABLE: Integrated in maven-plugins #5077 (See 
[https://builds.apache.org/job/maven-plugins/5077/])
[MSITE-765] upgraded Doxia from 1.6 to 1.7 (hboutemy: 
[http://svn.apache.org/viewvc/?view=rev&rev=1729456])
* maven-site-plugin/pom.xml


> upgrade Doxia from 1.6 to 1.7
> -
>
> Key: MSITE-765
> URL: https://issues.apache.org/jira/browse/MSITE-765
> Project: Maven Site Plugin
>  Issue Type: Improvement
>  Components: doxia integration
>Affects Versions: 3.4
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 3.5
>
>
> Release Notes - Maven Doxia - Version 1.7
> Bug
> * [DOXIA-482] - add an API to avoid translating APT source comments into 
> output comments
> * [DOXIA-485] - Outdated links in xdoc format description
> * [DOXIA-508] - Format characters are interpreted in monospaced text
> * [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
> * [DOXIA-537] - NullPointerException parsing table cell with single space
> Improvement
> * [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
> for sections
> * [DOXIA-492] - Add support for doxia macros in markdown documents.
> * [DOXIA-505] - Upgrade to Commons Configuration 1.10
> * [DOXIA-527] - Allow multiple extensions for given format
> * [DOXIA-529] - Add macro support to XHTML 
> New Feature
> * [DOXIA-224] - Add source name in parser
> * [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
> comments starting with #)
> * [DOXIA-535] - Support .markdown file extension (in addition to current 
> .md)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-148) Remove SiteRenderingContext#publishDate (undo DOXIASITETOOLS-63)

2016-02-09 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on DOXIASITETOOLS-148:
---

correct, as long as there are only 24 hours in a day and there are other 
projects which have higher prio for me :P

> Remove SiteRenderingContext#publishDate (undo DOXIASITETOOLS-63)
> 
>
> Key: DOXIASITETOOLS-148
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-148
> Project: Maven Doxia Sitetools
>  Issue Type: Task
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Michael Osipov
>Assignee: Michael Osipov
>
> The field {{publishDate}} has been introduced DOXIATESITETOOLS-63 but cannot 
> be set anywhere. Either provide a sane setting option including proper 
> documentation or drop it completely.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MSITE-765) upgrade Doxia from 1.6 to 1.7

2016-02-09 Thread JIRA

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

Hervé Boutemy updated MSITE-765:

Description: 
Release Notes - Maven Doxia - Version 1.7

Bug
* [DOXIA-482] - add an API to avoid translating APT source comments into 
output comments
* [DOXIA-485] - Outdated links in xdoc format description
* [DOXIA-508] - Format characters are interpreted in monospaced text
* [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
* [DOXIA-537] - NullPointerException parsing table cell with single space

Improvement
* [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
for sections
* [DOXIA-492] - Add support for doxia macros in markdown documents.
* [DOXIA-505] - Upgrade to Commons Configuration 1.10
* [DOXIA-527] - Allow multiple extensions for given format
* [DOXIA-529] - Add macro support to XHTML 

New Feature
* [DOXIA-224] - Add source name in parser
* [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
comments starting with #)
* [DOXIA-535] - Support .markdown file extension (in addition to current 
.md)


  was:
a lot of fixed issues and cleanup
and new features: todo copy release notes once released


> upgrade Doxia from 1.6 to 1.7
> -
>
> Key: MSITE-765
> URL: https://issues.apache.org/jira/browse/MSITE-765
> Project: Maven Site Plugin
>  Issue Type: Improvement
>  Components: doxia integration
>Affects Versions: 3.4
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 3.5
>
>
> Release Notes - Maven Doxia - Version 1.7
> Bug
> * [DOXIA-482] - add an API to avoid translating APT source comments into 
> output comments
> * [DOXIA-485] - Outdated links in xdoc format description
> * [DOXIA-508] - Format characters are interpreted in monospaced text
> * [DOXIA-524] - ISO-8859-15 encoding problem with markdown 
> * [DOXIA-537] - NullPointerException parsing table cell with single space
> Improvement
> * [DOXIA-203] - Add support for level 6 sections and generalize Sink API 
> for sections
> * [DOXIA-492] - Add support for doxia macros in markdown documents.
> * [DOXIA-505] - Upgrade to Commons Configuration 1.10
> * [DOXIA-527] - Allow multiple extensions for given format
> * [DOXIA-529] - Add macro support to XHTML 
> New Feature
> * [DOXIA-224] - Add source name in parser
> * [DOXIA-532] - Update handling of comments for Server Side Includes (ie 
> comments starting with #)
> * [DOXIA-535] - Support .markdown file extension (in addition to current 
> .md)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MSITE-755) upgrade Doxia Sitetools from 1.6 to 1.7

2016-02-09 Thread JIRA

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

Hervé Boutemy updated MSITE-755:

Summary: upgrade Doxia Sitetools from 1.6 to 1.7  (was: upgrade Doxia and 
Doxia Sitetools from 1.6 to 1.7)

> upgrade Doxia Sitetools from 1.6 to 1.7
> ---
>
> Key: MSITE-755
> URL: https://issues.apache.org/jira/browse/MSITE-755
> Project: Maven Site Plugin
>  Issue Type: Improvement
>  Components: doxia integration
>Affects Versions: 3.4
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 3.5
>
>
> a lot of fixed issues and cleanup
> and new features: todo copy release notes once released



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MSITE-765) upgrade Doxia from 1.6 to 1.7

2016-02-09 Thread JIRA
Hervé Boutemy created MSITE-765:
---

 Summary: upgrade Doxia from 1.6 to 1.7
 Key: MSITE-765
 URL: https://issues.apache.org/jira/browse/MSITE-765
 Project: Maven Site Plugin
  Issue Type: Improvement
  Components: doxia integration
Affects Versions: 3.4
Reporter: Hervé Boutemy
Assignee: Hervé Boutemy
 Fix For: 3.5


a lot of fixed issues and cleanup
and new features: todo copy release notes once released



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIA-540) Update/remove ancient information on Doxia site

2016-02-09 Thread JIRA

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

Hervé Boutemy commented on DOXIA-540:
-

no, no site Jira issue: no problem to keep this issue in DOXIA key

> Update/remove ancient information on Doxia site
> ---
>
> Key: DOXIA-540
> URL: https://issues.apache.org/jira/browse/DOXIA-540
> Project: Maven Doxia
>  Issue Type: Task
>  Components: Documentation
>Affects Versions: 1.7
>Reporter: Michael Osipov
>
> Tasks to perform:
> * Remove [Developer Docs|http://maven.apache.org/doxia/] for ancient versions.
> * Remove Doxia IDE intergration is there is none working
> * Remove [Upgrading|http://maven.apache.org/doxia/upgrading.html], it is 
> overhauled.
> * Check [FAQ|http://maven.apache.org/doxia/faq.html] for validity.
> * Merge [Apt 
> Enhancements|http://maven.apache.org/doxia/references/doxia-apt.html] into 
> [main site|http://maven.apache.org/doxia/references/apt-format.html].
> * Check [Issues|http://maven.apache.org/doxia/issues/index.html]  for 
> validity.
> * Update [External Resources|http://maven.apache.org/doxia/resources.html].
> * Remove ancient information from [Sink 
> API|http://maven.apache.org/doxia/doxia/doxia-sink-api/index.html].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Closed] (DOXIASITETOOLS-154) add encoding to skin descriptor to define the encoding of site.vm

2016-02-09 Thread JIRA

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

Hervé Boutemy closed DOXIASITETOOLS-154.

Resolution: Fixed

> add encoding to skin descriptor to define the encoding of site.vm
> -
>
> Key: DOXIASITETOOLS-154
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-154
> Project: Maven Doxia Sitetools
>  Issue Type: New Feature
>  Components: Site renderer, Skin model
>Affects Versions: 1.7
>Reporter: Hervé Boutemy
>Assignee: Hervé Boutemy
> Fix For: 1.7
>
>
> currently, Velocity parses site.vm template with its default encoding: a skin 
> should be able to define its template encoding



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MNG-5626) Avoid negative durations or handle them correctly

2016-02-09 Thread Christian Schulte (JIRA)

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

Christian Schulte commented on MNG-5626:


{{System.currentTimeMillis()}} is affected by everything changing the system 
clock. If e.g. a NTP client adjusts the system clock, negative durations can 
occur. The javadoc of class {{java.util.time.Clock}} contains further 
information (leap seconds, etc.). {{System.nanoTime()}} can only be used to 
calculate durations. It's more like a VM uptime counter in nanoseconds. There 
is no alternative to using {{System.currentTimeMillis()}} or 
{{java.util.time.Clock.systemUTC()}} whenever a real timestamp is needed.

> Avoid negative durations or handle them correctly
> -
>
> Key: MNG-5626
> URL: https://issues.apache.org/jira/browse/MNG-5626
> Project: Maven
>  Issue Type: Bug
>  Components: Logging
>Affects Versions: 3.2.1
>Reporter: Christian Jung
>Priority: Minor
>
> In issue MNG-5623 we reported an exception when printing the reactor summary 
> if one of the times was negative.
> I saw in one case, that the overall maven build time, as measured from 
> outside (i.e. by our QuickBuild system) was -10.8 seconds. The corresponding 
> reactor summary was:
> {code}
> 13:55:25,184 INFO  - Reactor Summary:
> 13:55:25,184 INFO  -
> 13:55:25,184 INFO  - module1 ... 
> SUCCESS [ 5.911 s]
> 13:55:25,184 INFO  - module2 ... 
> SUCCESS [ 0.255 s]
> 13:55:25,184 INFO  - gpPlaygroundBase-lnx-x64-gcc4 . 
> SUCCESS [-27.-64 s]
> 13:55:25,185 INFO  - 
> 
> 13:55:25,185 INFO  - BUILD SUCCESS
> 13:55:25,185 INFO  - 
> 
> 13:55:25,185 INFO  - Total time: -20.-73 s
> 13:55:25,185 INFO  - Finished at: 2014-04-28T13:55:25+01:00
> 13:55:25,572 INFO  - Final Memory: 32M/439M
> 13:55:25,572 INFO  - 
> 
> {code}
> The thing is quite hard to reproduce, the machines were virtual machines that 
> have been running for quite a long time.
> Our administrators suspected that just at this point, the local clock was 
> synchronized with some outer source.
> We should check if such negative durations can be avoided, and if not, they 
> should be handled correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MNG-5934) String handling issues identified by PMD

2016-02-09 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-5934:
-

Github user ChristianSchulte commented on the pull request:

https://github.com/apache/maven/pull/72#issuecomment-181984267
  
This has been committed. See MNG-5934. Please close this pull request 
manually.


> String handling issues identified by PMD
> 
>
> Key: MNG-5934
> URL: https://issues.apache.org/jira/browse/MNG-5934
> Project: Maven
>  Issue Type: Improvement
>Affects Versions: 3.3.9
>Reporter: Alex Henrie
>Assignee: Christian Schulte
>Priority: Minor
>  Labels: easyfix, newbie, patch, performance
> Fix For: 3.4.0
>
> Attachments: 
> 0001-Prefer-StringBuilder.append-char-to-StringBuilder.ap.patch, 
> 0002-Prefer-String.indexOf-char-to-String.indexOf-String.patch, 
> 0003-Combine-string-literals-instead-of-calling-String.ap.patch, 
> 0004-Construct-StringBuffers-with-enough-space-to-begin-w.patch
>
>
> The code linting tool [PMD|http://pmd.sourceforge.net/pmd-5.2.0/] identified 
> numerous places where Maven's string handling is inefficient. I've prepared a 
> set of patches that should help speed things up.
> The command I used was `pmd -d . -R java-strings`



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (MNG-5940) Change the maven-source-plugin jar goal into jar-no-fork in Maven Super POM

2016-02-09 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-5940:
-

Github user ChristianSchulte commented on the pull request:

https://github.com/apache/maven/pull/61#issuecomment-181983567
  
This has been committed. See MNG-5940. Please close this pull request 
manually.



> Change the maven-source-plugin jar goal into jar-no-fork in Maven Super POM
> ---
>
> Key: MNG-5940
> URL: https://issues.apache.org/jira/browse/MNG-5940
> Project: Maven
>  Issue Type: Improvement
>  Components: core
>Reporter: Karl Heinz Marbaise
>Assignee: Christian Schulte
>Priority: Minor
> Fix For: 3.4.0
>
>
> At the moment the {{maven-source-plugin:jar}} goal is defined in the Maven 
> super pom:
> {code:xml}
>   
> true
> maven-source-plugin
> 
>   
> attach-sources
> 
>   jar
> 
>   
> 
>   
> {code}
> where the goal of {{maven-source-plugin}} should be changed from {{jar}} into 
> {{jar-no-fork}}, cause most of the time you need to override this behaviour.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Assigned] (MNG-5863) default pom's release-profile should invoke source plugin with goal "jar-no-fork" instead of "jar"

2016-02-09 Thread Christian Schulte (JIRA)

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

Christian Schulte reassigned MNG-5863:
--

Assignee: Christian Schulte

> default pom's release-profile should invoke source plugin with goal 
> "jar-no-fork" instead of "jar"
> --
>
> Key: MNG-5863
> URL: https://issues.apache.org/jira/browse/MNG-5863
> Project: Maven
>  Issue Type: Bug
>  Components: POM
>Affects Versions: 3.3.3
>Reporter: Petr Kozelka
>Assignee: Christian Schulte
> Fix For: 3.4.0
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> in maven-model-builder, the file pom-4.0.0.xml defines  "release-profile" 
> which binds some executions to the lifecycle.
> One of them is source:jar - which forks the build. That can be a problem in 
> some configurations, and the forking is probably not necessary.
> One situation where the forked build hurts is this:
> - I have checkstyle:check attached to phase "validate"
> - some of my modules generate code, obviously not compliant to the checkstyle
> The problem is that, inside forked build, the checkstyle:check is called 
> again, but now it checks also the generated code (because target/ is no 
> longer empty). And of course fails.
> Even worse: during normal development iterations, everything is fine. But 
> when I have to issue a release (usually under some pressure), I hit this 
> problem.
> Fortunately, there _is_ a workaround: override the execution "attach-sources" 
> and assign it to a non-existing phase, and define execution with different id 
> for that.
> But it is too ugly and I believe that the simple fix would solve it - for the 
> meantime before the whole profile is removed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Closed] (MNG-5863) default pom's release-profile should invoke source plugin with goal "jar-no-fork" instead of "jar"

2016-02-09 Thread Christian Schulte (JIRA)

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

Christian Schulte closed MNG-5863.
--
   Resolution: Fixed
Fix Version/s: 3.4.0

> default pom's release-profile should invoke source plugin with goal 
> "jar-no-fork" instead of "jar"
> --
>
> Key: MNG-5863
> URL: https://issues.apache.org/jira/browse/MNG-5863
> Project: Maven
>  Issue Type: Bug
>  Components: POM
>Affects Versions: 3.3.3
>Reporter: Petr Kozelka
>Assignee: Christian Schulte
> Fix For: 3.4.0
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> in maven-model-builder, the file pom-4.0.0.xml defines  "release-profile" 
> which binds some executions to the lifecycle.
> One of them is source:jar - which forks the build. That can be a problem in 
> some configurations, and the forking is probably not necessary.
> One situation where the forked build hurts is this:
> - I have checkstyle:check attached to phase "validate"
> - some of my modules generate code, obviously not compliant to the checkstyle
> The problem is that, inside forked build, the checkstyle:check is called 
> again, but now it checks also the generated code (because target/ is no 
> longer empty). And of course fails.
> Even worse: during normal development iterations, everything is fine. But 
> when I have to issue a release (usually under some pressure), I hit this 
> problem.
> Fortunately, there _is_ a workaround: override the execution "attach-sources" 
> and assign it to a non-existing phase, and define execution with different id 
> for that.
> But it is too ugly and I believe that the simple fix would solve it - for the 
> meantime before the whole profile is removed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Reopened] (MNG-5863) default pom's release-profile should invoke source plugin with goal "jar-no-fork" instead of "jar"

2016-02-09 Thread Christian Schulte (JIRA)

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

Christian Schulte reopened MNG-5863:


> default pom's release-profile should invoke source plugin with goal 
> "jar-no-fork" instead of "jar"
> --
>
> Key: MNG-5863
> URL: https://issues.apache.org/jira/browse/MNG-5863
> Project: Maven
>  Issue Type: Bug
>  Components: POM
>Affects Versions: 3.3.3
>Reporter: Petr Kozelka
>Assignee: Christian Schulte
> Fix For: 3.4.0
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> in maven-model-builder, the file pom-4.0.0.xml defines  "release-profile" 
> which binds some executions to the lifecycle.
> One of them is source:jar - which forks the build. That can be a problem in 
> some configurations, and the forking is probably not necessary.
> One situation where the forked build hurts is this:
> - I have checkstyle:check attached to phase "validate"
> - some of my modules generate code, obviously not compliant to the checkstyle
> The problem is that, inside forked build, the checkstyle:check is called 
> again, but now it checks also the generated code (because target/ is no 
> longer empty). And of course fails.
> Even worse: during normal development iterations, everything is fine. But 
> when I have to issue a release (usually under some pressure), I hit this 
> problem.
> Fortunately, there _is_ a workaround: override the execution "attach-sources" 
> and assign it to a non-existing phase, and define execution with different id 
> for that.
> But it is too ugly and I believe that the simple fix would solve it - for the 
> meantime before the whole profile is removed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MENFORCER-247) Add a "require file checksum" rule

2016-02-09 Thread Lyubomyr Shaydariv (JIRA)

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

Lyubomyr Shaydariv updated MENFORCER-247:
-
Description: 
Hi.

Sometimes there is a need to have a rule to check a file against a predefined 
checksum. Let's say, a result of a generated file in the build output directory 
is expected to result in a required checksum, or some source/test files must 
result in predefined checksums. I've came across such a rule on GitHub 
https://github.com/esamson/checksum-enforcer-rule and I think it would be nice 
to have this one as a standard rule.

What I've done already:

* Adapted the original source to the source code base in  
http://svn.apache.org/repos/asf/maven/enforcer/trunk (1.4.2-SNAPSHOT)
* Added Apache Commons Codec 1.6 dependency thus adding support for MD5, SHA-1, 
SHA-256, SHA-384, SHA-512
* Added some tests
* Added an example for all hashing functions to the generated site.

What I'm not sure about:

* Would you accept this feature as one that's worth be a part of standard rules.
* If so, I'm not sure if https://github.com/apache/maven-enforcer accepts pull 
requests as I've noticed probably automatically generated `git-svn-id` field in 
`git log` output. So probably the patch might be committed to the Subversion 
repository first.

Here is the patch diff on GitHub: 
https://github.com/lyubomyr-shaydariv/maven-enforcer/commit/86ed97423b5c64625f700c3afe555c02d6c619df

  was:
Hi.

Sometimes there is a need to have a rule to check a file against a predefined 
checksum. Let's say, a result of a generated file in the build output directory 
is expected to result in a required checksum, or some source/test files must 
result in predefined checksums. I've came across such a rule on GitHub 
https://github.com/esamson/checksum-enforcer-rule and I think it would be nice 
to have this one as a standard rule.

What I've done already:

* Adapted the original source to the source code base in  
http://svn.apache.org/repos/asf/maven/enforcer/trunk
* Added Apache Commons Codec 1.6 dependency thus adding support for MD5, SHA-1, 
SHA-256, SHA-384, SHA-512
* Added some tests
* Added an example for all hashing functions to the generated site.

What I'm not sure about:

* Would you accept this feature as one that's worth be a part of standard rules.
* If so, I'm not sure if https://github.com/apache/maven-enforcer accepts pull 
requests as I've noticed probably automatically generated `git-svn-id` field in 
`git log` output. So probably the patch might be committed to the Subversion 
repository first.

Here is the patch diff on GitHub: 
https://github.com/lyubomyr-shaydariv/maven-enforcer/commit/86ed97423b5c64625f700c3afe555c02d6c619df


> Add a "require file checksum" rule
> --
>
> Key: MENFORCER-247
> URL: https://issues.apache.org/jira/browse/MENFORCER-247
> Project: Maven Enforcer Plugin
>  Issue Type: New Feature
>  Components: Standard Rules
>Reporter: Lyubomyr Shaydariv
>
> Hi.
> Sometimes there is a need to have a rule to check a file against a predefined 
> checksum. Let's say, a result of a generated file in the build output 
> directory is expected to result in a required checksum, or some source/test 
> files must result in predefined checksums. I've came across such a rule on 
> GitHub https://github.com/esamson/checksum-enforcer-rule and I think it would 
> be nice to have this one as a standard rule.
> What I've done already:
> * Adapted the original source to the source code base in  
> http://svn.apache.org/repos/asf/maven/enforcer/trunk (1.4.2-SNAPSHOT)
> * Added Apache Commons Codec 1.6 dependency thus adding support for MD5, 
> SHA-1, SHA-256, SHA-384, SHA-512
> * Added some tests
> * Added an example for all hashing functions to the generated site.
> What I'm not sure about:
> * Would you accept this feature as one that's worth be a part of standard 
> rules.
> * If so, I'm not sure if https://github.com/apache/maven-enforcer accepts 
> pull requests as I've noticed probably automatically generated `git-svn-id` 
> field in `git log` output. So probably the patch might be committed to the 
> Subversion repository first.
> Here is the patch diff on GitHub: 
> https://github.com/lyubomyr-shaydariv/maven-enforcer/commit/86ed97423b5c64625f700c3afe555c02d6c619df



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (MENFORCER-247) Add a "require file checksum" rule

2016-02-09 Thread Lyubomyr Shaydariv (JIRA)

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

Lyubomyr Shaydariv updated MENFORCER-247:
-
Affects Version/s: (was: 1.4.2)

> Add a "require file checksum" rule
> --
>
> Key: MENFORCER-247
> URL: https://issues.apache.org/jira/browse/MENFORCER-247
> Project: Maven Enforcer Plugin
>  Issue Type: New Feature
>  Components: Standard Rules
>Reporter: Lyubomyr Shaydariv
>
> Hi.
> Sometimes there is a need to have a rule to check a file against a predefined 
> checksum. Let's say, a result of a generated file in the build output 
> directory is expected to result in a required checksum, or some source/test 
> files must result in predefined checksums. I've came across such a rule on 
> GitHub https://github.com/esamson/checksum-enforcer-rule and I think it would 
> be nice to have this one as a standard rule.
> What I've done already:
> * Adapted the original source to the source code base in  
> http://svn.apache.org/repos/asf/maven/enforcer/trunk
> * Added Apache Commons Codec 1.6 dependency thus adding support for MD5, 
> SHA-1, SHA-256, SHA-384, SHA-512
> * Added some tests
> * Added an example for all hashing functions to the generated site.
> What I'm not sure about:
> * Would you accept this feature as one that's worth be a part of standard 
> rules.
> * If so, I'm not sure if https://github.com/apache/maven-enforcer accepts 
> pull requests as I've noticed probably automatically generated `git-svn-id` 
> field in `git log` output. So probably the patch might be committed to the 
> Subversion repository first.
> Here is the patch diff on GitHub: 
> https://github.com/lyubomyr-shaydariv/maven-enforcer/commit/86ed97423b5c64625f700c3afe555c02d6c619df



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (MENFORCER-247) Add a "require file checksum" rule

2016-02-09 Thread Lyubomyr Shaydariv (JIRA)
Lyubomyr Shaydariv created MENFORCER-247:


 Summary: Add a "require file checksum" rule
 Key: MENFORCER-247
 URL: https://issues.apache.org/jira/browse/MENFORCER-247
 Project: Maven Enforcer Plugin
  Issue Type: New Feature
  Components: Standard Rules
Affects Versions: 1.4.2
Reporter: Lyubomyr Shaydariv


Hi.

Sometimes there is a need to have a rule to check a file against a predefined 
checksum. Let's say, a result of a generated file in the build output directory 
is expected to result in a required checksum, or some source/test files must 
result in predefined checksums. I've came across such a rule on GitHub 
https://github.com/esamson/checksum-enforcer-rule and I think it would be nice 
to have this one as a standard rule.

What I've done already:

* Adapted the original source to the source code base in  
http://svn.apache.org/repos/asf/maven/enforcer/trunk
* Added Apache Commons Codec 1.6 dependency thus adding support for MD5, SHA-1, 
SHA-256, SHA-384, SHA-512
* Added some tests
* Added an example for all hashing functions to the generated site.

What I'm not sure about:

* Would you accept this feature as one that's worth be a part of standard rules.
* If so, I'm not sure if https://github.com/apache/maven-enforcer accepts pull 
requests as I've noticed probably automatically generated `git-svn-id` field in 
`git log` output. So probably the patch might be committed to the Subversion 
repository first.

Here is the patch diff on GitHub: 
https://github.com/lyubomyr-shaydariv/maven-enforcer/commit/86ed97423b5c64625f700c3afe555c02d6c619df



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIA-540) Update/remove ancient information on Doxia site

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on DOXIA-540:
--

You are right. The Doxia site does not have any project in JIRA or is there 
any? I'd like to track this issue anyway.

> Update/remove ancient information on Doxia site
> ---
>
> Key: DOXIA-540
> URL: https://issues.apache.org/jira/browse/DOXIA-540
> Project: Maven Doxia
>  Issue Type: Task
>  Components: Documentation
>Affects Versions: 1.7
>Reporter: Michael Osipov
>
> Tasks to perform:
> * Remove [Developer Docs|http://maven.apache.org/doxia/] for ancient versions.
> * Remove Doxia IDE intergration is there is none working
> * Remove [Upgrading|http://maven.apache.org/doxia/upgrading.html], it is 
> overhauled.
> * Check [FAQ|http://maven.apache.org/doxia/faq.html] for validity.
> * Merge [Apt 
> Enhancements|http://maven.apache.org/doxia/references/doxia-apt.html] into 
> [main site|http://maven.apache.org/doxia/references/apt-format.html].
> * Check [Issues|http://maven.apache.org/doxia/issues/index.html]  for 
> validity.
> * Update [External Resources|http://maven.apache.org/doxia/resources.html].
> * Remove ancient information from [Sink 
> API|http://maven.apache.org/doxia/doxia/doxia-sink-api/index.html].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DOXIASITETOOLS-106) Set language for customizable resources

2016-02-09 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on DOXIASITETOOLS-106:
---

That is possible as long as the module/resource has localized files. You have 
{{$locale.language}} which you can use. Provide a patch for the localizable 
spots, ideally with an IT, and I will apply it.

> Set language for customizable resources
> ---
>
> Key: DOXIASITETOOLS-106
> URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-106
> Project: Maven Doxia Sitetools
>  Issue Type: Improvement
>  Components: Site renderer
>Affects Versions: 1.6
>Reporter: Eric Barboni
>
> This idea came from issue MSKINS-118. Because any locale you have the gif 
> file for google cse will allways be the english one and it may be nice to 
> have the possibilty to use the locale for customizing resources.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)