[mythtv] Re: [mythtv-commits] mythtv commit: r7932 by nigel

[George Nassas]

On 23-Nov-05, at 1:23 AM, Nigel Pearson wrote:

>> I updated the script to do all the checkouts in one shot.
>
> 	Indeed. Looking good, but I think we can do better.
> This version will always do a svn co, and never an update.
> I suspect it should still try and update if the source dirs
> are already there. (unless there two commands are equivalent)

We're on the same wavelength, I initially tried an update but that  
command only works when you're in the actual source dir. In other  
words, you can't be in myth-svn and have update find all the lower  
directories and update them. You'd have to go into each and do separate  
updates which opens you to the small but not zero chance of mismatching  
revisions.

Sensing the current rev number and doing a co to that number is the  
safe equivalent of a multi-dir update and it's fewer lines so I  
resigned myself to the simpler approach.

> 	When I tried it last night, there were a few errors:
> ...
> svn: Failed to add directory  
> '/Volumes/MythOSXbuild/.osx-packager/src/myth-svn/mythplugins/ 
> mythdvd': object of the same name already exists

It sounds to me like you had a non-svn directory called mythdvd and  
it's refusing to clobber it but I'm not an svn expert. I dunno.

> 	No DarwinPorts here, and no known problems in configure.
>
> % sh
> sh-2.05b$ which zot
> no zot in . /Users/nigel/Bin /usr/bin /bin /usr/sbin /sbin  
> /Developer/Tools /Volumes/Backups/qt/bin /Volumes/Backups/bin
> sh-2.05b$ which which
> /usr/bin/which
>
>
> 	What problems did you encounter?

The problem is with line 325 of mythplugins/configure:

     if test x`which sdl-config` != x ; then

test expects one word on either side of the != so all that "no  
sdl-config in ..." crap gives it heartburn. Down at line 691 theres a  
which that redirects stderr to /dev/null. I assume that is to protect  
from chatty which programs but Apple's which writes to stdout. It's  
true we could change the test to pipe which's output to grep -v ^no but  
that goes against my principles on the grounds of uglyness. Still, it  
would work.

OTOH those two tests aren't very crucial so I don't see why they caused  
my build to fail. I'll remove the /sw/bin from the script's path and  
try again.

- George