RFC: python3 support (which would mean dropping atleast 2.5) about django-analytical

[jcassee]

Hi Tinnet,

Sorry for the delay. I would like to keep Python 2.6 support until Debian Wheezy is out, but I think it is absolutely required to start supporting Python 3.

The specific problem you cite luckily occurs only once in the code (templatetags/analytical.py, line 72). I think we can remove the exception logging in this particular case, because the exception should only occur if the code in django-analytical itself is buggy.

Are there any other fundamental problems that you are aware of? How far do you think we can go without dropping Python 2.6 support?

[tinnet]

This: https://docs.djangoproject.com/en/dev/topics/python3/ suggests targeting ">=2.6.5" and the (current) django 1.6 release notes do not mention dropping 2.6 support (https://docs.djangoproject.com/en/dev/releases/1.6/).

I think keeping 2.6 around will be ok for quite a while (no need to move faster than django itself, right?), but 2.5 will be tricky/impossible. This would be fine for debian squeeze and wheezy :)

[jcassee]

Sounds good, let's support versions 2.6.5 and up. I noticed you have been coding Python 3 stuff in your repo. Do you want to contribute these changes?

[tinnet]

Sure, i'll get the branch updated and open a PR soon.

[tinnet]

I took a quick look, but something about the override_settings decorator is broken with python3.2 and 3.3 (django version does not matter):

https://github.com/jcassee/django-analytical/blob/master/analytical/tests/utils.py#L86-88

File "~/devel/django-analytical/analytical/tests/utils.py", line 87, in __getattr__
    return self.__dict__["default_settings"]
KeyError: 'default_settings'

The decorator itself has changed in django, maybe pulling the current version in would fix things, but i'll not be able to try before later tonight.

https://github.com/django/django/blob/master/django/test/utils.py#L188

[laike9m]

What's the current state of django-analytical Python 3 support? @jcassee is it on your schedule?

[jcassee]

@laike9m I have not worked on this myself. @tinnet Do you still intend to work on this?

[jcassee]

Fixed by @amadornimbis in #52, merged as aab1cfc and release as 0.21.0.