Wireshark mailing list archives
Re: Handling of generated dissectors
From: Evan Huus <eapache () gmail com>
Date: Sun, 6 Oct 2013 16:27:50 -0400
On Sun, Oct 6, 2013 at 4:25 PM, Joerg Mayer <jmayer () loplof de> wrote:
Hello, Matthieu raised an issue that I've seen with asn2wrs generated dissectors too, just not to the extent of the pidl dissectors. To make it much harder to miss that the changes we are doing are to generated dissectors I see two approaches: - Name the files differently, e.g. prefix with gpacket- instead of packet-, so we know when we check that file in that there should be an accompanying basefile or generator patch. - Have a checkin script check for the term generated in the first few lines and annotate the commit message that a generated file was changed. Thoughts?
Ideally we wouldn't keep generated files in svn/git at all. We would keep the sources and (if necessary) the generating tool, and we would make generating them part of the build process. This slows down the initial build somewhat, but it makes it absolutely impossible for them to get out of sync. ___________________________________________________________________________ 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:
- Handling of generated dissectors Joerg Mayer (Oct 06)
- Re: Handling of generated dissectors Evan Huus (Oct 06)
- Re: Handling of generated dissectors Joerg Mayer (Oct 06)
- Re: Handling of generated dissectors Bálint Réczey (Oct 07)
- Re: Handling of generated dissectors Joerg Mayer (Oct 06)
- <Possible follow-ups>
- Re: Handling of generated dissectors mmann78 (Oct 07)
- Re: Handling of generated dissectors Ed Beroset (Oct 07)
- Re: Handling of generated dissectors Evan Huus (Oct 06)