{"action":"created","issue":{"url":"https://api.github.com/repos/open-mpi/ompi/issues/1017","labels_url":"https://api.github.com/repos/open-mpi/ompi/issues/1017/labels{/name}","comments_url":"https://api.github.com/repos/open-mpi/ompi/issues/1017/comments","events_url":"https://api.github.com/repos/open-mpi/ompi/issues/1017/events","html_url":"https://github.com/open-mpi/ompi/pull/1017","id":111234823,"number":1017,"title":"dynamics: fix OPAL_CR_EXIT_LIBRARY()","user":{"login":"jsquyres","id":617316,"avatar_url":"https://avatars.githubusercontent.com/u/617316?v=3","gravatar_id":"","url":"https://api.github.com/users/jsquyres","html_url":"https://github.com/jsquyres","followers_url":"https://api.github.com/users/jsquyres/followers","following_url":"https://api.github.com/users/jsquyres/following{/other_user}","gists_url":"https://api.github.com/users/jsquyres/gists{/gist_id}","starred_url":"https://api.github.com/users/jsquyres/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsquyres/subscriptions","organizations_url":"https://api.github.com/users/jsquyres/orgs","repos_url":"https://api.github.com/users/jsquyres/repos","events_url":"https://api.github.com/users/jsquyres/events{/privacy}","received_events_url":"https://api.github.com/users/jsquyres/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/open-mpi/ompi/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/open-mpi/ompi/milestones/1","html_url":"https://github.com/open-mpi/ompi/milestones/Future","labels_url":"https://api.github.com/repos/open-mpi/ompi/milestones/1/labels","id":810126,"number":1,"title":"Future","description":"This milestone is a parking lot for issues / ideas that are beyond the last scheduled future release.","creator":{"login":"ompiteam","id":2165735,"avatar_url":"https://avatars.githubusercontent.com/u/2165735?v=3","gravatar_id":"","url":"https://api.github.com/users/ompiteam","html_url":"https://github.com/ompiteam","followers_url":"https://api.github.com/users/ompiteam/followers","following_url":"https://api.github.com/users/ompiteam/following{/other_user}","gists_url":"https://api.github.com/users/ompiteam/gists{/gist_id}","starred_url":"https://api.github.com/users/ompiteam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ompiteam/subscriptions","organizations_url":"https://api.github.com/users/ompiteam/orgs","repos_url":"https://api.github.com/users/ompiteam/repos","events_url":"https://api.github.com/users/ompiteam/events{/privacy}","received_events_url":"https://api.github.com/users/ompiteam/received_events","type":"User","site_admin":false},"open_issues":88,"closed_issues":22,"state":"open","created_at":"2014-10-01T15:57:40Z","updated_at":"2015-10-13T17:58:36Z","due_on":null,"closed_at":null},"comments":5,"created_at":"2015-10-13T17:58:27Z","updated_at":"2015-10-13T19:58:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/open-mpi/ompi/pulls/1017","html_url":"https://github.com/open-mpi/ompi/pull/1017","diff_url":"https://github.com/open-mpi/ompi/pull/1017.diff","patch_url":"https://github.com/open-mpi/ompi/pull/1017.patch"},"body":"Noticed that these were wrong will working on a different pull\r\nrequest.  Submit these fixes indepdent of other changes, just to keep\r\nthings separated."},"comment":{"url":"https://api.github.com/repos/open-mpi/ompi/issues/comments/147834701","html_url":"https://github.com/open-mpi/ompi/pull/1017#issuecomment-147834701","issue_url":"https://api.github.com/repos/open-mpi/ompi/issues/1017","id":147834701,"user":{"login":"rhc54","id":2440928,"avatar_url":"https://avatars.githubusercontent.com/u/2440928?v=3","gravatar_id":"","url":"https://api.github.com/users/rhc54","html_url":"https://github.com/rhc54","followers_url":"https://api.github.com/users/rhc54/followers","following_url":"https://api.github.com/users/rhc54/following{/other_user}","gists_url":"https://api.github.com/users/rhc54/gists{/gist_id}","starred_url":"https://api.github.com/users/rhc54/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhc54/subscriptions","organizations_url":"https://api.github.com/users/rhc54/orgs","repos_url":"https://api.github.com/users/rhc54/repos","events_url":"https://api.github.com/users/rhc54/events{/privacy}","received_events_url":"https://api.github.com/users/rhc54/received_events","type":"User","site_admin":false},"created_at":"2015-10-13T19:58:19Z","updated_at":"2015-10-13T19:58:19Z","body":"I honestly don't know - if we are going to restore it, we probably would have to start from scratch anyway as the overall arch has changed so much (e.g., I don't think the CR system has any way of dealing with dynamic add_procs, or fenceless operations today). This is why the last attempted stalled as it was trying to force-fit itself into the prior implementation.\r\n\r\nProbably worth asking @bosilca and @jjhursey for their opinions. I'm ambivalent - was just curious as to why this PR existed given that the macro is supporting broken functionality."}}