[mythtv] [mythtv-commits] mythtv commit: r21764 - in trunk by gigem

Michael T. Dean mtdean at thirdcontact.com
Fri Sep 11 17:31:42 UTC 2009


On 09/10/2009 06:12 PM, mythtv at cvs.mythtv.org wrote:
>       Author: gigem
>         Date: 2009-09-10 22:12:06 +0000 (Thu, 10 Sep 2009)
> New Revision: 21764
>    Changeset: http://cvs.mythtv.org/trac/changeset/21764
>
> Modified:
>
>    trunk/mythplugins/cpsvndir
>    trunk/mythtv/themes/cpsvndir
>    trunk/oldthemes/cpsvndir
>    trunk/themes/cpsvndir
>
> Log:
>
> My last cpsvndir change, honest!  If this breaks anything, I give up.
> Refs #7010.

OK, David, I realize you've given up ;), but it seems we may need one 
more change.  Some of the themes (in themes) have symlinks, so the chown 
ends up erroring with:

chown: changing ownership of 
`/usr/local/share/mythtv/themes//blootubelite-wide/./trans-pl-background.png': 
No such file or directory

or similar.  Simply adding a -h (no dereference) to the chown fixes it, 
but I was afraid to commit that fix as I have no idea now idea how 
portable chown's -h is, and cpsvndir actually works as it is--its output 
is just a bit ugly when symlinks are involved.

So, basically, if anyone happens to know why we shouldn't use -h/where 
it won't work, please let me know.  If I don't hear reasons not to use 
it, I'll commit the attached patch sometime.

Thanks,
Mike
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: themes-cpsvndir-chown_link.patch
URL: <http://mythtv.org/pipermail/mythtv-dev/attachments/20090911/3f78d04e/attachment.txt>


More information about the mythtv-dev mailing list