grails / grails-data-mapping

GORM - Groovy Object Mapping
http://gorm.grails.org/
218 stars 198 forks source link

Revert classpath scanning change, use the same strategy as grails-core #1837

Closed jdaugherty closed 3 weeks ago

jdaugherty commented 3 weeks ago

I spoke to @jamesfredley and we decided to revert the initial change - there were further issues on the Spring side that didn't look fixed until 6.1.7.

This also lead to the discovery that AnnotationMetdataReader is copied from grails-core. Hence, this pull request will copy the deprecated spring classes as well. Worse, there's even an ASM bug fix on the data-mapping side, that isn't in the core side.

Long term we need a base support project that can be shared between grails & gorm, but that is out of scope for now. I'll bring this up in our weekly meeting.