[coreboot-gerrit] Patch set updated for coreboot: crossgcc: skip TARGETARCH for tools that don't use it

[Martin Roth]

Martin Roth ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14229

-gerrit

commit 8dc03374528ebc93194885e013a94c90a4d170b2
Author: Martin Roth 
Date:   Fri Apr 1 18:46:29 2016 -0600

crossgcc: skip TARGETARCH for tools that don't use it

Many of the tools and libraries don't use a target architecture, but
    they were still getting put in one. This change separates out the
    builds that need the target architecture from the ones that don't,
    and sets the build directory accordingly.

This will help keep from rebuilding the libraries when building all
    of the tools if you keep the temporary files around (-t option).

Change-Id: Id6c17719332f2244657f103f5f07ca7812d51af1
    Signed-off-by: Martin Roth 
---
 util/crossgcc/buildgcc | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index 525b574..7782739 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -286,13 +286,25 @@ is_package_enabled()
    echo "$PACKAGES" |grep -q "<$1>"
 }

+package_uses_targetarch()
+{
+	if [ "$1" = "GCC" ] || [ "$1" = "GDB" ] || [ "$1" = "BINUTILS" ] || \
+		[ "$1" = "PYTHON" ] || [ "$1" = "EXPAT" ]; then
+		true
+	else
+		false
+	fi
+}
+
 build() {
    package=$1

fn_exists build_$package || return

version="$(eval echo $$package"_VERSION")"
-	BUILDDIR=build-${TARGETARCH}-$package
+	package_uses_targetarch "$package" && \
+		BUILDDIR=build-${TARGETARCH}-$package || \
+		BUILDDIR=build-$package

mkdir -p ${BUILDDIR}

@@ -321,7 +333,7 @@ cleanup()
 {
    printf "Cleaning up temporary files... "
    for package in $PACKAGES; do
-		rm -rf build-${TARGETARCH}-$package $(eval echo $$package"_DIR")
+		rm -rf build-${TARGETARCH}-$package build-$package $(eval echo $$package"_DIR")
    done
    rm -f getopt
    printf "${green}ok${NC}\n"