diff mbox

[ovs-dev] debian; Avoid installing ovs-vswitchd.conf.db manpage as "db" language.

Message ID 20170110170948.16501-1-blp@ovn.org
State Accepted
Headers show

Commit Message

Ben Pfaff Jan. 10, 2017, 5:09 p.m. UTC
Usually, when the name of a manpage has a two-letter extension, it means
that the manpage is written in the language designated by that language
code.

Reported-by: Michael Stapelberg <stapelberg@debian.org>
Reported-at: https://bugs.debian.org/850631
Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 AUTHORS.rst  | 1 +
 debian/rules | 3 +++
 2 files changed, 4 insertions(+)

Comments

Ben Pfaff March 8, 2017, 6:22 p.m. UTC | #1
Simon, would you mind reviewing this, since it's about the Debian
packaging?

On Tue, Jan 10, 2017 at 09:09:48AM -0800, Ben Pfaff wrote:
> Usually, when the name of a manpage has a two-letter extension, it means
> that the manpage is written in the language designated by that language
> code.
> 
> Reported-by: Michael Stapelberg <stapelberg@debian.org>
> Reported-at: https://bugs.debian.org/850631
> Signed-off-by: Ben Pfaff <blp@ovn.org>
> ---
>  AUTHORS.rst  | 1 +
>  debian/rules | 3 +++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/AUTHORS.rst b/AUTHORS.rst
> index add95ef..9b21bd6 100644
> --- a/AUTHORS.rst
> +++ b/AUTHORS.rst
> @@ -444,6 +444,7 @@ Michael Hu                      mhu@nicira.com
>  Michael J. Smalley              michaeljsmalley@gmail.com
>  Michael Mao                     mmao@nicira.com
>  Michael Shigorin                mike@osdn.org.ua
> +Michael Stapelberg              stapelberg@debian.org
>  Mihir Gangar                    gangarm@vmware.com
>  Mike Bursell                    mike.bursell@citrix.com
>  Mike Kruze                      mkruze@nicira.com
> diff --git a/debian/rules b/debian/rules
> index 12fb94a..62cd14b 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -96,3 +96,6 @@ override_dh_strip:
>  	dh_strip --dbg-package=openvswitch-dbg
>  
>  override_dh_usrlocal:
> +
> +override_dh_installman:
> +	dh_installman --language=C
> -- 
> 2.10.2
>
nickcooper-zhangtonghao March 9, 2017, 5:30 a.m. UTC | #2
I have tested it. It is ok now. Simon, what do you think about it?


Tested-by: nickcooper-zhangtonghao <nic@opencloud.tech>

> On Mar 9, 2017, at 2:22 AM, Ben Pfaff <blp@ovn.org> wrote:
> 
> Simon, would you mind reviewing this, since it's about the Debian
> packaging?
Ben Pfaff March 17, 2017, 8:16 p.m. UTC | #3
Thanks for testing.  I applied this to master and branch-2.7.

On Thu, Mar 09, 2017 at 01:30:55PM +0800, nickcooper-zhangtonghao wrote:
> I have tested it. It is ok now. Simon, what do you think about it?
> 
> 
> Tested-by: nickcooper-zhangtonghao <nic@opencloud.tech>
> 
> > On Mar 9, 2017, at 2:22 AM, Ben Pfaff <blp@ovn.org> wrote:
> > 
> > Simon, would you mind reviewing this, since it's about the Debian
> > packaging?
>
diff mbox

Patch

diff --git a/AUTHORS.rst b/AUTHORS.rst
index add95ef..9b21bd6 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -444,6 +444,7 @@  Michael Hu                      mhu@nicira.com
 Michael J. Smalley              michaeljsmalley@gmail.com
 Michael Mao                     mmao@nicira.com
 Michael Shigorin                mike@osdn.org.ua
+Michael Stapelberg              stapelberg@debian.org
 Mihir Gangar                    gangarm@vmware.com
 Mike Bursell                    mike.bursell@citrix.com
 Mike Kruze                      mkruze@nicira.com
diff --git a/debian/rules b/debian/rules
index 12fb94a..62cd14b 100755
--- a/debian/rules
+++ b/debian/rules
@@ -96,3 +96,6 @@  override_dh_strip:
 	dh_strip --dbg-package=openvswitch-dbg
 
 override_dh_usrlocal:
+
+override_dh_installman:
+	dh_installman --language=C