Mercurial > public > mercurial-scm > hg
diff mercurial/subrepo.py @ 16529:3d5d204a08c7 stable
subrepo/svn: abort on commit with missing file (issue3029)
Previous code was printing a traceback because it expected some error output
from svn. But sometimes our definition of "changed" differs with the subversion
one. For instance, subversion ignores missing files when committing. And when
there are only missing files, svn commit will be a successful no-op with no
output. Still, we should stick to our definition including missing files in
changes as doing otherwise could cause surprising behaviour for the user.
| author | Patrick Mezard <patrick@mezard.eu> |
|---|---|
| date | Thu, 26 Apr 2012 11:55:07 +0200 |
| parents | 17a1f7690b49 |
| children | e37199a1f9d4 |
line wrap: on
line diff
--- a/mercurial/subrepo.py Thu Apr 26 21:32:48 2012 +0900 +++ b/mercurial/subrepo.py Thu Apr 26 11:55:07 2012 +0200 @@ -726,6 +726,12 @@ self._ui.status(commitinfo) newrev = re.search('Committed revision ([0-9]+).', commitinfo) if not newrev: + if not commitinfo.strip(): + # Sometimes, our definition of "changed" differs from + # svn one. For instance, svn ignores missing files + # when committing. If there are only missing files, no + # commit is made, no output and no error code. + raise util.Abort(_('failed to commit svn changes')) raise util.Abort(commitinfo.splitlines()[-1]) newrev = newrev.groups()[0] self._ui.status(self._svncommand(['update', '-r', newrev])[0])