[mythtv] [PATCH] Win32 frontend support

Isaac Richards ijr at po.cwru.edu
Tue Jan 20 04:28:44 EST 2004


On Monday 19 January 2004 03:11 pm, Ian Caulfield wrote:
> This patch should allow mythfrontend to run under Windows using cygwin.

Very cool.  I've applied most of your patch to CVS.

> Please could people test this, both on Windows systems and to check that it
> doesn't break the build on Linux machines.

It did indeed break execution on Linux machines -- you didn't change any of 
the video output classes over to the WId stuff, so the class functions 
weren't getting called.  I fixed this when I applied things, though.

> The most significant change I've made is to move the definition of the
> gContext variable into libmyth - this is necessary in order to allow
> compilation under windows, and I don't think it should affect Linux users.
> Please correct me if I'm wrong.

You can't have it export that symbol to the libs?  I'd like to keep things the 
way they are in this respect, unless really necessary..

> Current issues:
>  - no volume control
>  - surround sound may or may not work properly

You're missing the other AddSamples function, I noticed, which means it won't 
play back mp3-using software encodes.

>  - A/V sync currently doesn't work very well
>  - not sure the themes are all working properly
>
> Rough installation instructions are below.

In addition to the gContext change, I left out the vsync parts of NVP.cpp -- I 
would really prefer that those were wrapped into vsync.cpp, so that there's 
no logic duplication in NVP.

Isaac


More information about the mythtv-dev mailing list