yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #80907
[Bug 1855506] [NEW] Incorrect django version in requirements.txt
Public bug reported:
requirements.txt contains these lines:
Django<2,>=1.11;python_version<'3.0' # BSD
Django<2.1,>=1.11;python_version>='3.0' # BSD
First of all, it seems weird that there are two conflicting lines for
the same package. But, I'm seeing a more serious issue. Throughout the
code there are import lines like this:
openstack_dashboard/views.py:from django import urls
but django.urls is only present in django version 2 and later. In
version 1.8, for example, that module appears to be django.conf.urls.
I may be misunderstanding something fundamental but I think that
requrements line needs to specify a lower bound of 2.0. I don't know
how to reconcile that with there also being an upper bound of 2.
** Affects: horizon
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1855506
Title:
Incorrect django version in requirements.txt
Status in OpenStack Dashboard (Horizon):
New
Bug description:
requirements.txt contains these lines:
Django<2,>=1.11;python_version<'3.0' # BSD
Django<2.1,>=1.11;python_version>='3.0' # BSD
First of all, it seems weird that there are two conflicting lines for
the same package. But, I'm seeing a more serious issue. Throughout
the code there are import lines like this:
openstack_dashboard/views.py:from django import urls
but django.urls is only present in django version 2 and later. In
version 1.8, for example, that module appears to be django.conf.urls.
I may be misunderstanding something fundamental but I think that
requrements line needs to specify a lower bound of 2.0. I don't know
how to reconcile that with there also being an upper bound of 2.
To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1855506/+subscriptions
Follow ups