[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