Grails

PackagePlugin includes **BootStrap.groovy if present in plugin, which kills the app it is installed into

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.0.2
  • Fix Version/s: 1.0.3
  • Component/s: None
  • Labels:
    None

Activity

Hide
Sergey Nebolsin added a comment -

I just want to note that in Quartz plugin we depend on QuartzBootStrap file which is bundled in the plugin's distribution to make some required operations during bootstrap process.

Show
Sergey Nebolsin added a comment - I just want to note that in Quartz plugin we depend on QuartzBootStrap file which is bundled in the plugin's distribution to make some required operations during bootstrap process.
Hide
Marc Palmer added a comment -

Ok but the problem here is that if the name clashes with the bootstrap in the app, it kills the app.

So in that case perhaps we have to change the logic so that in applications it only loads BootStrap.groovy and in plugins they must use the plugin name as the bootstrap - i.e. QuartzBootStrap - and this is enforced in package-plugin.

The reason for this is that a plugin might have a LoaderBootStrap.groovy otherwise (for example) and more than one plugin might provide this and then boom the two plugins can't work together.

Show
Marc Palmer added a comment - Ok but the problem here is that if the name clashes with the bootstrap in the app, it kills the app. So in that case perhaps we have to change the logic so that in applications it only loads BootStrap.groovy and in plugins they must use the plugin name as the bootstrap - i.e. QuartzBootStrap - and this is enforced in package-plugin. The reason for this is that a plugin might have a LoaderBootStrap.groovy otherwise (for example) and more than one plugin might provide this and then boom the two plugins can't work together.
Hide
Graeme Rocher added a comment -

Bulk closing bunch of resolved issues

Show
Graeme Rocher added a comment - Bulk closing bunch of resolved issues

People

Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: