Re: [PATCH 5/5] adding completion to seadmin feature

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 07, 2013 at 06:43:53PM +0100, Dominick Grift wrote:
> On Thu, 2013-11-07 at 13:23 -0200, Leonidas Da Silva Barbosa wrote:
> > Signed-off-by: Leonidas Da Silva Barbosa <leosilva@xxxxxxxxxxxxxxxxxx>
> > ---
> >  policycoreutils/sepolicy/sepolicy-bash-completion.sh | 18 +++++++++++++++++-
> >  1 file changed, 17 insertions(+), 1 deletion(-)
> > 
> > diff --git a/policycoreutils/sepolicy/sepolicy-bash-completion.sh b/policycoreutils/sepolicy/sepolicy-bash-completion.sh
> > index 779fd75..2b85ad9 100644
> > --- a/policycoreutils/sepolicy/sepolicy-bash-completion.sh
> > +++ b/policycoreutils/sepolicy/sepolicy-bash-completion.sh
> > @@ -60,6 +60,9 @@ __get_all_domain_types () {
> >  __get_all_domains () {
> >      seinfo -adomain -x 2>/dev/null | sed 's/_t$//g'
> >  }
> > +__get_all_roles () {
> > +    seinfo -r 2>/dev/null | tail -n +3
> > +}
> 
> probably better to specify the commands with full paths?
>
tottaly agree, will propose this change in the next patch/review. 

Thanks! 

--
selinux mailing list
selinux@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/selinux





[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux