add <includeRubygemsInTestResources>false</includeRubygemsInTestResources>
in config. btw would like to revert your adding back the joda-time deps since came where travis did not fail on joda-time anymore. and it will break -Dtzdata=.... thingy. - christian On Sun, Jul 7, 2013 at 6:56 PM, Charles Oliver Nutter <head...@headius.com>wrote: > 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 > > >
