[mythtv] [mythtv-commits] Ticket #7010: ./cpsvndir causes errors with checkinstall for themes
Michael T. Dean
mtdean at thirdcontact.com
Tue Sep 8 01:24:52 UTC 2009
On 09/07/2009 08:43 PM, MythTV wrote:
> #7010: ./cpsvndir causes errors with checkinstall for themes
>
>
> Comment:
>
> Replying to [ticket:7010 ryan.goat at gmail.com]:
> > sh ./cpsvndir /home/ryan/mythtv_svn/trunk/mythtv/themes/blue
> /usr/share/mythtv/themes/
> > ./cpsvndir: 28: Bad substitution
>
> What /bin/sh are you using?
David, this is just a guess (as I don't use *buntu nor dash), but I'm
pretty certain Ubuntu is using dash as its default shell.
I'm also pretty certain that the problem is all the bashisms in the
current script.
the
IFS=$'\012'
is a bashism, that should be
IFS='
'
(where you put a single newline between the ticks) as long as we have
#!/bin/sh at the top of the file.
Also, $(<command>) is a bashism that should be, i.e.:
for directory in `find "$1" -path '*/.svn' -prune -or -type d -print`; do
(though I think that one works on dash).
Unfortunately, I don't know how to do the bash regexp-type stuff:
${1##*/}
without bash. I think these are causing some problems on dash.
If you open up the file in a vim with a sh syntax file and highlighting
enabled, you'll see all the bashism's (or, at least, non-standard sh
syntax) marked in "error" color (red on mine).
Mike
More information about the mythtv-dev
mailing list