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

Tim Harvey tharvey at alumni.calpoly.edu
Mon Jan 26 18:14:06 EST 2004


> -----Original Message-----
> From: mythtv-dev-bounces at mythtv.org
[mailto:mythtv-dev-bounces at mythtv.org]
> On Behalf Of Isaac Richards
> Sent: Thursday, January 22, 2004 9:37 PM
> To: Development of mythtv
> Subject: Re: [mythtv] [PATCH] abstract IMDB queries into external
scripts
> 
> On Thursday 22 January 2004 02:34 am, Tim Harvey wrote:
> > 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.
> 
> It looks essentially fine to me, though I'd like to have some feedback
in
> case
> of an error (couldn't run the script, for instance) to the user before
I
> check it in.

agreed.  I'm working on some enhancements currently and am hoping to
post a patch in the next day or so.  How do you get diff to show newly
added files so that I don't have to post them as a separate tgz?

> 
> As for using the IMDB::Movie perl module, well, it'd be nice not to
> introduce
> any more dependencies.

The page parsing is so simplistic right now I'm not bothering with this
module.  The only dependency is perl-libwww which is pretty standard. I
still haven't given it a very good look into, but I don't think it will
actually do what we need anyway.

Tim

> 
> Isaac
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev



More information about the mythtv-dev mailing list