GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

buildgcc: Define $CMAKE only if clang package is built 

This moves the CMAKE definition down into the case statement
for $PACKAGE so that it is only required when the user wants to
build clang.

With this approach, "./buildgcc -P clang" will error out with the
"ERROR: Missing tool:" message if cmake is not installed.

Change-Id: I1e5c1bd67ade8f93ba0390df7f234deb47b9b18a
Signed-off-by: David Hendricks <dhendrix@chromium.org>
Reviewed-on: http://review.coreboot.org/10556
Tested-by: build bot (Jenkins)
Reviewed-by: Francis Rowe <info@gluglug.org.uk>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
David Hendricks 2015-06-15 17:03:22 -07:00 committed by Stefan Reinauer
parent ae9f87fc1b
commit cd5cdd3ba4
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
util/crossgcc/buildgcc
View File

@ -165,7 +165,6 @@ searchtool()
TAR=$(searchtool tar) || exit $? TAR=$(searchtool tar) || exit $?
PATCH=$(searchtool patch) || exit $? PATCH=$(searchtool patch) || exit $?
MAKE=$(searchtool make) || exit $? MAKE=$(searchtool make) || exit $?
CMAKE=$(searchtool cmake "cmake") || exit $?
SHA1SUM=$(searchtool sha1sum) SHA1SUM=$(searchtool sha1sum)
SHA512SUM=$(searchtool sha512sum) SHA512SUM=$(searchtool sha512sum)
CHECKSUM=$SHA1SUM CHECKSUM=$SHA1SUM
@ -554,6 +553,7 @@ case "$PACKAGE" in
CLANG|clang) CLANG|clang)
NAME=clang NAME=clang
PACKAGES="LLVM CFE CRT CTE" PACKAGES="LLVM CFE CRT CTE"
CMAKE=$(searchtool cmake "cmake") || exit $?
;; ;;
IASL|iasl) IASL|iasl)
NAME="IASL ACPI compiler" NAME="IASL ACPI compiler"