I think it is OK to commit bug fixes, but not OK to introduce new concepts.
Just my 2cents.
Antoine
Matt Benson wrote:
> I also have a few quite minor things, so am interested in the
> community opinion here.
>
> On Jan 20, 2010, at 8:56 AM, Steve Loughran wrote:
>
>>
>> Having just checked this in, I was struck by a thought of guilt:
>> should I have done this given we are in the RC phase? I'm just
>> looking at what (minor) changes I have in my local ant build (this &
>> some better error and OOM handling in the junit result formatters).
>> I've stuck this in, but technically it's still a code change, right?
>>
>> I can move this patch to a branch instead.
>>
>> -steve
>>
>> stevel@apache.org wrote:
>>> Author: stevel
>>> Date: Wed Jan 20 14:41:43 2010
>>> New Revision: 901218
>>> URL: http://svn.apache.org/viewvc?rev=901218&view=rev
>>> Log:
>>> <scp> task didn't report build file location when a remote operation
>>> failed
>>> Bugzilla Report 48578.
>>> Modified:
>>> ant/core/trunk/WHATSNEW
>>>
>>> ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/Scp.java
>>>
>>> Modified: ant/core/trunk/WHATSNEW
>>> URL:
>>> http://svn.apache.org/viewvc/ant/core/trunk/WHATSNEW?rev=901218&r1=901217&r2=901218&view=diff
>>>
>>
>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
>> For additional commands, e-mail: dev-help@ant.apache.org
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org
|