Python >= 2.6 should be required for recaptcha2 module

[Ryan J Ollos]

Noted while reviewing ​gmessage:trac-users:ibiYYaqNok0/Sz_Nt6eTAAAJ, plugins/1.0/spam-filter/tracspamfilter/captcha/recaptcha2.py@14630:14​ imports json, which is only available in Python ≥ 2.6.

setup.py​ requires json for the tracspamfilter.filters.blogspam module, and json requires Python ≥ 2.6.

It's possible we could address the issue found on the mailing list as well, with the following change (untested):

../spam-filter/setup.py

@@ -12 +12 @@
 # individuals. For the exact contribution history, see the revision
 # history and logs, available at http://projects.edgewall.com/trac/.
 
+import sys
 from setuptools import setup, find_packages
 
 PACKAGE = 'TracSpamFilter'
@@ -60 +61 @@
         'dns': ['dnspython>=1.3.5'],
         'spambayes': ['spambayes'],
         'pillow': ['pillow'],
-        'json': ['python>=2.6'],
+        'json': ['simplejson' if sys.version_info < (2, 6) else ''],
         'account': ['TracAccountManager >= 0.4'],
         'oauth': ['oauth2'],
         'httplib2': ['httplib2']
@@ -98 +99 @@
         spamfilter.captcha.expression = tracspamfilter.captcha.expression
         spamfilter.captcha.rand = tracspamfilter.captcha.rand
         spamfilter.captcha.recaptcha = tracspamfilter.captcha.recaptcha
-        spamfilter.captcha.recaptcha2 = tracspamfilter.captcha.recaptcha2
+        spamfilter.captcha.recaptcha2 = tracspamfilter.captcha.recaptcha2[json]
         spamfilter.captcha.keycaptcha = tracspamfilter.captcha.keycaptcha
         spamfilter.captcha.mollom = tracspamfilter.captcha.mollom[oauth,httplib2]
     """,

../spam-filter/tracspamfilter/captcha/recaptcha2.py

@@ -11 +11 @@
 # individuals. For the exact contribution history, see the revision
 # history and logs, available at http://projects.edgewall.com/trac/.
 
-import json
 import urllib
 import urllib2
 from pkg_resources import get_distribution
+try:
+    import json
+except ImportError:
+    import simplejson as json
 
 from trac import __version__ as TRAC_VERSION
 from trac.config import Option

../spam-filter/tracspamfilter/filters/blogspam.py

@@ -15 +15 @@
 # Author: Dirk Stöcker <trac@dstoecker.de>
 
 from email.Utils import parseaddr
-import json
 from pkg_resources import get_distribution
+try:
+    import json
+except ImportError:
+    import simplejson as json
 
 from trac import __version__ as TRAC_VERSION
 from trac.config import IntOption, Option, ListOption

[Jun Omae]

I noticed tracspamfilter for Trac 1.0 with Python 2.5 has syntax errors due to with keywords without from __future__ import with_statement.