[PATCH] sync samba pidl with wireshark pidl
metze at samba.org
Mon Jun 20 06:26:13 UTC 2016
Am 14.06.2016 um 18:55 schrieb Jeremy Allison:
> 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>
>>>> 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
> 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.
But our recent changes should be synced to wireshark
and there're also a few new commits...
Can you prepare additional patches?
But there're new commits
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 836 bytes
Desc: OpenPGP digital signature
More information about the samba-technical