[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