From: Vitaly Fertman <vitaly.fertman@sun.com>
Date: Thu, 21 Oct 2010 23:27:19 +0000 (+0400)
Subject: b=20744 remedial fix for inkernel ofed handling
X-Git-Tag: 2.0.54.0~9
X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=b7029721f2a0e32e85c985f7fbda4bce600ddd90

b=20744 remedial fix for inkernel ofed handling

o=brian
i=yangsheng
i=yibin.wang

It seems that the patch to add inkernel handling for SLES10's OFED introduced a
bug for other inkernel uses.  This fixes that bug.
---

diff --git a/build/lbuild b/build/lbuild
index ab6c725..9177661 100755
--- a/build/lbuild
+++ b/build/lbuild
@@ -1335,19 +1335,25 @@ build_ofed() {
 
     # if an ofed version is given, then it means use OFED proper,
     # not any vendor specific "inkernel" version
-    if [ -z "$ofed_version" -o "$ofed_version" = "inkernel" ]; then
-        # first see if there is a distro specific override for this
-        # XXX we need to better integrate an distro specific override with
+    if [ -z "$ofed_version" ]; then
+        return 0
+    fi
+
+    if [ "$ofed_version" = "inkernel" ]; then
+        # see if there is a distro specific override for this and use
+        # that if it exists
+        # XXX we need to better integrate a distro specific override with
         #     the rest of this function so that all of the reuse cache
         #     stuff is leveraged given that 80% of this function is reuse
         if type -p build_ofed-${DISTRO}; then
             local ofed_location
             ofed_location=$(build_ofed-${DISTRO} ${STDOUT})
+            local rc=${PIPESTATUS[0]}
             CONFIGURE_FLAGS="--with-o2ib=${ofed_location} ${CONFIGURE_FLAGS}"
-            return ${PIPESTATUS[0]}
+            return $rc
+        else
+            return 0
         fi
-    else
-        return 0
     fi
 
     # build kernel-ib