{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/opm-common/issues/58","labels_url":"https://api.github.com/repos/OPM/opm-common/issues/58/labels{/name}","comments_url":"https://api.github.com/repos/OPM/opm-common/issues/58/comments","events_url":"https://api.github.com/repos/OPM/opm-common/issues/58/events","html_url":"https://github.com/OPM/opm-common/pull/58","id":112624105,"number":58,"title":"adapt Finddune-alugrid.cmake to the \"paramerge\" branch of dune-alugrid","user":{"login":"andlaus","id":1837735,"avatar_url":"https://avatars.githubusercontent.com/u/1837735?v=3","gravatar_id":"","url":"https://api.github.com/users/andlaus","html_url":"https://github.com/andlaus","followers_url":"https://api.github.com/users/andlaus/followers","following_url":"https://api.github.com/users/andlaus/following{/other_user}","gists_url":"https://api.github.com/users/andlaus/gists{/gist_id}","starred_url":"https://api.github.com/users/andlaus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andlaus/subscriptions","organizations_url":"https://api.github.com/users/andlaus/orgs","repos_url":"https://api.github.com/users/andlaus/repos","events_url":"https://api.github.com/users/andlaus/events{/privacy}","received_events_url":"https://api.github.com/users/andlaus/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-10-21T16:06:53Z","updated_at":"2015-11-09T13:30:51Z","closed_at":"2015-11-09T12:13:41Z","pull_request":{"url":"https://api.github.com/repos/OPM/opm-common/pulls/58","html_url":"https://github.com/OPM/opm-common/pull/58","diff_url":"https://github.com/OPM/opm-common/pull/58.diff","patch_url":"https://github.com/OPM/opm-common/pull/58.patch"},"body":"recently the `paramerge` branch has been made the new `master` version of dune-alugrid. this PR adapts the CMake module of the OPM build system to this change. Note that old versions of ALUGrid do not work with this anymore, but detecting whether a pre-merge or a post-merge version of dune-alugrid is installed would require a complete rewrite of Finddune-alugrid.cmake (as far as I can see). I'd appreciate if someone had a better solution."},"comment":{"url":"https://api.github.com/repos/OPM/opm-common/issues/comments/155062414","html_url":"https://github.com/OPM/opm-common/pull/58#issuecomment-155062414","issue_url":"https://api.github.com/repos/OPM/opm-common/issues/58","id":155062414,"user":{"login":"bska","id":1871438,"avatar_url":"https://avatars.githubusercontent.com/u/1871438?v=3","gravatar_id":"","url":"https://api.github.com/users/bska","html_url":"https://github.com/bska","followers_url":"https://api.github.com/users/bska/followers","following_url":"https://api.github.com/users/bska/following{/other_user}","gists_url":"https://api.github.com/users/bska/gists{/gist_id}","starred_url":"https://api.github.com/users/bska/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bska/subscriptions","organizations_url":"https://api.github.com/users/bska/orgs","repos_url":"https://api.github.com/users/bska/repos","events_url":"https://api.github.com/users/bska/events{/privacy}","received_events_url":"https://api.github.com/users/bska/received_events","type":"User","site_admin":false},"created_at":"2015-11-09T13:30:51Z","updated_at":"2015-11-09T13:30:51Z","body":"> With the release done, I'll merge this.\r\n\r\nAnd create a situation in which sources just-after-release are incompatible with sources just-before-release.  We had this kind of breaking change in opm-parser 2015.04 too and it did cause a bit of confusion in the issues list concerning which source combinations are supported.  This time the impact is likely to be smaller because, as @dr-robertk  says\r\n\r\n> all people that are interested in [dune-alugrid combined with OPM] are using the git sources anyway\r\n\r\nThat said, we should probably establish some kind of policy for when and how to introduce breaking changes and whether or not it is acceptable to combine release sources with Git master."}}