Re: Switch to automake *now*?
"Kristian G. Kvilekval" <[email protected]>
| Newsgroups | gmane.comp.audio.zinf.devel |
|---|---|
| Message-ID | <1048289292.23912.40.camel@merrimack> |
On Fri, 2003-03-21 at 10:42, Andreas Rottmann wrote: > "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). Thanks for the tip.. I originally looked in configure.ac but didn't see that define. I've added the LINE_END_MARKER there and removed my previous patch. > Andy -- email:[email protected] office:(805)893-4276 http://www.cs.ucsb.edu/~kris ------------------------------------------------------- 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