Hi all,

I am trying to build lustre 1.8.1 against OFED 1.4.2 and have uncovered a couple
of bugs regarding how BACKPORT_INCLUDES is handled in the ofa-kernel config.mk 
file.

The ofed_patch.sh script in the ofa_kernel package is incorrectly escaped, and
results in a mangled  BACKPORT_INCLUDES path.

The lustre ./configure script is also broken, and prepends and extra "-I"
infront of the BACKPORT_INCLUDES path.

Patches for both are attached.

Cheers,

Guy


-- 
Dr. Guy Coates,  Informatics System Group
The Wellcome Trust Sanger Institute, Hinxton, Cambridge, CB10 1HH, UK
Tel: +44 (0)1223 834244 x 6925
Fax: +44 (0)1223 496802



-- 
 The Wellcome Trust Sanger Institute is operated by Genome Research 
 Limited, a charity registered in England with number 1021457 and a 
 company registered in England with number 2742969, whose registered 
 office is 215 Euston Road, London, NW1 2BE. 
Index: ofed_patch.sh
===================================================================
--- ofed_patch.sh	(revision 443)
+++ ofed_patch.sh	(working copy)
@@ -183,7 +183,7 @@
                         if [ "X${WITH_BACKPORT_PATCHES}" == "Xyes" ]; then
                                 apply_backport_patches
                         fi
-                        BACKPORT_INCLUDES='-I${CWD}/kernel_addons/backport/'${BACKPORT_DIR}/include/
+                        BACKPORT_INCLUDES="-I${CWD}/kernel_addons/backport/"${BACKPORT_DIR}/include/
                 fi
 
 
--- lustre-1.8.1/configure	2009-07-24 23:28:51.000000000 +0100
+++ configure	2009-08-10 15:08:22.316488430 +0100
@@ -5595,7 +5595,7 @@
 		fi
 		if test -n "$BACKPORT_INCLUDES"; then
 			OFED_BACKPORT_PATH=`echo $BACKPORT_INCLUDES | sed "s#.*/src/ofa_kernel/#$O2IBPATH/#"`
-			EXTRA_LNET_INCLUDE="-I$OFED_BACKPORT_PATH $EXTRA_LNET_INCLUDE"
+			EXTRA_LNET_INCLUDE="$OFED_BACKPORT_PATH $EXTRA_LNET_INCLUDE"
 			echo "$as_me:$LINENO: result: yes" >&5
 echo "${ECHO_T}yes" >&6
 		else
_______________________________________________
Lustre-discuss mailing list
[email protected]
http://lists.lustre.org/mailman/listinfo/lustre-discuss

Reply via email to