Because of how the gradle dependency resolution works, it would very likely break or cause core to depend on an external change. This is why I viewed it as a must. If the tests removed the plugin, it could be removed.
On Thu, May 21, 2026 at 2:57 AM Mattias Reichel <[email protected]> wrote: > I think grails-mail is only used in functional tests, so I'm not sure > that grails-mail needs to be merged into grails-core. > > Den ons 20 maj 2026 kl 20:25 skrev James Fredley <[email protected] > >: > > > > Before we create a vote thread to consolidate grails-spring-security > into grails-core, grails-spring-security depends on > org.apache.grails:grails-redis and org.grails.plugins:grails-mail and given > the policies we apply to grails-core, I think both of those would have to > be merged into grails-core, also. > > > > Agree or disagree? > > > > James > > > > On 2026/05/18 12:04:43 James Daugherty wrote: > > > We briefly discussed this on the weekly, but I thought this deserves > its > > > own thread: I would like to propose we merge grails-spring-security > into > > > grails-core and its associated dependencies (means grails-redis). > > > > > > The reason we did not merge these before was the security build often > hung > > > and took 2+ hours. Recently Mattias has worked to make that more > stable. > > > It’s now running in 1-2min. > > > > > > We also have not done the best job releasing dependency updates on this > > > repo and by moving it into core we can ensure timely updates. We can > > > also easily identify dependencies that should really be in the bom. > > > > > > I would propose we move the 7.0 branch into 7.0.x so we can cease all > > > activity on spring security. > > > > > > What do others think? > > > > > > -James > > > >
