Skip to content

Remove potential self-dependency from gemspec#4

Open
brchristian wants to merge 1 commit into
spoiledmilk:masterfrom
brchristian:patch-2
Open

Remove potential self-dependency from gemspec#4
brchristian wants to merge 1 commit into
spoiledmilk:masterfrom
brchristian:patch-2

Conversation

@brchristian
Copy link
Copy Markdown

When updating my Rails 3.2.x Gemfile, I've noticed that with recent versions of Bundler, my bundle update has been failing with the following message:

Resolving dependencies...
Your Gemfile requires gems that depend on each other, creating an infinite loop.
Please remove gem 'casein' and try again.

Checking my Gemfile.lock shows that Casein3 declares itself as a dependency:

casein (3.1.11)
  authlogic (= 3.0.3)
  casein
  will_paginate (= 3.0.0)

Removing these lines in the gemspec appears to enable bundle update to succeed again.

When updating my Rails 3.2.x Gemfile, I've noticed that with recent versions of Bundler, my `bundle update` has been failing with the following message:

    Resolving dependencies...
    Your Gemfile requires gems that depend on each other, creating an infinite loop. Please remove gem 'casein' and try
again.

Checking my `Gemfile.lock` shows that Casein3 declares itself as a dependency:

    casein (3.1.11)
      authlogic (= 3.0.3)
      casein
      will_paginate (= 3.0.0)

Removing these lines in the gemspec appears to enable `bundle update` to succeed again.
@brchristian
Copy link
Copy Markdown
Author

@russellquinn Does this PR look good to you?

@russellquinn
Copy link
Copy Markdown
Contributor

Did you mean to post this to Casein v.3? I don't maintain this anymore. (Spoiled Milk do.)

Casein v.5 is here:
https://github.com/russellquinn/casein

@brchristian
Copy link
Copy Markdown
Author

Ah, sorry, I actually did mean to post to Casein 3. Cheers!

@spoiledmilk Does this PR look good to you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants