[PATCH] sync samba pidl with wireshark pidl

Stefan Metzmacher 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>
>>> 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.

Pushed :-)

But our recent changes should be synced to wireshark
and there're also a few new commits...

Can you prepare additional patches?

Thanks!
metze
But there're new commits

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160620/9502eae6/signature.sig>


More information about the samba-technical mailing list