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

Michael T. Dean mtdean at thirdcontact.com
Thu Dec 22 22:53:50 EST 2005


Tom Lichti wrote:

> Michael T. Dean wrote:
>
>> Thanks.  BTW, it's easier for the devs if you attach the file instead 
>> of putting it in the text (which can lead to copy/paste errors of 
>> varying severity from annoying to won't-apply to 
>> applies-but-won't-compile to 
>> applies-and-compiles-but-doesn't-function-properly).  Normally, you 
>> post your, "This patch ..." message and /after/ submitting it, view 
>> the ticket and use the "Attach File" button to attach the file.
>
> I knew that, but I figured for a 1 line patch it wasn't worth it. My bad.

You may be right.  I guess now they have a choice.  :)

>>> There are a couple of other things that mythlink did that mythrename 
>>> doesn't... :)
>>>
>>> 1) It doesn't delete old links for shows that have been deleted (I 
>>> just add an rm * but it's pretty brutal)
>>
>> Are you sure?  It should remove any links (even ones it didn't create 
>> ;) in the destination directory before starting to create links and 
>> should die if it fails to remove any of them (so you should get all 
>> or nothing).  Lines 226-229 - 
>> http://svn.mythtv.org/trac/browser/trunk/mythtv/contrib/mythrename.pl?rev=8364#L225 
>> .
>>
>> BTW, it works for me.
>
> Hrm. Might be related to the fact that I added that whole 
> sub-directory thing... :)

Yeah.  Since it only checks the destination directory (and not subdirs), 
that would be the problem.  The old mythlink.sh used to "rm -rf" the 
whole destination directory, so that's why it worked.

>>> 2) I like to organize the links by making a folder for each show, 
>>> and then just have the shows listed by title and date (again, I 
>>> hacked mine, but not nicely)
>>
>> I'm doing a slightly easier thing that may work for you.  Instead of 
>> grouping shows by folder, I create several folders with different 
>> "views" of the files--each sorted differently.  I have a script 
>> (mythlink.sh) which creates the views (so I can run a single cron job 
>> to create them all).
>>
>> When changing from the old mythlink.sh to mythlink.pl, I created the 
>> script as a "functional replacement" of the "upgraded" mythlink.sh 
>> (not the 0.18.1 and below version) in case packagers/etc. wanted it.  
>> Chris decided that the script didn't really warrant inclusion in the 
>> contrib directory and I agreed, but I'm including a copy here for 
>> your viewing.  (You'll need to edit the values at the top as 
>> described in the comments.)
>
> Perfect! I'll use this and then I'll stop complaining... :)

Although I'll admit that your approach had one benefit mine didn't: you 
could list the directories to quickly see/count what different titles 
you have available.  (Didn't lose sight of the forest for the trees.)

If you know of a not-so-hackish way to do this (you mentioned you hacked 
yours), you could add a ticket.  If not, maybe someone more creative 
than I will help figure one out.  Of course, as deep in the thread as 
we've taken this, the request probably won't be seen by anyone who could 
help.  If you really want the ability, you may want to start an "any 
ideas" thread on -users...

Mike


More information about the mythtv-dev mailing list