search

padelt / temper-python

libusb/PyUSB-based driver to read TEMPer USB HID devices (USB ID 0c45:7401) and serve as a NetSNMP passpersist module
Other
183 stars 78 forks source link

syntax error on debian buster #103

Closed ghost closed 3 years ago

ghost commented 3 years ago

root@pfv-vmsrv-03:~# temper-poll Traceback (most recent call last): File "/usr/local/bin/temper-poll", line 11, in load_entry_point('temperusb==1.5.3', 'console_scripts', 'temper-poll')() File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 489, in load_entry_point return get_distribution(dist).load_entry_point(group, name) File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 2793, in load_entry_point return ep.load() File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 2411, in load return self.resolve() File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 2417, in resolve module = import(self.module_name, fromlist=['name'], level=0) File "build/bdist.linux-x86_64/egg/temperusb/init.py", line 1, in File "build/bdist.linux-x86_64/egg/temperusb/temper.py", line 17, in File "/usr/local/lib/python2.7/dist-packages/temperusb-1.5.3-py2.7.egg/temperusb/device_library.py", line 21 temp_sens_offsets: list, ^ SyntaxError: invalid syntax

ghost commented 3 years ago

This is from a git clone about 10 minutes ago or so

padelt commented 3 years ago

The latest changes introduced syntax that is valid from Python 3.6 but not earlier versions. You seem to be running Python 2.7. A workaround would be switching to Python 3.6 if you can.

rdica commented 3 years ago

The latest changes introduced syntax that is valid from Python 3.6 but not earlier versions. You seem to be running Python 2.7. A workaround would be switching to Python 3.6 if you can.

Might want to update README, it says:

Compatibility with Python versions

This should work on both Python 2 and 3. It was tested with Python 2.7.3 and 3.2.3. The snmp_passpersist mode is Python 2 only because the upstream package is not ready yet.

:smiley:

padelt commented 3 years ago

You are right, that’s inconsistent and we‘ll try to decide upon a solution in #104 . Maybe you could comment there if Python 3.6 is an option for you. I‘d like to gather some data what versions people actually use.

ghost commented 3 years ago

Oh yeah I can switch. On Ubuntu it’s fine , just Debian buster seems to utilize 2.7 by default (this is all referring to stock system python )

——- Charles N Wyble Founder TSYS Group Email: charles@turnsys.com Phone: 818 280 7059 Matrix: @Charlesnw:turnsys.com

From: Philipp notifications@github.com Sent: Friday, February 26, 2021 1:22:39 PM To: padelt/temper-python temper-python@noreply.github.com Cc: Charles Wyble charles@turnsys.com; Author author@noreply.github.com Subject: Re: [padelt/temper-python] syntax error on debian buster (#103)

You are right, that’s inconsistent and we‘ll try to decide upon a solution in #104https://github.com/padelt/temper-python/issues/104 . Maybe you could comment there if Python 3.6 is an option for you. I‘d like to gather some data what versions people actually use.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/padelt/temper-python/issues/103#issuecomment-786845520, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAQSDGPJ7ZZPVRI5TSHY3ZLTA7YH7ANCNFSM4YIZSGLA.

davet2001 commented 3 years ago

On many distros the default python is 2.7. But python3 runs a python 3.x version. Try: which python which python3 To see what happens.

padelt commented 3 years ago

I guess the situation here is clear, so I'll close.