Bug#942056: [ovs-dev] [PATCH] debian: Update list of copyright holders.

2019-12-02 Thread Ben Pfaff
On Wed, Nov 27, 2019 at 10:25:36AM -0800, Yi-Hung Wei wrote: > On Wed, Oct 9, 2019 at 10:35 AM Ben Pfaff wrote: > > > > The list of copyright holders was incomplete and out of date. This > > updates it based on a "grep" for copyright notices, which I review

Bug#942056: [PATCH] debian: Update list of copyright holders.

2019-10-09 Thread Ben Pfaff
The list of copyright holders was incomplete and out of date. This updates it based on a "grep" for copyright notices, which I reviewed by hand. CC: 942...@bugs.debian.org Reported-by: Chris Lamb Reported-at: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=942056 Signed-off-by:

Bug#908978: openvswitch: FTBFS on armel, mips, mips64el, mipsel

2018-09-17 Thread Ben Pfaff
On Mon, Sep 17, 2018 at 09:41:50AM +0200, Thomas Goirand wrote: > On 09/17/2018 12:50 AM, Ivo De Decker wrote: > > package: openvswitch > > version: 2.10.0+2018.08.28+git.8ca7c82b7d+ds1-3 > > severity: serious > > > > Hi, > > > > The latest version of openvswitch in unstable fails on armel, mips,

Bug#860479: [ovs-dev] Bug#860479: openvswitch-datapath-dkms (and openvswitch-datapath-source) fail compiling on fresh install of 7.11

2017-05-01 Thread Ben Pfaff
On Mon, Apr 17, 2017 at 07:04:12PM +0300, Bogdan Ilisei wrote: > Package: openvswitch-datapath-dkms > Version: 1.4.2+git20120612-9.1~deb7u1.1 > Severity: grave > Justification: renders package unusable > > Dear Maintainer, > > Installing openvswitch-datapath-dkms (OR openvswitch-datapath-source)

Bug#768095: openvswitch-datapath-dkms fails to build on Debian 7.7 3.2.0-4-amd64 (3.2.63-2+deb7u1)

2014-11-21 Thread Ben Pfaff
On Mon, Nov 17, 2014 at 04:38:27PM +0100, Jonathan Dupart wrote: > Hi, > > I have been bittend by this bug. > > I am raising this bug severity to grave, as it renders openvswitch > unusable after upgrading to the last stable kernel. > > I tested the patch and it works flawlessly. It should be qu

Bug#750252: autoconf: FTBFS: error: pdfetex (file cm-super-t1.enc): cannot open encoding file for reading

2014-08-31 Thread Ben Pfaff
On Mon, Jun 02, 2014 at 08:32:05PM +0200, David Su??rez wrote: > Source: autoconf > Version: 2.69-6 > Severity: serious > Tags: jessie sid > User: debian...@lists.debian.org > Usertags: qa-ftbfs-20140601 qa-ftbfs > Justification: FTBFS on amd64 ... > > !pdfTeX error: pdfetex (file cm-super-t1.enc

Bug#740983: Bug #740983: use the upstream kernel module instead

2014-03-31 Thread Ben Pfaff
severity 740983 normal thanks This bug was submitted with: Severity: grave Justification: renders package unusable because the DKMS module does not build with Linux kernel 3.13. While this is true, it does not render the package unusable by any stretch of the imagination, because Linux

Bug#738402: FTBFS autoconf

2014-02-09 Thread Ben Pfaff
On Mon, Feb 10, 2014 at 09:00:09AM +0900, Norbert Preining wrote: > Anyway, Ben, it was *not* an unintended consequence. > > Please adjust the build deps to include > texlive-fonts-recommended Thanks for clarifying. I'll upload autoconf with the additional build dependency in a few minutes

Bug#738451: autoconf: FTBFS: I can't find file `ecrm1095'.

2014-02-09 Thread Ben Pfaff
reassign 738451 texlive-latex-recommended merge 738451 738402 thanks On Sun, Feb 09, 2014 at 11:10:13AM -0800, Daniel Schepler wrote: > Source: autoconf > Version: 2.69-2 > Severity: serious Same as #738402, reassigning and merging (with this email). -- To UNSUBSCRIBE, email to debian-bugs-rc-

Bug#738402: autoconf: FTBFS: grep: ecrm1095.log: No such file or directory

2014-02-09 Thread Ben Pfaff
reassign 738402 texlive-latex-recommended thanks > [...lots of output from texinfo.tex...] > kpathsea: Running mktexmf ecrm1095 > > ! I can't find file `ecrm1095'. > <*> ...ljfour; mag:=1; nonstopmode; input ecrm1095 > > Please type another input

Bug#732070: Switched severity to grave

2013-12-13 Thread Ben Pfaff
On Sat, Dec 14, 2013 at 12:22:46AM +0800, Thomas Goirand wrote: > Since this completely blocks the apt-get install process, and makes it > impossible to install openvswitch-switch without manual intervention, I > have set this bug severity to "grave", because it fits this definition: > > "makes th

Bug#719736: openvswitch: build FTBFS "make check" failures on most archs

2013-08-27 Thread Ben Pfaff
On Thu, Aug 15, 2013 at 01:03:14PM -0700, Ben Pfaff wrote: > On Wed, Aug 14, 2013 at 11:24:38AM -0700, Ben Pfaff wrote: > > A little work with gdb shows that 501 failed due to misaligned data > > access in a "packet out" OpenFlow command in packet_set_ipv4(). It > &g

Bug#719736: openvswitch: build FTBFS "make check" failures on most archs

2013-08-15 Thread Ben Pfaff
On Wed, Aug 14, 2013 at 11:24:38AM -0700, Ben Pfaff wrote: > A little work with gdb shows that 501 failed due to misaligned data > access in a "packet out" OpenFlow command in packet_set_ipv4(). It > looks like handle_packet_out() in ofproto.c needs to properly align > th

Bug#719736: openvswitch: build FTBFS "make check" failures on most archs

2013-08-14 Thread Ben Pfaff
Package: openvswitch Version: 1.9.2+git20130813 Severity: grave I'm filing this bug to track the various "make check" failures that caused the 1.9.2+git20130813 build to fail on most archs. I need to spend some time tracking down the exact causes, and this bug report seems to be a reasonable plac

Bug#711297: Patch tested

2013-08-11 Thread Ben Pfaff
On Sun, Aug 11, 2013 at 09:38:56AM -0700, Daniel Schepler wrote: > With the patch referred to by Wookey, the autoconf build works fine for me. > (I'm reattaching the patch, with the formatting issues introduced somewhere > along the line fixed, and refreshed according to the current version of t

Bug#710125: [ovs-dev] Bug#710125: openvswitch: unable to compile kernel module with linux-3.8

2013-08-08 Thread Ben Pfaff
On Thu, Aug 08, 2013 at 09:29:18AM -0700, Jesse Gross wrote: > On Thu, Aug 8, 2013 at 9:03 AM, Ben Hutchings wrote: > > Control: severity -1 grave > > Control: tag -1 jessie sid > > > > This is obviously grave, but not for stable. > > Ben - I know that you were working on packaging 1.9. That shou

Bug#708807: pspp: diff for NMU version 0.7.9+git20120620-1.2

2013-05-26 Thread Ben Pfaff
On Sun, May 26, 2013 at 01:36:38PM +0200, Sebastian Ramacher wrote: > Control: tags -1 + patch pending > > Dear maintainer, > > I've prepared an NMU for pspp (versioned as 0.7.9+git20120620-1.2) and > uploaded it to DELAYED/1. Please feel free to tell me if I > should delay it longer. The NMU is

Bug#699263: pspp: diff for NMU version 0.7.9+git20120620-1.1

2013-02-04 Thread Ben Pfaff
On Tue, Feb 05, 2013 at 12:37:15AM +0100, Sebastian Ramacher wrote: > Dear maintainer, > > I've prepared an NMU for pspp (versioned as 0.7.9+git20120620-1.1) and > uploaded it to DELAYED/2. Please feel free to tell me if I > should delay it longer. That is the correct patch. Thank you for the N

Bug#684057: openvswitch-switch - Does not start with upstream openvswitch module

2012-08-06 Thread Ben Pfaff
severity 684057 important thanks On Mon, Aug 06, 2012 at 05:26:28PM +0200, Bastian Blank wrote: > Package: openvswitch-switch > Version: 1.4.2+git20120612-8 > Severity: grave > > The openvswitch-switch init script fails if openvswitch included in > Linux should be used. The openvswitch kernel mo

Bug#683665: Bug #683665 - Fwd: openvswitch world writable directories (CVE-2012-3449)

2012-08-02 Thread Ben Pfaff
On Thu, Aug 02, 2012 at 09:16:25PM +0200, Andreas Beckmann wrote: > Even if an ordinary local user cannot list the contents of the > directory, he may correctly derive/guess filenames (unless they are > exclusively $(mktemp)) and delete and replace files in there. > > I don't know how openvswitch-

Bug#681880: [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-30 Thread Ben Pfaff
On Fri, Jul 27, 2012 at 11:39:47PM +0200, Bastian Blank wrote: > On Fri, Jul 27, 2012 at 10:21:10AM -0700, Ben Pfaff wrote: > > On Fri, Jul 27, 2012 at 10:28:08AM +0200, Bastian Blank wrote: > > > I was unclear. You should first use realpath(3) on the database filename > >

Bug#681880: [debian 3/9] debian: Move database from /etc/openvswitch to /var/lib/openvswitch.

2012-07-30 Thread Ben Pfaff
Debian bug #681880. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- REPORTING-BUGS |2 +- debian/automake.mk |1 + debian/openvswitch-switch.dirs |1 + debian/openvswitch-switch.postinst | 15

Bug#681880: [debian 2/9] ovsdb: Make "ovsdb-tool create" work through a dangling symlink.

2012-07-30 Thread Ben Pfaff
ce, but no one should be doing "ovsdb-tool create" in parallel anyhow; O_EXCL is just an idiot check here, not required to be fail-safe. Debian bug #681880. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- ovsdb/log.c | 13 +++-- 1 files ch

Bug#681880: [debian 1/9] lockfile: Fix hang locking through a dangling symlink.

2012-07-30 Thread Ben Pfaff
commit fixes the problem by dropping O_EXCL. I don't see any reason that it's actually necessary. That means that the loop itself is unnecessary, so this commit drops that too. Debian bug #681880. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- lib/

Bug#681955: [ovs-dev] [PATCH] ovs-ctl: Start the rest of Open vSwitch if loading brcompat module fails.

2012-07-27 Thread Ben Pfaff
On Fri, Jul 27, 2012 at 09:36:04AM +0900, Simon Horman wrote: > On Thu, Jul 26, 2012 at 03:01:26PM -0700, Ben Pfaff wrote: > > This may be more useful in practice than failing the entire OVS startup > > sequence. > > Acked-by: Simon Horman Thank, I added your ack and push

Bug#681880: [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-27 Thread Ben Pfaff
On Fri, Jul 27, 2012 at 10:28:08AM +0200, Bastian Blank wrote: > On Thu, Jul 26, 2012 at 11:53:25PM -0700, Ben Pfaff wrote: > > On Fri, Jul 27, 2012 at 08:39:42AM +0200, Bastian Blank wrote: > > > I would always use the lock file at the target location of the database > > &

Bug#681880: [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-26 Thread Ben Pfaff
On Fri, Jul 27, 2012 at 08:39:42AM +0200, Bastian Blank wrote: > On Thu, Jul 26, 2012 at 02:48:51PM -0700, Ben Pfaff wrote: > > open() with O_CREAT|O_EXCL yields EEXIST if the file being opened is a > > symlink. lockfile_try_lock() interpreted that error code to mean that > &g

Bug#681880: [ovs-dev] [bug 681880 2/3] ovsdb: Make "ovsdb-tool create" work through a dangling symlink.

2012-07-26 Thread Ben Pfaff
On Fri, Jul 27, 2012 at 09:47:49AM +0900, Simon Horman wrote: > On Thu, Jul 26, 2012 at 02:48:52PM -0700, Ben Pfaff wrote: > > open() with O_CREAT|O_EXCL yields EEXIST if the name passed in is a > > symlink, but we would like "ovsdb-tool create /etc/openvswitch/conf.db&

Bug#681880: [ovs-dev] Bug#681880: [bug 681880 3/3] debian: Move database from /etc/openvswitch to /var/lib/openvswitch.

2012-07-26 Thread Ben Pfaff
On Thu, Jul 26, 2012 at 04:47:53PM -0700, Ansis Atteka wrote: > On Thu, Jul 26, 2012 at 2:48 PM, Ben Pfaff wrote: > > > Debian bug #681880. > > CC: 681...@bugs.debian.org > > Reported-by: Bastian Blank > > Signed-off-by: Ben Pfaff ... > > --- a/debian/open

Bug#681955: [PATCH] ovs-ctl: Start the rest of Open vSwitch if loading brcompat module fails.

2012-07-26 Thread Ben Pfaff
This may be more useful in practice than failing the entire OVS startup sequence. Debian bug #681955. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- utilities/ovs-ctl.in |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/utilities

Bug#681955: openvswitch-switch - Enabling brcompat breaks normal operation

2012-07-26 Thread Ben Pfaff
On Thu, Jul 26, 2012 at 10:44:58AM +0200, Bastian Blank wrote: > On Wed, Jul 18, 2012 at 06:50:10AM -0700, Ben Pfaff wrote: > > On Wed, Jul 18, 2012 at 09:57:13AM +0200, Bastian Blank wrote: > > > Enabling brcompat without openvswitch-brcompat installed or the module > >

Bug#681880: [bug 681880 3/3] debian: Move database from /etc/openvswitch to /var/lib/openvswitch.

2012-07-26 Thread Ben Pfaff
Debian bug #681880. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- REPORTING-BUGS |2 +- debian/automake.mk |1 + debian/openvswitch-switch.dirs |1 + debian/openvswitch-switch.postinst | 15

Bug#681880: [bug 681880 2/3] ovsdb: Make "ovsdb-tool create" work through a dangling symlink.

2012-07-26 Thread Ben Pfaff
ce, but no one should be doing "ovsdb-tool create" in parallel anyhow; O_EXCL is just an idiot check here, not required to be fail-safe. Debian bug #681880. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- ovsdb/log.c | 13 +++-- 1 files ch

Bug#681880: [bug 681880 1/3] lockfile: Fix hang locking through a dangling symlink.

2012-07-26 Thread Ben Pfaff
commit fixes the problem by dropping O_EXCL. I don't see any reason that it's actually necessary. That means that the loop itself is unnecessary, so this commit drops that too. Debian bug #681880. CC: 681...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- lib/

Bug#681955: openvswitch-switch - Enabling brcompat breaks normal operation

2012-07-25 Thread Ben Pfaff
On Wed, Jul 18, 2012 at 06:50:10AM -0700, Ben Pfaff wrote: > On Wed, Jul 18, 2012 at 09:57:13AM +0200, Bastian Blank wrote: > > Enabling brcompat without openvswitch-brcompat installed or the module > > available breaks the init script and normal operation without brcompat. >

Bug#681880: openvswitch-switch - Automatic changed file in /etc/

2012-07-25 Thread Ben Pfaff
On Wed, Jul 18, 2012 at 07:01:42AM -0700, Ben Pfaff wrote: > On Wed, Jul 18, 2012 at 10:00:49AM +0200, Bastian Blank wrote: > > On Tue, Jul 17, 2012 at 09:31:44AM -0700, Ben Pfaff wrote: > > > On Tue, Jul 17, 2012 at 03:20:40PM +0200, Bastian Blank wrote: > > > > ope

Bug#682384: [ovs-dev] [PATCH] Fix race condition in parallel execution of "make install".

2012-07-23 Thread Ben Pfaff
Thanks. I pushed this fix to master and branch-1.[87654]. On Mon, Jul 23, 2012 at 12:34:33PM -0700, Ethan Jackson wrote: > Looks good, thanks. > > Ethan > > On Mon, Jul 23, 2012 at 9:58 AM, Ben Pfaff wrote: > > ovs-vsctl is listed, incorrectly, in both bin_PROGRAMS and

Bug#682384: [PATCH] Fix race condition in parallel execution of "make install".

2012-07-23 Thread Ben Pfaff
&arch=s390&ver=1.4.2%2Bgit20120612-5&stamp=1342851603 Debian bug #682384. CC: 682...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- I already uploaded this to Debian as -6. I'll wait for a review before pushing it to the OVS repository. utilities/aut

Bug#682384: odd s390 build failure for openvswitch

2012-07-23 Thread Ben Pfaff
On Sun, Jul 22, 2012 at 12:07:53PM +0200, Bastian Blank wrote: > And a quick check reveals that ovs-vsctl is installed two times. It is > included in bin_PROGRAMS and bin_SCRIPTS. They are evalutated in two > different make targets and may run parallel. Thanks for spotting the problem. I uploaded

Bug#682182: ERROR: could not insert 'brcompat_mod': Exec format error when inserting brcompat_mod.ko

2012-07-20 Thread Ben Pfaff
On Fri, Jul 20, 2012 at 03:27:25PM +0800, Aron Xu wrote: > I was trying to enable Linux bridge compatibility with openvswitch in > Wheezy, and when doing modprobe something appears to be wrong: > # modprobe brcompat_mod > ERROR: could not insert 'brcompat_mod': Exec format error What does "file" o

Bug#680537: [ovs-dev] [PATCH] debian: Do not change iptables rules by default.

2012-07-18 Thread Ben Pfaff
On Mon, Jul 16, 2012 at 04:57:48PM -0700, Ben Pfaff wrote: > On Fri, Jul 13, 2012 at 02:19:10PM +0900, Simon Horman wrote: > > On Thu, Jul 12, 2012 at 09:48:34PM -0700, Ben Pfaff wrote: > > > On Fri, Jul 13, 2012 at 01:46:39PM +0900, Simon Horman wrote: > > > > On T

Bug#681880: openvswitch-switch - Automatic changed file in /etc/

2012-07-18 Thread Ben Pfaff
On Wed, Jul 18, 2012 at 10:00:49AM +0200, Bastian Blank wrote: > On Tue, Jul 17, 2012 at 09:31:44AM -0700, Ben Pfaff wrote: > > On Tue, Jul 17, 2012 at 03:20:40PM +0200, Bastian Blank wrote: > > > openvswitch uses a db called /etc/openvswitch/conf.db. This file is > > >

Bug#681955: openvswitch-switch - Enabling brcompat breaks normal operation

2012-07-18 Thread Ben Pfaff
On Wed, Jul 18, 2012 at 09:57:13AM +0200, Bastian Blank wrote: > Enabling brcompat without openvswitch-brcompat installed or the module > available breaks the init script and normal operation without brcompat. I'm surprised to hear that, because the init script contains: if test X"$BRCOMPAT"

Bug#681880: openvswitch-switch - Automatic changed file in /etc/

2012-07-17 Thread Ben Pfaff
On Tue, Jul 17, 2012 at 03:20:40PM +0200, Bastian Blank wrote: > Package: openvswitch-switch > Version: 1.4.2+git20120612-2 > Severity: serious > > openvswitch uses a db called /etc/openvswitch/conf.db. This file is > programmatic modified and not user editable. This violates §10.7 of the > policy

Bug#680537: [ovs-dev] [PATCH] debian: Do not change iptables rules by default.

2012-07-16 Thread Ben Pfaff
On Fri, Jul 13, 2012 at 02:19:10PM +0900, Simon Horman wrote: > On Thu, Jul 12, 2012 at 09:48:34PM -0700, Ben Pfaff wrote: > > On Fri, Jul 13, 2012 at 01:46:39PM +0900, Simon Horman wrote: > > > On Thu, Jul 12, 2012 at 09:17:11PM -0700, Ben Pfaff wrote: > > > > De

Bug#680537: [ovs-dev] [PATCH] debian: Do not change iptables rules by default.

2012-07-12 Thread Ben Pfaff
On Fri, Jul 13, 2012 at 01:46:39PM +0900, Simon Horman wrote: > On Thu, Jul 12, 2012 at 09:17:11PM -0700, Ben Pfaff wrote: > > Debian kernel maintainer Bastian Blank writes, at > > http://bugs.debian.org/680537: > > > >The netfilter rules are a shared resource.

Bug#680537: [PATCH] debian: Do not change iptables rules by default.

2012-07-12 Thread Ben Pfaff
. Therefore, this patch simply turns off this behavior on Debian, given that in ordinary Debian installations it will have no adverse effect on Open vSwitch. Debian bug #680537. CC: 680...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- debian/openvswitch-switch.init |2

Bug#680536: openvswitch: oops, closed the wrong bug.

2012-07-09 Thread Ben Pfaff
# The upload of openvswitch 1.4.2+git20120612-3 fixed bug #680536 # but the upload incorrectly closed #680537 instead. Fix it up. reopen 680537 notfixed 680537 1.4.2+git20120612-3 fixed 680536 1.4.2+git20120612-3 close 680536 thanks -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debi

Bug#680537: [ovs-dev] Bug#680537: [PATCH] debian: Add "netbase" dependency for /etc/protocols.

2012-07-09 Thread Ben Pfaff
Thanks, I pushed this to all relevant branches. I'll do a Debian upload for this in a bit. On Fri, Jul 06, 2012 at 03:30:29PM -0700, Justin Pettit wrote: > Looks good. Thanks. > > --Justin > > > On Jul 6, 2012, at 3:08 PM, Ben Pfaff wrote: > > > ovs-ctl.in

Bug#680537: openvswitch - Plays with iptables without consent

2012-07-06 Thread Ben Pfaff
On Fri, Jul 06, 2012 at 06:13:25PM +0200, Bastian Blank wrote: > Package: openvswitch > Version: 1.4.2+git20120612-2 > Severity: serious > > The openvswitch-switch init script modifies filter rules using iptables > without consent by the admin. Where are the requirements spelled out? I couldn't

Bug#680537: [PATCH] debian: Add "netbase" dependency for /etc/protocols.

2012-07-06 Thread Ben Pfaff
ovs-ctl.in uses /etc/protocols, which is in the "netbase" package, so a dependency is required. Debian bug #680537. CC: 680...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- AUTHORS|1 + debian/control |2 +- 2 files changed, 2 insert

Bug#663051: [ovs-dev] Bug#663051: Lots of (sometimes serious)

2012-03-21 Thread Ben Pfaff
# My upload of OVS 1.4.0+git20120321 is at least intended to fix # all the important lintian violation reported in bug #663051 with # the exceptions of ones due to shipping files that aren't actually # used in the openvswitch-datapath-dkms package. In my judgment, # this is no longer a "serious" s

Bug#663051: [ovs-dev] Bug#663051: Lots of (sometimes serious)

2012-03-21 Thread Ben Pfaff
On Tue, Mar 20, 2012 at 10:25:39PM -0700, Ben Pfaff wrote: > On Wed, Mar 21, 2012 at 02:59:01AM +0800, Thomas Goirand wrote: > > BTW, it'd be really great if you could upload the fixed version of ovs. > > FYI, It's blocking the migration of xen-api in Testing... :) > &

Bug#663051: [ovs-dev] Bug#663051: Lots of (sometimes serious)

2012-03-21 Thread Ben Pfaff
On Wed, Mar 21, 2012 at 03:47:19PM +0800, Thomas Goirand wrote: > > [resending to address that doesn't bounce for me with SPF complaint] > > Could you please provide me with the SPF error, so that I can fix it? Or > is the issue on your side? > > If you send to z...@debian.org, it works because I

Bug#663051: [ovs-dev] Bug#663051: Lots of (sometimes serious)

2012-03-20 Thread Ben Pfaff
/openvswitch-1.4.0/build-aux/check-structs > [ ... many of the same ...] > > Ben Pfaff wrote: > > These are actually not serious, because the DKMS build doesn't > > actually use any of those files. I should add a lintian override. > > Please don't do that!

Bug#663051: [ovs-dev] Bug#663051: Lots of (sometimes serious)

2012-03-20 Thread Ben Pfaff
[resending to address that doesn't bounce for me with SPF complaint] On Tue, Mar 20, 2012 at 10:25:39PM -0700, Ben Pfaff wrote: > On Wed, Mar 21, 2012 at 02:59:01AM +0800, Thomas Goirand wrote: > > I missed that one email, just saw it when reviewing the bug. > > > > On

Bug#663051: [ovs-dev] [PATCH] debian: Use a different way to avoid failing install without kernel module.

2012-03-19 Thread Ben Pfaff
On Tue, Mar 20, 2012 at 07:11:33AM +0900, Simon Horman wrote: > On Mon, Mar 19, 2012 at 11:16:32AM -0700, Ben Pfaff wrote: > > On Fri, Mar 16, 2012 at 03:49:13PM -0700, Ben Pfaff wrote: > > > On Sat, Mar 17, 2012 at 07:16:04AM +0900, Simon Horman wrote: > > > > On F

Bug#663051: [ovs-dev] [PATCH] debian: Use a different way to avoid failing install without kernel module.

2012-03-19 Thread Ben Pfaff
On Fri, Mar 16, 2012 at 03:49:13PM -0700, Ben Pfaff wrote: > On Sat, Mar 17, 2012 at 07:16:04AM +0900, Simon Horman wrote: > > On Fri, Mar 16, 2012 at 02:19:31PM -0700, Ben Pfaff wrote: > > > On Thu, Mar 15, 2012 at 09:30:24AM +0900, Simon Horman wrote: > > > > On W

Bug#663051: [ovs-dev] [PATCH] debian: Use a different way to avoid failing install without kernel module.

2012-03-19 Thread Ben Pfaff
On Sat, Mar 17, 2012 at 01:16:53PM +0800, Thomas Goirand wrote: > On 03/17/2012 06:57 AM, Simon Horman wrote: > > Understood, in that case I agree that backporting makes sense. > > I agree that "backporting" fixes makes sense. I'd like also to highlight > that you'd be backporting a forwarded-port

Bug#663051: [ovs-dev] [PATCH] debian: Use a different way to avoid failing install without kernel module.

2012-03-16 Thread Ben Pfaff
On Sat, Mar 17, 2012 at 07:16:04AM +0900, Simon Horman wrote: > On Fri, Mar 16, 2012 at 02:19:31PM -0700, Ben Pfaff wrote: > > On Thu, Mar 15, 2012 at 09:30:24AM +0900, Simon Horman wrote: > > > On Wed, Mar 14, 2012 at 02:49:08PM -0700, Ethan Jackson wrote: > > > >

Bug#663051: [ovs-dev] [PATCH] debian: Use a different way to avoid failing install without kernel module.

2012-03-16 Thread Ben Pfaff
On Thu, Mar 15, 2012 at 09:30:24AM +0900, Simon Horman wrote: > On Wed, Mar 14, 2012 at 02:49:08PM -0700, Ethan Jackson wrote: > > This looks fine to me, I don't know much about debian though. If you > > feel confident in it I'm fine with merging it. Otherwise someone else > > should look at it.

Bug#663051: [PATCH] debian: Use a different way to avoid failing install without kernel module.

2012-03-12 Thread Ben Pfaff
-off-by: Ben Pfaff --- debian/openvswitch-switch.init |7 +++ debian/openvswitch-switch.postinst | 18 ++ debian/rules |3 +-- 3 files changed, 10 insertions(+), 18 deletions(-) diff --git a/debian/openvswitch-switch.init b/debian/openvswitch

Bug#663051: [ovs-dev] Bug#663051: Bug#663051: Lots of (sometimes serious) lintian errors

2012-03-12 Thread Ben Pfaff
On Fri, Mar 09, 2012 at 03:25:01PM -0800, Ben Pfaff wrote: > On Fri, Mar 09, 2012 at 01:47:45AM +0800, Thomas Goirand wrote: > > At the end, I did work on the packaging of openvswitch. > > I converted your patches to a series of commits and sent them out to > ovs-dev for revie

Bug#663051: [ovs-dev] Bug#663051: Bug#663051: Lots of (sometimes serious) lintian errors

2012-03-09 Thread Ben Pfaff
On Fri, Mar 09, 2012 at 04:50:32AM +0800, Thomas Goirand wrote: > On 03/09/2012 03:22 AM, Ben Pfaff wrote: > > On Fri, Mar 09, 2012 at 01:47:45AM +0800, Thomas Goirand wrote: > >> As I understand, you are also upstream for openvswitch. If that is the > >> case, then be

Bug#663051: [ovs-dev] Bug#663051: Bug#663051: Lots of (sometimes serious) lintian errors

2012-03-09 Thread Ben Pfaff
On Fri, Mar 09, 2012 at 12:06:03AM +0800, Thomas Goirand wrote: > >> E: openvswitch-datapath-dkms: python-script-but-no-python-dep > >> usr/src/openvswitch-1.4.0/build-aux/check-structs > >> E: openvswitch-datapath-dkms: python-script-but-no-python-dep > >> usr/src/openvswitch-1.4.0/build-aux/ext

Bug#663051: [ovs-dev] Bug#663051: Bug#663051: Lots of (sometimes serious) lintian errors

2012-03-09 Thread Ben Pfaff
On Fri, Mar 09, 2012 at 01:47:45AM +0800, Thomas Goirand wrote: > At the end, I did work on the packaging of openvswitch. Thank you very much! I converted your patches to a series of commits and sent them out to ovs-dev for review. I believe that you are CCed on most of them, but you can view th

Bug#663051: [ovs-dev] Bug#663051: Bug#663051: Lots of (sometimes serious) lintian errors

2012-03-08 Thread Ben Pfaff
On Fri, Mar 09, 2012 at 01:47:45AM +0800, Thomas Goirand wrote: > As I understand, you are also upstream for openvswitch. If that is the > case, then best would be to have the man page fixes to be applied > upstream, and the debian/patches folder to go away. Definitely. May I have a Signed-off-by

Bug#663051: [ovs-dev] Bug#663051: Lots of (sometimes serious) lintian errors

2012-03-08 Thread Ben Pfaff
On Thu, Mar 08, 2012 at 04:27:39PM +0800, Thomas Goirand wrote: > Before uploading version 1.4.0-2+nmu1, I ran Lintian, as I always do, and > I have found out that lots of Lintian warnings and errors were not > addressed: Thanks for the list. I didn't realize there were so many lintian warnings.

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

2012-03-07 Thread Ben Pfaff
On Wed, Mar 07, 2012 at 11:26:39PM +0800, Thomas Goirand wrote: > If you agree with my patch, I can do an NMU. Is that ok? Speaking as co-maintainer, yes, your patch looks correct, please NNU. -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe".

Bug#659685: [ovs-dev] Bug#659685: It still doesn't build

2012-02-16 Thread Ben Pfaff
On Thu, Feb 16, 2012 at 10:42:27AM -0800, Ben Pfaff wrote: > On Thu, Feb 16, 2012 at 05:45:24PM +0800, Thomas Goirand wrote: > > checking for Linux build directory... /usr/src/linux-headers-3.1.0-1-686-pae > > checking for Linux source directory... configure: error: can

Bug#659685: [ovs-dev] [PATCH] configure: Try to extract kernel source directory from build Makefile.

2012-02-16 Thread Ben Pfaff
On Thu, Feb 16, 2012 at 04:32:57PM -0800, Jesse Gross wrote: > On Thu, Feb 16, 2012 at 10:36 AM, Ben Pfaff wrote: > > OVS needs to inspect the headers in the kernel source directory at build > > time.  Debian keeps moving the source directory relative to the build > >

Bug#659685: [PATCH] configure: Try to extract kernel source directory from build Makefile.

2012-02-16 Thread Ben Pfaff
On Thu, Feb 16, 2012 at 10:36:34AM -0800, Ben Pfaff wrote: > OVS needs to inspect the headers in the kernel source directory at build > time. Debian keeps moving the source directory relative to the build > directory and doesn't provide an obvious way to find the source director

Bug#659685: [ovs-dev] Bug#659685: It still doesn't build

2012-02-16 Thread Ben Pfaff
On Thu, Feb 16, 2012 at 05:45:24PM +0800, Thomas Goirand wrote: > checking for Linux build directory... /usr/src/linux-headers-3.1.0-1-686-pae > checking for Linux source directory... configure: error: cannot find source > directory (please use --with-linux-source) It seems we need a new way to f

Bug#659685: [PATCH] configure: Try to extract kernel source directory from build Makefile.

2012-02-16 Thread Ben Pfaff
ommit extracts the source directory from that line. To avoid regressions this commit retains the older heuristics as fallbacks. CC: 659...@bugs.debian.org Reported-by: Thomas Goirand Signed-off-by: Ben Pfaff --- I've tested this on squeeze with linux-headers-3.2.0-0.bpo.1-amd64. I'm instal

Bug#659685: [ovs-dev] [PATCH] debian: Use provided kernel source dir instead of host kernel version.

2012-02-14 Thread Ben Pfaff
h-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

Bug#659685: [ovs-dev] Bug#659685: Bug#659685: fails to build the kernel module

2012-02-13 Thread Ben Pfaff
On Tue, Feb 14, 2012 at 12:16:25PM +0900, Simon Horman wrote: > On Mon, Feb 13, 2012 at 05:35:15PM -0800, Ben Pfaff wrote: > > On Tue, Feb 14, 2012 at 09:31:18AM +0900, Simon Horman wrote: > > > On Mon, Feb 13, 2012 at 10:10:18AM -0800, Ben Pfaff wrote: > > > > On T

Bug#659685: [ovs-dev] Bug#659685: Bug#659685: fails to build the kernel module

2012-02-13 Thread Ben Pfaff
On Tue, Feb 14, 2012 at 09:31:18AM +0900, Simon Horman wrote: > On Mon, Feb 13, 2012 at 10:10:18AM -0800, Ben Pfaff wrote: > > On Tue, Feb 14, 2012 at 01:54:28AM +0800, Thomas Goirand wrote: > > > On 02/14/2012 01:22 AM, Ben Pfaff wrote: > > > > Would it be po

Bug#659685: [PATCH] debian: Use provided kernel source dir instead of host kernel version.

2012-02-13 Thread Ben Pfaff
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 Signed-off-by: Ben Pfaff --- debian/dkms.conf.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a

Bug#659685: [PATCH] debian: Do not run "make" if "configure" fails during DKMS build.

2012-02-13 Thread Ben Pfaff
On Tue, Feb 14, 2012 at 07:50:23AM +0800, Thomas Goirand wrote: > On 02/14/2012 07:41 AM, Thomas Goirand wrote: > > On 02/14/2012 02:09 AM, Ben Pfaff wrote: > >> -MAKE="./configure --with-linux=/lib/modules/`uname -r`/build ; make -C > >> datapath/linux" >

Bug#659685: [PATCH] debian: Do not run "make" if "configure" fails during DKMS build.

2012-02-13 Thread Ben Pfaff
On Tue, Feb 14, 2012 at 07:41:05AM +0800, Thomas Goirand wrote: > On 02/14/2012 02:09 AM, Ben Pfaff wrote: > > There is no point in trying to run "make" if "configure" failed. The > > "make" will fail too and the log will be harder to read. > I

Bug#659685: [ovs-dev] Bug#659685: Bug#659685: fails to build the kernel module

2012-02-13 Thread Ben Pfaff
On Tue, Feb 14, 2012 at 01:54:28AM +0800, Thomas Goirand wrote: > On 02/14/2012 01:22 AM, Ben Pfaff wrote: > > Would it be possible for you to provide your make.log? > > I've attached it to this mail. Here's the root of the problem: checking for Linux build directory.

Bug#659685: [ovs-dev] Bug#659685: Bug#659685: fails to build the kernel module

2012-02-13 Thread Ben Pfaff
On Mon, Feb 13, 2012 at 05:49:54PM +0900, Simon Horman wrote: > On Mon, Feb 13, 2012 at 04:10:06PM +0800, Thomas Goirand wrote: > > Package: openvswitch-datapath-dkms > > Version: 1.4.0-1 > > Severity: grave > > > > Hi there! > > > > First, thanks for maintaining OVS, this is a very nice software

Bug#651798: [ovs-dev] Bug#651798: Fails to build against Linux 3.1

2011-12-12 Thread Ben Pfaff
On Mon, Dec 12, 2011 at 08:00:25AM +, Ben Hutchings wrote: > Package: openvswitch-datapath-source > Version: 1.2.2-2 > Severity: grave > > This module fails to build against Linux 3.1: The correct solution is just to upgrade the Debian package to Open vSwitch 1.3.0, which does support Linux 3

Bug#633652: emacs on a text console - please help me overcome the shock

2011-09-08 Thread Ben Pfaff
a Linux console, if gpm is running. I haven't used it in a long time. Look around for t-mouse.el. -- Ben Pfaff http://benpfaff.org -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Bug#636816: Bug #636816: fixed upstream in 1.2.1

2011-08-23 Thread Ben Pfaff
tags 636816 patch fixed-upstream thanks A patch to fix this was committed as part of upstream 1.2.1, so this problem will be fixed when 1.2.1 is uploaded. -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debi

Bug#636815: Bug #636815: fixed in upstream Git repo

2011-08-10 Thread Ben Pfaff
---------->8-- From: Ben Pfaff Date: Wed, 10 Aug 2011 09:13:12 -0700 Subject: [PATCH] debian: Avoid installing duplicate files in ovsdbmonitor package. This is just a typo introduced in commit 57483aeda (debian: Fix bug from commit 211b05b5 "debian: Modernize use of dh_install.) tha

Bug#636816: [ovs-dev] Bug#636816: ovsdbmonitor and openvswitch-switch: error when trying to install together

2011-08-10 Thread Ben Pfaff
0-1 Thank you for the report. I sent out a fix for review: http://openvswitch.org/pipermail/dev/2011-August/010394.html Here's a copy. --8<--cut here------>8-- From: Ben Pfaff Date: Wed, 10 Aug 2011 09:13:12 -0700 Subject: [PATCH] d

Bug#630325: [ovs-dev] Bug#630325: Bug#630325: openvswitch: FTBFS on big-endian architectures

2011-06-15 Thread Ben Pfaff
On Tue, Jun 14, 2011 at 10:05:35PM -0700, Ben Pfaff wrote: > On Wed, Jun 15, 2011 at 02:04:39PM +0900, Simon Horman wrote: > > On Tue, Jun 14, 2011 at 07:56:52PM -0700, Ben Pfaff wrote: > > > On Wed, Jun 15, 2011 at 10:51:24AM +0900, Simon Horman wrote: > > > >

Bug#630325: [ovs-dev] Bug#630325: Bug#630325: openvswitch: FTBFS on big-endian architectures

2011-06-14 Thread Ben Pfaff
On Wed, Jun 15, 2011 at 02:04:39PM +0900, Simon Horman wrote: > On Tue, Jun 14, 2011 at 07:56:52PM -0700, Ben Pfaff wrote: > > On Wed, Jun 15, 2011 at 10:51:24AM +0900, Simon Horman wrote: > > > Unfortunately I'm not familiar enough with the output to determine > >

Bug#630325: [ovs-dev] Bug#630325: Bug#630325: openvswitch: FTBFS on big-endian architectures

2011-06-14 Thread Ben Pfaff
On Wed, Jun 15, 2011 at 10:51:24AM +0900, Simon Horman wrote: > Unfortunately I'm not familiar enough with the output to determine > the problem without digging much further. Perhaps you have some ideas. > > The detailed output from sparc is below. > I believe that the output is similar if not the

Bug#621986: pspp: patch for FTBFS

2011-04-14 Thread Ben Pfaff
maintainer, I agree. I applied this patch to PSPP's "stable" branch back in December. http://git.savannah.gnu.org/cgit/pspp.git/log/?h=stable Because it's the *only* commit on "stable" past the 0.6.2 release, I haven't released a 0.6.3. (But I could, if

Bug#609506: [Y2011 3/3] tests: Fix Y2011 bug in testsuite.

2011-01-10 Thread Ben Pfaff
The tests have been failing for a few days now, because the PKI expired a few days into 2011. This commit instead generates the PKI at "make check" time, which has the additional benefit of getting some test exposure for the ovs-pki program. Reported-by: Aaron M. Ucko CC: 609...@bugs.debian.org

Bug#602891: [ovs-dev] Bug#602891 closed by Simon Horman (Bug#602891: fixed in openvswitch 1.1.0~pre2-4)

2010-11-22 Thread Ben Pfaff
On Sun, Nov 21, 2010 at 02:37:31PM +, Hector Oron wrote: > Hello, > > 2010/11/16 Ben Pfaff : > > I fixed the originally reported bug in openvswitch/1.1.0~pre2-3. > > Hector Oron reopened > > the bug without any comment, so I can only speculate why, but it is >

Bug#602891: [ovs-dev] Bug#602891 closed by Simon Horman (Bug#602891: fixed in openvswitch 1.1.0~pre2-4)

2010-11-21 Thread Ben Pfaff
On Sun, Nov 21, 2010 at 6:37 AM, Hector Oron wrote: > Let's forget blames and get it fixed :) For what it's worth, I believe that I've now fixed all the buildd bugs that I know about so far. I guess updating the control file to build only on linux is one straggler. But we should be able to fix

Bug#602891: [ovs-dev] Bug#602891 closed by Simon Horman (Bug#602891: fixed in openvswitch 1.1.0~pre2-4)

2010-11-19 Thread Ben Pfaff
On Mon, Nov 15, 2010 at 11:49 PM, Sebastian Andrzej Siewior wrote: >>#602891: openvswitch: FTBFS on all arches (unix socket path name too long) > but .changes says: > >>   * Add procps to Build-Depends >>     (closes: #602891) >> > So I guess one used wrong bug number. I fixed the originally repo

Bug#602891: [buildd-tools-devel] Bug#602891: [ovs-dev] Bug#602891: openvswitch: FTBFS on all arches

2010-11-09 Thread Ben Pfaff
On Tue, Nov 09, 2010 at 09:33:33PM +, Roger Leigh wrote: > We've had this issue before. Ultimately: > - we can mitigate the restriction by making the path shorter, but > we can't make any guarantees even than since package names and > version numbers don't have any fixed limit. You still

Bug#602891: [buildd-tools-devel] Bug#602891: Bug#602891: [ovs-dev] Bug#602891: openvswitch: FTBFS on all arches

2010-11-09 Thread Ben Pfaff
reassign 602891 src:openvswitch thanks On Tue, Nov 09, 2010 at 09:39:06PM +, Roger Leigh wrote: > This isn't an sbuild issue, and we can't reliably work around it in > sbuild either, for the reasons stated above. Thank you. I agree, I'm reassigning it back to openvswitch (with this email).

Bug#602891: [ovs-dev] Bug#602891: openvswitch: FTBFS on all arches

2010-11-09 Thread Ben Pfaff
On Tue, Nov 09, 2010 at 10:30:17PM +0100, Sebastian Andrzej Siewior wrote: > * Ben Pfaff | 2010-11-09 13:21:59 [-0800]: > > >Thanks for the hint. I was able to reproduce the problem locally with > >sbuild. I'm sure that I can track it down now. > > You got the par

Bug#602891: [ovs-dev] Bug#602891: openvswitch: FTBFS on all arches

2010-11-09 Thread Ben Pfaff
On Tue, Nov 09, 2010 at 09:26:25PM +0100, Sebastian Andrzej Siewior wrote: > I just built via > | sbuild --apt-update --batch --dist=unstable --arch=powerpcspe \ > |openvswitch_1.1.0~pre2-2 > and got the same error message. On buildds it hang in the perl test case > but here it continued. I jus

Bug#602911: [ovs-dev] Bug#602911: openvswitch: FTBFS: unit tests fail

2010-11-09 Thread Ben Pfaff
reassign 602911 src:openvswitch merge 602891 602911 thanks On Tue, Nov 09, 2010 at 11:01:42AM +, Hector Oron wrote: > Your package fails to build on many architectures: Thank you for the report. Is there any way to obtain the testsuite logs? They should appear in _debian/tests/testsuite.d

  1   2   >