Wireshark-dev: Re: [Wireshark-dev] Handling of generated dissectors
From: Joerg Mayer <[email protected]>
Date: Mon, 7 Oct 2013 01:00:35 +0200
On Sun, Oct 06, 2013 at 04:27:50PM -0400, Evan Huus wrote:
> On Sun, Oct 6, 2013 at 4:25 PM, Joerg Mayer <[email protected]> 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.

True. OK, looks like another solution which looks more convincing than
the ones I proposed.

Ciao
     Jörg
-- 
Joerg Mayer                                           <[email protected]>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.