commit | ada886dd43ccc07f48d3a82d8d3d840fe5096c03 | [log] [tgz] |
---|---|---|
author | Ian Wienand <iwienand@redhat.com> | Wed Oct 07 14:06:26 2015 +1100 |
committer | Ian Wienand <iwienand@redhat.com> | Wed Oct 07 17:03:32 2015 +1100 |
tree | 93d62f1c82edc08d813b7f9f7eb9270e7024e055 | |
parent | 433a9b10ddd6fa67d7459c4943a92ce4f488cebc [diff] [blame] |
Don't mix declaration and set of locals Ia0957b47187c3dcadd46154b17022c4213781112 proposes to have bashate find instances of setting a local value. The issue is that "local" always returns 0, thus hiding any failure in the commands running to set the variable. This is an automated replacement of such instances Depends-On: I676c805e8f0401f75cc5367eee83b3d880cdef81 Change-Id: I9c8912a8fd596535589b207d7fc553b9d951d3fe
diff --git a/lib/neutron_plugins/embrane b/lib/neutron_plugins/embrane index 6b4819e..2028496 100644 --- a/lib/neutron_plugins/embrane +++ b/lib/neutron_plugins/embrane
@@ -10,7 +10,8 @@ source $TOP_DIR/lib/neutron_plugins/openvswitch function save_function { - local ORIG_FUNC=$(declare -f $1) + local ORIG_FUNC + ORIG_FUNC=$(declare -f $1) local NEW_FUNC="$2${ORIG_FUNC#$1}" eval "$NEW_FUNC" }