Re: [tools-discuss] Quick preliminary review for http://cr.opensolaris.org/~gisburn/cr6842892_webrev_cleanup_002/ ?

[James C. McPherson]

...

  68 STDHEAD=$'<meta http-equiv="cache-control" conStent="no-cache"></meta>

I think you've got an extra "S" there :-)


I'm pretty sure that these lines won't work with /usr/bin/sed:

  2947  $SED -e '/^#/d' -e $'/^[ \t]*$/d' $REGFILE | while read LINE; do
  2978  $SED -e '/^#/d' -e $'/^[ \t]*$/d' $cf | while read LINE; do


Apart from that it looks ok to me.


James
--
Senior Kernel Software Engineer, Solaris
Sun Microsystems
http://blogs.sun.com/jmcp       http://www.jmcp.homeunix.com/blog
Kernel Conference Australia - http://au.sun.com/sunnews/events/2009/kernel
_______________________________________________
tools-discuss mailing list
tools-discuss@opensolaris.org