[PATCH 2 of 4] eol: rename 'error' to 'haserror'

Yuya Nishihara yuya at tcha.org
Fri Oct 9 10:05:08 CDT 2015


On Thu, 08 Oct 2015 12:56:13 -0700, Pierre-Yves David wrote:
> # HG changeset patch
> # User Pierre-Yves David <pierre-yves.david at fb.com>
> # Date 1444110564 25200
> #      Mon Oct 05 22:49:24 2015 -0700
> # Node ID 938ed1b297ac735888da73310d2821b3d4219fa5
> # Parent  5d09ff6fb1868d1d379b80d9e06765c4af87d2ef
> eol: rename 'error' to 'haserror'
> 
> The variable 'error' conflict with the module name that we would like to import
> and use in a coming changeset.
> 
> diff --git a/hgext/eol.py b/hgext/eol.py
> --- a/hgext/eol.py
> +++ b/hgext/eol.py
> @@ -331,11 +331,11 @@ def reposetup(ui, repo):
>                      # as modified. But if we cannot lock the
>                      # repository, then we can also not make a commit,
>                      # so ignore the error.
>                      pass
>  
> -        def commitctx(self, ctx, error=False):
> +        def commitctx(self, ctx, haserror=False):

Perhaps localrepo.commitctx and other wrappers should be updated as well.


More information about the Mercurial-devel mailing list