Thanks Ravindra,
On Fri, Jan 29, 2010 at 10:31:49AM -0800, U.V. Ravindra wrote:
> Jan Friedel wrote on Fri Jan 29 2010 07:41:02 GMT-0800 (PST):
> >
> > Hi all,
> >
> > I'd like to ask you for reviewing changes proposed as a fix for
> > "6920552 audit_record_attr validation test case should be
> > created" [1]. The proposed script extends the current Solaris
> > Audit test suite integrated into stcnv-gate by "6879767 Solaris
> > Audit test suite inception" [2] sometime last month.
> >
> > Webrev:
> > http://cr.opensolaris.org/~jf_/webrev.6920552.20100129.01/
> >
> > I'd like to set the timer to expire next Wed, Feb 3rd, 2010.
>
> The webrev looks fine to me, except that you may want to remove
> remove the text about setting WS_ROOT from the README. It's not
> an environment variable required by CTI-TET.
The WS_ROOT, IMO, is quite usable in the README context. Even
thought it's not CTI-TET related variable, it's there to
simplify and uniquely identify the work space root through out
the entire README; do you really think it should be removed?
Please, do you see some other efficient way how to specify the
path?
Thanks in advance,
/j.
