On 21/03/15 13:18, Andrea Aime wrote: > By the looks of it, we need apacheds-all. The strange thing is that > apparently maven uses pom.xml order to > order classpath entries and gs-main, from which we inherit bouncycastle, is > declared before the spring-ldap-test > from which we get apacheds-all ... but wait, aren't test dependecies loaded > before the normal one in test runs?
Since Maven 2.0.9 (IIRC), dependencies are on the classpath in pom order, with all inherited dependencies afterwards. gs-main:test is a dependency of the parent pom, so spring-ldap-test and its apacheds-all dependency won the race. The other change required for success was to change the Travis CI build from "clean test" to "clean install" to make sure changes can be resolved across modules. Otherwise, we are testing changed modules but not integration between them; this was resulting in much Travis CI breakage. According to the docs, pull requests do not result in repository cache update. > If so, then maybe declaring a bouncycastle dependency before the > spring-ldap-test one will do the trick Done and merged: https://github.com/geoserver/geoserver/pull/1001 Andrea, your bouncycastle security provider dependency order fix worked, but there is still an (apparently) unrelated failure for Oracle JDK 7 in wps-core ExecuteTest. Intermittent problem? Another victim of -T2? I have made another test pull request to see if we get the same result. Kind regards, -- Ben Caradoc-Davies <[email protected]> Director Transient Software Limited <http://transient.nz> New Zealand ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
