[PATCH] sync samba pidl with wireshark pidl

Jeremy Allison jra at samba.org
Tue Jun 14 16:55:36 UTC 2016


On Tue, May 24, 2016 at 07:44:59PM +0200, David Disseldorp wrote:
> On Wed, 18 May 2016 17:28:40 +0200, Aurélien Aptel wrote:
> 
> > On Wed, 18 May 2016 14:03:32 +0200 Aurélien Aptel <aaptel at suse.com>
> > wrote:
> > > It also happens in the wireshark repo. Will look into it.
> 
> Thanks for fixing this.
> 
> > 
> > I've added 2 patches to the patchset.
> > 
> > #19 fixes 2 test failures in the Pidl test suite. 
> >     - commit 02cd7808 changed the error the test was expecting
> >       => update expected output in the test  
> >     - commit c76b65e changed whitespaces in the output
> >       => revert to the old output in the generator  
> > 
> > #20 is from the Wireshark repo that and fixes the generated MAPI
> > dissector (couldnt compile previously). Note: it is not generated in samba.
> 
> These two new changes look good to me. @Metze: please push if you're
> happy.

Ping metze. At first glance this looks good to me too (going to
go through it more carefully). If you don't have time please
let us know so patches don't get indefinately backed up.

Cheers,

	Jeremy.



More information about the samba-technical mailing list