[PATCH] Missing config options in gentoo init script

[Jakub Hrozek]

Hi,

the attached patch was kindly contributed by Tyler Gates. It looks good
to me, so I'll go ahead and merge it unless anyone opposes.

The tracking ticket is:
    https://fedorahosted.org/sssd/ticket/2722

[Lukas Slebodnik]

On (22/07/15 11:41), Jakub Hrozek wrote:
> Hi,
>
> the attached patch was kindly contributed by Tyler Gates. It looks good
> to me, so I'll go ahead and merge it unless anyone opposes.
>
> The tracking ticket is:
>    https://fedorahosted.org/sssd/ticket/2722
> From d91d6792013068363e7456231f25d4aa63d268bf Mon Sep 17 00:00:00 2001
> From: Tyler Gates 
> Date: Mon, 20 Jul 2015 17:14:12 -0400
> Subject: [PATCH] CONTRIB: Gentoo daemon startup options as declared in
> conf.d/sssd
>
> Resolves:
>    https://fedorahosted.org/sssd/ticket/2722
>
> src/sysv/gentoo/sssd.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/sysv/gentoo/sssd.in b/src/sysv/gentoo/sssd.in
> index 30d7559af5a8d6752634436399b5a3689734c6dd..7ced44f4b03a2076df59e1bdc99815637637dec4 100644
> --- a/src/sysv/gentoo/sssd.in
> +++ b/src/sysv/gentoo/sssd.in
> @@ -7,7 +7,7 @@ depend(){
>
> start(){
>     ebegin "Starting sssd"
> start-stop-daemon --start --exec @sbindir@/sssd -- -Df
> start-stop-daemon --start --exec @sbindir@/sssd -- -Df ${SSSD_OPTIONS}
> eend ${?}
>
> }
>
> -- 
> 2.4.3

I cannot see such patch in portage
https://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/sys-auth/sssd/files...

I would prefer if it was first in portage and then in upstream
or at least gentoo maintainer confirm this patch.

LS

[Tyler Gates]

https://bugs.gentoo.org/show_bug.cgi?id=553678

I originally submitted as a bug to gentoo and the dev asked me to try this
route first. I will see if I can get him to confirm here.

On Wed, Jul 22, 2015 at 11:20 AM, Lukas Slebodnik 
wrote:
> On (22/07/15 11:41), Jakub Hrozek wrote:
> > Hi,
> >
> > the attached patch was kindly contributed by Tyler Gates. It looks good
> > to me, so I'll go ahead and merge it unless anyone opposes.
> >
> > The tracking ticket is:
> >    https://fedorahosted.org/sssd/ticket/2722
> > From d91d6792013068363e7456231f25d4aa63d268bf Mon Sep 17 00:00:00 2001
> > From: Tyler Gates 
> > Date: Mon, 20 Jul 2015 17:14:12 -0400
> > Subject: [PATCH] CONTRIB: Gentoo daemon startup options as declared in
> > conf.d/sssd
> >
> > Resolves:
> >    https://fedorahosted.org/sssd/ticket/2722
> >
> > src/sysv/gentoo/sssd.in | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/sysv/gentoo/sssd.in b/src/sysv/gentoo/sssd.in
> > index
>
> 30d7559af5a8d6752634436399b5a3689734c6dd..7ced44f4b03a2076df59e1bdc99815637637dec4
> 100644
> > --- a/src/sysv/gentoo/sssd.in
> > +++ b/src/sysv/gentoo/sssd.in
> > @@ -7,7 +7,7 @@ depend(){
> >
> > start(){
> >     ebegin "Starting sssd"
> > start-stop-daemon --start --exec @sbindir@/sssd -- -Df
> > start-stop-daemon --start --exec @sbindir@/sssd -- -Df
>
> ${SSSD_OPTIONS}
> > eend ${?}
> >
> > }
> >
> > --
> > 2.4.3
>
> I cannot see such patch in portage
>
> https://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/sys-auth/sssd/files...
>
> I would prefer if it was first in portage and then in upstream
> or at least gentoo maintainer confirm this patch.
>
> LS

[Lukas Slebodnik]

On (22/07/15 11:59), Tyler Gates wrote:
> https://bugs.gentoo.org/show_bug.cgi?id=553678

According to BZ, it seems that Markos is fine with patch.
So we can push it.
> I originally submitted as a bug to gentoo and the dev asked me to try this
> route first. I will see if I can get him to confirm here.

Thank you very much for patch.

LS

[Jakub Hrozek]

On Wed, Jul 22, 2015 at 06:47:57PM +0200, Lukas Slebodnik wrote:
> On (22/07/15 11:59), Tyler Gates wrote:
> > https://bugs.gentoo.org/show_bug.cgi?id=553678
>
> According to BZ, it seems that Markos is fine with patch.
> So we can push it.
> > I originally submitted as a bug to gentoo and the dev asked me to try this
> > route first. I will see if I can get him to confirm here.
>
> Thank you very much for patch.
>
> LS

* master: cbe70d47d13d0da1acef83f0c81a0f2d81418ed8