[coreboot-gerrit] New patch to review for coreboot: board_status/getrevision.sh: get rid of colons in dir names
Martin Roth (martinroth@google.com)
gerrit at coreboot.org
Wed Jan 27 18:04:37 CET 2016
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
More information about the coreboot-gerrit
mailing list