devtools: fix lookup commit fixing a fix of many commits
authorThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 17 Jan 2017 14:24:20 +0000 (15:24 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Wed, 18 Jan 2017 15:50:18 +0000 (16:50 +0100)
There was a bug when looking at a commit fixing a commit which
itself was fixing many commits:

% devtools/git-log-fixes.sh 12ee45a36~..12ee45a36
devtools/git-log-fixes.sh: 96: local: 5499c1fc9baa: bad variable name

In this case, the list of commits was not quoted in variable assignment.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
devtools/git-log-fixes.sh

index 4824c7f..d590735 100755 (executable)
@@ -90,7 +90,7 @@ origin_version () # <origin_hash> ...
                git rev-parse -q --verify $1 >&- || continue
                # get version of this bug origin
                local origver=$(commit_version $origin)
-               local roothashes=$(origin_filter $origin)
+               local roothashes="$(origin_filter $origin)"
                if [ -n "$roothashes" ] ; then
                        # look chained fix of fix recursively
                        local rootver="$(origin_version $roothashes)"