On Fri, Mar 20, 2015 at 01:32:05PM +0100, Jakub Hrozek wrote:
> On Fri, Mar 20, 2015 at 12:26:15PM +0100, Jakub Hrozek wrote:
> > On Fri, Mar 20, 2015 at 12:02:55PM +0100, Sumit Bose wrote:
> > > On Fri, Mar 20, 2015 at 11:48:28AM +0100, Jakub Hrozek wrote:
> > > > On Tue, Mar 17, 2015 at 04:46:22PM +0100, Sumit Bose wrote:
> > > > > On Tue, Mar 10, 2015 at 12:31:59PM +0100, Sumit Bose wrote:
> > > >
> > > > My only nitpick would be that we should either add a comment or
define a
> > > > const int in the function with a descriptive name to avoid
"magical"
> > > > constants.
> > >
> > > new version with GUID_BIN_LENGTH attached.
> > >
> > > bye,
> > > Sumit
> >
> > ACK
>
> * master: 1d93029624d708119bbf803e6647a2cbb271f001
>
> In order to give out test builds I backported the patch to 1-12:
>
https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?h=revie...
>
> The only conflict was in src/tests/cmocka/test_utils.c
Sorry to spam the list but I realized I forgot to send the CI link prior
to pushing. Here it is:
http://sssd-ci.duckdns.org/logs/job/11/28/summary.html
I also pushed the patch to sssd-1-12:
4619742836ec22edf8f9d274d928bc896c5b0883