{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-tinkerpop/issues/186","labels_url":"https://api.github.com/repos/apache/incubator-tinkerpop/issues/186/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-tinkerpop/issues/186/comments","events_url":"https://api.github.com/repos/apache/incubator-tinkerpop/issues/186/events","html_url":"https://github.com/apache/incubator-tinkerpop/pull/186","id":123782728,"number":186,"title":"Osgify","user":{"login":"gdelafosse","id":4673837,"avatar_url":"https://avatars.githubusercontent.com/u/4673837?v=3","gravatar_id":"","url":"https://api.github.com/users/gdelafosse","html_url":"https://github.com/gdelafosse","followers_url":"https://api.github.com/users/gdelafosse/followers","following_url":"https://api.github.com/users/gdelafosse/following{/other_user}","gists_url":"https://api.github.com/users/gdelafosse/gists{/gist_id}","starred_url":"https://api.github.com/users/gdelafosse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gdelafosse/subscriptions","organizations_url":"https://api.github.com/users/gdelafosse/orgs","repos_url":"https://api.github.com/users/gdelafosse/repos","events_url":"https://api.github.com/users/gdelafosse/events{/privacy}","received_events_url":"https://api.github.com/users/gdelafosse/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-12-24T09:48:43Z","updated_at":"2016-01-07T14:08:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-tinkerpop/pulls/186","html_url":"https://github.com/apache/incubator-tinkerpop/pull/186","diff_url":"https://github.com/apache/incubator-tinkerpop/pull/186.diff","patch_url":"https://github.com/apache/incubator-tinkerpop/pull/186.patch"},"body":"The goal of this request is to start bundleizing tinkerpop.\r\nIn this request, I have bundleized the following modules : \r\n- gremlin-core\r\n- gremlin-groovy\r\n- tinkerpop-gremlin\r\nThey are enough to start playing with TinkerPop in an OSGI environment.\r\n\r\nTo be the less intrusive as possible, I had to create a new module called gremlin-osgi-deps which aggregate and export packages from tinkerpop dependencies which are not bundles for the moment.\r\nAs a result, the only impact my request has, is to add entries in the MANIFESTs to make those modules valid bundles.\r\n\r\nTo illustrate the usage of those bundles, I'm developping a Wisdom-Framework extension (https://github.com/gdelafosse/wisdom-tinkerpop)."},"comment":{"url":"https://api.github.com/repos/apache/incubator-tinkerpop/issues/comments/169674201","html_url":"https://github.com/apache/incubator-tinkerpop/pull/186#issuecomment-169674201","issue_url":"https://api.github.com/repos/apache/incubator-tinkerpop/issues/186","id":169674201,"user":{"login":"gdelafosse","id":4673837,"avatar_url":"https://avatars.githubusercontent.com/u/4673837?v=3","gravatar_id":"","url":"https://api.github.com/users/gdelafosse","html_url":"https://github.com/gdelafosse","followers_url":"https://api.github.com/users/gdelafosse/followers","following_url":"https://api.github.com/users/gdelafosse/following{/other_user}","gists_url":"https://api.github.com/users/gdelafosse/gists{/gist_id}","starred_url":"https://api.github.com/users/gdelafosse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gdelafosse/subscriptions","organizations_url":"https://api.github.com/users/gdelafosse/orgs","repos_url":"https://api.github.com/users/gdelafosse/repos","events_url":"https://api.github.com/users/gdelafosse/events{/privacy}","received_events_url":"https://api.github.com/users/gdelafosse/received_events","type":"User","site_admin":false},"created_at":"2016-01-07T14:08:13Z","updated_at":"2016-01-07T14:08:13Z","body":"Hi all,\r\nconcerning Grapes, it doesn't seem to use the jar MANIFEST file, so I don't think it will be disturbed by my change.\r\n\r\nConcenring the TinkerPop dependencies, I've created a pull request for each of them to turn them into OSGI bundle, so that gremlin-osgi-deps won't be required anymore.\r\nSee\r\nhttps://github.com/svenkubiak/jBCrypt/pull/4\r\nhttps://github.com/jcabi/jcabi-log/pull/69\r\nhttps://github.com/jcabi/jcabi-aspects/pull/201\r\nhttps://github.com/jcabi/jcabi-manifests/pull/29\r\nhttps://github.com/carrotsearch/hppc/pull/5\r\nhttps://github.com/javatuples/javatuples/pull/4\r\n\r\nConcerning testing, I'm implementing an it test for each impacted sub module to check that they are valid bundles installable into an OSGI container thanks to PaxExam. I will join those tests to this pull request soon.\r\n\r\nConcerning the documentation, where can I document how to install TinkerPop into an OSGI container?"}}