On Fri, 2004-01-16 at 15:24, Eric Giguere wrote:
> Hi all
> 
> A simple modification proposed for the pluggin.jelly for the jar plugin.
> As it is now, it calls ant:jar but only with the exclusions. Listing 
> exclusions is sometime longer than inclusions. By adding a property in 
> the build.properties (maven.jar.includes) and modifying plugin.jelly 
> like this:
> 
> cvs diff plugin.jelly (in directory 
> F:\java\apache-public-cvs\maven-plugins\jar\)
> Index: plugin.jelly
> ===================================================================
> RCS file: /home/cvspublic/maven-plugins/jar/plugin.jelly,v
> retrieving revision 1.27
> diff -r1.27 plugin.jelly
> 32c32,33
> <       excludes="${maven.jar.excludes}">
> ---
>  >       excludes="${maven.jar.excludes}"
>  >       includes="${maven.jar.includes}">
> 
> allows usage of both includes and excludes which can be very usefull in 
> certain circumptances.

Do you have some examples?

We definitely appreciate the patch but reasons are always helpful :-)

Jira is also a more appropriate place for patches and suggested
enhancements:

http://jira.codehaus.org

> My first enhancement proposal, is it the right place to do it? If not, 
> sorry for the unwanted traffic.
> Didn't call it a patch either, neither did I open a bug for this since 
> its not a bug ;)
> 
> Eric.
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
-- 
jvz.

Jason van Zyl
[EMAIL PROTECTED]
http://maven.apache.org

In short, man creates for himself a new religion of a rational
and technical order to justify his work and to be justified in it.

-- Jacques Ellul, The Technological Society


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to