Make B.E.N.J.I. compatible with Linux

the-ethan-hunt / B.E.N.J.I.

B.E.N.J.I.- The Impossible Missions Force's digital assistant

MIT License

91 stars 94 forks source link

Make B.E.N.J.I. compatible with Linux #4

Closed the-ethan-hunt closed 6 years ago

the-ethan-hunt commented 7 years ago

Currently, the digital assistant is compatible only with Windows.

kaushal-py commented 6 years ago

I would love to work on this!

kaushal-py commented 6 years ago

Should a new file be created for the linux version? Because the imports and functions could vary! Eg. benji-linux.py

Or do we continue on the same file and make it compatible for linux.

the-ethan-hunt commented 6 years ago

Making a new file is better. You can name it as linux/benji.py and mention about this in the readme

kaushal-py commented 6 years ago

Made a PR. I used pyttsx which is supposed to be cross-platform. So the file I wrote is platform-independent. (is supposed run on a mac too, haven't tested it). I feel keeping the Linux version in a separate file may be a problem, as the features added to the main file are not reflected here.

the-ethan-hunt commented 6 years ago

Closing this issue for now