{"action":"created","issue":{"url":"https://api.github.com/repos/icecc/icecream/issues/410","repository_url":"https://api.github.com/repos/icecc/icecream","labels_url":"https://api.github.com/repos/icecc/icecream/issues/410/labels{/name}","comments_url":"https://api.github.com/repos/icecc/icecream/issues/410/comments","events_url":"https://api.github.com/repos/icecc/icecream/issues/410/events","html_url":"https://github.com/icecc/icecream/issues/410","id":348207088,"node_id":"MDU6SXNzdWUzNDgyMDcwODg=","number":410,"title":"ICECC_REMOTE_CPP and -Werror -pedantic causes build failure","user":{"login":"dionorgua","id":534357,"node_id":"MDQ6VXNlcjUzNDM1Nw==","avatar_url":"https://avatars2.githubusercontent.com/u/534357?v=4","gravatar_id":"","url":"https://api.github.com/users/dionorgua","html_url":"https://github.com/dionorgua","followers_url":"https://api.github.com/users/dionorgua/followers","following_url":"https://api.github.com/users/dionorgua/following{/other_user}","gists_url":"https://api.github.com/users/dionorgua/gists{/gist_id}","starred_url":"https://api.github.com/users/dionorgua/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dionorgua/subscriptions","organizations_url":"https://api.github.com/users/dionorgua/orgs","repos_url":"https://api.github.com/users/dionorgua/repos","events_url":"https://api.github.com/users/dionorgua/events{/privacy}","received_events_url":"https://api.github.com/users/dionorgua/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-08-07T07:51:41Z","updated_at":"2018-10-26T11:54:57Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"It looks like `ICECC_REMOTE_CPP` doesn't work with `-Werror -pedantic` (gcc)\r\n\r\nI have no idea how to fix it.. I see two possible options:\r\n1. ignore `ICECC_REMOTE_CPP`\r\n2. remove `-pedantic` (or `-pedantic-errors`) from command line\r\n\r\nMaybe it makes sense to extend `ICECC_REMOTE_CPP` a bit and add two more value that will do proposed above things\r\n\r\na.cpp:\r\n```#include <iostream>\r\nint main(int argc, char*argv[])\r\n{\r\n        std::cout << \"hello world!\" << std::endl;\r\n        return 0;\r\n}\r\n```\r\n\r\n```\r\n% export ICECC_REMOTE_CPP=1\r\n% export ICECC_PREFERRED_HOST=1.2.3.4 # make sure we compiling remotely\r\n\r\n% g++ -c -O0 -Werror -pedantic a.cpp -o a.o # success\r\n% icecc g++ -c -O0 -Werror -pedantic a.cpp -o a.o\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\nIn file included from /usr/include/c++/7/bits/stl_algobase.h:61:0,\r\n                 from /usr/include/c++/7/bits/char_traits.h:39,\r\n                 from /usr/include/c++/7/ios:40,\r\n                 from /usr/include/c++/7/ostream:38,\r\n                 from /usr/include/c++/7/iostream:39,\r\n                 from a.cpp:1:\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\nIn file included from /usr/include/c++/7/bits/stl_algobase.h:61:0,\r\n                 from /usr/include/c++/7/bits/char_traits.h:39,\r\n                 from /usr/include/c++/7/ios:40,\r\n                 from /usr/include/c++/7/ostream:38,\r\n                 from /usr/include/c++/7/iostream:39,\r\n                 from a.cpp:1:\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\nIn file included from /usr/include/c++/7/bits/basic_string.h:6575:0,\r\n                 from /usr/include/c++/7/string:52,\r\n                 from /usr/include/c++/7/bits/locale_classes.h:40,\r\n                 from /usr/include/c++/7/bits/ios_base.h:41,\r\n                 from /usr/include/c++/7/ios:42,\r\n                 from /usr/include/c++/7/ostream:38,\r\n                 from /usr/include/c++/7/iostream:39,\r\n                 from a.cpp:1:\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\n<built-in>: error: ISO C++ does not support '__int128' for 'type name' [-Werror=pedantic]\r\ncc1plus: all warnings being treated as errors\r\n```"},"comment":{"url":"https://api.github.com/repos/icecc/icecream/issues/comments/433383116","html_url":"https://github.com/icecc/icecream/issues/410#issuecomment-433383116","issue_url":"https://api.github.com/repos/icecc/icecream/issues/410","id":433383116,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMzM4MzExNg==","user":{"login":"mostynb","id":6337887,"node_id":"MDQ6VXNlcjYzMzc4ODc=","avatar_url":"https://avatars1.githubusercontent.com/u/6337887?v=4","gravatar_id":"","url":"https://api.github.com/users/mostynb","html_url":"https://github.com/mostynb","followers_url":"https://api.github.com/users/mostynb/followers","following_url":"https://api.github.com/users/mostynb/following{/other_user}","gists_url":"https://api.github.com/users/mostynb/gists{/gist_id}","starred_url":"https://api.github.com/users/mostynb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mostynb/subscriptions","organizations_url":"https://api.github.com/users/mostynb/orgs","repos_url":"https://api.github.com/users/mostynb/repos","events_url":"https://api.github.com/users/mostynb/events{/privacy}","received_events_url":"https://api.github.com/users/mostynb/received_events","type":"User","site_admin":false},"created_at":"2018-10-26T11:54:57Z","updated_at":"2018-10-26T11:54:57Z","author_association":"CONTRIBUTOR","body":"GCC uses a language standard that allows GNU extensions by default (something like gnu11 or gnu++14 for C and C++, depending on the GCC version).  This explains why -pedantic gives warnings (though I am no yet sure why it does not give these warnings locally also).\r\n\r\nIf you specify an official language standard like -std=c++11, then the testcase passes.  I think we should not disable ICECC_REMOTE_CPP in this case, since its purpose is to avoid a different class of remote compilation issue."}}
{"actor":{"display_login":"mostynb"}}