[PATCH 5/5] adding completion to seadmin feature

[Leonidas Da Silva Barbosa]

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 at linux.vnet.ibm.com>
> > ---
> >  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!