Bug#659685: [ovs-dev] Bug#659685: Can we have a fix please?

2012-03-07 Thread Simon Horman
On Wed, Mar 07, 2012 at 04:48:55PM +0800, Thomas Goirand wrote:
 Hi,
 
 I had a look to the current packaging of openvswitch, in order to fix
 this bug (eg: #659685) With all due respect... it's a mess.
 
 The only reason why you are using Quilt is to patch files that are in
 your openvswitch_version.debian.tar.gz. Please don't abuse quilt like
 this! There's no reason to patch things under the debian folder, it
 should come already patched.
 
 Your debian-changes-1.4.0-2 is reverting some of the patches previously
 applied, and which would have otherwise let the DKMS module.
 
 I have attached a patch which removes all patches in debian/patches,
 since they aren't needed at all. Also, with this patch,
 openvswitch-datapath-dkms works as expected (eg: it build the kernel
 module), because using ${kernel_source_dir} as it should be. Please
 apply this patch and upload a new 1.4.0-3 of openvswitch.

Hi Thomas,

It is my understanding that placing patches under debian/patches is part of
the 3.0 (quilt) package format. Is OVS using the format incorrectly in some
way? Does that format interact with DKMS poorly in some way?



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



Bug#659685: [ovs-dev] Bug#659685: Can we have a fix please?

2012-03-07 Thread Thomas Goirand
Hi,

I'm sorry, I think I didn't express myself correctly. Please read this:
http://lintian.debian.org/tags/patch-modifying-debian-files.html

As you can see, lintian did catch issues in your openvswitch package! :)

So, if you have changes to make in let's say debian/control, then edit
the file, don't produce a patch for it in debian/patches. Same for
debian/python-openvswitch.install, or debian/dkms.conf.in.

Did you get it this time? Or should I explain further?

I hope that helps.

Thomas



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



Bug#659685: [ovs-dev] Bug#659685: Can we have a fix please?

2012-03-07 Thread Thomas Goirand
If you agree with my patch, I can do an NMU. Is that ok?

Thomas



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