Sriram Natarajan wrote: > > Thanks for your comments. I have posted the updated webrev at > http://cr.opensolaris.org/~sn123202/6630059/webrev
File under "nits".. but consistency is nice, so I noticed this has "memcache-2.2.2.tgz" as the tarball instead of .tar.gz as all the other upstream sources in that directory (ok and I now just noticed xdebug also went in as tgz so another exception.. oh well I guess.) Anyway, diffs look good to me now. -- Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems
