<div dir="ltr">On Fri, Jul 3, 2015 at 8:00 AM, Augie Fackler <span dir="ltr"><<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On Jul 3, 2015, at 9:48 AM, Yuya Nishihara <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>> wrote:<br>
><br>
> # HG changeset patch<br>
> # User Yuya Nishihara <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>><br>
> # Date 1435929436 -32400<br>
> #      Fri Jul 03 22:17:16 2015 +0900<br>
> # Node ID 506956b94ca90eec47dfd7d9746e129bc3ccf884<br>
> # Parent  d3d2ba265e17af1d9700e3e99de4d012b1b31ab8<br>
> import-checker: exclude dist-packages tree from stdlib as well<br>
><br>
> On Debian, packaged modules are installed into dist-packages, not to<br>
> site-packages. Because of this, if we have mercurial installed, the test<br>
> failed as 'mercurial.node' was listed in stdlib_modules:<br>
<br>
</span>Hmm. I think this means that the import-checker is running with too many things on its sys.path - does that make sense to you?<br>
<br>
I’m not sure what we should do about it, but it definitely seems a bit undesirable to pick things up from diet-packages when we’re trying to analyze a working tree. Thoughts? <br></blockquote><div><br></div><div>Having recently touched this code, I believe the only place where "is stdlib module" comes into play is when checking for mixed imports between mercurial modules and stdlib modules on the same line. Instead of worrying about sys.path, site-packages, etc, how about simply excluding "mercurial.*" and "hgext.*" modules from the stdlib modules list? <br></div></div><br></div></div>