Pulp

The ideal way to fix this would be to find a way to express the urls in one way that is common to 1.6 through 1.10 and just switch to that.

I suspect a common way across that many versions is not possible. If it's not, then creating a compatibility layer to allow the code to express the urls once, but have them handed to Django in the way that specific version needs would be good.

Either way, we should follow the Don't Repeat Yourself (DRY) principle and only have our URLs represented once and not duplicated.

There has been some previous work for this compatibility.

After grep'ing the codebase, I found a couple places that could also use a version check guard.

./pulp_puppet/pulp_puppet_plugins/pulp_puppet/forge/urls.py:urlpatterns = patterns('',
./pulp/server/pulp/server/content/web/urls.py:urlpatterns = patterns(

Given what patterns() does in django 1.6[1], I think we're safe using just a list of url() calls. That's actually what we're doing already here:

https://github.com/pulp/pulp/blob/master/server/pulp/server/webservices/urls.py#L84

Also, I think I am going to leave the compat_urls.py files alone since they're only loaded in django < 1.6.

[1] https://git.io/v91F8