rdoc should be a dependency about mapknitter HOT 4 CLOSED

i think i ran into this too recently -- thanks for the catch and will pull
in the PR when you send it, thanks.

On Tue, Jul 29, 2014 at 5:20 PM, Justin Manley [email protected]
wrote:

I'm trying to get MapKnitter set up and ran into some issues. I tried to
run rake --tasks but kept getting the following error:

rake aborted!
LoadError: no such file to load -- rdoc/task
/home/justin/prog/mapknitter/Rakefile:8
/home/justin/.rvm/gems/ruby-1.8.7-head/bin/ruby_executable_hooks:15
(See full trace by running task with --trace)

I was able to fix it by installing rdoc:

gem install rdoc

rdoc should be listed as a dependency in the README file. I'll add it at
some point in the next few days and submit a pull request. I'm also looking
in to using Bundler http://bundler.io/bundle_install.html to make
MapKnitter installation easier.

Let me know if there are specific version requirements for rdoc.

—
Reply to this email directly or view it on GitHub
#14.

from mapknitter.

Bundler is good stuff. plots2 has it if you're looking for examples.

from mapknitter.

@jywarren - Notes on trying to modernize MapKnitter's dependency handling:

I've got the gem dependencies mostly set up with Bundler using:

ruby 1.8.7
rails 2.3.15
rubygems 1.8.25

There are just two warnings/ugly hacks that I'd like to fix if possible before I submit a PR:

require "recaptcha/rails"

The recaptcha package is causing problems with Bundler.

I've commented out all of the config.gem lines in config/environment.rb. When I try to start passenger, I get the following error:

Message from application: uninitialized constant Recaptcha (NameError)

(along with a long stack trace).

I can make this error go away by either uncommenting the config.gem line in config/environment.rb for the recaptcha gem, or by adding require "recaptcha/rails" at the beginning of the config/initializers/recaptcha.rb.

I'd like to eliminate all of the config.gem expressions from config/environment.rb - but it also seems to be to be bad practice to put the require statement at the top of config/initializers/recaptcha.rb. I've tried putting the require statement in other places (like at the top of app/controllers/map_controller.rb) but that doesn't work there because the initializers are called before all other code.

Adding a :require directive in the Gemfile doesn't seem to make a difference:

gem "recaptcha", "0.3.6", :require => "recaptcha/rails"

Any ideas? @btbonval? What's the best place to require "recaptcha/rails"?

Gem.source_index is deprecated

Whenever I run passenger with passenger start -p 3000, I get a warning:

App 1312 stderr: NOTE: Gem.source_index is deprecated, use Specification. It will be removed on or after 2011-11-01.
App 1312 stderr: Gem.source_index called from /home/justin/prog/mapknitter/vendor/bundle/ruby/1.8/gems/rails-2.3.15/lib/rails/gem_dependency.rb:21.
[ 2014-07-31 15:25:17.8569 1272/7fc3439bd700 Pool2/SmartSpawner.h:298 ]: Preloader for /home/justin/prog/mapknitter started on PID 1312, listening on unix:/tmp/passenger.1.0.1268/generation-0/backends/preloader.1312

It's just a warning, so the site runs just fine, but it would be nice to get rid of it if I can. Do you recognize this warning or have any idea what it is?

When I switched to ruby 2.1.2, rails 1.8.7, rubygems 2.2.2, this morphed into a full-fledged error and crashed passenger:

  Message from application: undefined method `source_index' for Gem:Module (NoMethodError)
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/rails-2.3.15/lib/rails/gem_dependency.rb:21:in `add_frozen_gem_path'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/rails-2.3.15/lib/initializer.rb:298:in `add_gem_load_paths'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/rails-2.3.15/lib/initializer.rb:132:in `process'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/rails-2.3.15/lib/initializer.rb:113:in `run'
  /home/justin/prog/mapknitter/config/environment.rb:13:in `<top (required)>'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/passenger-4.0.48/helper-scripts/classic-rails-preloader.rb:96:in `require'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/passenger-4.0.48/helper-scripts/classic-rails-preloader.rb:96:in `preload_app'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/passenger-4.0.48/helper-scripts/classic-rails-preloader.rb:184:in `<module:App>'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/passenger-4.0.48/helper-scripts/classic-rails-preloader.rb:29:in `<module:PhusionPassenger>'
  /home/justin/prog/mapknitter/vendor/bundle/ruby/2.1.0/gems/passenger-4.0.48/helper-scripts/classic-rails-preloader.rb:28:in `<main>'

Any ideas? I'd like to get rid of the warning if possible. I'd rather not downgrade rubygems as this SO answer suggests.

from mapknitter.

Believe this is in a much better state now.

from mapknitter.