Another oddity... Benoit is trying to build on master and ran into compilation issues in the redcar-javamateview gem. I was able to reproduce by installing the gem and then doing a clean build.
It appears that the maven gem plugin is copying all gems from our normal gem home into test/target/test-classes. Because that gem has java sources, it tries to compile them and fails. After installing the redcar-javamateview gem (which pulls in the swt gem), I see this in test/target/test-classes/gems: $ ls test/target/test-classes/gems/ diff-lcs-1.1.3/ minitest-4.7.5/ rdoc-3.12/ rspec-core-2.12.2/ swt-0.16/ jruby-launcher-1.0.18.pre1-java/ minitest-excludes-1.0.2/ redcar-javamateview-0.2/ rspec-expectations-2.12.1/ json-1.7.3-java/ rake-10.1.0/ And I get the same compilation errors Benoit sees: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:testCompile (default-testCompile) on project jruby-tests: Compilation failure: Compilation failure: [ERROR] /Users/headius/projects/jruby/test/target/test-classes/gems/redcar-javamateview-0.2/test/com/redcareditor/mate/MateTextTest.java:[3,30] error: package org.eclipse.swt.widgets does not exist [ERROR] /Users/headius/projects/jruby/test/target/test-classes/gems/redcar-javamateview-0.2/src/com/redcareditor/mate/MateText.java:[12] error: package org.eclipse.jface.text does not exist [ERROR] /Users/headius/projects/jruby/test/target/test-classes/gems/redcar-javamateview-0.2/src/com/redcareditor/mate/MateText.java:[13] error: package org.eclipse.jface.text.source does not exist [ERROR] /Users/headius/projects/jruby/test/target/test-classes/gems/redcar-javamateview-0.2/src/com/redcareditor/mate/MateText.java:[14,22] error: package org.eclipse.swt does not exist ... This is definitely *not* behavior we want. Ideally no gems would get into test/target/test-classes *at all* and at the very least we shouldn't be copying gems we don't depend on for running tests. What can we do, kristian? - Charlie On Sun, Jul 7, 2013 at 1:40 PM, Charles Oliver Nutter <head...@headius.com> wrote: > Master appears to be broken due to the move of joda-time out of normal > dependencies. I'm going to move it back so we have a working build > again. > > - Charlie > > On Sun, Jul 7, 2013 at 9:09 AM, Christian Meier > <meier.krist...@gmail.com> wrote: >> Yes, I had the same thought to get joda-timezone out of the normal build. I >> will try this. >> - christian >> >> >> >> >> Hirotsugu Asari <asari.r...@gmail.com> schrieb: >>> >>> >>> On Jul 7, 2013, at 7:47 AM, kristian <m.krist...@web.de> wrote: >>> >>>> @headius could you push joda-timezones-2012j which is needed for >>>> core/pom.xml even though it will not be packed with the shade-plugin but >>>> needed to populate the Constants.java correctly. >>>> >>>> - christian >>> >>> >>> I see that tzdata.version gets reset (and subsequent builds fail, due to >>> the issue that you raise above). So I feel that we are *almost* there. >>> >>> But 2012j is distributed with joda-time. Is it possible to convince maven >>> that we don't need that artifact under certain circumstances? >>> >>> Hiro >>> ________________________________ >>> >>> To unsubscribe from this list, please visit: >>> >>> http://xircles.codehaus.org/manage_email >>> >>> >> >> -- >> Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet. --------------------------------------------------------------------- To unsubscribe from this list, please visit: http://xircles.codehaus.org/manage_email
