Wireshark · Ethereal-dev: Re: [Ethereal-dev] Re: [Ethereal-cvs] cvs commit: ethereal/gtk Makefile.am

[Gerald Combs <gerald@xxxxxxxxxxxx>]

Joerg Mayer wrote:

> On Tue, Jun 08, 2004 at 10:08:13PM -0500, Gerald Combs wrote:
> 
>>  Modified files:
>>    gtk                  Makefile.am 
>>  Log:
>>  Add Automake/Autoconf Doxygen targets.
> 
> 
> and Makefile.am says:
> 
> doxygen.tmp: ../config.nmake doxygen.cfg
>         sed -e s/@VERSION@/$(VERSION)/ \
>             < doxygen.cfg > $@
> 
> Wouldn't renameing doxygen.cfg to doxygen.cfg.in and fixing the version
> by configure be a better way? The resulting file would quite probably
> be called doxygen.cfg afterwards.

Using configure to set the version would definitely be better.  There
should probably be a "--with-doxygen" flag as well, so that people can
disable Doxygen builds or point to a non-standard location for the
executable.

BTW, should the doxygen-generated files be part of the distribution?

> Also: What's the idea of adding the depenency on config.nmake to the
> unix build process?

Cut and paste error.  It's been fixed.