PiSupply / PaPiRus

Resources for PaPiRus ePaper eInk displays
https://www.pi-supply.com/product/papirus-epaper-eink-screen-hat-for-raspberry-pi/
Other
347 stars 84 forks source link

papirus-write complained about missing parentheses #67

Closed CleoQc closed 7 years ago

CleoQc commented 7 years ago

It wouldn't run without this change, tested on a brand new Raspbian card

francesco-vannini commented 7 years ago

Hi Cleo, isn't it because you are using Python 3 that you get that?

CleoQc commented 7 years ago

Hi @francesco-vannini, fancy meeting you here :) Nope I'm using Python 2. In fact, it's on a fresh Jessie (from Nov 25th), so SD card is not dirty. When I type python, I do get python 2. When I run python papirus-write, it complains and crashes When I run the one in /usr/local/bin on its own, it complains and crashes. Looks like you're pulling from future import print_function from somewhere but I didn't investigate that.

shawaj commented 7 years ago

Strange - never seen that error before. How odd.

I have an old version sat on my desk with software already installed - will test with that.

@francesco-vannini have you tested with a new Raspbian as well?

If it works on both fine, then no reason to not add the ()

CleoQc commented 7 years ago

Also those files already use the parentheses for the print statements as this link shows: https://github.com/PiSupply/PaPiRus/search?utf8=%E2%9C%93&q=print

So right now it's inconsistent the way it's currently done.