On Mon, Aug 20, 2007 at 02:23:20PM +0100, Darren Salt wrote:
> > Uwe Hermann <[EMAIL PROTECTED]> writes:
> >> here's a patch which fixes the xine build on Debian GNU/kFreeBSD.
> 
> > Thanks for the patch, Uwe! I'm forwarding it to the xine developers, so
> > it can be included in the next release.
> [snip patch]
> 
> Why is only the vcd input plugin patched? Why not other files which check for
> __FreeBSD__?

Good question. For me it was enough to be able to build the Debian
package, but I guess we should fix all files...


> Would it be better to have include/config.h cause __FreeBSD__ to
> be defined if __FreeBSD_kernel__ is defined?

Yes, I think that should work, but I haven't tested anything, yet.


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org

Attachment: signature.asc
Description: Digital signature

Reply via email to