search

Esri / ArcREST

python package for REST API (AGS, AGOL, webmap JSON, etc..)
Apache License 2.0
192 stars 155 forks source link

Discussion: Adding Six Support to Package #120

Closed achapkowski closed 8 years ago

achapkowski commented 8 years ago

Should ArcREST incorporate Six?

https://pythonhosted.org/six/

This would make creating a 2/3 version easier. The biggest impact would be the web._base.py where the web operation class is stored.

I think the proposed changes in code: -- web._base.py -- setup.py to add in required packages --or-- package six into ArcREST -- securityHandlers - make sure the ntlm works -- make sure all reference to urllib2, urlparse, urlunparse are reference properly.

There would also need verification of ArcPy functions as well, but I think that can come later.

What does the community think?

achapkowski commented 8 years ago

Began developing v3.5.0 on main branch. Six support added.