I have tested this patch and the path and resulting package
appear to be correct.

I see ./configure --with-linux=/usr/src/linux-headers-3.1.0-1-amd64
in the resulting config.log

On Mon, Feb 13, 2012 at 06:05:19PM -0800, Justin Pettit wrote:
> Assuming it's the fully correct path, it looks reasonable to me.
> 
> --Justin
> 
> 
> On Feb 13, 2012, at 4:15 PM, Ben Pfaff wrote:
> 
> > DKMS passes in an explicit variable for the kernel source directory, so we
> > should use that instead of `uname -r`.
> > 
> > CC: 659...@bugs.debian.org
> > Reported-by: Thomas Goirand <tho...@goirand.fr>
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> > ---
> > debian/dkms.conf.in |    2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/debian/dkms.conf.in b/debian/dkms.conf.in
> > index ae1fc7a..d5bc37e 100644
> > --- a/debian/dkms.conf.in
> > +++ b/debian/dkms.conf.in
> > @@ -1,6 +1,6 @@
> > PACKAGE_NAME="openvswitch"
> > PACKAGE_VERSION="__VERSION__"
> > -MAKE="./configure --with-linux=/lib/modules/`uname -r`/build && make -C 
> > datapath/linux"
> > +MAKE="./configure --with-linux='${kernel_source_dir}' && make -C 
> > datapath/linux"
> > BUILT_MODULE_NAME[0]=openvswitch_mod
> > BUILT_MODULE_NAME[1]=brcompat_mod
> > BUILT_MODULE_LOCATION[0]=datapath/linux/
> > -- 
> > 1.7.2.5
> > 
> > _______________________________________________
> > dev mailing list
> > d...@openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> 
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
> 



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to