HEADS UP: xorg 7.2 ready for testing

[Dejan Lesjak]

On Saturday 12 May 2007 00:42:09 Andrew Partan wrote:
> On Fri, May 11, 2007 at 05:11:05PM -0400, Andrew Partan wrote:
> > Tools/scripts/mergebase.sh correctly disabled local_startup but it
> > didn't correctly disable local_periodic.
>
> This turns out to be a bug in /etc/defaults/periodic.conf.
> For some reason, when called from mergebase.sh, this bit does not work:
> 	[ -r $i ] && . $i
> but this does:
> 	if [ -r $i ]; then
> 		. $i
> 	fi
>

Yet using

. /etc/defaults/periodic.conf ; source_periodic_confs ; echo ${local_periodic}

seems to do the right thing (ie. it still sources /etc/periodic.conf) while 
using "&&" fails as source_periodic_confs returns 1. I'm not sure why, but I 
guess this would do for the needs of mergebase.sh.
Thanks for digging into this!

Ah, the problem would be that since /etc/periodic.conf.local (the last file 
checked by default) is not readable, the version
  [ -r $i ] && . $i
returns 1 and if..fi version returns 0:

% sh  
# if [ -r /foo ]; then echo bar; fi; echo $?
0
# [ -r /foo ] && echo bar ; echo $?
1


Dejan