{"action":"created","issue":{"url":"https://api.github.com/repos/bloomberg/bde_verify/issues/6","repository_url":"https://api.github.com/repos/bloomberg/bde_verify","labels_url":"https://api.github.com/repos/bloomberg/bde_verify/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/bloomberg/bde_verify/issues/6/comments","events_url":"https://api.github.com/repos/bloomberg/bde_verify/issues/6/events","html_url":"https://github.com/bloomberg/bde_verify/issues/6","id":256007820,"number":6,"title":"Building bde_verify fails against clang 3.8, succeeds against 4.0","user":{"login":"weyfonk","id":2477066,"avatar_url":"https://avatars1.githubusercontent.com/u/2477066?v=4","gravatar_id":"","url":"https://api.github.com/users/weyfonk","html_url":"https://github.com/weyfonk","followers_url":"https://api.github.com/users/weyfonk/followers","following_url":"https://api.github.com/users/weyfonk/following{/other_user}","gists_url":"https://api.github.com/users/weyfonk/gists{/gist_id}","starred_url":"https://api.github.com/users/weyfonk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/weyfonk/subscriptions","organizations_url":"https://api.github.com/users/weyfonk/orgs","repos_url":"https://api.github.com/users/weyfonk/repos","events_url":"https://api.github.com/users/weyfonk/events{/privacy}","received_events_url":"https://api.github.com/users/weyfonk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-09-07T17:08:05Z","updated_at":"2017-09-07T18:07:13Z","closed_at":null,"author_association":"NONE","body":"EDIT: Just realised I had been using branch `debian` all along instead of `master`; it seems the former is the default branch. `master` builds against `clang` 3.8. Keeping this in case some of it is of interest to you, otherwise sorry for the noise.\r\n\r\nI have tried to build `bde_verify` against clang 3.8 as required by the `README`, but got the following output:\r\n```\r\nMakefile:369: Linux-g++/make.depend: No such file or directory\r\nanalysing dependencies\r\nmake[1]: Entering directory '/home/weyfonk/bde_verify/groups/csa/csabase'\r\ncompiling csabase_analyser.cpp\r\ncsabase_analyser.cpp: In member function ‘int csabase::Analyser::ReplaceText(llvm::StringRef, unsigned int, unsigned int, llvm::StringRef)’:\r\ncsabase_analyser.cpp:750:14: error: ‘clang::tooling::Replacements {aka class std::set<clang::tooling::Replacement>}’ has no member named ‘add’\r\n     if (!one.add(r)) {\r\n              ^~~\r\ncsabase_analyser.cpp:751:39: error: ‘clang::tooling::Replacements {aka class std::set<clang::tooling::Replacement>}’ has no member named ‘merge’\r\n         replacements_ = replacements_.merge(one);\r\n                                       ^~~~~\r\nMakefile:82: recipe for target 'Linux-g++/csabase_analyser.o' failed\r\nmake[1]: *** [Linux-g++/csabase_analyser.o] Error 1\r\nmake[1]: Leaving directory '/home/weyfonk/bde_verify/groups/csa/csabase'\r\nMakefile:271: recipe for target 'groups/csa/csabase/Linux-g++/libbde-verify.a' failed\r\nmake: *** [groups/csa/csabase/Linux-g++/libbde-verify.a] Error 2\r\n```\r\nTrying it again with `clang` 3.9 (both 3.8 and 3.9 are packaged for Debian Stretch, which I am using) gave the same result, however a build against a manually built `clang` 4.0 was successful."},"comment":{"url":"https://api.github.com/repos/bloomberg/bde_verify/issues/comments/327879041","html_url":"https://github.com/bloomberg/bde_verify/issues/6#issuecomment-327879041","issue_url":"https://api.github.com/repos/bloomberg/bde_verify/issues/6","id":327879041,"user":{"login":"hyrosen","id":4554328,"avatar_url":"https://avatars3.githubusercontent.com/u/4554328?v=4","gravatar_id":"","url":"https://api.github.com/users/hyrosen","html_url":"https://github.com/hyrosen","followers_url":"https://api.github.com/users/hyrosen/followers","following_url":"https://api.github.com/users/hyrosen/following{/other_user}","gists_url":"https://api.github.com/users/hyrosen/gists{/gist_id}","starred_url":"https://api.github.com/users/hyrosen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hyrosen/subscriptions","organizations_url":"https://api.github.com/users/hyrosen/orgs","repos_url":"https://api.github.com/users/hyrosen/repos","events_url":"https://api.github.com/users/hyrosen/events{/privacy}","received_events_url":"https://api.github.com/users/hyrosen/received_events","type":"User","site_admin":false},"created_at":"2017-09-07T18:07:13Z","updated_at":"2017-09-07T18:07:13Z","author_association":"CONTRIBUTOR","body":"I've been making changes to move bde_verify to LLVM 4.\nI'll check the README.\n\nreply@reply.github.com At: 09/07/17 13:08:07\" data-digest=\"From: reply@reply.github.com At: 09/07/17 13:08:07\" style=\"\">\nFrom: reply@reply.github.com At: 09/07/17 13:08:07\nTo: bde_verify@noreply.github.com\nCc: subscribed@noreply.github.com\nSubject: Re:[bloomberg/bde_verify] Building bde_verify fails against clang 3.8, succeeds against 4.0 (#6)\n\n\nI have tried to build bde_verify against clang 3.8 as required by the README, but got the following output: Makefile:369: Linux-g++/make.depend: No such file or directory\nanalysing dependencies\nmake[1]: Entering directory '/home/weyfonk/bde_verify/groups/csa/csabase'\ncompiling csabase_analyser.cpp\ncsabase_analyser.cpp: In member function ‘int csabase::Analyser::ReplaceText(llvm::StringRef, unsigned int, unsigned int, llvm::StringRef)’:\ncsabase_analyser.cpp:750:14: error: ‘clang::tooling::Replacements {aka class std::set<clang::tooling::Replacement>}’ has no member named ‘add’\n     if (!one.add(r)) {\n              ^~~\ncsabase_analyser.cpp:751:39: error: ‘clang::tooling::Replacements {aka class std::set<clang::tooling::Replacement>}’ has no member named ‘merge’\n         replacements_ = replacements_.merge(one);\n                                       ^~~~~\nMakefile:82: recipe for target 'Linux-g++/csabase_analyser.o' failed\nmake[1]: *** [Linux-g++/csabase_analyser.o] Error 1\nmake[1]: Leaving directory '/home/weyfonk/bde_verify/groups/csa/csabase'\nMakefile:271: recipe for target 'groups/csa/csabase/Linux-g++/libbde-verify.a' failed\nmake: *** [groups/csa/csabase/Linux-g++/libbde-verify.a] Error 2\n \nTrying it again with clang 3.9 (both 3.8 and 3.9 are packaged for Debian Stretch, which I am using) gave the same result, however a build against a manually built clang 4.0 was successful.\nI am not sure whether this is of interest to you, but thought I would report it in case the README needs updating (which I can of course do via a PR if relevant).  \n—\nYou are receiving this because you are subscribed to this thread.\nReply to this email directly, view it on GitHub, or mute the thread. \n       \n\n"}}
{"actor":{"display_login":"hyrosen"}}