Hi!

----

Can anyone review the patch in
http://cr.opensolaris.org/~gisburn/cr_6712500_nightly_use_ksh93_001/ ,
please ?

The basic changes are:
- Switch the shell interpreter from ksh88 to ksh93
- Replace "foo()" with "function foo" since this is a difference between
ksh88 and ksh93 - functions with "foo()" have a sepeate scope while
ksh93 conforms to POSIX and does "foo()" not give a seperate scope (see
http://www.opensolaris.org/os/project/shell/shellstyle/#use_ksh_style_function_syntax)
- Add shell version information to the nightly log
- "sort | uniq" ---> "sort -u"

Notes:
- The patch is the first of four patches to incrementally clean-up
"nightly.sh".
- This patch is _not_ intended to improve performace. This patch only
switches "nightly.sh" from ksh88 to ksh93 to be in sync with "bldenv.sh"
that users can actually use ksh93 statements in the "opensolaris.sh"
config file and to prepare for later cleanup steps (my fear is that this
patch will trigger a larger discussion again and a large patch may
suffer from bitrot again (at least my last two nightly cleanup patches
suffered that fate and I am not exactly willing anymore to write patches
for /dev/null)). As said I now try to split the original monster-patch
into smaller pieces to get them added to the tree faster.

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) roland.ma...@nrubsig.org
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 3992797
 (;O/ \/ \O;)
_______________________________________________
opensolaris-code mailing list
opensolaris-code@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/opensolaris-code

Reply via email to