Skip to content
This repository has been archived by the owner on Jun 20, 2021. It is now read-only.

Bundler fails unless sinatra is already installed. #2

Merged
merged 1 commit into from Mar 24, 2013
Merged

Bundler fails unless sinatra is already installed. #2

merged 1 commit into from Mar 24, 2013

Conversation

jwoffindin
Copy link
Contributor

The dubai.gemspec file requires dubai.rb, and this breaks unless all dependencies are
already installed. E.g. a bundle install in a vanilla install breaks with

There was a LoadError while evaluating dubai.gemspec:
cannot load such file -- sinatra/base from
blah blah ..../bundler/gems/dubai-918236eaa64b/dubai.gemspec:3:in `<main>'

Gemspec only needs version, so split it into a separate file and include that only.

Also bumped the version, 0.0.1 -> 0.0.2; not sure if that's okay.

mattt added a commit that referenced this pull request Mar 24, 2013
Bundler fails unless sinatra is already installed.
@mattt mattt merged commit e26bda2 into nomad-cli:master Mar 24, 2013
@mattt
Copy link
Contributor

mattt commented Mar 24, 2013

Thanks for the patch, @jwoffindin. It's only recently that I realized what exactly was going on with the version convention in Ruby gems. Unfortunately misleading that gem installation works locally, you know?

@jwoffindin
Copy link
Contributor Author

No problem & thanks for the great gem !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants