Browse code

cleanup comments

Change-Id: I685f726178722e58ccfd008bad16a671cc8b8e2a

Sean Dague authored on 2017/06/21 23:55:16
Showing 1 changed files
... ...
@@ -2,7 +2,7 @@
2 2
 #
3 3
 # lib/oslo
4 4
 #
5
-# Functions to install **Oslo** libraries from git
5
+# Functions to install libraries from git
6 6
 #
7 7
 # We need this to handle the fact that projects would like to use
8 8
 # pre-released versions of oslo libraries.
... ...
@@ -13,10 +13,10 @@
13 13
 
14 14
 # ``stack.sh`` calls the entry points in this order:
15 15
 #
16
-# - install_oslo
16
+# - install_libraries
17 17
 
18 18
 # Save trace setting
19
-_XTRACE_LIB_OSLO=$(set +o | grep xtrace)
19
+_XTRACE_LIB_LIBRARIES=$(set +o | grep xtrace)
20 20
 set +o xtrace
21 21
 
22 22
 
... ...
@@ -54,12 +54,11 @@ GITDIR["python-openstacksdk"]=$DEST/python-openstacksdk
54 54
 GITDIR["stevedore"]=$DEST/stevedore
55 55
 GITDIR["taskflow"]=$DEST/taskflow
56 56
 GITDIR["tooz"]=$DEST/tooz
57
-# TODO(mriedem): This is a common pattern so even though os-traits isn't
58
-# officially an oslo library, it is nice to re-use this script for non-oslo
59
-# things like os-traits. We should rename this script to be more generic
60
-# and then fold os-brick into it also.
61
-GITDIR["os-traits"]=$DEST/os-traits
57
+
58
+# Non oslo libraries are welcomed below as well, this prevents
59
+# duplication of this code.
62 60
 GITDIR["os-brick"]=$DEST/os-brick
61
+GITDIR["os-traits"]=$DEST/os-traits
63 62
 
64 63
 # Support entry points installation of console scripts
65 64
 OSLO_BIN_DIR=$(get_python_exec_prefix)
... ...
@@ -120,18 +119,20 @@ function install_libs {
120 120
     # installation of additional libraries
121 121
     #
122 122
     # os-traits for nova
123
-    _install_lib_from_source "os-traits"
124
-    # os-brick for nova/cinder
125 123
     _install_lib_from_source "os-brick"
124
+    _install_lib_from_source "os-traits"
126 125
 
127 126
 
128 127
     # etcd (because tooz does not have a hard dependency on these)
128
+    #
129
+    # NOTE(sdague): this is currently a work around because tooz
130
+    # doesn't pull in etcd3.
129 131
     pip_install etcd3
130 132
     pip_install etcd3gw
131 133
 }
132 134
 
133 135
 # Restore xtrace
134
-$_XTRACE_LIB_OSLO
136
+$_XTRACE_LIB_LIBRARIES
135 137
 
136 138
 # Tell emacs to use shell-script-mode
137 139
 ## Local variables: