Log file Viewer
Help
|
Karma
| Search:
#pypa logs for Saturday the 4th of October, 2014
(
Back to #pypa overview
) (
Back to channel listing
) (
Animate logs
)
[
17:16:00
] <
buck11
> dstufft: what command were you running when you saw the pylint-distutils problem?
[
17:16:09
] <
buck11
> tox -e lint should be clean
[
17:17:15
] <
dstufft
> buck11: I was invoking pylint directly, I wasn't running the PR itself, I just did pylint pip/baseparser.py
[
17:17:37
] <
dstufft
> E: 8, 0: No name 'util' in module 'distutils' (no-name-in-module)
[
17:17:37
] <
dstufft
> F: 8, 0: Unable to import 'distutils.util' (import-error)
[
17:18:04
] <
buck11
> dstufft: the issue is that virtualenv does black magic in disutils namespace, that can't be statically analyzed
[
17:18:45
] <
buck11
> its simple to configure it to ignore distutils
[
17:22:05
] <
buck11
> same with six.moves =/
[
17:25:04
] <
dstufft
> buck11: how?
[
17:25:33
] <
buck11
> making a patch one sec
[
17:36:58
] <
buck11
> dstufft: i find a lot of methods that could be functions
[
17:37:02
] <
buck11
> is this a design intent or no?
[
17:37:42
] <
dstufft
> pip wasn't really "designed", but I find @staticmethod to be pointless
[
17:46:07
] <
dstufft
> buck11: I tried to add distutils to ignored-modules=distutils which removed the no-name-in-module check
[
17:46:22
] <
dstufft
> but the F: 8, 0: Unable to import 'distutils.util' (import-error) is still there
[
17:46:29
] <
buck11
> yea i think there's a usability bug there
[
17:46:36
] <
buck11
> i've got a workaround but it's ugly =/
[
17:46:49
] <
buck11
> i think that's a bug that i can get patched
[
17:46:58
] <
buck11
> import-error on ignored module
[
17:51:24
] <
buck11
> dstufft: https://github.com/bukzor/pip/commit/523a8f327bbc82ecf4044062b0b3ecbccad941a2
[
18:15:31
] <
buck11
> dstufft: https://bitbucket.org/logilab/pylint/issue/223/ignored-modules-should-turn-no-name-in#comment-12774331