search

tjvr / kurt

Python library for reading/writing MIT's Scratch file format.
https://kurt.tjvr.org
GNU General Public License v3.0
86 stars 24 forks source link

Cannot import OptionalGreedyRepeater #10

Closed alandmoore closed 11 years ago

alandmoore commented 11 years ago

I installed kurt in a virtualenv using pip. Construct 2.5.1 was automatically installed. When I run kurtc.py I get:

Traceback (most recent call last): File "/home/alanm/kurt/bin/kurtc.py", line 55, in from kurt.compiler import cmd_compile File "/home/alanm/kurt/local/lib/python2.7/site-packages/kurt/init.py", line 43, in from kurt.objtable import File "/home/alanm/kurt/local/lib/python2.7/site-packages/kurt/objtable.py", line 32, in from fixed_objects import File "/home/alanm/kurt/local/lib/python2.7/site-packages/kurt/fixed_objects.py", line 25, in from construct import Value, Switch, If, IfThenElse, OptionalGreedyRepeater ImportError: cannot import name OptionalGreedyRepeater

I can't find any reference to this class/function in the construct code.

tjvr commented 11 years ago

Argh — thanks for pointing this out!

It looks like the construct.text module was removed in 2.5. I'll have to update kurt's requirements.

For now, could you try using an earlier version of construct? Thanks again!

On Saturday, 4 May 2013 at 22:42, Alan D Moore wrote:

I installed kurt in a virtualenv using pip. Construct 2.5.1 was automatically installed. When I run kurtc.py I get: Traceback (most recent call last): File "/home/alanm/kurt/bin/kurtc.py", line 55, in
from kurt.compiler import cmd_compile File "/home/alanm/kurt/local/lib/python2.7/site-packages/kurt/init.py", line 43, in
from kurt.objtable import File "/home/alanm/kurt/local/lib/python2.7/site-packages/kurt/objtable.py", line 32, in
from fixed_objects import File "/home/alanm/kurt/local/lib/python2.7/site-packages/kurt/fixed_objects.py", line 25, in
from construct import Value, Switch, If, IfThenElse, OptionalGreedyRepeater ImportError: cannot import name OptionalGreedyRepeater
I can't find any reference to this class/function in the construct code.

— Reply to this email directly or view it on GitHub (https://github.com/blob8108/kurt/issues/10).

alandmoore commented 11 years ago

I tried installing 2.0.0, it's now telling me "FloatNumber" is not defined. What version of construct is this known to work with?

EDIT: got it working with 2.0.6.

tjvr commented 11 years ago

2.06 (the one directly before 2.5.0). On 5 May 2013 00:41, "Alan D Moore" notifications@github.com wrote:

I tried installing 2.0.0, it's now telling me "FloatNumber" is not defined. What version of construct is this known to work with?

— Reply to this email directly or view it on GitHubhttps://github.com/blob8108/kurt/issues/10#issuecomment-17443262 .

tjvr commented 11 years ago

I checked with the Construct people, and I pushed a new version (1.4.7) with an updated dependency that should fix the problem. Could you test it again for me and check it fixes the problem? Thanks!

alandmoore commented 11 years ago

Yes it seems to work correctly now. Thanks!

tjvr commented 11 years ago

Thanks again for reporting this! On 15 May 2013 17:45, "Alan D Moore" notifications@github.com wrote:

Yes it seems to work correctly now. Thanks!

— Reply to this email directly or view it on GitHubhttps://github.com/blob8108/kurt/issues/10#issuecomment-17950770 .