[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 22:26:03 EST 2005
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.
>
>> 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... :)
>> 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... :)
Thanks!
Tom
More information about the mythtv-dev
mailing list