nbmake and chk are also good


On Sat, Apr 4, 2009 at 10:26 AM, Piotr Jasiukajtis <estseg at gmail.com> wrote:
> Hello,
>
> http://cr.opensolaris.org/~jmellors/ohac-tools-ksh88-93/usr/src/tools/scripts/nbuild.ksh.html
>
> nbuild changes look good at least for me ;)
>
> But, what about some changes I would suggest?
>
> 470 ? ? ? ? task=""
> 474 ? ? ? ? ? ? ? ? export RELEASE_BUILD=""
> 1305 ? ? ? ? whyfailed=""
> 1411 ? ? ? ? (cd $dir && $func "$@"; echo $? >$xfile) 2>&1 | $TEE $log
> 1433 ? ? ? ? ? ? ? ? (cd $dir && $pfunc $log; echo $? >$xfile) 2>&1 > $pplog
>
> btw, there are a lot of 'cd $foo; do_something'
>
> instead of
>
> 470 ? ? ? ? task=
> 474 ? ? ? ? ? ? ? ? export RELEASE_BUILD=
> 1305 ? ? ? ? whyfailed=
> 1411 ? ? ? ? (cd $dir; $func "$@"; echo $? >$xfile) 2>&1 | $TEE $log
> 1433 ? ? ? ? ? ? ? ? (cd $dir; $pfunc $log; echo $? >$xfile) 2>&1 > $pplog
>
>
> On Sat, Apr 4, 2009 at 2:52 AM, Jonathan Mellors
> <Jonathan.Mellors at sun.com> wrote:
>> Hi Folks,
>>
>> When we started Colorado on opensolaris we discovered that some of our ksh
>> scripts didn't work with ksh93, so we temporarily switched them to
>> /usr/xpg4/bin/sh. I've now updated our build tools to be compatible with
>> both ksh88 and ksh93.
>>
>> The changes are:
>> - Change "local" to "typeset"
>> - Change function declarations from "foo()" to "function foo" to ensure
>> local variables don't leak out of their functions.
>> - Fix incorrect variable declarations
>> - Change "`...`" to "$(...)"
>> - Internal ksh93 tee sometimes receives interrupts so force use of /bin/tee.
>>
>> http://cr.opensolaris.org/~jmellors/ohac-tools-ksh88-93/
>>
>> Thanks
>> Jonathan
>> _______________________________________________
>> ha-clusters-discuss mailing list
>> ha-clusters-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/ha-clusters-discuss
>>
>
>
>
> --
> Regards,
> Piotr Jasiukajtis | estibi | SCA OS0072
> http://estseg.blogspot.com
>



-- 
Regards,
Piotr Jasiukajtis | estibi | SCA OS0072
http://estseg.blogspot.com

Reply via email to