Amanda Waite wrote: > Looks ok but with a few comments: > > usr/src/cmd/mysql-5-1/METADATA > > - You might want to look at Norm's update to the METADATA guidelines > at http://wikis.sun.com/display/SFWNotes/METADATA They seem to be > using a single letter in the Support field now.
I'll take a look at this and correct.Thanks > > usr/src/cmd/mysql-5-1/Makefile.sfw > > - Why all of the touching of files and the sleep statements? It's hard > to see what this achieves aside from adding 20-odd seconds to the > build time. Since we are patching the mysql-5.1.37 source base ,some files gets modified like the Makefile.am,configure.in and other input files which causes an automatic rerun of the GNU autotools and which caused the nightly builds to fail with messages such as posted earlier in this forum ==== Build errors (non-DEBUG) ==== aclocal.m4:14: error: this file was generated for autoconf 2.61. aclocal.m4:14: error: this file was generated for autoconf 2.61. ==== Elapsed build time (non-DEBUG) ==== By doing these changes,we are just avoiding retrigger runs of GNU autotools. > > usr/src/pkgdefs/SUNWmysql51test/prototype_com > > - What happened to all of the tests that are being removed? Are they > no longer included in the MySQL distribution? yes some tests gets added or modified and some deprecated with every mysql version changes. > > Thanks > > Amanda > > On 14/12/2009 11:46, sunanda menon wrote: >> Hi, >> >> Please review the code changes for mysql-5.1.37 upgrade at >> http://cr.opensolaris.org/~sunandam/6857014/ >> >> Let me know your comments. >> >> Thanks Sunanda >> > -- Sunanda Menon Database Technology Group BLR03, x87098/91-80-66937098 http://blogs.sun.com/smenon
