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
_______________________________________________
mythtv-dev mailing list
[email protected]
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

Reply via email to