explicitly adds dependency for benchmarkable

[mejackreed]

Closes #30

This explicitly requires 'activesupport-core-ext' as a dependency. Having issues in other projects where travis will report that Benchmarkable is not loaded.

NameError: uninitialized constant ActiveSupport::Benchmarkable

https://travis-ci.org/geoblacklight/geoblacklight/builds/45956482

[mejackreed]

cc @atz

[jcoyne]

activesupport-core-ext is not compatible with activesupport. It's an either/or sort of thing.

[mejackreed]

Ah ok.. ls there an explicit way then to include ActiveSupport::Benchmarkable ? This include is failing for me?

https://github.com/projecthydra/jettywrapper/blob/master/lib/jettywrapper.rb#L20

[jcoyne]

There's already a dependency on ActiveSupport. Try adding require 'activesupport' up on line 5? I'm surprised you see an error on line 20, because 'benchmarkable' is required on line 7. Maybe it's because you aren't using the master branch which has this commit: 92653cc

[mejackreed]

👍 @jcoyne ... thanks. That seemed to solve the problem.