Add workaround where SVN.Revert() deletes the checkout.

Add workaround where SVN.Revert() deletes the checkout.

This would cause a few exceptions on the CQ until it realizes the checkout is
not present anymore.

R=petermayo@chromium.org
BUG=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=173916

[Peter Mayo, 2012-12-19 14:05:19]

Reading this makes be believe that scm is a poor abstraction, and should really
be called svn_utils or some such, since the funatonality to accomplish scm
objectives (or even vcs ones) is spread through it and above it.

This change LGTM, incremenetally.

https://codereview.chromium.org/11649011/diff/1/checkout.py
File checkout.py (right):

https://codereview.chromium.org/11649011/diff/1/checkout.py#newcode483
checkout.py:483: # This call may as well remove the whole checkout.
I think you mean "This call may remove the whole checkout as well."

or even "This may remove any part (or all) of the checkout."

[M-A Ruel, 2012-12-19 14:18:24]

On 2012/12/19 14:05:19, Peter Mayo wrote:
> Reading this makes be believe that scm is a poor abstraction, and should
really
> be called svn_utils or some such, since the funatonality to accomplish scm
> objectives (or even vcs ones) is spread through it and above it.

I'm not arguing that. :) but things likely won't change at that point, at least
not by me.

https://codereview.chromium.org/11649011/diff/1/checkout.py
File checkout.py (right):

https://codereview.chromium.org/11649011/diff/1/checkout.py#newcode483
checkout.py:483: # This call may as well remove the whole checkout.
On 2012/12/19 14:05:19, Peter Mayo wrote:
> I think you mean "This call may remove the whole checkout as well."
> 
> or even "This may remove any part (or all) of the checkout."

done.

[commit-bot: I haz the power, 2012-12-19 14:18:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/11649011/4001

[commit-bot: I haz the power, 2012-12-19 14:21:15]

Change committed as 173916