Skip to content

Instantly share code, notes, and snippets.

@tednaleid
Created July 22, 2012 03:07
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save tednaleid/3158146 to your computer and use it in GitHub Desktop.
Save tednaleid/3158146 to your computer and use it in GitHub Desktop.
removing redundant `packageApp()` call in bootstrap target shaves ~1sec off many grails commands for me
Not sure if this causes issues elsewhere, but `test-app`, `run-app`, and `package` seem fine. The `packageApp()` method is called again in `loadApp()` so this one seems redundant.
Shaves off about a second for me on a larger app as it removes a redundant call. YMMV, put it back if you see any issues:
diff --git a/scripts/_GrailsBootstrap.groovy b/scripts/_GrailsBootstrap.groovy
index 4c44aa3..218a41a 100644
--- a/scripts/_GrailsBootstrap.groovy
+++ b/scripts/_GrailsBootstrap.groovy
@@ -126,7 +126,7 @@ target(monitorApp:"Monitors an application for changes using the PluginMana
}
target(bootstrap: "Loads and configures a Grails instance") {
- packageApp()
+ //packageApp()
loadApp()
configureApp()
}
Seems to be a lot of redundant and messy calls in the current gant scripts, I'm really looking forward to the gradle build system that's on the roadmap for grails 3.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment