search

LightTable / Python

Python language plugin for Light Table
MIT License
98 stars 51 forks source link

Exception when running python 3 on windows 7 #54

Closed Vitruvia closed 7 years ago

Vitruvia commented 7 years ago

I just installed LightTable and used it fine with JavaScript, but when I tried using python 3 I got this exception:

Traceback (most recent call last): File "P:\LightTable\resources\app\plugins\Python\py-src\ltmain.py", line 193, in handleEval code= compile(ensureUtf(code), ensureUtf(data[2]["name"]), 'eval') File "P:\LightTable\resources\app\plugins\Python\py-src\ltmain.py", line 50, in ensureUtf if type(s) == unicode: NameError: name 'unicode' is not defined

During handling of the above exception, another exception occurred:

Traceback (most recent call last): File "P:\LightTable\resources\app\plugins\Python\py-src\ltmain.py", line 197, in handleEval code= compile(ensureUtf(code), ensureUtf(data[2]["name"]), 'exec') File "P:\LightTable\resources\app\plugins\Python\py-src\ltmain.py", line 50, in ensureUtf if type(s) == unicode: NameError: name 'unicode' is not defined

What I did was assign a = 3 just to see if there were no problems, and I got this after running with ctr+shift+enter.

sbauer322 commented 7 years ago

@Vitruvia could you pull down the work from PR https://github.com/LightTable/Python/pull/47 and give it a try? It might address the problem.

Unfortunately, the Python plugin is in need of a maintainer and has a backlog of PRs and issues to address. The position is open to pretty much anyone who is willing and able to work on it.

Vitruvia commented 7 years ago

@sbauer322 Thanks for the help!

Well... I wouldn't mind helping out but I only begun programming a month ago. Is there anything I could do at this level?

sbauer322 commented 7 years ago

Sure, there are few ways to help out at that stage!

Since you are still learning programming, the biggest help would reporting any bugs you might come across. Attempting to reproduce new issues would be beneficial also, but of lesser importance.

Another helpful thing would be testing out pull requests to make sure they work and do not introduce any new problem.

Vitruvia commented 7 years ago

Great, I'll keep reporting anything I find. I'll also check for pull requests later.

By the way, the solution you suggested works, so in case anyone is experiencing this bug just refer to PR #47.

sbauer322 commented 7 years ago

Thanks for confirming the PR works!