<p dir="ltr">I'm pretty sure I had a change for py 3 or pypy or something which changed these lines because one wasn't available, so I'm glad to see the import indented, although the inline comment explanation isn't wonderful.</p>
<div class="gmail_quote">On May 14, 2016 12:34 AM, "Yuya Nishihara" <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Yuya Nishihara <<a href="mailto:yuya@tcha.org">yuya@tcha.org</a>><br>
# Date 1463204025 -32400<br>
#      Sat May 14 14:33:45 2016 +0900<br>
# Node ID 4bbd966a9ebc63eedbc0c576cab9e8f2e851c8c8<br>
# Parent  bf2b2d35f363c27372de2c5ebd3eef4cdbc9f304<br>
py3: make contrib/import-checker.py get along with itself<br>
<br>
Indent these imports to disable the rule of "not lexically sorted."<br>
<br>
diff --git a/contrib/import-checker.py b/contrib/import-checker.py<br>
--- a/contrib/import-checker.py<br>
+++ b/contrib/import-checker.py<br>
@@ -11,8 +11,9 @@ import sys<br>
 # Import a minimal set of stdlib modules needed for list_stdlib_modules()<br>
 # to work when run from a virtualenv.  The modules were chosen empirically<br>
 # so that the return value matches the return value without virtualenv.<br>
-import BaseHTTPServer<br>
-import zlib<br>
+if True: # disable lexical sorting checks<br>
+    import BaseHTTPServer<br>
+    import zlib<br>
<br>
 # Whitelist of modules that symbols can be directly imported from.<br>
 allowsymbolimports = (<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@mercurial-scm.org">Mercurial-devel@mercurial-scm.org</a><br>
<a href="https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div>