spring-view-component icon indicating copy to clipboard operation
spring-view-component copied to clipboard

hard dependency on devtools

Open Brunwo opened this issue 1 year ago • 4 comments

the dependency graph for spring-view-component-core shows devtools :

| +--- de.tschuehly:spring-view-component-core:0.8.1 | | +--- org.springframework.boot:spring-boot-starter-web -> 3.2.5 () | | +--- org.springframework.boot:spring-boot-starter-aop -> 3.2.5 () | | +--- org.springframework.boot:spring-boot-devtools -> 3.2.5 | | | +--- org.springframework.boot:spring-boot:3.2.5 () | | | --- org.springframework.boot:spring-boot-autoconfigure:3.2.5 ()

https://github.com/tschuehly/spring-view-component/blob/3ea2a9af8d4c68423526d7d025a27b830dc2c0f1/core/build.gradle.kts#L28

I"m pointing this because I wanted to run your lab repository, that relies on this, but had runtime clashing classloader issues, couldn't remove devtools to try to fix

Brunwo avatar Sep 06 '24 18:09 Brunwo

Yeah the livereload implementation relies on devtools, I don't know how to make it optional tho. Have to figure that out sometime

tschuehly avatar Sep 06 '24 18:09 tschuehly

could replace 'implementation' by developmentOnly , unless you want it also present for launches other than bootrun

Brunwo avatar Sep 11 '24 17:09 Brunwo

could replace 'implementation' by developmentOnly , unless you want it also present for launches other than bootrun

The problem is that the https://github.com/tschuehly/spring-view-component/blob/master/core/src/main/kotlin/de/tschuehly/spring/viewcomponent/core/processor/ViewComponentChangeListener.kt directly depends on devtools

tschuehly avatar Oct 25 '24 21:10 tschuehly

This problem can be solved with checked exception. Maybe it can throw more clear exception message.

sorry for lack of test in previous commit. I made new version using custom condition and this version does not fail on bean creation time.

Clickin avatar Dec 28 '24 02:12 Clickin