Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-09 Thread anita kulshreshtha
inline.. --- Jason Dillon [EMAIL PROTECTED] wrote: On Aug 7, 2006, at 10:33 PM, anita kulshreshtha wrote: This code is from servlets-examples-jetty config (rev 429124): resources resource directory${pom.basedir}/src/conf/directory

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-09 Thread Matt Hogstrom
I get the sense that both of you are a bit frustrated. The transition to the new RTC development model has been challenging for all. The PMC has not kept up with the number of reviews and that has allowed the codebase to drift while patches then get stale. Recently we've made several steps

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-09 Thread Jason Dillon
On Aug 9, 2006, at 8:49 AM, Matt Hogstrom wrote: Also, it sounds like some of the changes are preferences based on style. It would be a fair debate as to one should use Plexus or Geronimo infrastructure for the file delete activity. Are you kidding me? Maven2 is built upon Plexus...

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-09 Thread Jason Dillon
On Aug 9, 2006, at 8:04 AM, anita kulshreshtha wrote: The patch on July 27th was for m2migration and it is clearly written. Where is it? I keep getting lost in all of the patches. --jason

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-09 Thread Matt Hogstrom
Well, you got me. Given that so many people are heavily into Maven I have to admit ignorance in there. I think Anita had made a comment that she was reusing existing Geronimo code and that you were using something from Plexus. Not everyone has that same mighty mass of knowledge you take for

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-08 Thread Jason Dillon
On Aug 7, 2006, at 10:33 PM, anita kulshreshtha wrote: This code is from servlets-examples-jetty config (rev 429124): resources resource directory${pom.basedir}/src/conf/directory targetPathMETA-INF/targetPath includes

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-07 Thread anita kulshreshtha
--- Jason Dillon [EMAIL PROTECTED] wrote: The patch was not used, instead the plugin code was modified to add this functionality. I only added a few changes you had in the original patch since they were the only ones that could be added cleanly. ? As you mentioned, then

M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-06 Thread anita kulshreshtha
Jason and others, The car-maven-plugin (trunk rev 429115) still does not add the geronimo-plugin.xml file to the generated car. I had submitted a working patch for the m2migration branch rev 425727 to include geronimo-plugin.xml file to the zipped archive car. The patch was not used, instead the

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-06 Thread Jason Dillon
Anita, I am aware of this issue. I have not had time to resolve it completely. More comments below... On Aug 6, 2006, at 9:31 AM, anita kulshreshtha wrote: The car-maven-plugin (trunk rev 429115) still does not add the geronimo-plugin.xml file to the generated car. I had submitted a working

Re: M2 : car-maven-plugin and geronimo-plugin.xml files

2006-08-06 Thread David Blevins
Huh, I see you and Jason working *together* on the code. I know you're not saying you don't want to work collaboratively with Jason and co, so maybe you can give some suggestions on how you both might collaborate better? Just a comment about the whole no justification concept, that's