All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Further clarification of the usage of run_init in the manual page
@ 2011-07-19 19:51 Guido Trentalancia
  2011-07-19 20:12 ` Eric Paris
  0 siblings, 1 reply; 3+ messages in thread
From: Guido Trentalancia @ 2011-07-19 19:51 UTC (permalink / raw)
  To: selinux

It's a very minor thing really, but I believe (on the basis of an
off-list question) that the manual page for policycoreutils/run_init can
be improved by the following short patch which aims to further clarify
the intended usage of such tool and mention that it caters for one
(somewhat hidden) compile-time option.

diff -pruN selinux-09062011/policycoreutils/run_init/run_init.8 selinux-09062011-run_init-auth-manpage/policycoreutils/run_init/run_init.8
--- selinux-09062011/policycoreutils/run_init/run_init.8        2011-05-09 01:04:17.075842476 +0200
+++ selinux-09062011-run_init-auth-manpage/policycoreutils/run_init/run_init.8  2011-06-09 23:06:53.350242852 +0200
@@ -9,6 +9,8 @@ run_init \- run an init script in the pr
 .PP
 Run a init script under the proper context, which is specified in
 /etc/selinux/POLICYTYPE/contexts/initrc_context.
+It is generally used interactively as it requires either shadow or
+PAM user authentication (depending on compile-time options).
 .SH FILES
 /etc/passwd - user account information
 .br



--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Further clarification of the usage of run_init in the manual page
  2011-07-19 19:51 [PATCH] Further clarification of the usage of run_init in the manual page Guido Trentalancia
@ 2011-07-19 20:12 ` Eric Paris
  2011-07-19 20:34   ` Guido Trentalancia
  0 siblings, 1 reply; 3+ messages in thread
From: Eric Paris @ 2011-07-19 20:12 UTC (permalink / raw)
  To: Guido Trentalancia; +Cc: selinux

Patch added to my 'soon to be in fedora' tree.  I hope to slowly move
everything from my tree into the upstream tree and review from others
is done.

side note: I tend to add pam_rootok to run_init so I don't have to be
interactive.

-Eric

On Tue, Jul 19, 2011 at 3:51 PM, Guido Trentalancia
<guido@trentalancia.com> wrote:
> It's a very minor thing really, but I believe (on the basis of an
> off-list question) that the manual page for policycoreutils/run_init can
> be improved by the following short patch which aims to further clarify
> the intended usage of such tool and mention that it caters for one
> (somewhat hidden) compile-time option.
>
> diff -pruN selinux-09062011/policycoreutils/run_init/run_init.8 selinux-09062011-run_init-auth-manpage/policycoreutils/run_init/run_init.8
> --- selinux-09062011/policycoreutils/run_init/run_init.8        2011-05-09 01:04:17.075842476 +0200
> +++ selinux-09062011-run_init-auth-manpage/policycoreutils/run_init/run_init.8  2011-06-09 23:06:53.350242852 +0200
> @@ -9,6 +9,8 @@ run_init \- run an init script in the pr
>  .PP
>  Run a init script under the proper context, which is specified in
>  /etc/selinux/POLICYTYPE/contexts/initrc_context.
> +It is generally used interactively as it requires either shadow or
> +PAM user authentication (depending on compile-time options).
>  .SH FILES
>  /etc/passwd - user account information
>  .br
>
>
>
> --
> This message was distributed to subscribers of the selinux mailing list.
> If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
> the words "unsubscribe selinux" without quotes as the message.
>


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Further clarification of the usage of run_init in the manual page
  2011-07-19 20:12 ` Eric Paris
@ 2011-07-19 20:34   ` Guido Trentalancia
  0 siblings, 0 replies; 3+ messages in thread
From: Guido Trentalancia @ 2011-07-19 20:34 UTC (permalink / raw)
  To: Eric Paris; +Cc: selinux

Hi Eric,

thanks very much for your time.

On Tue, 2011-07-19 at 16:12 -0400, Eric Paris wrote:
> Patch added to my 'soon to be in fedora' tree.  I hope to slowly move
> everything from my tree into the upstream tree and review from others
> is done.
> 
> side note: I tend to add pam_rootok to run_init so I don't have to be
> interactive.

It's a nice idea/tip which could be potentially appreciated by many and
therefore I'd be quite keen to create a new patch and add a note about
it to the manual page, if it wasn't for the uncertainty of what would
happen if one day the pam_rootok module disappears, changes its name or
whatever else. At the end it's an external dependency that has nothing
to do with SELinux...

> -Eric

Regards,

Guido

> On Tue, Jul 19, 2011 at 3:51 PM, Guido Trentalancia
> <guido@trentalancia.com> wrote:
> > It's a very minor thing really, but I believe (on the basis of an
> > off-list question) that the manual page for policycoreutils/run_init can
> > be improved by the following short patch which aims to further clarify
> > the intended usage of such tool and mention that it caters for one
> > (somewhat hidden) compile-time option.
> >
> > diff -pruN selinux-09062011/policycoreutils/run_init/run_init.8 selinux-09062011-run_init-auth-manpage/policycoreutils/run_init/run_init.8
> > --- selinux-09062011/policycoreutils/run_init/run_init.8        2011-05-09 01:04:17.075842476 +0200
> > +++ selinux-09062011-run_init-auth-manpage/policycoreutils/run_init/run_init.8  2011-06-09 23:06:53.350242852 +0200
> > @@ -9,6 +9,8 @@ run_init \- run an init script in the pr
> >  .PP
> >  Run a init script under the proper context, which is specified in
> >  /etc/selinux/POLICYTYPE/contexts/initrc_context.
> > +It is generally used interactively as it requires either shadow or
> > +PAM user authentication (depending on compile-time options).
> >  .SH FILES
> >  /etc/passwd - user account information
> >  .br
> >
> >
> >
> > --
> > This message was distributed to subscribers of the selinux mailing list.
> > If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
> > the words "unsubscribe selinux" without quotes as the message.
> >
> 



--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-07-19 20:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-19 19:51 [PATCH] Further clarification of the usage of run_init in the manual page Guido Trentalancia
2011-07-19 20:12 ` Eric Paris
2011-07-19 20:34   ` Guido Trentalancia

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.