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
build: Remove -DBOOST_SPIRIT_THREADSAFE #6316
build: Remove -DBOOST_SPIRIT_THREADSAFE #6316
Conversation
ACK FYI src/rpcprotocol.cpp has a mention of json-spirit in a comment (a URL) also |
ACK. |
There also is a comment in univalue.h, perhaps related? // The following were added for compatibility with json_spirit. |
utACK |
@Diapolo: right. UniValue has some json_spirit compatibility functions which should be removed/changed (but not in this PR). Also some functions should be renamed |
Now that boost spirit is no longer used, `-DBOOST_SPIRIT_THREADSAFE` doesn't need to be passed to the compiler anymore.
ce6ebe3
to
b666820
Compare
ACK |
1 similar comment
ACK |
b666820 build: Remove -DBOOST_SPIRIT_THREADSAFE (Wladimir J. van der Laan)
Bitcoin 0.12 misc PRs 1 Cherry-picked from the following upstream PRs: - bitcoin/bitcoin#6198 - bitcoin/bitcoin#6206 - bitcoin/bitcoin#5927 - bitcoin/bitcoin#6213 - bitcoin/bitcoin#6061 - bitcoin/bitcoin#6283 (partial, remainder was pulled in #929) - bitcoin/bitcoin#6272 - bitcoin/bitcoin#6316 - bitcoin/bitcoin#6133 - bitcoin/bitcoin#6387 - bitcoin/bitcoin#6401 - bitcoin/bitcoin#6434 - bitcoin/bitcoin#6372 - bitcoin/bitcoin#6447 - bitcoin/bitcoin#6149 - bitcoin/bitcoin#6468 Part of #2074.
Now that boost spirit is no longer used,
-DBOOST_SPIRIT_THREADSAFE
doesn't need to be passed to the compiler anymore.