get NameError: name 'patterns' is not defined when following install instructions on centos7

[chicks-net]

While following https://github.com/sii/siptrackweb/wiki/Install-personal-dev-environment I ran into NameError: name 'patterns' is not defined when running python manage.py migrate. According to stackoverflow this can be fixed by replacing

from django.conf.urls import url, include

with

from django.conf.urls import url, include, patterns

which got me past that issue, but then I see

/var/venv/siptrack/lib/python2.7/site-packages/siptrackweb/urls.py:253: RemovedInDjango110Warning: django.conf.urls.patterns() is deprecated and will be removed in Django 1.10. Update your urlpatterns to be a list of django.conf.urls.url() instances instead.
  (r'^prototype.js$', 'root.prototypejs'),

Should the instructions be updated to avoid this deprecation?

[chicks-net]

Tivix/django-rest-auth#136 shows how another team handled this.

[stemid]

Thanks, seems to me the main problem was in urls.py that django-admin generates. I've updated the docs to reflect its new appeareance.

Also might add some note in the docs about which Django version is recommended for that guide.

[chicks-net]

I updated my stweb/urls.py to follow the docs and the warning is still there.

/var/venv/siptrack/lib/python2.7/site-packages/siptrackweb/urls.py:253: RemovedInDjango110Warning: django.conf.urls.patterns() is deprecated and will be removed in Django 1.10. Update your urlpatterns to be a list of django.conf.urls.url() instances instead.
  (r'^prototype.js$', 'root.prototypejs'),

I also see many lines of

/var/venv/siptrack/lib/python2.7/site-packages/django/conf/urls/__init__.py:89: RemovedInDjango110Warning: Support for string view arguments to url() is deprecated and will be removed in Django 1.10 (got view.add_post). Pass the callable instead.
  t = url(prefix=prefix, *t)

I'm guessing both of these are due to what is in siptrackweb/urls.py which still uses patterns().

[stemid]

Will definitely move away from patterns() and towards using a urlpatterns list in the master branch too Chris, thanks for pointing that out. I'll make a PR asap.

All I did for now, and which was enough to get siptrack working on my CentOS 7 install, was to update the docs.

[chicks-net]

Agreed. I was able to proceed with testing with it as-is.

Thanks for responding so quickly. 👍

[stemid]

@sii this is most likely solved now considering you've upgraded urls.py to be more modern using list(urlpatterns) instead of patterns.