Subject: [PATCH] tags.sh: Add missing quotes
authorStephen Boyd <sboyd@codeaurora.org>
Sun, 1 Apr 2012 06:41:07 +0000 (23:41 -0700)
committerMichal Marek <mmarek@suse.cz>
Mon, 2 Apr 2012 09:28:17 +0000 (11:28 +0200)
When $remove_structs is empty a test for empty string will turn
into test -n with no arguments meaning true. Add quotes so an
empty string is tested and so that make cscope works again.

Reported-and-tested-by: Jike Song <albcamus@gmail.com>
Reported-by: Prarit Bhargava <prarit@redhat.com>
Acked-by: Yang Bai <hamo.by@gmail.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Michal Marek <mmarek@suse.cz>

scripts/tags.sh

index 0d6004e..cf7b12f 100755 (executable)
@@ -254,6 +254,6 @@ case "$1" in
 esac
 
 # Remove structure forward declarations.
-if [ -n $remove_structs ]; then
+if [ -n "$remove_structs" ]; then
     LANG=C sed -i -e '/^\([a-zA-Z_][a-zA-Z0-9_]*\)\t.*\t\/\^struct \1;.*\$\/;"\tx$/d' $1
 fi