Merge lp:~fu-wei/lava-dispatcher/multinode into lp:lava-dispatcher/multinode
Proposed by
Fu Wei
Status: | Merged |
---|---|
Approved by: | Neil Williams |
Approved revision: | no longer in the source branch. |
Merged at revision: | 687 |
Proposed branch: | lp:~fu-wei/lava-dispatcher/multinode |
Merge into: | lp:lava-dispatcher/multinode |
Diff against target: |
11 lines (+1/-1) 1 file modified
lava_test_shell/multi_node/lava-multi-node.lib (+1/-1) |
To merge this branch: | bzr merge lp:~fu-wei/lava-dispatcher/multinode |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Neil Williams | Pending | ||
Review via email: mp+181806@code.launchpad.net |
Description of the change
Fix the bug of disabling debug info in multi-node API shell scritps
To post a comment you must log in.
> === modified file 'lava_test_ shell/multi_ node/lava- multi-node. lib' shell/multi_ node/lava- multi-node. lib 2013-08-01 12:42:35 +0000 shell/multi_ node/lava- multi-node. lib 2013-08-23 12:10:34 +0000 node_debug () { NODE_DEBUG ] ; then MULTI_NODE_ DEBUG" ] ; then COMMAND} _DEBUG $@ $(date)>"
> --- lava_test_
> +++ lava_test_
> @@ -23,7 +23,7 @@
>
> _lava_multi_
>
> -if [ -n $LAVA_MULTI_
> +if [ -n "$LAVA_
> echo "${MESSAGE_
> fi
most of the time you use variables in a shell script, they should be
surrounded by double quotes, so it would be nice to do this all over
those shell scripts.