[GitHub] maven-surefire pull request: Adding individual rerun times to XML ...

2015-09-28 Thread agudian
Github user agudian commented on the pull request: https://github.com/apache/maven-surefire/pull/96#issuecomment-143957705 I agree that it makes sense to add the same information to the xml report that we provide in the console output. Regarding what time to take for the test case,

[GitHub] maven-surefire pull request: Adding individual rerun times to XML ...

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/96#issuecomment-143955811 @agudian I would like to include you into our discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] maven-surefire pull request: Option to rerun failures at the end o...

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/102#issuecomment-143954421 We can try with junit4 of course. On Tue, Sep 29, 2015 at 12:59 AM, lamyaa wrote: > @Tibor17 I just tried junit47 f

Re: [GitHub] maven-wagon pull request: Add Data URI (RFC 2397) Wagon provider

2015-09-28 Thread Dan Tran
yeah, not sure what is use case to link Maven and Wagon via this provider -D On Mon, Sep 28, 2015 at 11:23 AM, michael-o wrote: > Github user michael-o commented on the pull request: > > https://github.com/apache/maven-wagon/pull/10#issuecomment-143834031 > > Is there any real world exa

[GitHub] maven pull request: MNG-5877: Ensure that snapshot versions use th...

2015-09-28 Thread josephw
Github user josephw commented on the pull request: https://github.com/apache/maven/pull/65#issuecomment-143918594 I've verified the behaviour. To reproduce: MAVEN_OPTS="-Duser.country=th -Duser.language=th" mvn deploy It doesn't fail, but the snapshot dates are inc

Re: [1/3] maven git commit: Package and configure log4J 2.4 by default. Replace the content of conf/logging/log4j2.xml by the one from conf/logging/log4j2-color.xml to enjoy the colorised console

2015-09-28 Thread Arnaud Héritier
Hi Dan, In this version the color isn't activated by default. You have to switch the config files in conf/logging to activate it. Maybe we could have command line parameters to enforce the activation or deactivation . But in 2015 not having the color is so ... The main issue isn't, from my P

Re: [1/3] maven git commit: Package and configure log4J 2.4 by default. Replace the content of conf/logging/log4j2.xml by the one from conf/logging/log4j2-color.xml to enjoy the colorised console

2015-09-28 Thread Dan Tran
do we have option to disable the colorize? -D On Mon, Sep 28, 2015 at 4:08 PM, wrote: > Repository: maven > Updated Branches: > refs/heads/slf4j-log4j2.4 [created] 2c78decb9 > > > Package and configure log4J 2.4 by default. > Replace the content of conf/logging/log4j2.xml by the one from > co

[GitHub] maven-surefire pull request: Adding individual rerun times to XML ...

2015-09-28 Thread lamyaa
Github user lamyaa commented on the pull request: https://github.com/apache/maven-surefire/pull/96#issuecomment-143900518 @Tibor17 I just tested this PR with your recent SUREFIRE-1144 fix. It still works. Again, the point here was really just to provide the same level of informat

[GitHub] maven-surefire pull request: Option to rerun failures at the end o...

2015-09-28 Thread lamyaa
Github user lamyaa commented on the pull request: https://github.com/apache/maven-surefire/pull/102#issuecomment-143896973 @Tibor17 I just tried junit47 for the first time. Indeed, it seems to already run test at the end like I was attempting to do in this PR with junit4. Do you think

Re: Help getting started contributing to maven

2015-09-28 Thread Bernd Eckenfels
Hello, Am Mon, 28 Sep 2015 21:40:54 +0100 schrieb ivange larry : > 1 - Is the code hosted here > https://git-wip-us.apache.org/repos/asf/maven.git, the same as the > code hosted on github with this url https://github.com/apache/maven. Yes, github receives a copy of the ASF repository. You can se

Help getting started contributing to maven

2015-09-28 Thread ivange larry
Hi all My name is Ivange Larry, am from Cameroon and new to maven community. I have followed the instructions on getting started as a developer in the Apache maven community and below are a few of the things i have done so far. I did a git clone on this url https://git-wip-us.apache.org/repos/asf

[GitHub] maven-pom pull request: [MPOM-47] Use assembly descriptor which al...

2015-09-28 Thread ctubbsii
Github user ctubbsii closed the pull request at: https://github.com/apache/maven-pom/pull/1 --- 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 e

[GitHub] maven-wagon pull request: Add Data URI (RFC 2397) Wagon provider

2015-09-28 Thread michael-o
Github user michael-o commented on the pull request: https://github.com/apache/maven-wagon/pull/10#issuecomment-143834031 Is there any real world example for this? --- 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 p

[GitHub] maven pull request: MNG-5877: Ensure that snapshot versions use th...

2015-09-28 Thread michael-o
Github user michael-o commented on the pull request: https://github.com/apache/maven/pull/65#issuecomment-143833287 Can you ping me by Thursday, I will review and merge?! Did you really observe it to fail with Thai locale? --- If your project is set up for it, you can reply to this e

[GitHub] maven-scm pull request: Move System.println(..) to Logger.debug(.....

2015-09-28 Thread michael-o
Github user michael-o commented on the pull request: https://github.com/apache/maven-scm/pull/35#issuecomment-143832739 Can you create a JIRA issue for that depicting what trouble it causes. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143793736 There is already version range [5.3,6.4] you should split which means to split in [5.3,5.10) and [5.10,6.4] update only `TestNGMapConfigurator`. Do y

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143791336 The style with reflection is usually in junit provider, but you'r right, forget reflection in testng. --- If your project is set up for it, you can reply to thi

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143790836 Alright create `TestNG510Configurator` and register it in `AbstractSurefireMojo` and increate the version in surefire-testng POM. --- If your project is set up

[GitHub] maven-plugins pull request: Compute the set of clazzes of a Clazzp...

2015-09-28 Thread BenoitPerrot
Github user BenoitPerrot closed the pull request at: https://github.com/apache/maven-plugins/pull/61 --- 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 fea

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread juherr
Github user juherr commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143768585 It's not a problem of time but I just want to understand what I'm doing. I'm talking about an compile dependency (scoped provided) where, as I understand,

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143766756 Because we were always dependent on 5.7 and the users may still rely on it. It is question of one minute to crate Utility class in the module surefire-testng-

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread juherr
Github user juherr commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143764774 For my information, why not updating the testng dependency as I did it in #105? I'll try to add some tests for suite thread count. --- If your project i

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143760174 Is there the same principle with thread count for suites? I did not test suite thread count because I do not have IT. Can you try? On Mon, Sep 28,

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143758786 All you need to do is to call this method with Java Reflection. There is `surefire-testng-util` module. You should create reflection utility in there.

[GitHub] maven-scm pull request: Fixed SVN branching with svn+ssh://usernam...

2015-09-28 Thread rwegmann
GitHub user rwegmann opened a pull request: https://github.com/apache/maven-scm/pull/38 Fixed SVN branching with svn+ssh://username@host URL. This fixes https://issues.apache.org/jira/browse/SCM-734 You can merge this pull request into a Git repository by running: $ git pull ht

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread juherr
Github user juherr commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143743363 Sometime code is easier than explainations :) I updated #105. Feel free to comment. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread juherr
Github user juherr commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143733741 Since 5.10, `suite.setDataProviderThreadCount()` is available but surefire doesn't use it. I'd like to add ``` String dataProviderThr

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread Tibor17
Github user Tibor17 commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143683397 Since 5.10? Hm, I was testing with 6.9.x and the number of threads was 10 instead of 30 configured in `dataproviderthreadCount`. Pending question - we use

[GitHub] maven-plugins pull request: Compute the set of clazzes of a Clazzp...

2015-09-28 Thread BenoitPerrot
GitHub user BenoitPerrot opened a pull request: https://github.com/apache/maven-plugins/pull/61 Compute the set of clazzes of a ClazzpathUnit only once The method `getClazzes()` in `org.vafer.jdependency.ClazzpathUnit` actually builds a hash set and fills it up, each time it is c

[GitHub] maven-surefire pull request: Quick fix for SUREFIRE-1177 and SUREF...

2015-09-28 Thread juherr
Github user juherr commented on the pull request: https://github.com/apache/maven-surefire/pull/105#issuecomment-143677994 I have some time but the support of `dataproviderthreadcount` is not as easy as expected. You can see my comment: https://github.com/apache/maven-surefire/pull/10

[GitHub] maven-surefire pull request: [SUREFIRE-1179] integration test

2015-09-28 Thread juherr
Github user juherr commented on the pull request: https://github.com/apache/maven-surefire/pull/106#issuecomment-143677343 I had time to analyze and I found that `dataProviderThreadCount` is taken into account only when `m_commandLineTestClasses != null || m_commandLineMethods != null

Re: FOSDEM 2016

2015-09-28 Thread Kristian Rosenvold
Jan 30-31 is in the prime skiing season. No can do :) Kristian 2015-09-27 22:25 GMT+02:00 Arnaud Héritier : > I think that Jenkins will be here too, thus I hope to have the opportunity > to come > > On Sun, Sep 27, 2015 at 7:13 PM, Robert Scholte > wrote: > >> Hi, >> >> Hervés arguments count