[GitHub] maven-surefire issue #157: SUREFIRE-1383 dependenciesToScan Does Not Leverag...

2017-07-04 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/157 Would you add integration test? See the module `surefire-integration-tests` and `src/test/java/org/apache/maven/surefire/its/jiras`. It's easy. Pickup some existing test, e.g.

[GitHub] maven-enforcer pull request #25: [MENFORCER-276]

2017-07-04 Thread oleg-nenashev
GitHub user oleg-nenashev opened a pull request: https://github.com/apache/maven-enforcer/pull/25 [MENFORCER-276] We have recently adopted `RequireUpperBoundDeps` in the Jenkins project (https://github.com/jenkinsci/plugin-pom/pull/67). In order to implement it, Jesse Glick added

[GitHub] maven pull request #125: [MNG-5965] Parallel build multiplies work if multip...

2017-07-04 Thread dbmeneses
GitHub user dbmeneses opened a pull request: https://github.com/apache/maven/pull/125 [MNG-5965] Parallel build multiplies work if multiple goals are given This p/r should also fix MNG-5705. When multiple tasks are given so that they are grouped in different task segments,

Re: maven-wagon git commit: [WAGON-488] Upgrade WebDAV Wagon to a more recent HttpClient version Fix checkstyle reported errors

2017-07-04 Thread Olivier Lamy
On 5 July 2017 at 04:31, wrote: > Repository: maven-wagon > Updated Branches: > refs/heads/master 3464da867 -> dfadb0b78 > > > [WAGON-488] Upgrade WebDAV Wagon to a more recent HttpClient version > Fix checkstyle reported errors > > > Project:

[GitHub] maven-doxia-sitetools pull request #4: DOXIASITETOOLS-178 upgrade to commons...

2017-07-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/maven-doxia-sitetools/pull/4 --- 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

[GitHub] maven-doxia pull request #3: DOXIA-558 upgrade to commons-lang 3

2017-07-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/maven-doxia/pull/3 --- 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

[GitHub] maven-enforcer pull request #26: Add Unit tests for RequireUpperBoundDeps

2017-07-04 Thread oleg-nenashev
GitHub user oleg-nenashev opened a pull request: https://github.com/apache/maven-enforcer/pull/26 Add Unit tests for RequireUpperBoundDeps I decoupled the test improvements commit from #25. It just adds new tests for the existing functionality. @reviewbybees @jglick

[RESULT] [VOTE] Release Maven Resolver version 1.1.0

2017-07-04 Thread Michael Osipov
Hi, The vote has passed with the following result: +1: Michael Osipov, Manfred Moser, Hervé Boutemy, Karl Heinz Marbaise, Olivier Lamy, Tamás Cservenák PMC quorum: reached I will promote the artifacts to the central repo, the source release ZIP file and add this release the board report.

[GitHub] maven issue #125: [MNG-5965] Parallel build multiplies work if multiple goal...

2017-07-04 Thread ifedorenko
Github user ifedorenko commented on the issue: https://github.com/apache/maven/pull/125 Can you provide a regression test that demonstrates the problem and the fix? Semi-related, I recently fixed similar problem in [Takari Smart

[GitHub] maven-surefire issue #157: SUREFIRE-1383 dependenciesToScan Does Not Leverag...

2017-07-04 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/157 No, no. No worries. You should build entire project `mvn install` and not the individual IT. The ITs cannot run separately, you can trigger it from building entire project from

[GitHub] maven-surefire issue #157: SUREFIRE-1383 dependenciesToScan Does Not Leverag...

2017-07-04 Thread owenfarrell
Github user owenfarrell commented on the issue: https://github.com/apache/maven-surefire/pull/157 I don't follow. When I run an install from any level, the `org.apache.maven.surefire:it-parent:1.0:pom` artifact (surefire-integration-tests/src/test/resources/pom.xml) does not get

[GitHub] maven-surefire issue #157: SUREFIRE-1383 dependenciesToScan Does Not Leverag...

2017-07-04 Thread owenfarrell
Github user owenfarrell commented on the issue: https://github.com/apache/maven-surefire/pull/157 I've put together an integration test that can validate the execution of [SUREFIRE-1383](https://issues.apache.org/jira/browse/SUREFIRE-1383). And all is good on that front.

[GitHub] maven-surefire issue #157: SUREFIRE-1383 dependenciesToScan Does Not Leverag...

2017-07-04 Thread owenfarrell
Github user owenfarrell commented on the issue: https://github.com/apache/maven-surefire/pull/157 Looking at it a bit deeper, looks like the root cause is that the `it-parent` POM is not installed in the IT repo. The issue isn't the dependency itself. --- If your project is set up

[GitHub] maven-surefire issue #157: SUREFIRE-1383 dependenciesToScan Does Not Leverag...

2017-07-04 Thread owenfarrell
Github user owenfarrell commented on the issue: https://github.com/apache/maven-surefire/pull/157 One workaround for this would be to install the missing `it-parent` POM as part of my first lifecycle (i.e. mid-test). But this seems kind of hacky. ```