openstack/devstack
Code Issues Proposed changes

Restrict requests to fedora

Browse Source 
The os_RELEASE for RHEL is 7.1 (for example). Which does not work for comparisons
to an integer.  And, while I am at it, change base_path to not use a hard-coded
directory.

Change-Id: I64a04810cc7ba4668c2cb7a8df79c206301e9e16
This commit is contained in:
Mark Hamzy 2015-10-14 13:42:18 -05:00
parent 188b38fc80
commit 746e72d4c3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/fixup_stuff.sh
View File

@ -135,7 +135,7 @@ if is_fedora; then
fi fi
fi fi
if [[ "$os_RELEASE" -ge "21" ]]; then if [[ "$os_VENDOR" == "Fedora" ]] && [[ "$os_RELEASE" -ge "21" ]]; then
# requests ships vendored version of chardet/urllib3, but on # requests ships vendored version of chardet/urllib3, but on
# fedora these are symlinked back to the primary versions to # fedora these are symlinked back to the primary versions to
# avoid duplication of code on disk. This is fine when # avoid duplication of code on disk. This is fine when
@ -152,9 +152,9 @@ if is_fedora; then
# https://bugs.launchpad.net/glance/+bug/1476770 # https://bugs.launchpad.net/glance/+bug/1476770
# https://bugzilla.redhat.com/show_bug.cgi?id=1253823 # https://bugzilla.redhat.com/show_bug.cgi?id=1253823
base_path=/usr/lib/python2.7/site-packages/requests/packages base_path=$(get_package_path requests)/packages
if [ -L $base_path/chardet -o -L $base_path/urllib3 ]; then if [ -L $base_path/chardet -o -L $base_path/urllib3 ]; then
sudo rm -f /usr/lib/python2.7/site-packages/requests/packages/{chardet,urllib3} sudo rm -f $base_path/{chardet,urllib3}
# install requests with the bundled urllib3 to avoid conflicts # install requests with the bundled urllib3 to avoid conflicts
pip_install --upgrade --force-reinstall requests pip_install --upgrade --force-reinstall requests
fi fi