Jyri Virkki wrote: >Ritu Kamboj wrote: > > >>I have updated webrev; >> >>http://cr.opensolaris.org/~rkamboj/MySQLWebRev/ >> >> > >usr/src/cmd/mysql-5-0/Makefile.sfw: > >In the option list for ./configure, the convention is to sort them >alphabetically. Makes changes easier later. > > >usr/src/cmd/mysql-5-0/Solaris/mysql: > > 40 #set your root password here > 41 password=""; > >First, this is not a user editable file so it can't have editable content. >Aside from that, a password certainly doesn't belong there! >What's the intent? It's only used by stop. > >
When the user disables the service,it invokes mysqladmin tool to shutdown the database which needs a password. Hence if the default password (which is no password) is changed, it needs to be set here. It is recommended that database tool (client) is used to bring down the server , rather than killing the server as killing the server causes the next restart to recover from the old crash.....(which is lengthy process and can also lead to data corruption) >mysql_restart() sleeps until the process disappears. Doesn't this hang >forever if the process doesn't exit gracefully for whatever reason? >Look at /lib/svc/method/nfs-server for an alternative which seems >more reliable? > >usr/src/pkgdefs/SUNWmysql5r/prototype_com: > >The config file needs to be editable "e renamenew" instead of "f none" >(and then the CLASSES needs to have renamenew as well) > 49 f none etc/mysql/5.0/my.cnf 0644 root bin > > > > >
