Fedora17 and Django 1.4 #372

wants to merge 8 commits into from

1 participant

Autotest member

The move to Django 1.4 turned out easier than expected, and since 1.3 and 1.4 are similar enough, this patchset make the rpc server to run in both 1.3 and 1.4. Also, update the dependencies and make changes to the install script, so autotest installs fine on a Fedora 17 server.

lmr added some commits May 31, 2012
@lmr lmr install-autotest-server.sh: Add java-1.7.0-openjdk-devel to pkg list
This way, the appropriate java tools will be installed
to compile the web interface on Fedora 17.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr frontend.db: Pass connection to DatabaseOperations
In Django 1.4, the DatabaseOperations constructor takes
a mandatory connection argument, so let's handle the
TypeError, and make the application compatible with both
Django 1.3 and 1.4.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr frontend.afe.feeds: Update use of deprecated code
In Django 1.2, the access to the feeds object was moved
to from django.contrib.feeds from django.contrib.syndication
and marked it as deprecated. Django 1.4 removed it entirely,
so this change makes the web app actually compatible with
both versions of Django.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr frontend.afe.feeds: Fix coding style
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr apache/django-directives: Update to Django 1.4
Django 1.4 does change the location of the CSS files for
the admin interface, so let's add an extra rule for

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr utils/external_packages.py: Update Django built to 1.4
But stablish a minimum version of 1.3. This way systems
with Django at least 1.3 can use the distro installed
django instead of a newer one.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr frontend.settings: Add MessageMiddleware among the middleware classes
So we can effectively show up popup messages in the Django
admin view. As this class was introduced in 1.2, this change
should also allow to use Django 1.3 as a supported dep.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
@lmr lmr install-autotest-server.sh: Fix IP grabbing expression
So the IP displays correctly on Fedora 17.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
Autotest member
lmr commented Jun 5, 2012

Ok, this all was tested on F16 and F17, and it's working well with both Django 1.3 and 1.4. Applied to next.

@lmr lmr closed this Jun 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment