Wireshark mailing list archives
Re: Wireshark PIDL generated dissectors
From: Matthieu Patou <mat () samba org>
Date: Mon, 07 Oct 2013 00:13:03 -0700
On 10/06/2013 01:19 PM, Joerg Mayer wrote:
Sorry this is not an excuse I mean you can complain that it's hard to run pidl but if a file is autogenerated with a big fat warning at the top you don't modify it by hand. I would have understood if for the previous changes the dev have asked us and we would have remained silent that they decided to do the manual modification but here it looks more like a bit of lazyness sure doing the right fix is more complicated but in the long run it's a big win for everybody.On Sun, Oct 06, 2013 at 01:29:50AM -0700, Matthieu Patou wrote:For me there is only one pidl and it's in the samba repository. Then we have a 2 big use case: * Samba * Wireshark By the way I take the chance of this thread to underline that it's very hard to get the attention of the wireshark developpers to actually push patches (much harder than the Samba actually ...).Which bugnumber(s) are still open?By the way it would be nice if other dev in wireshark understand the term "autogenerated" because I spotted on the packet-dcerpc-frsrpc.c not less than 5 manual changes since the last time ronnie pushed the last version of the regenerated file:It works quite fine with asn2wrs generated, because that generator is in a maintained state, while it's hard to maintain the pidl dissector in Wireshark: While samba was still maintained via svn,
I used to (every few weeks or months) sync the samba changes into Wireshark, including the commit history. As I didn't have the disk space at that time to keep a copy of the samba repo, I stopped doing that. I'd like to have a current and working pidl generator in the Wireshark sources again.
As I said in another thread:"We might have some work to do to insure that the regenerating the files for wireshark is as simple as checking out the samba-tree and running the pidl command. "
Also maybe it would be nice that I can modify the page http://wiki.wireshark.org/Pidl to have a simple guide there too, this page is way too complicated and ihmo not completely accurate.
Matthieu -- Matthieu Patou Samba Team http://samba.org ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Wireshark PIDL generated dissectors mmann78 (Oct 01)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Andrew Bartlett (Oct 05)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Andrew Bartlett (Oct 05)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 06)
- Re: Wireshark PIDL generated dissectors Evan Huus (Oct 06)
- Re: Wireshark PIDL generated dissectors Joerg Mayer (Oct 06)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 07)
- Re: Wireshark PIDL generated dissectors Jeff Morriss (Oct 07)
- Re: Wireshark PIDL generated dissectors Joerg Mayer (Oct 07)
- Re: Wireshark PIDL generated dissectors Anders Broman (Oct 07)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 08)
- Re: Wireshark PIDL generated dissectors Joerg Mayer (Oct 07)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 21)
- Re: Wireshark PIDL generated dissectors mmann78 (Oct 21)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 26)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Jeff Morriss (Oct 07)
- Re: Wireshark PIDL generated dissectors Guy Harris (Oct 07)