> I'm attaching the entire files, since that last patch was a little ugly. I'm
> gzipping them to conserve bandwidth.
I've checked those files, and most of your most recent patch (except for
the patch to "packet-diameter.c", obviously, as the message to which
this is a reply had "packet-diameter.c" in it), and most of the files
from an earlier message (except for "xmlstub.[ch]", obviously, as those
were also in the message to which I replied).
I also updated "packaging/nsis/ethereal.nsi.in" to install
"dictionary.dtd", "dictionary.xml", "mobileipv4.xml", "nasreq.xml", and
"sunping.xml" into the main installation directory for Ethereal. I
think NSIS may let us make the installation of that stuff optional; I've
left it as mandatory for now.
We might want to update "rpm/ethereal.spec" for those files as well; I'm
not sufficiently familiar with RPM to know what the right thing to do
there is. I infer, perhaps incorrectly, that we'd add
cp {file name} $RPM_BUILD_ROOT/etc
lines for each of the files, and also add
/etc/{file name}
lines to the %config subsection of the %files section, along the lines
of what's done for the "manuf" file. (I'm also inferring that %config
is a section; if not, we'd presumably add a bunch of
%config {file name}
lines.)
We may want to update various bits of documentation to reflect the new
XML-based dictionary, e.g. mentioning the files in the man pages for
Ethereal and Tethereal.