Re: [ovs-dev] [debian 2/2] ovs-ofctl: Avoid groff warning due to too-long line.

2013-08-13 Thread Alex Wang
Thanks for the answer, On Tue, Aug 13, 2013 at 9:29 AM, Ben Pfaff wrote: > "dpkg-buildpackage -b" then lintian on the .changes file that produces. > > On Mon, Aug 12, 2013 at 05:47:41PM -0700, Alex Wang wrote: > > This change looks good to me, > > > > During my run of "fakeroot debian/rules bin

Re: [ovs-dev] [debian 2/2] ovs-ofctl: Avoid groff warning due to too-long line.

2013-08-13 Thread Ben Pfaff
Thanks for the reviews, I applied these to master and branch-1.9, branch-1.10, branch-1.11. On Tue, Aug 13, 2013 at 09:29:19AM -0700, Ben Pfaff wrote: > "dpkg-buildpackage -b" then lintian on the .changes file that produces. > > On Mon, Aug 12, 2013 at 05:47:41PM -0700, Alex Wang wrote: > > This

Re: [ovs-dev] [debian 2/2] ovs-ofctl: Avoid groff warning due to too-long line.

2013-08-13 Thread Ben Pfaff
"dpkg-buildpackage -b" then lintian on the .changes file that produces. On Mon, Aug 12, 2013 at 05:47:41PM -0700, Alex Wang wrote: > This change looks good to me, > > During my run of "fakeroot debian/rules binary", there is no warning. > > Could you hint me how to reproduce it? > > > On Mon,

Re: [ovs-dev] [debian 2/2] ovs-ofctl: Avoid groff warning due to too-long line.

2013-08-12 Thread Alex Wang
This change looks good to me, During my run of "fakeroot debian/rules binary", there is no warning. Could you hint me how to reproduce it? On Mon, Aug 12, 2013 at 3:13 PM, Ben Pfaff wrote: > Avoids these warnings from groff: > > :1037: warning [p 14, 6.0i]: cannot adjust line > :1037: warning

[ovs-dev] [debian 2/2] ovs-ofctl: Avoid groff warning due to too-long line.

2013-08-12 Thread Ben Pfaff
Avoids these warnings from groff: :1037: warning [p 14, 6.0i]: cannot adjust line :1037: warning [p 14, 6.2i]: can't break line Found by lintian. Signed-off-by: Ben Pfaff --- utilities/ovs-ofctl.8.in |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-ofctl.8.i