-
Notifications
You must be signed in to change notification settings - Fork 82
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
configure forces -std=c++98 but that should be moved to relevant Makefiles or sga code adjusted #156
Comments
Thanks @mmokrejs for the detailed report. Do you remember how you configured your installation of sparsehash? It should not be looking for std::hash and should have no problems compiling with |
It was built with No, the issue in abyss was not solved. |
Thanks. I don't see anything there would cause a problem. Can you paste the contents of your sparsehash/internal/sparseconfig.h file? |
|
Ok, that confirms your sparsehash is configured to assume c++11 hash functions are available. I am reluctant to change the dialect to c++11 just over this issue. SGA is no longer used for assembly very often but some subprograms it contains are still useful. Perhaps I should default to using the standard hash map if sparsehash is unavailable instead of dying. |
FWIW, I was able to compile with c++11 with the following changes: configure.ac ClusterReader.cpp rmdup.cpp |
For me configure does not detect sparsehash-2.0.3 not even with current git master at commit af73db6 :
The
config.log
states:I dropped the
-std=c++98
and now configure checks passed:Now
configure
works:But, the
sga
code needs some more tweaks orsrc/Util/Makefile and src/SGA/Makefile
need to append-std=c++98
toCXXFLAGS
:The text was updated successfully, but these errors were encountered: