[coreboot-gerrit] Patch set updated 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 69c614541bce6c16cf3080759717aa239af16957
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.
    
    When the makefile expands a variable containing a dirctory name that
    has colons in it, it seems to interpret that as a makefile target, and
    fails the build.
    
    Many other characters also confuse the makefiles, including spaces,
    ampersand symbols, dollar signs, etc.
    
    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