I patched the file manually myself. But I had misdiagnosed the
problem: the I was using the wrong location for linux build. Turns out
opensuse is like debian. Under /lib/modules/2.6.31.12-0.2-default for
example there are two dirs: build and source. The correct option to
use is --with-l26=/lib/modules/2.6.31.12-0.2-default/build/

Sorry for the inconvenience. So, Ben, you should reject the last patch
looking at *-obj. 43255dc9de6ac699a9ecd70be7abbdff3afb5ed8 I think.

So now it finally builds. On to figure out how to do this in the rpm spec file.

Thanks,
Jad.


On Fri, Jul 2, 2010 at 6:05 PM, Jad Naous <[email protected]> wrote:
> 1) The automatic tarball creation from git is not working, I get "404
> - Object does not exist". So I checked the src out from git.
> 2) The patch is getting rejected. At the risk of sounding like a complete 
> noob:
>
> cd openvswitch
> patch -p1 < ../split-linux-src-obj.patch
>
> Gives:
>
> patching file acinclude.m4
> Hunk #1 FAILED at 20.
> Hunk #2 FAILED at 44.
> Hunk #3 FAILED at 80.
> 3 out of 3 hunks FAILED -- saving rejects to file acinclude.m4.rej
>
> I've attached acinclude.m4.rej and the patch. What am I doing wrong?
>
> Jad.
>
> On Fri, Jul 2, 2010 at 2:15 PM, Ben Pfaff <[email protected]> wrote:
>> On Fri, Jul 02, 2010 at 01:53:17PM -0700, Jad Naous wrote:
>>> Sure! For opensuse, the source dir is /usr/src/linux and the build dir
>>> is /usr/src/linux-obj. I want to build an rpm for this to make it easy
>>> to install. I'll let you know how it goes, but making an rpm requires
>>> a standard way of building modules, which I don't think ovs uses.
>>>
>>> So I should checkout the latest tarball I assume to get this fix?
>>
>> I didn't push anything yet.  I'm waiting on Ian for approval.
>>
>> Here's a patch to apply on top of the previous one.  I think it should
>> handle the OpenSUSE case automatically.
>>
>> To try this out, check out the "master" branch from openvswitch.org (or
>> retrieve
>> http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=snapshot;sf=tgz
>> for an equivalent tar.gz), apply the previously posted patch, then this
>> one, then run ./boot.sh.
>>
>> --8<--------------------------cut here-------------------------->8--
>>
>> From 43255dc9de6ac699a9ecd70be7abbdff3afb5ed8 Mon Sep 17 00:00:00 2001
>> From: Ben Pfaff <[email protected]>
>> Date: Fri, 2 Jul 2010 14:13:14 -0700
>> Subject: [PATCH] configure: Automatically detect OpenSUSE kernel source 
>> directory.
>>
>> CC: Jad Naous <[email protected]>
>> ---
>>  acinclude.m4 |    3 +++
>>  1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/acinclude.m4 b/acinclude.m4
>> index 80794da..9248ddc 100644
>> --- a/acinclude.m4
>> +++ b/acinclude.m4
>> @@ -66,6 +66,9 @@ AC_DEFUN([OVS_CHECK_LINUX26], [
>>          */build)
>>            KSRC26=`echo "$KBUILD26" | sed 's,/build/*$,/source,'`
>>            ;; # (
>> +         *-obj)
>> +           KSRC26=`echo "$KBUILD26" | sed 's,-obj$,,'`
>> +           ;; # (
>>          *)
>>            KSRC26=`(cd $KBUILD26 && pwd -P) | sed 's,-[[^-]]*$,-common,'`
>>            ;;
>> --
>> 1.7.1
>>
>>
>>
>

_______________________________________________
discuss mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/discuss_openvswitch.org

Reply via email to