Re: [VFS] Any news on VFS-500 (implement VFSClassLoader#findResources)

2014-02-04 Thread Gary Gregory
Bernd, Please see my comments in the Jira. Gary On Mon, Feb 3, 2014 at 1:22 PM, Bernd wrote: > Hello, > > just a quick reminder, the Patch which completes the VFSClassLoader > interface (which also includes a testcase) seems to be still unassigned. > Anything I can do to help with that to get

Re: [VFS] Fix up pom so it works with Commons release profile

2014-02-04 Thread sebb
Is the multi-module VFS pom really needed? Commons Parent seems to work fine with single module poms. On 4 February 2014 22:48, Bernd Eckenfels wrote: > Am Tue, 04 Feb 2014 23:20:45 +0100 > schrieb Jörg Schaible : >> > This is a good thing to have. What about OSGi bundle >> > plugin? I am not su

Re: [VFS] Fix up pom so it works with Commons release profile

2014-02-04 Thread Bernd Eckenfels
Am Tue, 04 Feb 2014 23:20:45 +0100 schrieb Jörg Schaible : > > This is a good thing to have. What about OSGi bundle > > plugin? I am not sure if we need the pluginManagement > > preconfiguration at all. > > Does it harm? At least I dont get it to work in the VFS POM. A Stand-alone POM with a test

Re: [VFS] Fix up pom so it works with Commons release profile

2014-02-04 Thread Jörg Schaible
Bernd Eckenfels wrote: > Am Tue, 04 Feb 2014 21:53:24 +0100 > schrieb Jörg Schaible : > >> We could move the compiler and jar plugin into an own profile of the >> parent, that is automatically activated if src/main/java exists. >> That, however, enforces Maven 3 and standard layout for all Java >

Re: [VFS] Fix up pom so it works with Commons release profile

2014-02-04 Thread Bernd Eckenfels
Am Tue, 04 Feb 2014 21:53:24 +0100 schrieb Jörg Schaible : > We could move the compiler and jar plugin into an own profile of the > parent, that is automatically activated if src/main/java exists. > That, however, enforces Maven 3 and standard layout for all Java > projects. Yes, I thought about

Re: [VFS] Fix up pom so it works with Commons release profile

2014-02-04 Thread Jörg Schaible
Hi Bernd, Bernd Eckenfels wrote: > Am Tue, 21 Jan 2014 16:10:35 + > schrieb sebb : > >> I have done some work on this.I can fix up the poms so that the >> Commons "release" profile works. >> And it looks like the distribution module is not really needed. The >> assembly plugin can be run fro

Re: commons-lang pull request: add isNull and isNotNull methods

2014-02-04 Thread Bernd Eckenfels
Hello, I agree, for a simple "isNull(Object)" predicate, this is really an anti pattern. I can understand validate() methods which throw a application specific cleaned up exception and do some logging and stuff. But this is nothing for a external dependency. This is different from isEmpty(String)

Re: [VFS] Fix up pom so it works with Commons release profile

2014-02-04 Thread Bernd Eckenfels
Am Tue, 21 Jan 2014 16:10:35 + schrieb sebb : > I have done some work on this.I can fix up the poms so that the > Commons "release" profile works. > And it looks like the distribution module is not really needed. The > assembly plugin can be run from the top level pom. (sorry for picking this

Re: commons-lang pull request: add isNull and isNotNull methods

2014-02-04 Thread Adrian Crum
Personally, I view this as an anti-pattern. You have a null reference in your hand, and instead of checking the reference that is in your hand, you give it to an unrelated class and have that class check to see if it is a null reference. That just doesn't make any sense to me. Adrian Crum Sand

commons-lang pull request: add isNull and isNotNull methods

2014-02-04 Thread otaviojava
GitHub user otaviojava opened a pull request: https://github.com/apache/commons-lang/pull/16 add isNull and isNotNull methods Sugestion to add two simple and useful methods, with tests, on ObjectsUtil to get the code cleaner. isnull and isNotNull. So you can use if (ObjectUtil.isNu

[continuum] BUILD FAILURE: Apache Commons DbUtils - Apache Commons (Build using Java 1.6)

2014-02-04 Thread Apache Continuum
Online report : https://continuum-ci.apache.org/continuum/buildResult.action?buildId=28010&projectId=74 Build statistics: State: Failed Previous State: Failed Started at: Tue 4 Feb 2014 15:20:18 + Finished at: Tue 4 Feb 2014 15:20:38 + Total time: 20s Build Trigger: Schedule

[continuum] BUILD FAILURE: Apache Commons DbUtils - Apache Commons (Build using Java 1.6)

2014-02-04 Thread Apache Continuum
Online report : https://continuum-ci.apache.org/continuum/buildResult.action?buildId=28008&projectId=74 Build statistics: State: Failed Previous State: Failed Started at: Tue 4 Feb 2014 14:06:16 + Finished at: Tue 4 Feb 2014 14:06:35 + Total time: 19s Build Trigger: Forced B

[continuum] BUILD FAILURE: Apache Commons Parent - Apache Commons (CP cannot use -Prelease)

2014-02-04 Thread Apache Continuum
Online report : https://continuum-ci.apache.org/continuum/buildResult.action?buildId=28007&projectId=131 Build statistics: State: Failed Previous State: Ok Started at: Tue 4 Feb 2014 13:20:55 + Finished at: Tue 4 Feb 2014 13:21:01 + Total time: 6s Build Trigger: Schedule Bui

[continuum] BUILD FAILURE: Apache Commons DbUtils - Apache Commons (Build using Java 1.6)

2014-02-04 Thread Apache Continuum
Online report : https://continuum-ci.apache.org/continuum/buildResult.action?buildId=28006&projectId=74 Build statistics: State: Failed Previous State: Failed Started at: Tue 4 Feb 2014 13:19:18 + Finished at: Tue 4 Feb 2014 13:19:36 + Total time: 17s Build Trigger: Forced B

[continuum] BUILD FAILURE: Apache Commons DbUtils - Apache Commons (Build using Java 1.6)

2014-02-04 Thread Apache Continuum
Online report : https://continuum-ci.apache.org/continuum/buildResult.action?buildId=28005&projectId=74 Build statistics: State: Failed Previous State: Failed Started at: Tue 4 Feb 2014 13:16:49 + Finished at: Tue 4 Feb 2014 13:17:05 + Total time: 16s Build Trigger: Forced B

[continuum] BUILD FAILURE: Apache Commons DbUtils - Apache Commons (Build using Java 1.6)

2014-02-04 Thread Apache Continuum
Online report : https://continuum-ci.apache.org/continuum/buildResult.action?buildId=28004&projectId=74 Build statistics: State: Failed Previous State: Failed Started at: Tue 4 Feb 2014 13:12:19 + Finished at: Tue 4 Feb 2014 13:12:46 + Total time: 27s Build Trigger: Forced B