Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Building bde_verify fails against clang 3.8, succeeds against 4.0 #6

Open
weyfonk opened this issue Sep 7, 2017 · 1 comment
Open

Comments

@weyfonk
Copy link

weyfonk commented Sep 7, 2017

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.

I 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
analysing dependencies
make[1]: Entering directory '/home/weyfonk/bde_verify/groups/csa/csabase'
compiling csabase_analyser.cpp
csabase_analyser.cpp: In member function ‘int csabase::Analyser::ReplaceText(llvm::StringRef, unsigned int, unsigned int, llvm::StringRef)’:
csabase_analyser.cpp:750:14: error: ‘clang::tooling::Replacements {aka class std::set<clang::tooling::Replacement>}’ has no member named ‘add’
     if (!one.add(r)) {
              ^~~
csabase_analyser.cpp:751:39: error: ‘clang::tooling::Replacements {aka class std::set<clang::tooling::Replacement>}’ has no member named ‘merge’
         replacements_ = replacements_.merge(one);
                                       ^~~~~
Makefile:82: recipe for target 'Linux-g++/csabase_analyser.o' failed
make[1]: *** [Linux-g++/csabase_analyser.o] Error 1
make[1]: Leaving directory '/home/weyfonk/bde_verify/groups/csa/csabase'
Makefile:271: recipe for target 'groups/csa/csabase/Linux-g++/libbde-verify.a' failed
make: *** [groups/csa/csabase/Linux-g++/libbde-verify.a] Error 2

Trying 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.

@hyrosen
Copy link
Contributor

hyrosen commented Sep 7, 2017 via email

hyrosen added a commit that referenced this issue Aug 2, 2018
Changelog for version 1.2.18
hyrosen added a commit that referenced this issue Aug 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants