{"action":"created","issue":{"url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/issues/2222","labels_url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/issues/2222/labels{/name}","comments_url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/issues/2222/comments","events_url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/issues/2222/events","html_url":"https://github.com/hpcugent/easybuild-easyconfigs/pull/2222","id":122034921,"number":2222,"title":"added new Cray* toolchain versions with dependency pinning","user":{"login":"gppezzi","id":4578156,"avatar_url":"https://avatars.githubusercontent.com/u/4578156?v=3","gravatar_id":"","url":"https://api.github.com/users/gppezzi","html_url":"https://github.com/gppezzi","followers_url":"https://api.github.com/users/gppezzi/followers","following_url":"https://api.github.com/users/gppezzi/following{/other_user}","gists_url":"https://api.github.com/users/gppezzi/gists{/gist_id}","starred_url":"https://api.github.com/users/gppezzi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gppezzi/subscriptions","organizations_url":"https://api.github.com/users/gppezzi/orgs","repos_url":"https://api.github.com/users/gppezzi/repos","events_url":"https://api.github.com/users/gppezzi/events{/privacy}","received_events_url":"https://api.github.com/users/gppezzi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/milestones/26","html_url":"https://github.com/hpcugent/easybuild-easyconfigs/milestones/v2.6.0","labels_url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/milestones/26/labels","id":1460306,"number":26,"title":"v2.6.0","description":"","creator":{"login":"boegel","id":620876,"avatar_url":"https://avatars.githubusercontent.com/u/620876?v=3","gravatar_id":"","url":"https://api.github.com/users/boegel","html_url":"https://github.com/boegel","followers_url":"https://api.github.com/users/boegel/followers","following_url":"https://api.github.com/users/boegel/following{/other_user}","gists_url":"https://api.github.com/users/boegel/gists{/gist_id}","starred_url":"https://api.github.com/users/boegel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/boegel/subscriptions","organizations_url":"https://api.github.com/users/boegel/orgs","repos_url":"https://api.github.com/users/boegel/repos","events_url":"https://api.github.com/users/boegel/events{/privacy}","received_events_url":"https://api.github.com/users/boegel/received_events","type":"User","site_admin":false},"open_issues":91,"closed_issues":70,"state":"open","created_at":"2015-12-14T22:37:19Z","updated_at":"2016-01-18T19:49:21Z","due_on":"2016-01-25T23:00:00Z","closed_at":null},"comments":10,"created_at":"2015-12-14T12:40:15Z","updated_at":"2016-01-18T20:05:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/pulls/2222","html_url":"https://github.com/hpcugent/easybuild-easyconfigs/pull/2222","diff_url":"https://github.com/hpcugent/easybuild-easyconfigs/pull/2222.diff","patch_url":"https://github.com/hpcugent/easybuild-easyconfigs/pull/2222.patch"},"body":"This PR adds eb files for the new CrayGNU, CrayIntel and CrayCCE toolchains (with dep. pinning). For validation, it adds also easyconfig files for HPL.\r\n\r\nRequires: EB 2.5.0 and [hpcugent/easybuild-easyblocks/pull/766](https://github.com/hpcugent/easybuild-easyblocks/pull/766)"},"comment":{"url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/issues/comments/172639220","html_url":"https://github.com/hpcugent/easybuild-easyconfigs/pull/2222#issuecomment-172639220","issue_url":"https://api.github.com/repos/hpcugent/easybuild-easyconfigs/issues/2222","id":172639220,"user":{"login":"boegel","id":620876,"avatar_url":"https://avatars.githubusercontent.com/u/620876?v=3","gravatar_id":"","url":"https://api.github.com/users/boegel","html_url":"https://github.com/boegel","followers_url":"https://api.github.com/users/boegel/followers","following_url":"https://api.github.com/users/boegel/following{/other_user}","gists_url":"https://api.github.com/users/boegel/gists{/gist_id}","starred_url":"https://api.github.com/users/boegel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/boegel/subscriptions","organizations_url":"https://api.github.com/users/boegel/orgs","repos_url":"https://api.github.com/users/boegel/repos","events_url":"https://api.github.com/users/boegel/events{/privacy}","received_events_url":"https://api.github.com/users/boegel/received_events","type":"User","site_admin":false},"created_at":"2016-01-18T20:05:06Z","updated_at":"2016-01-18T20:05:06Z","body":"@gppezzi: I wouldn't bother splitting this up, it's not really going to help much (if anything, it'll add more overhead, for the both of us)\r\n\r\nThe general rule-of-thumb is to keep easyconfig PRs below 10 new easyconfigs.\r\n\r\nThis is kind of an exception, since the easyconfigs are so similar to each other (in two groups).\r\nThe HPL easyconfigs are pretty much a no-brainer since they're (near) identical to each other but also to existing HPL easyconfigs (only exception is the extra patch in the CCE ones).\r\n\r\nThe others are simple enough to review in a single PR, since it's basically just a matter of the toolchain definition."}}