[coreboot-gerrit] New patch to review for coreboot: board_status/getrevision.sh: get rid of colons in dir names

[Martin Roth (martinroth@google.com)]

Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13490

-gerrit

commit 3c6a1b6194f1af397363d2ab871b4b5c7b449aae
Author: Martin Roth <martinroth at google.com>
Date:   Wed Jan 27 09:53:45 2016 -0700

    board_status/getrevision.sh: get rid of colons in dir names
    
    Gnu make won't build in directories that have a colon in their name.
    
    As far as I can tell without extensive debug, this has to do with
    make's variable expansion.  When we try to expand the directory name
    of a file in the tree, the colon turns the directory into a target.
    
    I've started including scripts into the board-status directories to
    do the build of the rom that was tested, and this is preventing them
    from working without renaming the directory before doing the build.
    
    Change-Id: I9dd8e4027be21363015cd8df9918610e206afce2
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/board_status/getrevision.sh | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/util/board_status/getrevision.sh b/util/board_status/getrevision.sh
index 6559df3..168c972 100755
--- a/util/board_status/getrevision.sh
+++ b/util/board_status/getrevision.sh
@@ -96,7 +96,10 @@ timestamp() {
 	if [ -z "$t" ]; then
 		echo "Warning: Could not determine timestamp." 2>/dev/null
 	fi
-	echo "${t}"
+
+	# output the time, changing colons to underscores.
+	# gnu make doesn't work in directories with colons
+	echo "${t}" | tr ':' '_'
 }
 
 # Retrieve local SCM revision info. This is useful if we're working in a different SCM than upstream and/or