Re: [PATCH 1 of 3] svnsubrepo: decorate dirty method with annotatesubrepoerror

2018-01-17 Thread Yuya Nishihara
On Tue, 16 Jan 2018 08:42:15 -0500, Jordi Gutiérrez Hermoso wrote:
> # HG changeset patch
> # User Jordi Gutiérrez Hermoso 
> # Date 1515949528 18000
> #  Sun Jan 14 12:05:28 2018 -0500
> # Node ID fde3cd2487e1f4d5867bf8c6e79139b6fe475779
> # Parent  58fda95a0202fc6327d1f5d9df26f7ff16538d57
> svnsubrepo: decorate dirty method with annotatesubrepoerror

Looks good. Queued, thanks.
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


[PATCH 1 of 3] svnsubrepo: decorate dirty method with annotatesubrepoerror

2018-01-16 Thread Jordi Gutiérrez Hermoso
# HG changeset patch
# User Jordi Gutiérrez Hermoso 
# Date 1515949528 18000
#  Sun Jan 14 12:05:28 2018 -0500
# Node ID fde3cd2487e1f4d5867bf8c6e79139b6fe475779
# Parent  58fda95a0202fc6327d1f5d9df26f7ff16538d57
svnsubrepo: decorate dirty method with annotatesubrepoerror

This function invokes svn commands which can error out in any number
of ways, so it's helpful to know in which subrepo this error happens.

diff --git a/mercurial/subrepo.py b/mercurial/subrepo.py
--- a/mercurial/subrepo.py
+++ b/mercurial/subrepo.py
@@ -1331,6 +1331,7 @@ class svnsubrepo(abstractsubrepo):
 return True, True, bool(missing)
 return bool(changes), False, bool(missing)
 
+@annotatesubrepoerror
 def dirty(self, ignoreupdate=False, missing=False):
 wcchanged = self._wcchanged()
 changed = wcchanged[0] or (missing and wcchanged[2])
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel