diff mbox

[2/3] libselinux: Fix line wrapping in selabel_file.5

Message ID 20180528214633.19374-2-bigon@debian.org (mailing list archive)
State Not Applicable
Headers show

Commit Message

Laurent Bigonville May 28, 2018, 9:46 p.m. UTC 
From: Laurent Bigonville <bigon@bigon.be>

Fix line wrapping with limited to 80 columns

Fix lintian error:
W: selinux-utils: manpage-has-errors-from-man usr/share/man/man5/selabel_file.5.gz 104: warning [p 2, 10.0i]: cannot adjust line

Signed-off-by: Laurent Bigonville <bigon@bigon.be>
---
 libselinux/man/man5/selabel_file.5 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Nicolas Iooss May 30, 2018, 8:38 p.m. UTC | #1 
On Mon, May 28, 2018 at 11:46 PM, Laurent Bigonville <bigon@debian.org> wrote:
> From: Laurent Bigonville <bigon@bigon.be>
>
> Fix line wrapping with limited to 80 columns
>
> Fix lintian error:
> W: selinux-utils: manpage-has-errors-from-man usr/share/man/man5/selabel_file.5.gz 104: warning [p 2, 10.0i]: cannot adjust line
>
> Signed-off-by: Laurent Bigonville <bigon@bigon.be>
> ---
>  libselinux/man/man5/selabel_file.5 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libselinux/man/man5/selabel_file.5 b/libselinux/man/man5/selabel_file.5
> index e7388242..e97bd826 100644
> --- a/libselinux/man/man5/selabel_file.5
> +++ b/libselinux/man/man5/selabel_file.5
> @@ -92,7 +92,7 @@ The optional local and distribution substitution files that perform any path ali
>  .RE
>  .sp
>  The default file context series of files are:
> -.RS
> +.RS 6
>  .I /etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts
>  .br
>  .I  /etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts.local
> --
> 2.17.0

All the manpage patches look good to me (and I give them an ack).

For information, I didn't know what the default indent value of ".RS"
[1] is and it seems to be 8. I tested running "man ./selabel_file.5"
in a 80-char-wide terminal and the line
"/etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts.subs_dist" is
no longer cut after this patch.

Thanks,
Nicolas

[1] For those who are unfamiliar with manpage code, this instruction
increases the indent by moving the left margin to the right.
diff mbox

Patch

diff --git a/libselinux/man/man5/selabel_file.5 b/libselinux/man/man5/selabel_file.5
index e7388242..e97bd826 100644
--- a/libselinux/man/man5/selabel_file.5
+++ b/libselinux/man/man5/selabel_file.5
@@ -92,7 +92,7 @@  The optional local and distribution substitution files that perform any path ali
 .RE
 .sp
 The default file context series of files are:
-.RS
+.RS 6
 .I /etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts
 .br
 .I  /etc/selinux/{SELINUXTYPE}/contexts/files/file_contexts.local