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

Jython 2.2.1 support #80

Merged
merged 1 commit into from
Apr 12, 2012
Merged

Jython 2.2.1 support #80

merged 1 commit into from
Apr 12, 2012

Conversation

clach04
Copy link
Contributor

@clach04 clach04 commented Apr 9, 2012

Jython 2.2.1 support added (I've not tested CPython 2.2)

@@ -1,6 +1,7 @@
#!/usr/bin/env python
# Copyright (c) 2007-2008 ActiveState Corp.
# License: MIT (http://www.opensource.org/licenses/mit-license.php)
from __future__ import generators
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this part necessary? Or just the property change below?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes it is needed for pre py2.3, see commit comment on clach04@0290a1e

@trentm
Copy link
Owner

trentm commented Apr 12, 2012

Frankly I'm surprised that markdown2 works with Python 2.2 with only that change. I only test back to Python 2.4 these days.

trentm added a commit that referenced this pull request Apr 12, 2012
@trentm trentm merged commit 4878301 into trentm:master Apr 12, 2012
trentm added a commit that referenced this pull request Apr 12, 2012
@trentm
Copy link
Owner

trentm commented Apr 12, 2012

Pulled and included in a markdown2 1.3.1 release. Thanks!

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

Successfully merging this pull request may close these issues.

2 participants