{"action":"created","issue":{"url":"https://api.github.com/repos/openssl/openssl/issues/4755","repository_url":"https://api.github.com/repos/openssl/openssl","labels_url":"https://api.github.com/repos/openssl/openssl/issues/4755/labels{/name}","comments_url":"https://api.github.com/repos/openssl/openssl/issues/4755/comments","events_url":"https://api.github.com/repos/openssl/openssl/issues/4755/events","html_url":"https://github.com/openssl/openssl/pull/4755","id":275093305,"number":4755,"title":"Configure: use a better method to identify gcc and derivates [1.0.2]","user":{"login":"levitte","id":698918,"avatar_url":"https://avatars1.githubusercontent.com/u/698918?v=4","gravatar_id":"","url":"https://api.github.com/users/levitte","html_url":"https://github.com/levitte","followers_url":"https://api.github.com/users/levitte/followers","following_url":"https://api.github.com/users/levitte/following{/other_user}","gists_url":"https://api.github.com/users/levitte/gists{/gist_id}","starred_url":"https://api.github.com/users/levitte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/levitte/subscriptions","organizations_url":"https://api.github.com/users/levitte/orgs","repos_url":"https://api.github.com/users/levitte/repos","events_url":"https://api.github.com/users/levitte/events{/privacy}","received_events_url":"https://api.github.com/users/levitte/received_events","type":"User","site_admin":false},"labels":[{"id":441165257,"url":"https://api.github.com/repos/openssl/openssl/labels/1.0.2","name":"1.0.2","color":"bfdadc","default":false},{"id":314766253,"url":"https://api.github.com/repos/openssl/openssl/labels/ready","name":"ready","color":"009800","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2017-11-18T17:03:43Z","updated_at":"2018-04-19T12:36:16Z","closed_at":"2017-12-10T09:32:37Z","author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/openssl/openssl/pulls/4755","html_url":"https://github.com/openssl/openssl/pull/4755","diff_url":"https://github.com/openssl/openssl/pull/4755.diff","patch_url":"https://github.com/openssl/openssl/pull/4755.patch"},"body":"Looking for 'gcc' and 'clang' in the output from the C compiler is\r\nuncertain.  Some versions report argv[0], which might be /usr/bin/cc\r\n(for example), and others might mention gcc without being gcc or a\r\nderivate.\r\n    \r\nBetter then to fetch predefined macros and checking if `__GNUC__` and\r\n`__clang__` are defined.\r\n\r\n-----\r\n\r\nThis is an alternative to #1524"},"comment":{"url":"https://api.github.com/repos/openssl/openssl/issues/comments/382717568","html_url":"https://github.com/openssl/openssl/pull/4755#issuecomment-382717568","issue_url":"https://api.github.com/repos/openssl/openssl/issues/4755","id":382717568,"user":{"login":"tmshort","id":10981912,"avatar_url":"https://avatars1.githubusercontent.com/u/10981912?v=4","gravatar_id":"","url":"https://api.github.com/users/tmshort","html_url":"https://github.com/tmshort","followers_url":"https://api.github.com/users/tmshort/followers","following_url":"https://api.github.com/users/tmshort/following{/other_user}","gists_url":"https://api.github.com/users/tmshort/gists{/gist_id}","starred_url":"https://api.github.com/users/tmshort/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmshort/subscriptions","organizations_url":"https://api.github.com/users/tmshort/orgs","repos_url":"https://api.github.com/users/tmshort/repos","events_url":"https://api.github.com/users/tmshort/events{/privacy}","received_events_url":"https://api.github.com/users/tmshort/received_events","type":"User","site_admin":false},"created_at":"2018-04-19T12:36:16Z","updated_at":"2018-04-19T12:36:16Z","author_association":"CONTRIBUTOR","body":"The 10.8 compiler `cc -dM -E -x c` partial output (I have a complete list available):\r\n```\r\n#define __GNUC__ 4\r\n#define __APPLE__ 1\r\n#define __APPLE_CC__ 5658\r\n#define __llvm__ 1\r\n```\r\nIt does not define `__clang__`. Would it suffice to just check `__llvm__`?"}}
{"actor":{"display_login":"tmshort"}}