Skip to content

Commit

Permalink
Simplify mkmf.rb.
Browse files Browse the repository at this point in the history
There is no need to check for CAPI version in mkmf.rb, since Rubinius already
loaded mkmf.rb from version dependent location.
  • Loading branch information
voxik committed Jun 29, 2012
1 parent d26abd0 commit ebad23a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
6 changes: 1 addition & 5 deletions lib/18/mkmf.rb
Expand Up @@ -132,11 +132,7 @@ def map_dir(dir, map = nil)

# ---------------------- Changed for Rubinius --------------------------------
if dir = ENV['RBX_CAPI_DIR']
if Rubinius.ruby18?
$topdir = "#{dir}/vm/capi/18/include"
else
$topdir = "#{dir}/vm/capi/19/include"
end
$topdir = "#{dir}/vm/capi/18/include"
else
$topdir = RbConfig::CONFIG["rubyhdrdir"]
end
Expand Down
6 changes: 1 addition & 5 deletions lib/19/mkmf.rb
Expand Up @@ -156,11 +156,7 @@ def map_dir(dir, map = nil)

# ---------------------- Changed for Rubinius --------------------------------
if dir = ENV['RBX_CAPI_DIR']
if Rubinius.ruby18?
$topdir = "#{dir}/vm/capi/18/include"
else
$topdir = "#{dir}/vm/capi/19/include"
end
$topdir = "#{dir}/vm/capi/19/include"
else
$topdir = RbConfig::CONFIG["rubyhdrdir"]
end
Expand Down

0 comments on commit ebad23a

Please sign in to comment.