documentation: tzinfo needed as explicit dependencies for rails 3.2.8 #71

Merged
merged 1 commit into from Oct 31, 2012

3 participants

@froderik

activesupport needs the symbol TZInfo. tzinfo is a dependent of active record so rails server won't start without it.

@froderik froderik Added tzinfo
activesupport needs the symbol TZInfo. tzinfo is a dependent of active record so rails server won't start without it.
9809cf6
@awendt

That's just great. Gotta love the Rails guys. I once filed a bug report saying that AR has a dependency on tzinfo while activesupport didn't. The issue was fixed by moving the dependency.

They must've changed it back in some recent release.

@langalex
Owner

Any way we can add a test for this?

@froderik

Well - the pull request only involves documentation.

(I guess it would be possible to have a test that makes sure that the suggested gems for rails interop works as intended. But then we would have two lists of gems to maintain. Unless we want to insert the list into the documentation automatically. No idea how that would work.)

@langalex
Owner

Oops, sorry. I thought you had touched the Gemfile.

@langalex langalex merged commit df02e1a into langalex:master Oct 31, 2012

1 check passed

Details default The Travis build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment