Re: [SVNMERGE][PATCH] svnmerge rollback

Madan U Sreenivasan wrote:

>>>> + # make sure theres some revision to
>> rollback
>>>> + if len(revs) == 0:
>>>> + warn("Nothing to rollback in
>> revision range r%s" %
>> opts["revision"])
>>>> + return
>> I'd rather this be a report for now, and
>> then have a separate commit pass
>> which introduces warn() and uses it
>> consistenly across the program (once the
>> semantic is clear).
>
> I will revert this warn() function and use report() for now.... but I
> would like to know what you mean by 'semantic', so that I could
> provide that patch sooner, than later.

We should first agree on a strict definition of warn(), so that it becomes
obvious when to use warn() rather than report(). Once we have this
agreement, the mechanical patch to change some of the report() calls into
warn() is obviously ok.

-- 
Giovanni Bajo
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org