Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

Use rbconfig's Config::CONFIG['host_os'] instead of RUBY_PLATFORM

#4477

With reference to ticket & patch #3999 Rails 2.3.5 on Windows: script/dbconsole cannot find sqlite3. And as per suggested here proper-way-to-detect-windows-platform-in-ruby/. I have replaced other occurrences of RUBY_PLATFORM with Config::CONFIG['host_os'].

See the pastie http://pastie.org/935086 for values of both.

I haven't changed the file which #3999 Rails 2.3.5 on Windows: script/dbconsole cannot find sqlite3 patch fixes, so #3999 Rails 2.3.5 on Windows: script/dbconsole cannot find sqlite3 can be applied too.

This patch is against 3.0 master branch.

Do inform me, if there are further changes needed in patch. I will update the patch.

Reported by Anil Wadghule · April 26th, 2010 @ 10:48 AM

State: resolved
Milestone: none
Assigned to: Jeremy Kemper Jeremy Kemper
Importance: none

Activity

  1. Anil Wadghule
    Anil Wadghule
    • Assigned user set to Jeremy Kemper

    Patch file is here use_rbconfig_host_os.diff as getting some problems while uploading patch via LH.

    April 26th, 2010 @ 11:07 AM

  2. Anil Wadghule
    Anil Wadghule

    Here is updated patch.

    April 26th, 2010 @ 04:29 PM

  3. Repository
    Repository
    • State changed from new to resolved

    April 26th, 2010 @ 07:13 PM

  4. Loren Segal
    Loren Segal

    This patch isn't 100% correct (though it's due to the initial implementation also being wrong). Config::CONFIG['host_os'] can also return "cygwin" under a cygwin build of Ruby. The regex does not take this into account.

    May 18th, 2010 @ 08:56 AM