[SSSD] [sssd PR#91][comment] sssctl: Fix missing declaration

2016-11-25 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/91
Title: #91: sssctl: Fix missing declaration

lslebodn commented:
"""
On (25/11/16 01:52), fidencio wrote:
>On Fri, Nov 25, 2016 at 10:22 AM, lslebodn  wrote:
>
>> On (24/11/16 15:12), fidencio wrote:
>> >Please, mention that the breakage was noticed when building SSSD on
>> FreeBSD on this patch as well.
>> >
>> The commit message has sufficient information.
>>
>
>For you, not for me. Otherwise I wouldn't have asked. :-\
>
I added following sentence to commit mesasge to make you
a litlle bit happier :-)
"For example the libc on FreeBSD does not provide WEXITSTATUS in stdlib.h"
>>
>> LS
>>
>> .
>>
>Acked-by: Fabiano Fidêncio 
>
master:
* 73c9330fa3de6912e45c1ab686d5290f143b8352

sssd-1-14:
* 161ddc1f24082c735801775802a483e96909152c

LS

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/91#issuecomment-262941656
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#91][comment] sssctl: Fix missing declaration

2016-11-25 Thread fidencio
  URL: https://github.com/SSSD/sssd/pull/91
Title: #91: sssctl: Fix missing declaration

fidencio commented:
"""
On Fri, Nov 25, 2016 at 10:22 AM, lslebodn  wrote:

> On (24/11/16 15:12), fidencio wrote:
> >Please, mention that the breakage was noticed when building SSSD on
> FreeBSD on this patch as well.
> >
> The commit message has sufficient information.
>

For you, not for me. Otherwise I wouldn't have asked. :-\


> The same situation can happen on ther unix platforms
> I do not see a reason for mentioning BSD there.
>

Okay, then.


>
> LS
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
> 
> .
>
Acked-by: Fabiano Fidêncio 

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/91#issuecomment-262919608
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#91][comment] sssctl: Fix missing declaration

2016-11-25 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/91
Title: #91: sssctl: Fix missing declaration

lslebodn commented:
"""
On (24/11/16 15:12), fidencio wrote:
>Please, mention that the breakage was noticed when building SSSD on FreeBSD on 
>this patch as well.
>
The commit message has sufficient information.
The same situation can happen on ther unix platforms
I do not see a reason for mentioning BSD there.

LS

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/91#issuecomment-262913735
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#91][comment] sssctl: Fix missing declaration

2016-11-24 Thread fidencio
  URL: https://github.com/SSSD/sssd/pull/91
Title: #91: sssctl: Fix missing declaration

fidencio commented:
"""
Please, mention that the breakage was noticed when building SSSD on FreeBSD on 
this patch as well.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/91#issuecomment-262854618
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org