Wayne does and so do I.
I don't agree with the complexity. Per execution it's pretty clear what you are doing, but the fact that you need multiple execution-blocks might look like there is a lot of overhead. If you think there's a way to minimize it, we're all interested. Looking at your example there's one thing you need to know: Maven2 doesn't support xml-attributes in the pom, only tags. So to solve this you'll have to subtags. And that could be a reason to keep it as flat is it is right now. If you check the goal-information page (like http://mojo.codehaus.org/sql-maven-plugin/execute-mojo.html ) you can see exactly what type is expected, what it's default value is, etc. Like Wayne said, you can always submit a patch. Maybe it'll inspire us and we can indeed improve the plugin. - Robert > Date: Wed, 23 Jun 2010 22:15:17 +0100 > From: [email protected] > To: [email protected] > Subject: Re: [mojo-user] sql-maven-plugin: sql files need different delimiter > configurations for packages and stored proceduees > > Hi Wayne, thanks for the reply, > > Would such functionality be accepted? I understood that if changes do > not fit in the "maven way" then they are refused. > I'd happily submit a patch if it had any chance of being accepted. Do > you have any authority in this area? > > Thanks again, > Tim > > > > On 23 June 2010 22:09, Wayne Fay <[email protected]> wrote: > >> to reply. But this is very complex for something that should be > >> simple. (See further comments after the pom extract) > >> > >> THIS pom REALLY FRICKIN' SUCKS FROM A COMPLEXITY POINT OF VIEW > >> > >> Maven seems to make this common use case really hard! > > > > Feel free to submit patches to improve the functionality of the plugin. > > > > Wayne > > > > --------------------------------------------------------------------- > > To unsubscribe from this list, please visit: > > > > http://xircles.codehaus.org/manage_email > > > > > > > > > > -- > -o- > > --------------------------------------------------------------------- > To unsubscribe from this list, please visit: > > http://xircles.codehaus.org/manage_email > > _________________________________________________________________ New Windows 7: Find the right PC for you. Learn more. http://windows.microsoft.com/shop
