search

gosu-lang / gradle-gosu-plugin

BSD 3-Clause "New" or "Revised" License
5 stars 7 forks source link

Add an optional 'orderClasspath' property to GosuCompile task #13

Closed DPUkyle closed 8 years ago

DPUkyle commented 8 years ago

Guidewire's build has a unique requirement to do depth-first traversal of the dependency tree.

Guidewire has a separate process to implement that logic, but the results of that process need to influence the build classpath (specifically, its ordering).

If the optional orderClasspath property is set, it should supercede/replace the normal classpath argument.

DPUkyle commented 8 years ago

As mentioned in the log for df5fb6c, I think we can manage this in configuration. Closing as won't fix.

DPUkyle commented 8 years ago

Reopening; turns out we actually need a closure for execution-time resolution of dependencies in the Guidewire build.

DPUkyle commented 8 years ago

Fixed and appears to be working; will continue to monitor in Guidewire's test environment.

DPUkyle commented 8 years ago

Just a thought - couldn't we also make the classpath property a Closure too? Then we could set it dynamically at execution time and avoid the need to use orderedClasspath