{"issue":{"created_at":"2011-12-01T15:29:27Z","state":"open","updated_at":"2011-12-01T21:42:00Z","comments":4,"user":{"gravatar_id":"e86c5d5b142a13248cbbce010fea141a","id":723889,"url":"https://api.github.com/users/chrispbradley","avatar_url":"https://secure.gravatar.com/avatar/e86c5d5b142a13248cbbce010fea141a?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"chrispbradley"},"body":"Needs to be done with pull request 50.\r\n","assignee":null,"id":2415696,"labels":[],"number":56,"pull_request":{"patch_url":"https://github.com/OpenCMISS/cm/pull/56.patch","html_url":"https://github.com/OpenCMISS/cm/pull/56","diff_url":"https://github.com/OpenCMISS/cm/pull/56.diff"},"title":"Mpichange","html_url":"https://github.com/OpenCMISS/cm/issues/56","closed_at":null,"url":"https://api.github.com/repos/OpenCMISS/cm/issues/56","milestone":null},"action":"created","comment":{"created_at":"2011-12-01T21:42:00Z","updated_at":"2011-12-01T21:42:00Z","user":{"gravatar_id":"c48ed994ee7731ac99147d0161bd81e1","id":626438,"url":"https://api.github.com/users/adamreeve","avatar_url":"https://secure.gravatar.com/avatar/c48ed994ee7731ac99147d0161bd81e1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"adamreeve"},"body":"If you look at the pull requests though, they all have the same list of commits except for the last one, which is different for the two branches. So this one still includes all the previous FieldML commits.\r\n\r\nI have no problem with pulling everything in, but like I said, pulling in this wouldn't achieve what you want because it doesn't actually have the Makefile change in it. If you look at the diff tab on the right, you'll see there aren't any MPI Makefile changes to be added as they were all removed when the utils/MakefileCommon.inc request was merged.\r\n\r\nWhat I suggest is that we pull #59 which now has the MPI changes merged properly and close this request. Then we can pull the FieldML request when people are happy with that (or now if that's fine). Then the GPU branch has some conflicts with #59 but they should be easy to fix.","id":2981106,"url":"https://api.github.com/repos/OpenCMISS/cm/issues/comments/2981106"}}