----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67427/#review204243 -----------------------------------------------------------
contrib/clients/python/lens/client/auth.py Lines 17 (patched) <https://reviews.apache.org/r/67427/#comment286647> Can you add "future" module to setup.py install_requires as well? - Barun Kumar On June 3, 2018, 2:51 p.m., Michael McCarthy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67427/ > ----------------------------------------------------------- > > (Updated June 3, 2018, 2:51 p.m.) > > > Review request for lens. > > > Repository: lens > > > Description > ------- > > Fix of urlparse import in auth.py to make compatibiliy with Python 2 and 3. > > > Diffs > ----- > > contrib/clients/python/lens/client/auth.py fccc75c9 > > > Diff: https://reviews.apache.org/r/67427/diff/1/ > > > Testing > ------- > > Tested with Python 2.7.12 and Python 3.5.2. > > > Thanks, > > Michael McCarthy > >