Re: Switch to automake *now*?

Andreas Rottmann <[email protected]>
Newsgroups gmane.comp.audio.zinf.devel
Message-ID <[email protected]>
"Kristian G. Kvilekval" <[email protected]> writes:

> On Wed, 2003-03-19 at 10:13, Andreas Rottmann wrote:
>> "Kristian G. Kvilekval" <[email protected]> writes:
>> 
>> > Andreas,
>> >
>> >     I was playing with the new build system and got
>> > the m3u  plugin to work.
>> >
>> > I had to concatenate the definitions in config/config.win32 to
>> > config/config.h in order to get m3u.cpp to compile.
>> >
>> config/config.h* are generated files. You should rather fix
>> configure.ac and/or m3u.cpp.
>
> The problem was platform specific DIR separators etc.
> The automake manual suggested placing some macro defs
> in a common header file.   These definition were in 
> the config.h.in before.   I saw that you had moved
> them to their config.win32..   Not wanting to 
> add another include to all the source files,
> I just concatenate the extra defines onto the config.h.in
> after you have generated it with autoheader..
>
There is already code like for things like this in configure.ac -
search for DIR_MARKER. AGAIN: config.h.in is generated by autoheader
from configure.ac and will be overwritten when configure.ac
changes. Modify configure.ac, not config.h.in. (Note that this has
changed from the old build system).

Andy
-- 
Andreas Rottmann         | Rotty@ICQ      | 118634484@ICQ | [email protected]
http://www.8ung.at/rotty | GnuPG Key: http://www.8ung.at/rotty/gpg.asc
Fingerprint              | DFB4 4EB4 78A4 5EEE 6219  F228 F92F CFC5 01FD 5B62

It's GNU/Linux dammit!


-------------------------------------------------------
This SF.net email is sponsored by:Crypto Challenge is now open! 
Get cracking and register here for some mind boggling fun and 
the chance of winning an Apple iPod:
http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.