[Ironruby-core] Code Review: gems

Shri Borde Shri.Borde at microsoft.com
Tue Mar 9 13:41:00 EST 2010


  tfpt review "/shelveset:gems;REDMOND\sborde"

  Comment  : 
  When installing gems from a remote server that is using MRI, the gem information that is
   sent down to the local machine is incorrect since MRI is not able to parse the platform
   string of "universal-.net" until a patch is applied to RubyGems in Gem::Platform#initialize,
   similar to the way other platforms are handled in that method. IronRuby does include
   this patch. For now, we work around the problem in Gem::Specification._load

  Adds a case for "universal-.net" to Gem::Platform#=~. This may not be strictly needed to fix the problem above, but is included for completeness

  Running word_spec was leaving behind a Word dialog box asking whether to save the Word document or not. Fixed the spec to close Word without prompting.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: gems.diff
Type: application/octet-stream
Size: 3770 bytes
Desc: gems.diff
URL: <http://rubyforge.org/pipermail/ironruby-core/attachments/20100309/4b6bba55/attachment.obj>


More information about the Ironruby-core mailing list