search

matthewwithanm / pilkit

Utilities and processors built for, and on top of PIL
BSD 3-Clause "New" or "Revised" License
196 stars 54 forks source link

Error: string argument expected, got 'str' #5

Closed m0sth8 closed 11 years ago

m0sth8 commented 11 years ago

io.StringIO raise exception "TypeError: string argument expected, got 'str'" when pilkit saved Pillow img

matthewwithanm commented 11 years ago

This is probably related to the Python 3 stuff I did last night. 1.0 should be okay if you're using 2.x. Can you share the whole stack trace?

m0sth8 commented 11 years ago

Yes, pilkit 1.0 with cStringIO works for me.

Stack trace: http://bpaste.net/show/118162/

jacobh commented 11 years ago

I just got this error too, traceback: https://gist.github.com/hzy/6102063

matthewwithanm commented 11 years ago

Really sorry about that guys. Can you try the io-stringio-fix branch? If that's good, I'll merge it and publish it to PyPI as 1.1.1.

jacobh commented 11 years ago

after $ pip install git+https://github.com/matthewwithanm/pilkit.git@75e324fb3f492a7d3a34c7273de765e5fd0819fb (latest commit on that branch) all my tests went green!

matthewwithanm commented 11 years ago

@hzy Awesome, thanks. It's up on PyPI as 1.1.1. Let me know if you have any more issues!

jacobh commented 11 years ago

1.1.1 works a treat! Thankyou very much!

matthewwithanm commented 11 years ago

Np. Thanks for the gist and quick testing!

m0sth8 commented 11 years ago

Good work. Thank you =)

matthewwithanm commented 11 years ago

Thanks (: