adriank / ObjectPath

The agile query language for semi-structured data
http://objectpath.org
MIT License
380 stars 93 forks source link

pytz dependency missing in current pypi latest version (0.6.1) #96

Closed nastasi closed 4 years ago

nastasi commented 4 years ago

This is what I can see on my console without installing pytz, probably there is a missing dependency in setup.py file, cheers.

(venv) $ objectpath '$' data/nepal_exposure.json.in
Traceback (most recent call last):
  File "/home/nastasi/venv_objectslt/lib/python3.6/site-packages/objectpath/shell.py", line 19, in <module>
    import pytz
ModuleNotFoundError: No module named 'pytz'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/nastasi/venv_objectslt/bin/objectpath", line 7, in <module>
    from objectpath.shell import main
  File "/home/nastasi/venv_objectslt/lib/python3.6/site-packages/objectpath/shell.py", line 21, in <module>
    if os.isatty(sys.stdin.fileno()) and sys.stdout.isatty():
NameError: name 'os' is not defined
(venv) $ python --version
Python 3.6.9
adriank commented 4 years ago

PR welcomed. :)

Greetings, Adrian Kalbarczyk

https://kalbarczyk.co | https://devyard.io

On Sat, May 9, 2020 at 2:30 PM Matteo Nastasi notifications@github.com wrote:

This is what I can see on my console without installing pytz, probably there is a missing dependency in setup.py file, cheers.

(venv) $ objectpath '$' data/nepal_exposure.json.in Traceback (most recent call last): File "/home/nastasi/venv_objectslt/lib/python3.6/site-packages/objectpath/shell.py", line 19, in import pytz ModuleNotFoundError: No module named 'pytz'

During handling of the above exception, another exception occurred:

Traceback (most recent call last): File "/home/nastasi/venv_objectslt/bin/objectpath", line 7, in from objectpath.shell import main File "/home/nastasi/venv_objectslt/lib/python3.6/site-packages/objectpath/shell.py", line 21, in if os.isatty(sys.stdin.fileno()) and sys.stdout.isatty(): NameError: name 'os' is not defined (venv) $ python --version Python 3.6.9

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/adriank/ObjectPath/issues/96, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABLE4QD273GQFKJT2ZR7ZDRQVEGBANCNFSM4M4ZADSQ .

nastasi commented 4 years ago

Done lazy man.

adriank commented 4 years ago

Merged hard working man.