Ok going to merge @thauvin’s PR, then you can test...

# kobalt

[cedric]

Ok going to merge @thauvin’s PR, then you can test it @rhencke

[rhencke]

@ethauvin I'll take a look into this when I get in tonight.

[rhencke]

@ethauvin One thing I noticed reading over this is that compile-only dependencies are being added to dependenciesTest, if I'm reading that right?

[ethauvin]

I think so, don't remember now.

[ethauvin]

I know it's not on --projectInfo

[rhencke]

OK - let me poke a little more. I may be misreading it. I was just trying to grok it all before I ran it 🙂

[rhencke]

Gradle has a bit of a different stance on it - compile-only dependencies are not visible to tests. They have a brain-breaking chart here: https://docs.gradle.org/current/userguide/java_plugin.html#N161E0

[rhencke]

Not saying that Kobalt has to do what Gradle does. Just that I haven't thought through the ramifications of exposing it to tests.

[ethauvin]

Hmmm. You may have a point there. Let me know what you find, I can fix it if needed.

[rhencke]

OK - I mean, it's not really that that's broken, per se. I can see it argued both ways. But it's worth noting it's different. But let me actually try instead of just reading. 😉

[ethauvin]

Yeah, it might work like Gradle, I frankly don't remember where I included the compile only dependencies. There are few places. 😉

[ethauvin]

Whatever made sense at the time, now my mind is fried. 😉

[rhencke]

haha I know the feeling

[ethauvin]

Dealing with file permissions on windows right now, trying to make a test that insures that kobaltw is executable. Windows sucks.

[rhencke]

Yeahhhh that's not fun.

[rhencke]

You could probably check the attributes it's stored with in the zip file?

[rhencke]

@ethauvin Thinking about it more... what's the difference between a 'provided' reference and a 'compileOnly' reference? From my end it was just the metadata in the POM. But this would resolve differently than 'provided' right?

[rhencke]

I may have to take a fresh look in the morning. My brain's too spent. 🙂

[ethauvin]

compileOnly will not show in any dependencies.

[ethauvin]

It's just for compiling.

[rhencke]

@ethauvin Ah, got it. Thanks.

[ethauvin]

Have you had a chance to try it out?

[rhencke]

@ethauvin Sorry - work hit like a freight train and I've been swamped. I should get to try it this weekend - tomorrow hopefully.

[rhencke]

Sorry for the late reply.

[ethauvin]

No worries. Been there, done that. 😉

[rhencke]

Trying to fight refactoring a 20+ year old codebase. It's so bad. It's like everything you shouldn't do in Java.

[rhencke]

trying to get a proof of concept before a deadline, and I'm stubborn so I keep poking at it way past when I should give up.

[rhencke]

But I digress. I'll fill up 200 pages talking about it

[rhencke]

Thanks for the understanding though. Definitely want to dig in and try it