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
182 stars 76 forks source link

Use of Python 3.6 syntax #104

Closed padelt closed 3 years ago

padelt commented 3 years ago

103 surfaced that we have syntax in the codebase that requires Python 3.6.

At the same time, the Readme mentions compatibility with Python 2.7 and 3.2.

As Python 2 is EOL just as is Python 3.2, I am inclined to up the minimum version to 3.6.

Is there demand for keeping compatibility with earlier versions? Especially 3.5 which seems to be the default in major Linux distributions.

rdica commented 3 years ago

Works on Fedora 32:

ardy@daw3:~$ (0) python -V
Python 3.8.7

ardy@daw3:~$ (0) temper-poll 
Found 1 devices
Device #0: 24.4°C 76.0°F 
ps-jay commented 3 years ago

I would bump the minimum supported version up, for the following reasons:

padelt commented 3 years ago

That's less controversial than I thought. So let's do it - in #105.