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

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: Gerald Combs <gerald@xxxxxxxxxxxx>
Date: Wed, 09 Jun 2004 14:31:11 -0500
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.