search

chrisjbillington / zprocess

A collection of utilities for multiprocessing using zeromq.
BSD 2-Clause "Simplified" License
2 stars 5 forks source link

Bug in Python3 #3

Closed chrisjbillington closed 7 years ago

chrisjbillington commented 7 years ago

Original report by Jan Werkmann (Bitbucket: 557058:a70cc9cf-684e-4849-a61a-9ade4d7218b5, GitHub: PhyNerd).

In line 455 there is as python 3 incompatibility as under python 3 unicode is not defined.

chrisjbillington commented 7 years ago

Original comment by Jan Werkmann (Bitbucket: 557058:a70cc9cf-684e-4849-a61a-9ade4d7218b5, GitHub: PhyNerd).

maybe try this:

#!python

if isinstance(s, six.string_types):

That fixed it for me.

chrisjbillington commented 7 years ago

Original comment by Chris Billington (Bitbucket: 557058:cbf1bc43-1dc2-477b-9e25-1a8f40fd7ee3, GitHub: cbillington).

Fix issue #3, string regression in Python 3.

Patch version bump for PyPI release

chrisjbillington commented 7 years ago

Original comment by Chris Billington (Bitbucket: 557058:cbf1bc43-1dc2-477b-9e25-1a8f40fd7ee3, GitHub: cbillington).

Ah yikes. Sorry about that. I thought I had aliased unicode = str in Python 3, but actually I had aliased str = unicode in Python 2 (which makes sense - better practice to make old Python look like new Python instead of making new look like old).