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

thirdparty: fix build failure on boost-1.68 enviroment #195

Merged
merged 2 commits into from
Oct 25, 2018

Conversation

neverchanje
Copy link
Contributor

In file included from /home/wutao1/work/boost_1_68_0/output/include/boost/asio.hpp:118:0,
                 from /home/wutao1/work/pegasus-boost68/src/reporter/pegasus_counter_reporter.h:11,
                 from /home/wutao1/work/pegasus-boost68/src/server/info_collector_app.cpp:6:
/home/wutao1/work/boost_1_68_0/output/include/boost/asio/signal_set.hpp:114:58: error: macro "signal_set" requires 4 arguments, but only 1 given
   explicit signal_set(boost::asio::io_context& io_context)
                                                          ^
/home/wutao1/work/boost_1_68_0/output/include/boost/asio/signal_set.hpp:132:70: error: macro "signal_set" requires 4 arguments, but only 2 given
   signal_set(boost::asio::io_context& io_context, int signal_number_1)
                                                                      ^
/home/wutao1/work/boost_1_68_0/output/include/boost/asio/signal_set.hpp:157:26: error: macro "signal_set" requires 4 arguments, but only 3 given
       int signal_number_2)
                          ^
/home/wutao1/work/boost_1_68_0/output/include/boost/asio/signal_set.hpp:205:15: error: macro "signal_set" requires 4 arguments, but only 1 given
   ~signal_set()

@qinzuoyan qinzuoyan merged commit 63321c2 into apache:master Oct 25, 2018
vagetablechicken pushed a commit to vagetablechicken/pegasus that referenced this pull request Nov 1, 2018
neverchanje pushed a commit to neverchanje/pegasus that referenced this pull request Jul 13, 2019
Former-commit-id: cc38ae591788e08e46fafe125367822948ea31bc [formerly 63321c2]
Former-commit-id: c96e9164efa02c5e643b59c196ef89e3ecee2c8e
acelyc111 pushed a commit that referenced this pull request Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug-fix This PR fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants