----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35497/#review88066 -----------------------------------------------------------
proton-c/bindings/python/setuputils/misc.py (line 43) <https://reviews.apache.org/r/35497/#comment140479> this works, but isn't errno a top level builtin? os.errno doesn't appear in the python docs can't the patch simply import errno instead? - Kenneth Giusti On June 16, 2015, 7:46 a.m., Flavio Percoco wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35497/ > ----------------------------------------------------------- > > (Updated June 16, 2015, 7:46 a.m.) > > > Review request for qpid and Kenneth Giusti. > > > Repository: qpid-proton-git > > > Description > ------- > > PROTON-895: errno is part of `os` > > > Diffs > ----- > > proton-c/bindings/python/setuputils/misc.py > cf6b97f041baf02371cd6847be6506ea81a5e412 > > Diff: https://reviews.apache.org/r/35497/diff/ > > > Testing > ------- > > > Thanks, > > Flavio Percoco > >
