On Tue, Apr 28, 2015 at 05:54:57PM +0200, Jakub Hrozek wrote:
On Tue, Apr 28, 2015 at 02:53:30PM +0200, Sumit Bose wrote:
> On Mon, Apr 27, 2015 at 01:30:42PM +0200, Jakub Hrozek wrote:
> > On Mon, Apr 27, 2015 at 12:24:28PM +0200, Sumit Bose wrote:
> > > Hi,
> > >
> > > this patch fixes another issue related to idviews and idoverrides found
> > > by Steeve while testing this feature.
> > >
> > > bye,
> > > Sumit
> >
> > I could reproduce this bug and confirm the patch fixes the issue. Can
> > you add a by-SID testcase to test_sysdb_get_real_name as well so that we
> > have all branches covered?
> >
> > Also, I wonder if we should split the patch into two, but I don't really
> > mind either way.
>
> I added the test but kept the single patch to make porting to older
> versions and downstream easier.
OK, thanks for the test.
ACK.
* master: f70a1adbfc30b9acc302027439fb8157e0c6ea2a
* sssd-1-12: 85287a6b897d818d279171a83aa3c8a0de66f13b