Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SyntaxError: invalid syntax #10

Closed
juh2 opened this issue Sep 29, 2016 · 8 comments
Closed

SyntaxError: invalid syntax #10

juh2 opened this issue Sep 29, 2016 · 8 comments

Comments

@juh2
Copy link

juh2 commented Sep 29, 2016

I downloaded the release 0.1 installed the requirements via pip and run make.

make
mkdir -p example_output && \
cd example_output && \
../noteshrink.py -O -g -w -s 20 -v 30 -o notesA.pdf ../examples/notesA*.jpg
  File "../noteshrink.py", line 159
    print '  running "{}"...'.format(cmd),
                            ^
SyntaxError: invalid syntax
makefile:9: die Regel für Ziel „example_output/notesA.pdf“ scheiterte
make: *** [example_output/notesA.pdf] Fehler 1
@juh2
Copy link
Author

juh2 commented Sep 29, 2016

I tried it with Python 3 and obviously the software depends on Python 2. So perhaps turn this issue into a feature request: Make it Python 3 compliant.

@mzucker
Copy link
Owner

mzucker commented Sep 29, 2016

I believe we are now Python 3 compliant as of 0701e13 -- but I don't know how to update on PyPI -- if anyone can provide me instructions, I will increment the version number and re-upload.

@michi88
Copy link

michi88 commented Sep 29, 2016

@mzucker
Copy link
Owner

mzucker commented Sep 29, 2016

Think it's re-uploaded, not sure if my modifications to setup.py were correct; feel free to try again.

@LEXUSAPI
Copy link

Good to me ! officelens will die

@mzucker
Copy link
Owner

mzucker commented Oct 12, 2016

Thanks!
@juh2 can you confirm whether the latest PyPi fix worked for you?

@juh2
Copy link
Author

juh2 commented Oct 12, 2016

I pip installed it in my py3 env and ran it on the examples. It worked. Thanks!

@mzucker
Copy link
Owner

mzucker commented Oct 12, 2016

Great, closing this issue.

@mzucker mzucker closed this as completed Oct 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants