[GRADLE-1558] improve application plugin to support to copy more files to distribution Created: 17/May/11 Updated: 04/Jan/13 Resolved: 15/Jul/11 |
|
Status: | Resolved |
Project: | Gradle |
Affects Version/s: | 1.0-milestone-3 |
Fix Version/s: | 1.0-milestone-4 |
Type: | Improvement | ||
Reporter: | Sky Ao | Assignee: | Luke Daley |
Resolution: | Fixed | Votes: | 0 |
Description |
For task installApp and distZip of application plugin, it only copies bin/lib to target distribution. One suggestion about this, since we always need to package more files into our application distribution, not only bin script and libs, but also some configuration files and other files like readme. Why we don't add this support into the application plugin : add a file list properties for the additional directory / files, if set, copy them into the distribution. I think this is a general requirement for almost everyone and every application. So it will be helpful if it is supported by this plugin. |
Comments |
Comment by René Gröschke (Inactive) [ 06/Jun/11 ] |
I've just started working on this issue (see https://github.com/breskeby/gradle/tree/GRADLE-1558). As a solution I would suggest to add a customizable directory to the application plugin convention, that points to a directory containing additional resources for the application (default could be "src/dist"). This directory is added to the distSpec used in the distZip and in the installApp Task. Opinions/Suggestions? |
Comment by Szczepan Faber [ 08/Jun/11 ] |
Makes sense. The newly added ear plugin does something similar - deploys into the ear the contents of src/main/application (by default) |
Comment by René Gröschke (Inactive) [ 09/Jun/11 ] |
just send a pull request ( https://github.com/gradle/gradle/pull/35 ) As a default I use src/main/dist |
Comment by Luke Daley [ 15/Jul/11 ] |
Added in https://github.com/gradle/gradle/commit/4f08d88e257d7f62c6cad0e61f1f351092d76375 Thanks for the contribution I ended up changing it slightly to use a CopySpec, which offers more flexibility. |