Re: patch for review

2012-07-03 Thread Milos Kleint
done https://github.com/mkleint/maven-3/commit/2ca8e13135e34f5df7cde0a86e37b533de3be676 Milos On Mon, Jul 2, 2012 at 12:19 AM, Olivier Lamy ol...@apache.org wrote: 2012/7/1 Milos Kleint mkle...@gmail.com: On Sun, Jul 1, 2012 at 9:30 AM, Olivier Lamy ol...@apache.org wrote: 2012/6/29 Milos

Re: patch for review

2012-07-03 Thread Olivier Lamy
sounds good (at least for me :-) ). 2012/7/3 Milos Kleint mkle...@gmail.com: done https://github.com/mkleint/maven-3/commit/2ca8e13135e34f5df7cde0a86e37b533de3be676 Milos On Mon, Jul 2, 2012 at 12:19 AM, Olivier Lamy ol...@apache.org wrote: 2012/7/1 Milos Kleint mkle...@gmail.com: On Sun,

Re: patch for review

2012-07-03 Thread Milos Kleint
Can I proceed in usual github way and request a pull? or I just create a diff file and apply to the svn source base myself? Milos On Tue, Jul 3, 2012 at 5:00 PM, Olivier Lamy ol...@apache.org wrote: sounds good (at least for me :-) ). 2012/7/3 Milos Kleint mkle...@gmail.com: done

Re: patch for review

2012-07-03 Thread Olivier Lamy
2012/7/3 Milos Kleint mkle...@gmail.com: Can I proceed in usual github way and request a pull? or I just create a diff file and apply to the svn source base myself? As you prefer. Perso I use: git svn dcommit. Did you setup your local clone to use git svn ? Milos On Tue, Jul 3, 2012 at 5:00

Re: patch for review

2012-07-01 Thread Olivier Lamy
2012/6/29 Milos Kleint mkle...@gmail.com: I forgot to mention in previous reply that one important constraint is that Every single addition needs to fill out the Version value. The default maven processing makes no use of it and proceeds as before. Only in the IDE's subclass we will use it to

Re: patch for review

2012-07-01 Thread Milos Kleint
On Sun, Jul 1, 2012 at 9:30 AM, Olivier Lamy ol...@apache.org wrote: 2012/6/29 Milos Kleint mkle...@gmail.com: I forgot to mention in previous reply that one important constraint is that Every single addition needs to fill out the Version value. The default maven processing makes no use of it

Re: patch for review

2012-07-01 Thread Olivier Lamy
2012/7/1 Milos Kleint mkle...@gmail.com: On Sun, Jul 1, 2012 at 9:30 AM, Olivier Lamy ol...@apache.org wrote: 2012/6/29 Milos Kleint mkle...@gmail.com: I forgot to mention in previous reply that one important constraint is that Every single addition needs to fill out the Version value. The

patch for review

2012-06-29 Thread Milos Kleint
hello, I've prepared a patch for MavenModelBuilder and related code that hopefully will improve the performance of NetBeans integration/embedding. The basic idea is to have all validation problems collected but only fail to build the Mavenproject instance when serious base problems are found

Re: patch for review

2012-06-29 Thread Olivier Lamy
Hi, The main issue I see is non backward comp for tools implementing their own ModelProblemCollector. I don't have issue to change signature but for future enhancement if needed here, I would prefer to see something more easy to change and don't break again backward comp in the future. So instead

Re: patch for review

2012-06-29 Thread Milos Kleint
Is ModelProblemCollector intended for use outside of maven codebase? MavenModelBuilder is hardcoding reference on DefaultMPC and there's a few other implementations in tests or compat module only.. Milos On Fri, Jun 29, 2012 at 11:49 AM, Olivier Lamy ol...@apache.org wrote: Hi, The main issue

Re: patch for review

2012-06-29 Thread Olivier Lamy
Agree it's hard to inject that. So that reduce possible backward comp issues. BTW what about using this bean/data structure rather than adding new parameters ? 2012/6/29 Milos Kleint mkle...@gmail.com: Is ModelProblemCollector intended for use outside of maven codebase? MavenModelBuilder is

Re: patch for review

2012-06-29 Thread Milos Kleint
I forgot to mention in previous reply that one important constraint is that Every single addition needs to fill out the Version value. The default maven processing makes no use of it and proceeds as before. Only in the IDE's subclass we will use it to throw exception or not. If a request or

surefire patch needs review

2006-05-17 Thread Mike Perham
I have fixed http://jira.codehaus.org/browse/MSUREFIRE-113 but don't have commit privileges. This is a major regression versus 2.1.3 and for those of us who run Junit tests via suites a blocker. - To unsubscribe, e-mail: [EMAIL

Re: surefire patch needs review

2006-05-17 Thread Brett Porter
It looks fine, but incomplete. I'd forgotten about this better base class, and also referenced TestCase in the TestNG provider. If you can update that as well, then it should be fine. I've given you commit rights on surefire. Cheers, Brett Mike Perham wrote: I have fixed

RE: surefire patch needs review

2006-05-17 Thread Mike Perham
: surefire patch needs review It looks fine, but incomplete. I'd forgotten about this better base class, and also referenced TestCase in the TestNG provider. If you can update that as well, then it should be fine. I've given you commit rights on surefire. Cheers, Brett Mike Perham wrote: I have fixed

Re: surefire patch needs review

2006-05-17 Thread Brett Porter
( clazz ) ) { ... } -Original Message- From: Brett Porter [mailto:[EMAIL PROTECTED] Sent: Wednesday, May 17, 2006 12:32 PM To: Maven Developers List Subject: Re: surefire patch needs review It looks fine, but incomplete. I'd forgotten about this better base class, and also referenced

Re: surefire patch needs review

2006-05-17 Thread Jesse Kuhnert
Developers List Subject: Re: surefire patch needs review It looks fine, but incomplete. I'd forgotten about this better base class, and also referenced TestCase in the TestNG provider. If you can update that as well, then it should be fine. I've given you commit rights on surefire. Cheers, Brett

RE: surefire patch needs review

2006-05-17 Thread Mike Perham
- From: Jesse Kuhnert [mailto:[EMAIL PROTECTED] Sent: Wednesday, May 17, 2006 1:21 PM To: Maven Developers List Subject: Re: surefire patch needs review To be fair, there was definitely more than a ~little~ bit of pressure being put on Brett at the time...This might have been something I had in one

Re: surefire patch needs review

2006-05-17 Thread Jesse Kuhnert
, 2006 1:21 PM To: Maven Developers List Subject: Re: surefire patch needs review To be fair, there was definitely more than a ~little~ bit of pressure being put on Brett at the time...This might have been something I had in one of the original patches submitted as well I don't think extending

Re: surefire patch needs review

2006-05-17 Thread Brett Porter
Mike Perham wrote: Sorry, I wasn't trying to denigrate anyone, esp Brett. I think we've all written dodgy code in our day. I just got a chuckle out of the particular word choice. Certainly no offence taken. I fully admit to its dodginess (it was an improvement over not working at all,

Re: surefire patch needs review

2006-05-17 Thread Brett Porter
Jesse Kuhnert wrote: P.S. Not using TestNGClassFinder.isTestNGClass is the exact reason (probably?..) why someone's base class with no test methods and only @Configuration methods might not be found. Unlikely, as it is going to assume it's a TestNG class as long as it doesn't extend TestCase,

Re: surefire patch needs review

2006-05-17 Thread Jesse Kuhnert
Hmmm...I'm not remembering so well these days either. (I'm also a bit disconnected from what the current surefire code looks like) When I double checked the isTestNGClass logic though it confirmed that the basic function is for it to look for ~any~ annotations on the class in question. (whether

Re: surefire patch needs review

2006-05-17 Thread Brett Porter
Mike Perham wrote: Brett, I'm still getting a 403 Forbidden when I try to check in. Is there a cron job I need to wait for? There shouldn't be a delay. Have you got it checked out as https? - Brett - To unsubscribe,

RE: surefire patch needs review

2006-05-17 Thread Mike Perham
Well, there's a delay cause now it works. :-) Thanks. -Original Message- From: Brett Porter [mailto:[EMAIL PROTECTED] Sent: Wednesday, May 17, 2006 2:28 PM To: Maven Developers List Subject: Re: surefire patch needs review Mike Perham wrote: Brett, I'm still getting a 403 Forbidden