Hi Josh, I found id3v2 didn't work on v1.1.0. I checked source codes, macros- (in such cases as FLAC_HAVE_ID3LIB) ware moved to config.h. That is good. But the place to do include is the after of #ifdef FLAC_HAVE_ID3LIB. The result of configure script isn't reflected. And it doesn't always have id3lib with this. Move place of 'include "config.h"' to the top. I confirmed it worked good on linux. -- Daisuke Shimamura
Its file name is plugin_common/id3v2.c. I forgot to write. Sorry. On Sat, 29 Mar 2003 02:34:30 +0900 Daisuke Shimamura <Daisuke_Shimamura@nifty.com> wrote:> Hi Josh, > > I found id3v2 didn't work on v1.1.0. I checked source codes, macros- (in > such cases as FLAC_HAVE_ID3LIB) ware moved to config.h. That is good. > But the place to do include is the after of #ifdef FLAC_HAVE_ID3LIB. > The result of configure script isn't reflected. And it doesn't always > have id3lib with this. > > Move place of 'include "config.h"' to the top. I confirmed it worked > good on linux. > > -- > Daisuke Shimamura > > > > ------------------------------------------------------- > This SF.net email is sponsored by: > The Definitive IT and Networking Event. Be There! > NetWorld+Interop Las Vegas 2003 -- Register today! > http://ads.sourceforge.net/cgi-bin/redirect.pl?keyn0001en > _______________________________________________ > Flac-dev mailing list > Flac-dev@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/flac-dev-- Daisuke Shimamura
Ok, thanks, I'll look into it. Josh --- Daisuke Shimamura <Daisuke_Shimamura@nifty.com> wrote:> Its file name is plugin_common/id3v2.c. I forgot to write. > Sorry. > > On Sat, 29 Mar 2003 02:34:30 +0900 > Daisuke Shimamura <Daisuke_Shimamura@nifty.com> wrote: > > Hi Josh, > > > > I found id3v2 didn't work on v1.1.0. I checked source codes, > macros- (in > > such cases as FLAC_HAVE_ID3LIB) ware moved to config.h. That is > good. > > But the place to do include is the after of #ifdef > FLAC_HAVE_ID3LIB. > > The result of configure script isn't reflected. And it doesn't > always > > have id3lib with this. > > > > Move place of 'include "config.h"' to the top. I confirmed it > worked > > good on linux.__________________________________________________ Do you Yahoo!? Yahoo! Platinum - Watch CBS' NCAA March Madness, live on your desktop! http://platinum.yahoo.com
Thanks, I checked in the fix. Josh --- Daisuke Shimamura <Daisuke_Shimamura@nifty.com> wrote:> Its file name is plugin_common/id3v2.c. I forgot to write. > Sorry. > > On Sat, 29 Mar 2003 02:34:30 +0900 > Daisuke Shimamura <Daisuke_Shimamura@nifty.com> wrote: > > Hi Josh, > > > > I found id3v2 didn't work on v1.1.0. I checked source codes, > macros- (in > > such cases as FLAC_HAVE_ID3LIB) ware moved to config.h. That is > good. > > But the place to do include is the after of #ifdef > FLAC_HAVE_ID3LIB. > > The result of configure script isn't reflected. And it doesn't > always > > have id3lib with this. > > > > Move place of 'include "config.h"' to the top. I confirmed it > worked > > good on linux. > > > > -- > > Daisuke Shimamura > > > > -- > Daisuke Shimamura__________________________________ Do you Yahoo!? The New Yahoo! Search - Faster. Easier. Bingo. http://search.yahoo.com