Not working on linux systems where python defaults to python3

[despairblue]

$ touch test
$ trash test
Command failed: Traceback (most recent call last):
  File "./trash-put", line 4, in <module>
    from trashcli.put import main
  File "/home/despairblue/node_modules/trash/vendor/trash-cli/trashcli/put.py", line 255
    except (IOError, OSError), error:
                             ^
SyntaxError: invalid syntax

[sindresorhus]

Does changing

trash/vendor/trash-put

Line 1 in dc88ffc

#!/usr/bin/env python

to #!/usr/bin/env python2 fix it?

[danielchatfield]

Should probs change the exception handler to except (IOError, OSError) as error: since that works with python 2 and 3.

Edit: I can now see that it is in a third party lib.

[sindresorhus]

@danielchatfield Yeah, but please do open a ticket about Python 3 support on the third party lib ;)

[despairblue]

@sindresorhus changing it does fix it. But I'm not sure whether this would break it on systems where there is no python2, but python and python3.

Yeah, it would be best to fix it in trash-cli directly. (andreafrancia/trash-cli#42)

[danielchatfield]

Currently python 2 is a requirement for that third party lib.

On Monday, 4 August 2014, Danny Arnold notifications@github.com wrote:

@sindresorhus https://github.com/sindresorhus changing it does fix it.
But I'm not sure whether this would break it on systems where there is no
python2, but python and python3.

Yeah, it would be best to fix it in trash-cli directly.

Reply to this email directly or view it on GitHub
#10 (comment).

For contact info visit: www.danielchatfield.com

[sindresorhus]

@despairblue According to some PEP python2 is a requirement. But I guess I could somehow check.

[despairblue]

@sindresorhus just tested it on an ubuntu 14.04 server. /usr/bin/env python2 works there.