URL: https://github.com/SSSD/sssd/pull/92
Title: #92: UTIL: Fix compilation of sss_utf8 with libunistring

fidencio commented:
"""
On Fri, Nov 25, 2016 at 10:20 AM, lslebodn <notificati...@github.com> wrote:

> On (24/11/16 15:14), fidencio wrote:
> >Please, include some info in the commit message about how the breakage
> was noticed.
> >Is this part of the "Fixing SSSD build on FreeBSD"? Was this found using
> some "non-usual" configure option?
> >
> If you had read description of ticket properly
> then you would have noticed it's not BSD realted.
>

This comment was really unnecessary, at least in the way it's been written.
It's really far from being polite.
A suggestion of a similar answer would be: "Try building your code passing
'--with-unicode-lib=libunistring' to the configure."

Anyways, Acked-by: Fabiano Fidêncio <fiden...@redhat.com>


>
> >Also, the whole series about "Fixing SSSD build on FreeBSD" could have
> been sent together instead of one patch per PR.
> >
> BTW I do not see a reason for one FreeBSD patch set.
> They are all independent fixes with different root cause.
> or should we create series "Fixing SSSD with Active Directory"
> This request does not make any sense too me.
>
> LS
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/SSSD/sssd/pull/92#issuecomment-262913463>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/AAG4eqnfJADQ4VoWqbCabMf5IQJixuxJks5rBqhsgaJpZM4K7rm4>
> .
>

"""

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

Reply via email to