These are chat archives for jruby-gradle/jruby-gradle-plugin

30th
Oct 2014
R. Tyler Croy
@rtyler
Oct 30 2014 01:45
I am seeing it now :P
Schalk W. Cronjé
@ysb33r
Oct 30 2014 13:51
Is it something you can reproduce easily?
R. Tyler Croy
@rtyler
Oct 30 2014 15:13
I've reproduced it, but I'm not entirely sure what's wrong
jruby debugging flags are not turning on
Schalk W. Cronjé
@ysb33r
Oct 30 2014 17:34
This is what I reckon. PATH is propagated from the the environment via Gradle to JRubyExec. If you supply setEnvironment [] as the first line in the JRubyExec-based task, do you still see the problem?
Schalk W. Cronjé
@ysb33r
Oct 30 2014 17:44
I think your rspec case works. because rspec is already in jruby-complete, but compass is not.
ysb33r @ysb33r could be wrong (obviously)
R. Tyler Croy
@rtyler
Oct 30 2014 18:02
hrm
@ysb33r you're smert!
it does look like org.jruby.Main is relying on PATH somewhere
Schalk W. Cronjé
@ysb33r
Oct 30 2014 18:42
I left a braindump on GitHub #70.
R. Tyler Croy
@rtyler
Oct 30 2014 19:51
@ysb33r for your review: jruby-gradle/jruby-gradle-plugin#72 tested manually with success
R. Tyler Croy
@rtyler
Oct 30 2014 20:23
@ysb33r updated with a test, mind if I merge?
rtyler @rtyler counts to twelve
Schalk W. Cronjé
@ysb33r
Oct 30 2014 21:00
Sure.
R. Tyler Croy
@rtyler
Oct 30 2014 21:00
I'm going to cut a 0.1.4 release with this fix for @robfletcher
hopefully get him unblocked
ysb33r @ysb33r has been on the road for the last hour
R. Tyler Croy
@rtyler
Oct 30 2014 21:01
beep beep
Schalk W. Cronjé
@ysb33r
Oct 30 2014 21:01
Yes, do that
I would liek to have another change made, but can leave that for 0.1.5
R. Tyler Croy
@rtyler
Oct 30 2014 21:02
what change?
Schalk W. Cronjé
@ysb33r
Oct 30 2014 21:02
I think I would like the ability to control inheritance from system environment
but I need to think about that
R. Tyler Croy
@rtyler
Oct 30 2014 21:03
ah, hokay
cutting
R. Tyler Croy
@rtyler
Oct 30 2014 21:26
@ysb33r can I get you to run the integration tests on your machine?
I'm getting failures with HEAD of master and I'mn not sure they're legit
R. Tyler Croy
@rtyler
Oct 30 2014 21:36
hrm, this might have been failing for a while
Schalk W. Cronjé
@ysb33r
Oct 30 2014 21:36
OK I’ll run then in about an hour when I get home. They will run too slow over 3G.
rtyler @rtyler nods
Schalk W. Cronjé
@ysb33r
Oct 30 2014 21:37
Do they get run on BuildHive?
R. Tyler Croy
@rtyler
Oct 30 2014 21:41
I don't believe so
checking
they're not right now, just added the check task to the list
they're failing for me on 0.1.2 so I'm thinking this might be a environment thing specific to me perhaps
Schalk W. Cronjé
@ysb33r
Oct 30 2014 21:42
THat would a quicker way to check (no pun intended)
Schalk W. Cronjé
@ysb33r
Oct 30 2014 22:01
I’m getting on the road again
R. Tyler Croy
@rtyler
Oct 30 2014 22:58
heh
Schalk W. Cronjé
@ysb33r
Oct 30 2014 23:21
home now - running ./gradlew build
R. Tyler Croy
@rtyler
Oct 30 2014 23:30
looks like it failed in buildhive too
Schalk W. Cronjé
@ysb33r
Oct 30 2014 23:33
so far i’m seeing
R. Tyler Croy
@rtyler
Oct 30 2014 23:33
hrm
Schalk W. Cronjé
@ysb33r
Oct 30 2014 23:33
    new File(prepTask.outputDir,'gems/temple-0.6.8').exists()
    |        |        |                              |
    |        |        |                              false
    |        |        /Users/schalkc/Projects/jruby-gradle-dev/base/build/tmp/test/unittests/jpgis
    |        task ':RubyWax'
    /Users/schalkc/Projects/jruby-gradle-dev/base/build/tmp/test/unittests/jpgis/gems/temple-0.6.8
        at com.github.jrubygradle.JRubyPrepareGemsIntegrationSpec.Unpack our gem as normal(JRubyPrepareGemsIntegrationSpec.groovy:56)
R. Tyler Croy
@rtyler
Oct 30 2014 23:34
HAH, I think I know the issue
I don't think the integration tests are using a pinned version
I have temple-0.6.10 installed :P
wait, they are o_O
how the fak did 0.6.10 get installed
actually, I have no idea
aha
I was right the first time
Schalk W. Cronjé
@ysb33r
Oct 30 2014 23:38
transitives?
R. Tyler Croy
@rtyler
Oct 30 2014 23:38
it's an unpinned transitive dependency
yeah
I'm going to just update the test for now and file an issue
I want to get 0.1.4 out
Schalk W. Cronjé
@ysb33r
Oct 30 2014 23:39
ok