[MPlayer-dev-eng] current CVS doesn't compile :(

[Nick Kurshev]

Hello, pl!

On Mon, 7 Jan 2002 18:23:03 +0100 you wrote:

> On Mon, Jan 07, 2002 at 07:30:01PM +0300, Nick Kurshev wrote:
> 
> > > FreeBSD 4.5
> > > gcc version 2.95.3 20010315 (release) [FreeBSD]                                           
> > > 
> > > 
> > > gmake[2]: Entering directory `/root/src/main/vidix/drivers'
> > > gcc -c -O4 -march=i686 -mcpu=i686 -pipe -ffast-math -fomit-frame-pointer
> > > -D_THREAD_SAFE -D
> > > _LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -fPIC -I.
> > > -I.. -Wall -W -o
> > > radeon_vid.o radeon_vid.c
> > > radeon_vid.c:8: linux/pci_ids.h: No such file or directory
> > > radeon_vid.c: In function `vixProbe':
> > > radeon_vid.c:640: `PCI_VENDOR_ID_ATI' undeclared (first use in this
> > > function)
> > > radeon_vid.c:640: (Each undeclared identifier is reported only once
> > > radeon_vid.c:640: for each function it appears in.)
> > > 
> > On FreeBSD you should find replacement for that.
> > But it's strange that simplest commenting out linux/pci_ids.h solves this problem.
> > Maybe we should has own version of pci_isd.h?
> > (From http://pciids.sourceforge.net/)
> 
> The fix should be in the other way: #if linux/#include <linux...> is
> better than #if !freebsd/#include <linux/...
> 
> I guess bsdos/sunos/qnx/etc people will appreciate ;)
> 
Fixed for all OSes.
> -- 
> Best regards,
>   pl
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> 


Best regards! Nick