[mythtv] [PATCH] abstract IMDB queries into external scripts

[Tim Harvey]

Any feedback on this?  I'm waiting to see if its generally liked before
I enhance the IMDB queries in the perl script.  If there are not any
complaints I think this should get into CVS so that next time it breaks
it can be fixed quicker/easier.

Tim

> -----Original Message-----
> From: mythtv-dev-bounces at mythtv.org
[mailto:mythtv-dev-bounces at mythtv.org]
> On Behalf Of Tim Harvey
> Sent: Monday, January 19, 2004 3:10 AM
> To: mythtv-dev at mythtv.org
> Subject: [mythtv] [PATCH] abstract IMDB queries into external scripts
> 
> Attached is a patch that abstracts out the IMDB queries in the
MythVideo
> plugin and supports the functionality via external scripts.  There are
3
> files:
>    Mythvideo.txt - patch for videomanager.cpp/.h and mythvideo.pro
>    Scripts.bz2 - default imdb script and readme.  Extract in the
> mythvideo directory with a 'tar -xjvf scripts.bz2' before you do a
'make
> install'.
> 
> (By the way, I create my patches using cvs -diff, but this doesn't
seem
> to deal with new files such as the scripts.  How can I take care of
this
> so that the new files get created with the patch?)
> 
> I've patched the MythVideo Makefile so that it creates a directory for
> the scripts in /usr/local/share/mythtv/mythvideo/scripts.
> 
> The actual IMDB query functionally should not have 'changed' with this
> patch, just the underlying architecture changes.  This will allow
easier
> updates to the IMDB query if say they change things 'again' and it
> breaks.  No recompiles will be needed.  In addition alternate scripts
> could be created which use other sites or different types of queries.
> 
> Please let me know what you think.  Any feedback is greatly
appreciated.
> 
> If the general consensus is that this is a good thing, I'll followup
> with some enhancements to the script and a settings menu to allow the
> choosing of available scripts etc.
> 
> Tim