{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3059","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3059/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3059/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3059/events","html_url":"https://github.com/ManageIQ/manageiq/pull/3059","id":85125594,"number":3059,"title":"Towards JRuby support","user":{"login":"djberg96","id":78529,"avatar_url":"https://avatars.githubusercontent.com/u/78529?v=3","gravatar_id":"","url":"https://api.github.com/users/djberg96","html_url":"https://github.com/djberg96","followers_url":"https://api.github.com/users/djberg96/followers","following_url":"https://api.github.com/users/djberg96/following{/other_user}","gists_url":"https://api.github.com/users/djberg96/gists{/gist_id}","starred_url":"https://api.github.com/users/djberg96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djberg96/subscriptions","organizations_url":"https://api.github.com/users/djberg96/orgs","repos_url":"https://api.github.com/users/djberg96/repos","events_url":"https://api.github.com/users/djberg96/events{/privacy}","received_events_url":"https://api.github.com/users/djberg96/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/gem%20changes","name":"gem changes","color":"6A92BC"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2015-06-04T14:36:57Z","updated_at":"2015-07-02T17:29:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/3059","html_url":"https://github.com/ManageIQ/manageiq/pull/3059","diff_url":"https://github.com/ManageIQ/manageiq/pull/3059.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/3059.patch"},"body":"This patch is a step towards JRuby support. Mostly this means the following Gemfile modifications:\r\n\r\n* Don't bundle psych (JRuby has it's own version)\r\n* Use puma instead of thin\r\n* Use jruby-prof instead of ruby-prof (dev)\r\n* Use activerecord-jdbcpostgresql-adapter instead of pg\r\n\r\nThere's also a problem with using the Platform gem because it uses RUBY_PLATFORM under the hood instead of RbConfig. JRuby treats the platform as \"java\", not the operating system, so it doesn't work. For now I'm leveraging FFI::Platform instead since that's already in the Gemfile as a dependency, but I could easily replace it with RbConfig instead if necessary.\r\n\r\nDoes the app work then? Eh, I can't get past \"bundle exec rake evm:db:reset\" at the moment because it runs out of heap space, but I thought this would be a worthwhile first step."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/118100194","html_url":"https://github.com/ManageIQ/manageiq/pull/3059#issuecomment-118100194","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3059","id":118100194,"user":{"login":"kbrock","id":1930,"avatar_url":"https://avatars.githubusercontent.com/u/1930?v=3","gravatar_id":"","url":"https://api.github.com/users/kbrock","html_url":"https://github.com/kbrock","followers_url":"https://api.github.com/users/kbrock/followers","following_url":"https://api.github.com/users/kbrock/following{/other_user}","gists_url":"https://api.github.com/users/kbrock/gists{/gist_id}","starred_url":"https://api.github.com/users/kbrock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kbrock/subscriptions","organizations_url":"https://api.github.com/users/kbrock/orgs","repos_url":"https://api.github.com/users/kbrock/repos","events_url":"https://api.github.com/users/kbrock/events{/privacy}","received_events_url":"https://api.github.com/users/kbrock/received_events","type":"User","site_admin":false},"created_at":"2015-07-02T17:29:48Z","updated_at":"2015-07-02T17:29:48Z","body":"Love the questions raised in this PR.\r\n\r\na) Are we planning on fixing the gems one by one so the conditionals are not needed?\r\nb) Or do we want to merge this code and then remove the conditionals one by one?"}}