{"action":"created","issue":{"url":"https://api.github.com/repos/envelope-project/laik/issues/132","repository_url":"https://api.github.com/repos/envelope-project/laik","labels_url":"https://api.github.com/repos/envelope-project/laik/issues/132/labels{/name}","comments_url":"https://api.github.com/repos/envelope-project/laik/issues/132/comments","events_url":"https://api.github.com/repos/envelope-project/laik/issues/132/events","html_url":"https://github.com/envelope-project/laik/pull/132","id":308981230,"number":132,"title":" Dockerfile: Stop installing the \"openmpi-bin\" package manually.","user":{"login":"AlexanderKurtz","id":8996356,"avatar_url":"https://avatars2.githubusercontent.com/u/8996356?v=4","gravatar_id":"","url":"https://api.github.com/users/AlexanderKurtz","html_url":"https://github.com/AlexanderKurtz","followers_url":"https://api.github.com/users/AlexanderKurtz/followers","following_url":"https://api.github.com/users/AlexanderKurtz/following{/other_user}","gists_url":"https://api.github.com/users/AlexanderKurtz/gists{/gist_id}","starred_url":"https://api.github.com/users/AlexanderKurtz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AlexanderKurtz/subscriptions","organizations_url":"https://api.github.com/users/AlexanderKurtz/orgs","repos_url":"https://api.github.com/users/AlexanderKurtz/repos","events_url":"https://api.github.com/users/AlexanderKurtz/events{/privacy}","received_events_url":"https://api.github.com/users/AlexanderKurtz/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-03-27T13:58:59Z","updated_at":"2018-03-27T16:36:54Z","closed_at":"2018-03-27T16:36:51Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/envelope-project/laik/pulls/132","html_url":"https://github.com/envelope-project/laik/pull/132","diff_url":"https://github.com/envelope-project/laik/pull/132.diff","patch_url":"https://github.com/envelope-project/laik/pull/132.patch"},"body":"Both Debian and Ubuntu's ```libopenmpi-dev``` package already pull in\r\n```openmpi-bin``` on standard installation (```libopenmpi-dev``` depends on\r\n```libopenmpi``` which recommends ```openmpi-bin``` [0-3]), so installing this\r\n manually is unnecessary.\r\n\r\nFurthermore, this means there's now only a single mention of OpenMPI in the\r\nDockerfile which should make adding a MPICH test much easier.\r\n\r\n[0] https://packages.debian.org/stretch/libopenmpi-dev\r\n[1] https://packages.debian.org/stretch/libopenmpi2\r\n[2] https://packages.ubuntu.com/xenial/libopenmpi-dev\r\n[3] https://packages.ubuntu.com/xenial/libopenmpi1.10"},"comment":{"url":"https://api.github.com/repos/envelope-project/laik/issues/comments/376591582","html_url":"https://github.com/envelope-project/laik/pull/132#issuecomment-376591582","issue_url":"https://api.github.com/repos/envelope-project/laik/issues/132","id":376591582,"user":{"login":"weidendo","id":7921996,"avatar_url":"https://avatars1.githubusercontent.com/u/7921996?v=4","gravatar_id":"","url":"https://api.github.com/users/weidendo","html_url":"https://github.com/weidendo","followers_url":"https://api.github.com/users/weidendo/followers","following_url":"https://api.github.com/users/weidendo/following{/other_user}","gists_url":"https://api.github.com/users/weidendo/gists{/gist_id}","starred_url":"https://api.github.com/users/weidendo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/weidendo/subscriptions","organizations_url":"https://api.github.com/users/weidendo/orgs","repos_url":"https://api.github.com/users/weidendo/repos","events_url":"https://api.github.com/users/weidendo/events{/privacy}","received_events_url":"https://api.github.com/users/weidendo/received_events","type":"User","site_admin":false},"created_at":"2018-03-27T16:36:54Z","updated_at":"2018-03-27T16:36:54Z","author_association":"COLLABORATOR","body":"Just merged, but I have no idea why mentioning openmpi in less packages should change anything at all. And why not just also install MPICH? They do not conflict (I have both installed on my laptop).\r\n\r\nIt's just \"MPICC=mpicc.openmpi ./configure\" vs. \"MPICC=mpicc.mpich ./configure\". It should be similar when using cmake?\r\n"}}
{"actor":{"display_login":"weidendo"}}