Skip to content

Commit b639f54

Browse files
committed
[FIXUP] Final form
I think this is the route I am going to go with, and will most likely rebase these [FIXUP] and [DEBUG] commits out. It makes use of the `bundle install` that is already done by travis anyway (since we did a `before_install` instead of overwriting the `install`, this was being done when I switched the language back to `ruby`), it makes for a simpler diff to digest. Also... I think the `bundle exec rake ...` might be required...
1 parent f5dfa8a commit b639f54

File tree

2 files changed

+7
-16
lines changed

2 files changed

+7
-16
lines changed

.travis.yml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,8 @@ env:
1414
- RVM_RUBY_VERSION=2.6
1515
- RVM_RUBY_VERSION=ruby-head
1616

17-
before_install:
18-
- sudo --help
19-
- sudo -i which ruby
20-
- sudo -i which rake
21-
- bundle install
22-
- rake --trace test:docker:$RVM_RUBY_VERSION:build
17+
after_install:
18+
- bundle exec rake test:docker:$RVM_RUBY_VERSION:build
2319

2420
script:
25-
- rake --trace test:docker:$RVM_RUBY_VERSION
21+
- bundle exec rake test:docker:$RVM_RUBY_VERSION

Rakefile

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,11 @@ end
1414
# Ruby Extension
1515
# ==========================================================
1616

17-
begin
18-
require 'rake/extensiontask'
19-
Rake::ExtensionTask.new('stackprof', GEMSPEC) do |ext|
20-
ext.lib_dir = 'lib/stackprof'
21-
end
22-
task :build => :compile
23-
rescue LoadError => e
24-
# rake-compiler not installed. No problem if on CI.
25-
raise e unless ENV["TRAVIS"]
17+
require 'rake/extensiontask'
18+
Rake::ExtensionTask.new('stackprof', GEMSPEC) do |ext|
19+
ext.lib_dir = 'lib/stackprof'
2620
end
21+
task :build => :compile
2722

2823
# ==========================================================
2924
# Testing

0 commit comments

Comments
 (0)