Chris, I checked with Suresh Chandrashekaran from the Globalization team, and he assured me that SUNWiconv-extra is available in Nevada.
On the other hand, it seems pertinent to stay with your current webrev since (0) it is not critical to the putback (1) ?dding this dependency in will not in itself fix the iconv failures (2) we should decide which of the other iconv packages should also be added in to the dependency list. -ps Chris Zhu wrote: > > Prashant Srinivasan wrote: >> Chris, Yes, it's not a Ruby problem. Lets just put in iconv-extra >> for now - since this will help the Ruby unit tests. It still won't >> make the tests pass, but its a necessary part of the solution. >> >> About iconv-extra not being available, I just looked on a machine >> that we recently upgraded, and it's present there.( This is a >> question for the ETC folk who set up our machines. ) > Are you sure we will add iconv-extra to package dependency this time. > I checked two machines v40z-241-04.sfbay and v490-240-05.sfbay, which > are upgrade to nv107 this week, they don't have iconv-extra. > > Chris >> >> I'm not sure what conversions the other iconv packages bring in. >> Lets think about adding them in after the 1.8.7 putback. >> >> Your comment on GNU iconv is a good one - we may at some point want >> to move to using GNU Iconv, and hence avoid a whole class of issues. >> I don't think we have time for that this in putback - we need to be >> quick in order to make nv 109. >> >> thanks >> -ps >> >> >> Chris Zhu wrote: >>> Although the ruby test failed in some iconv conversion, but i don't >>> think it's a problem of ruby. It's the problem of Solaris iconv, >>> when we run iconv in command line, the same conversion failed >>> either. Add SUNWiconv-extra as a package dependency can just solve >>> part of the problem of code set conversion, and how about other >>> iconv modules such as UNWiconv-extra2 and SUNWiconv-unicode? BTW, >>> SUNWiconv-extra is not installed in nv107 by default, I wondered if >>> we can add it as package dependency for ruby. >>> It seems Gnu-iconv gives better support than the Solaris-iconv, but >>> it's not include in solaris nv yet. >>> >>> Regards >>> Chris >>> >>> Prashant Srinivasan wrote: >>>> Can we add SUNWiconv-extra as a package dependency? The following >>>> conversion doesn't work if it's not in there. >>>> >>>> require 'iconv' >>>> iconv = Iconv.new('SHIFT_JIS', 'EUC-JP') >>>> >>>> thanks >>>> -ps >>>> >>>> >>>> Chris Zhu wrote: >>>>> Hi all, >>>>> >>>>> Here's the webrev for CR6764580 & CR6788394, see >>>>> http://cr.opensolaris.org/~chriszhu/ruby187_gem131/ >>>>> "6764580 RubyGems needs updating to work with the latest versions >>>>> of Ruby on Rails" >>>>> "6788394 Upgrade Ruby to 1.8.7", >>>>> Now ruby 1.8.7 and rubygem 1.3.1 are widely used in most of the >>>>> web apps, we'd like to put both of them into webstack, please help >>>>> to review. >>>>> It depends on the rubygem upgrade to 1.3.1 which has been raised >>>>> by Amanda, following link gives more detail info: >>>>> http://cr.opensolaris.org/~tekgrrl/gems131-CR6764580-2/ >>>>> http://www.opensolaris.org/jive/thread.jspa?threadID=89892&tstart=0, >>>>> then we added "upgrade ruby to 1.8.7" into the same workspace. >>>>> >>>>> Regards >>>>> Chris >>>> >>> >> >