[mythtv] [mythtv-commits] mythtv commit: r12180 - in trunk/mythplugins/mytharchive by paulh

[Paul Harrison]

Chris Pinkham wrote:
> * On Sun Dec 03, 2006 at 11:58:11AM +0000, mythtv at cvs.mythtv.org wrote:
>   
>>       Author: paulh
>>         Date: 2006-12-03 11:58:10 +0000 (Sun, 03 Dec 2006)
>> New Revision: 12180
>>    Changeset: http://cvs.mythtv.org/trac/changeset/12180
>>
>> Modified:
>>
>>    trunk/mythplugins/mytharchive/mytharchive/archiveutil.cpp
>>    trunk/mythplugins/mytharchive/mytharchive/archiveutil.h
>>    trunk/mythplugins/mytharchive/mytharchive/mythburnwizard.cpp
>>    trunk/mythplugins/mytharchive/mytharchive/mythburnwizard.h
>>    trunk/mythplugins/mytharchive/mytharchive/recordingselector.cpp
>>    trunk/mythplugins/mytharchive/mytharchivehelper/main.cpp
>>    trunk/mythplugins/mytharchive/mythburn/scripts/mythburn.py
>>
>> Log:
>>
>> More fixes to MythArchive to get it working again after the
>> addition of Storage Groups.
>>     
>
> I guess grepping for RecordFilePrefix and fixing the places I found
> that addressed wasn't enough. :(  Sorry about that.
>
>   
No problem.
> On another related note, can you add calls to ProgramInfo::MarkAsInUse()
> and ProgramInfo::UpdateInUseMark() inside MythArchive?  We use the
> inuseprograms table to know what recordings are in use so we can take
> appropriate action in a few places like when we schedule recording
> storage in Storage Groups.
>
> Basically, when you start using a MythTV recording file, just add this:
>
> 	pginfo->MarkAsInUse(true, "mytharchive");
>
> Then periodically (if you can, every 15 minutes or less) do this:
>
> 	pginfo->UpdateInUseMark();
>
> Then whe you're done:
>
> 	pginfo->MarkAsInUse(false);
>
>
>   
That's not going to be as easy as you would think because there are two 
separate parts to MythArchive. There is the UI and then there is the  
script that does the actual  DVD creation.  The  DVD creation part is a 
python script so there is no way to  make those ProgramInfo calls 
directly. 

> I have a note on my TODO to add these, but obviously I'd miss a few
> places. :(
>
> --
> Chris
>
>   
Paul H.