[mythtv] Re: [mythtv-commits] Ticket #857: Patch to mythrename.pl to work correctly with slave backends

Tom Lichti tom at redpepperracing.com
Thu Dec 22 20:00:50 EST 2005


Michael T. Dean wrote:
> MythTV wrote:
>
>> #857: Patch to mythrename.pl to work correctly with slave backends
>> -------------------------------------+-------------------------------------- 
>>
>> Reporter:  tom at redpepperracing.com  |       Owner:  ijr     Type:  
>> patch                    |      Status:  new Priority:  
>> minor                    |   Milestone:      Component:  
>> mythtv                   |     Version:  head
>> Severity:  low                      |  
>> -------------------------------------+-------------------------------------- 
>>
>> This small patch updates mythrename.pl to use the hostname when deciding
>> what records to process. I only use the --link option, so further 
>> testing
>> may be required.
>>  
>>
> If you check for hostname with the query, we won't get links to 
> recordings recorded on other backends, even if the file exists on the 
> filesystem (i.e. multiple backends share a recordings directory using 
> NFS).  Since mythrename.pl checks for the existence of the file before 
> creating the link, we shouldn't need to do this check.  As is, we 
> (should) only get links to recordings that are available through the 
> filesystem and recordings whose filies are not available are (should 
> be) ignored.
>
> If there's a specific problem/error you're trying to fix, more info 
> would be nice...  :)
>
> Mike
Hrm. Well, in my case, using the --link option on both my master and 
slave backends creates the same links on both, so something isn't 
working. I'll see if I can narrow it down some more. At a quick glance, 
I don't see where the check for the file existence is being done, but 
I'm no perl expert.

One other thing, both my backends use the same path (/myth/tv) but the 
files are local in both cases, not shared. Not sure if that matters.

Thanks
Tom


More information about the mythtv-dev mailing list