mvexel / overpass-api-python-wrapper

Python bindings for the OpenStreetMap Overpass API
Apache License 2.0
367 stars 90 forks source link

API(debug=True) is not python3 compatible #53

Closed sandrotosi closed 6 years ago

sandrotosi commented 8 years ago
In [39]: api = overpass.API(timeout=3600, debug=True)
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-39-67bc1e87daf5> in <module>()
----> 1 api = overpass.API(timeout=3600, debug=True)

/usr/lib/python3/dist-packages/overpass/api.py in __init__(self, *args, **kwargs)
     26 
     27         if self.debug:
---> 28             import httplib
     29             import logging
     30             httplib.HTTPConnection.debuglevel = 1

ImportError: No module named 'httplib'

thats because httplib has been renamed to http.client in py3k, so the usual try/import/except/import dance is needed here

tbolender commented 6 years ago

Was fixed with 3ea6db13980c4ee4c2a4a8b9d9ff751bc8c282a7.

sandrotosi commented 6 years ago

thanks! it would be great if you could release a new version with that fix, so debian can pick that up

tbolender commented 6 years ago

Any plans for this @mvexel? I was no active for a long time, so I do not know how much has changed since the last version.

mvexel commented 6 years ago

Yes, a new release is due. I will take care of it shortly, but I want to go over open tickets to see if there's any that can be easily addressed before we do. Can you help out?

On Thu, Sep 28, 2017 at 6:58 AM, Tim Bolender notifications@github.com wrote:

Any plans for this @mvexel https://github.com/mvexel? I was no active for a long time, so I do not how much has changed since the last version.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mvexel/overpass-api-python-wrapper/issues/53#issuecomment-332828107, or mute the thread https://github.com/notifications/unsubscribe-auth/AAHWhAkmsJ-vXLCraP_EtuwWyZWAxYeYks5sm5fagaJpZM4IU5uw .

tbolender commented 6 years ago

I will do so next week (if that is fine with you), but I probably will not be able to implement much.