[mythtv] Mac OS X patch 17

Nigel Pearson nigel at ind.tansu.com.au
Sun Jul 4 20:54:19 EDT 2004

On Wednesday 30 June 2004 02:52 am, Nigel Pearson wrote:
> V4L is not on Mac OS X, so #ifndef _WIN32 is inappropriate
> Use config.h to determine if this platform has V4L APIs

On Jul 3, 2004, at 1:50 PM, Isaac Richards wrote:
> config.h isn't generated by mythtv code, so it shouldn't be relied on.

	True, but the two alternatives are:

a) In any source that uses V4L, add
    #if defined(__linux__) || defined(__FreeBSD__)
    plus any other future platforms that people decide to port
    the backend to. I figured that using configure was more
    likely to "just work" in the future.

b) Duplicating the login of configure in settings.pro
    e.g. adding -DCONFIG_VIDEO4LINUX in linux, freebsd, netbsd


1) Most people compiling Unix source code do:
    without even thinking
    (and then read the doco that says to qmake mythtv.pro :-)

2) Anyone porting to a new platform _should_ realise that
    they must run configure before compiling

3) We could put code into libmythtv.pro to check that config.h
    has been generated (like currently in libav*.pro)

	Anyway, it sounds like you and Matt would prefer option b),
so here is another patch

-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.macosx.18
Type: application/octet-stream
Size: 1626 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20040705/22daaa23/patch.macosx.obj
-------------- next part --------------

Nigel Pearson, nigel at ind.tansu.com.au | "Reality is that which,
Telstra BI&D, Sydney, Australia       |  when you stop believing
Office: 8255 4222    Fax:  8255 3153  |  in it, doesn't go away."
Mobile: 0408 664435  Home: 9792 6998  |  Philip K. Dick - 'Valis'

More information about the mythtv-dev mailing list