Hi Valery, Valery Sigalov wrote: > Hello, > > I am still waiting for feedback on my submit request. May be the better and > proper way is including it in CVN repository?
First of all, thank you for releasing this dissector, writing up the wiki pages, submitting the sample captures, etc. A lot of companies wouldn't release a dissector for their proprietary protocol - so the contribution is much appreciated. However, like Jaap, the external tpncp.dat file worries me. First of all, if you're happy for tpncp.dat to be distributed under the GPL, please could it have a copyright header at the top of it saying so? Secondly, I don't like the idea that someone has to download a file from the wiki to make it work. We don't have a lot of control over what goes on there, and this would effectively sanction whatever happened to be there. What I think would be ok would be if the dissector worked fine "out of the box", and the wiki said "if you want to dissect more recent versions of the TPNCP protocol, download this file". I'd have thought the configuration directory would make more sense than the plugins directory, too. The other maintainers may feel more strongly about this, though... I must say it's not an ideal precedent to be setting. Finally, please could you follow the advice in the developer's guide for submitting changes (http://www.wireshark.org/docs/wsdg_html/#ChSrcContribute) and submit a complete patch to bugzilla. Thanks, Richard _______________________________________________ Wireshark-dev mailing list [email protected] http://www.wireshark.org/mailman/listinfo/wireshark-dev
