I just hit the problem today with one of my colleague. Yes, System props should definitely win.
Thanks, Stéphane On Sat, Feb 23, 2008 at 11:28 PM, Olivier Lamy <[EMAIL PROTECTED]> wrote: > Now the question is "do we have to change this order ?". > Have a look at MRESOURCES-39 [1]. > > Users complains about system properties (-D in the mvn cli) doesn't win. > IMHO, system props should wins. > > This was the case with maven1 [2]. > > Thougths ? > > Thanks, > -- > Olivier > > [1] http://jira.codehaus.org/browse/MRESOURCES-39 > [2] http://maven.apache.org/maven-1.x/reference/properties.html > > 2008/2/19, Olivier Lamy <[EMAIL PROTECTED]>: > > > > Yep sure. > > Currently, the new component use the same strategy as the > > maven-resources-plugin. > > > > > > -- > > Olivier > > > > 2008/2/19, Stephane Nicoll <[EMAIL PROTECTED]>: > > > I guess we should have a look to the way this is done currently to > > > avoid breaking the backward compat. > > > > > > On Feb 19, 2008 11:30 AM, Olivier Lamy <[EMAIL PROTECTED]> wrote: > > > > Sure, could be better. > > > > But we have to agree on the order : > > > > * System Properties > > > > * pom.properties > > > > * List of properties ( the method has a parameter which accept a List > > > > of String -> path properties files ) (war plugin use it to pass a list > > > > of properties file). > > > > * pom.filters > > > > * pom.build.filters > > > > > > > > ? > > > > > > > > -- > > > > Olivier > > > > > > > > > > > > 2008/2/19, Stephane Nicoll <[EMAIL PROTECTED]>: > > > > > > > > > This one was very much expected, thanks for doing this. > > > > > > > > > > However, I think that we should use a first-win strategy, otherwise > > > > > there is no way to control the actual value of a property. First-win > > > > > strategy is mainly used everywhere in the war plugin and ease a lot > of > > > > > stuff > > > > > > > > > > Is there any reason you choose doing this? > > > > > > > > > > Thanks, > > > > > Stéphane > > > > > > > > > > > > > > > On Feb 19, 2008 10:17 AM, Olivier Lamy <[EMAIL PROTECTED]> wrote: > > > > > > Hi, > > > > > > As you know some plugins made some filtering base on the code > coming > > > > > > from the maven-resources-plugin. > > > > > > This means there are some copy and paste from the resources plugin > > > > > > source to other plugins. > > > > > > To prevent this, the code from the resources plugin has been put > in a > > > > > > plexus component (currently in shared sandbox [1]). > > > > > > A documentation has been started [2]. > > > > > > It has been integrated in the maven-war-plugin trunk. > > > > > > Before calling a vote on a first alpha-1 release, I'd like to have > > > > > > some comments/thoughts on it. > > > > > > > > > > > > The final goal of this component should be : using it in all > plugins > > > > > > which need filtering. > > > > > > > > > > > > Thanks, > > > > > > -- > > > > > > Olivier > > > > > > > > > > > > [1] : > https://svn.apache.org/repos/asf/maven/sandbox/trunk/shared/maven-filtering/ > > > > > > [2] : > http://people.apache.org/~olamy/staging-sites/maven-filtering/ > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Large Systems Suck: This rule is 100% transitive. If you build one, > > > > > you suck" -- S.Yegge > > > > > > > > > > > --------------------------------------------------------------------- > > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > > > > > > > -- > > > Large Systems Suck: This rule is 100% transitive. If you build one, > > > you suck" -- S.Yegge > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Large Systems Suck: This rule is 100% transitive. If you build one, you suck" -- S.Yegge --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]