[coreboot] New patch to review for coreboot: cedbdcd lint: Get absolute path in compare_output
Zheng Bao (zheng.bao@amd.com)
gerrit at coreboot.org
Fri Sep 28 12:28:49 CEST 2012
Zheng Bao (zheng.bao at amd.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1551
-gerrit
commit cedbdcdae7f045dc2b69da8b224298f70254db35
Author: Zheng Bao <fishbaozi at gmail.com>
Date: Fri Sep 28 20:14:38 2012 +0800
lint: Get absolute path in compare_output
The classes in $(top)/Makefile uses $(abspath) to get the path.
The $(abspath) can not resolve symlink. If the coreboot is located
in a symlink directory, the run_printall produces the absolute
path while the $PWD just produces the path with symlink. Use
`pwd -P` to get the abs path.
Change-Id: Icf6b364d030c14a9c78991767b17dafc701baf3c
Signed-off-by: Zheng Bao <zheng.bao at amd.com>
Signed-off-by: Zheng Bao <fishbaozi at gmail.com>
---
util/lint/lint-stable-002-build-dir-handling | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/util/lint/lint-stable-002-build-dir-handling b/util/lint/lint-stable-002-build-dir-handling
index 9350907..ca61a35 100755
--- a/util/lint/lint-stable-002-build-dir-handling
+++ b/util/lint/lint-stable-002-build-dir-handling
@@ -58,7 +58,8 @@ rm -f $TMPCONFIG
$MAKE DOTCONFIG=$TMPCONFIG allyesconfig >/dev/null
# look up parent directory
-PARENTDIR=`dirname $PWD`
+CURRENTDIR=`pwd -P`
+PARENTDIR=`dirname $CURRENTDIR`
compare_output "`run_printall build`" "build/.../static.c build/.../static.c build/.../static.ramstage.o build/.../static.romstage.o"
compare_output "`run_printall ../obj`" "$PARENTDIR/obj/.../static.c $PARENTDIR/obj/.../static.c $PARENTDIR/obj/.../static.ramstage.o $PARENTDIR/obj/.../static.romstage.o"
More information about the coreboot
mailing list