search

Xuxe / Sinusbot-File-Importer

A CLI Tool to Import audio files via the HTTP API. Written in Python.
GNU General Public License v2.0
6 stars 7 forks source link

Allow for specification of a folder using it's UUID and updated Readme to include new commands #6

Open Beachman4 opened 7 years ago

Beachman4 commented 7 years ago

I also changed the print statement to use print() instead of how it was. This may have been an issue for me, but the print statement was not working for me.

Xuxe commented 7 years ago

Looks good so far for #5 . The difference between print and print as function is python2 and python3. For python2 it is print for python3 print() .

Personally i only tested it on python2, so far i unterstand it is working for you on python3 with the changes?

I'm currently not sure if we should hold python2 support. Because Debian 8 there is python2 still default AFAIK, it may causes confusion for the user.

If we go to python3 we should alse change the interpreter header to python3 for systems where python2 is still default.

irgendwr commented 7 years ago

Using the print function is fine for both versions (and makes sense for compatibility reasons) BUT you have to import the new print function by adding

from __future__ import print_function

as the very first import to make it work for python 2