[PATCH 3 of 3 V2] test-pyflakes: detect undefined name error
Matt Mackall
mpm at selenic.com
Thu May 8 11:34:55 CDT 2014
On Fri, 2014-05-09 at 01:25 +0900, FUJIWARA Katsunori wrote:
> At Fri, 02 May 2014 19:14:37 +0900,
> Yuya Nishihara wrote:
> >
> > # HG changeset patch
> > # User Yuya Nishihara <yuya at tcha.org>
> > # Date 1398743641 -32400
> > # Tue Apr 29 12:54:01 2014 +0900
> > # Node ID b7b37547eec4246671f7ae28843a6931813b810d
> > # Parent de513875f008f333f320aca0b236b6e8af54cdbf
> > test-pyflakes: detect undefined name error
> >
> > It should be able to catch the following mistakes at 2606e7f227f6:
> >
> > mercurial/exchange.py:590: undefined name 'UnknownPartError'
> > mercurial/match.py:346: undefined name 'pat'
> > mercurial/win32.py:365: undefined name '_ERROR_NO_MORE_FILES'
> > tests/killdaemons.py:46: undefined name 'check'
> >
> > diff --git a/tests/filterpyflakes.py b/tests/filterpyflakes.py
> > --- a/tests/filterpyflakes.py
> > +++ b/tests/filterpyflakes.py
> > @@ -32,6 +32,7 @@ for line in sys.stdin:
> > r"imported but unused",
> > r"local variable '.*' is assigned to but never used",
> > r"unable to detect undefined names",
> > + r"undefined name '.*'",
> > ]
> > for msgtype, pat in enumerate(pats):
> > if re.search(pat, line):
>
> According to my bisect-ing, this change (0768cda8b579 in master repo)
> seems to break "test-check-pyflakes.t" as below:
>
> ====================
> @@ -17,5 +17,7 @@
> tests/hghave.py:*: 'pygments' imported but unused (glob)
> tests/hghave.py:*: 'ssl' imported but unused (glob)
> contrib/win32/hgwebdir_wsgi.py:93: 'from isapi.install import *' used; unable to detect undefined names (glob)
> + mercurial/util.py:116: undefined name 'memoryview'
> + mercurial/windows.py:34: undefined name 'WindowsError'
>
> #endif
> ====================
>
> Is this failure specific to my environment (python 2.6.6 on debian
> 2.6.32), or just expected failure ?
Probably a version issue, possibly with pyflakes.
--
Mathematics is the supreme nostalgia of our time.
More information about the Mercurial-devel
mailing list