chris zhu wrote: > Hi all, > The review for upgrade of ruby to p287 has been blocked for a period > of time, as we want to solve the problem of "db:migrate core dump > after install mysql gem" before upgrade. > After we found out the problem of db:migrate core dump after install > mysql gem. We raise the request for review the CR6721879 > <http://monaco.sfbay.sun.com/detail.jsp?cr=6721879> again. > > The problem of db:migrate core dump after install mysql gem is shown > as following: > 1) installed mysql gem by "gem install mysql -- > --with-mysql-dir=/usr/mysql/5.0" > 2) run the db migrate test in web20bench_ror by > "/usr/ruby/1.8/bin/rake db:migrate", ruby core dump. > > The bug is proved to the problem of install mysql gem not the problem > of ruby itself. > I test the db migrate in 3 way > 1) install mysql gem by " gem install mysql -- > --with-mysql-include=/usr/mysql/5.0/include > --with-mysql-lib=/usr/mysql/5.0/lib/mysql" > db migrate runs well, and ruby used the libmysqlclient.so in > /usr/mysql/5.0/lib/mysql which is compatible with define files in > /usr/mysql/5.0/include > 2) install mysql gem by " gem install mysql" > db migrate runs well, and ruby used the libmysqlclient.so in > /usr/sfw/lib which is compatible with define files in /usr/sfw/include > 3) install mysql gem by "gem install mysql -- > --with-mysql-dir=/usr/mysql/5.0" > db migrate core dumped, because ruby used the libmysqlclient. in > /usr/sfw/lib but using the /usr/mysql/5.0/include for mysql/mysql.h > files, the structure used in running is incompatible with what's > defined in the include file. Can't we make "gem install" to look for MySQL in /usr/mysql/5.0 ? For various unfortunate reasons, we don't deliver MySQL libraries in the common location which is visible to upstack like gem. User really doesn't have to learn this internal details to use gem. At least, a linux customer won't expect to.
- Sriram > > So we prefer not change ruby at all. The webrev is what it looks like > before. See http://cr.opensolaris.org/~chriszhu/CR6721879-webrev/ for > webrev. > This webrev looks good to me. If you would like to take this "core dump" bug separately that would be fine as well > Thanks > Chris > > > > Chris Zhu wrote: >> Hi all, >> >> Please help to review the CR6721879 >> <http://monaco.sfbay.sun.com/detail.jsp?cr=6721879> Ruby patch level update >> to p287. >> >> And the webrev is ready on >> http://cr.opensolaris.org/~chriszhu/CR6721879-webrev/ >> >> >> Thanks >> Chris >> ------------------------------------------------------------------------ >> >> _______________________________________________ >> sfwnv-discuss mailing list >> sfwnv-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss >> > > ------------------------------------------------------------------------ > > _______________________________________________ > > > webstack-discuss mailing list > webstack-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/webstack-discuss >
