Mercurial Mercurial > mercurial / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
statichttprepo: use URLError.reason directly
authorGregory Szorc <gregory.szorc@gmail.com>
Sat, 26 Jan 2019 10:53:10 -0800
changeset 41406 2bf689b13a17
parent 41405 9b3be572ff0c
child 41407 6e9bebb65ce7
statichttprepo: use URLError.reason directly 0b3f4be5c5bf changed str(inst) to inst.reason[0] all the way back in 2006. URLError.reason is a str and we should have taken that attribute in its entirety. I think the code was supposed to be inst.args[1] for compatibility with ancient Python versions. Python 2.7 always sets .reason, so it should be safe to use directly. Differential Revision: https://phab.mercurial-scm.org/D5715
mercurial/statichttprepo.py file | annotate | diff | comparison | revisions 
--- a/mercurial/statichttprepo.py	Sat Jan 26 10:40:37 2019 -0800
+++ b/mercurial/statichttprepo.py	Sat Jan 26 10:53:10 2019 -0800
@@ -59,7 +59,7 @@
             num = inst.code == 404 and errno.ENOENT or None
             raise IOError(num, inst)
         except urlerr.urlerror as inst:
-            raise IOError(None, inst.reason[1])
+            raise IOError(None, inst.reason)
 
         if code == 200:
             # HTTPRangeHandler does nothing if remote does not support
mercurial