On Tue, 2008-03-18 at 15:06 -0500, Jerone Young wrote: > +###################################### > +# libfdt probe > +# > +if test -z "$device_tree_support" -a \ > + "$cpu" = "powerpc"; then > + device_tree_support="no" > + cat > $TMPC << EOF > +#include <libfdt.h> > +/* XXX uncomment later when libfdt is built before this test */ > +//int main(void) { void *fdt; return fdt_create(fdt, 1024); } > +int main (void) {return 0;} > +EOF > +# XXX for now do not try to link to libfdt and just check for header */ > +# if $cc $ARCH_CFLAGS $CFLAGS $LDFLAGS -o $TMPE $TMPC -lfdt 2> /dev/null ; > then > + if $cc $ARCH_CFLAGS $CFLAGS $LDFLAGS -o $TMPE $TMPC 2> /dev/null; then > + device_tree_support="yes" > + else > + echo > + echo "Error: Could not find libfdt" > + echo "Make sure to have the libfdt libs and headers installed." > + echo > + exit 1 > + fi > +fi
What is the problem here? Should you describe it in the patch description? Did you mean to fix this before committing? -- Hollis Blanchard IBM Linux Technology Center ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel