Skip to content

Commit

Permalink
fixed Rake deprecation warning
Browse files Browse the repository at this point in the history
  • Loading branch information
@nazgob
nazgob committed May 19, 2012
1 parent bdcdee2 commit 95bb2fe
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions Rakefile
View file
Original file line number Original file line Diff line number Diff line change
@@ -1,6 +1,8 @@
# NOTE! When updating this file, also update INSTALL, if necessary. # NOTE! When updating this file, also update INSTALL, if necessary.
# NOTE! Please keep your tasks grouped together. # NOTE! Please keep your tasks grouped together.


include Rake::DSL

if ENV["RUBYLIB"] if ENV["RUBYLIB"]
STDERR.puts "ERROR: Please unset RUBYLIB to build Rubinius" STDERR.puts "ERROR: Please unset RUBYLIB to build Rubinius"
exit 1 exit 1
Expand Down

4 comments on commit 95bb2fe

@dbussink
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This breaks with Rake 0.8.7. @brixen do you feel like we still want to support Rake 0.8.7 for building Rubinius?

@nazgob
Copy link
Member Author

@nazgob nazgob commented on 95bb2fe May 19, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rake 0.8.7 was released in May 2009 so it's not THAT old. We can go with conditional include using Rake::VERSION for Rake 0.9+. Sounds ok?

@evanphx
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert this commit.

@nazgob
Copy link
Member Author

@nazgob nazgob commented on 95bb2fe May 19, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Please sign in to comment.