Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert tracks to use Bundler for gem resolution #5

Merged
merged 11 commits into from
Aug 27, 2011
Merged

Convert tracks to use Bundler for gem resolution #5

merged 11 commits into from
Aug 27, 2011

Conversation

mattr-
Copy link
Member

@mattr- mattr- commented Aug 20, 2011

This issue number may also refer to the older ticket #5, now #1472.

This set of changes converts tracks from using Rails 2.3.x own gem management to using Bundler for gem management. This provides two benefits in my opinion:

  1. Moves tracks closer towards converting to Rails 3, since Rails 3 has Bundler baked in and uses it by default.
  2. Removes a lot of vendored gems that the tracks team has to maintain in the tree

Add the Gemfile and Gemfile.lock files used to specify the dependencies
of the app.

The Gemfile specifies the dependencies and the Gemfile.lock is a
snapshot of the dependencies. If the Gemfile changes, then Bundler will
look at the differences between the Gemfile and the Gemfile.lock to
determine what other gems to install or upgrade
Bundler allows us to specify the exact versions of gems which eliminates
the need for vendoring gems (IMO)
The default version of actionwebservice removes wsdl_service_name. The
dejan/actionwebservice repository from git is a fork of the datanoise
repository that we were using that has the gemspec fixed to not use a
specific version of Rails.
Fixes the errors I was seeing when trying to run the unit tests
Bundler does a better job of manage dependencies and is used by default
in Rails 3 so this will move Tracks a bit closer towards a Rails 3
conversion.
Since we're not explictly requiring datanoise-actionwebservice anymore,
we need to modify this to match what we are using.
There were several places where newer versions of gems were being
pulled.  Those newer versions were breaking the functional tests and the
cucumber features.

Go back to known working versions of the gems that were causing failures
With rake 0.8.7, rake/rdoctask isn't deprecated and actually works, so
use that.
Since Bundler allows to specify exactly what versions of gems we need,
the vendored version isn't needed anymore.
Apparently this I missed this when I cleaned up the other frozen gems
@lrbalt
Copy link
Member

lrbalt commented Aug 22, 2011

is a cleanup of the config.gem lined in environment.rb possible after migrating to bundler

@mattr-
Copy link
Member Author

mattr- commented Aug 22, 2011

The Bundler docs don't mention anything about being able to remove those config.gem lines so I didn't try. I'll dig into this a little bit and find out.

@02strich
Copy link
Member

you can remove them as they are no longer needed. Furthermore you should be able to relax a some of the version constraints (also see my Gemfile for reference) and why do you need rubyforge (as it often seems outdated anyway)?

@lrbalt
Copy link
Member

lrbalt commented Aug 27, 2011

I can't find the time right now to test. I'll merge anyway and we will see :-)

lrbalt added a commit that referenced this pull request Aug 27, 2011
Convert tracks to use Bundler for gem resolution
@lrbalt lrbalt merged commit 5cbc0b2 into TracksApp:master Aug 27, 2011
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.

None yet

4 participants