breese / trial.protocol

Network wire protocols
11 stars 4 forks source link

Tests fail to build on boost 1.76 #51

Closed vinipsmaker closed 3 years ago

vinipsmaker commented 3 years ago
include/trial/protocol/detail/lightweight_test.hpp:42:28: error: no member named 'report_errors_remind' in namespace 'boost::detail'
            boost::detail::report_errors_remind();
breese commented 3 years ago

Already fixed on develop. I guess it is time to merge to master

vinipsmaker commented 3 years ago

Oops, I was looking at the wrong branch.

Okay, I'm having a different error on branch develop (this time on Boost 1.76):

/home/vinipsmaker/Projetos/trial.protocol/test/dynamic/std/partial_sum_suite.cpp:194:5: error: too few arguments to function call, expected 5, have 4
    TRIAL_PROTOCOL_TEST_THROW_EQUAL(op(),
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/vinipsmaker/Projetos/trial.protocol/cmake/../include/trial/protocol/core/detail/lightweight_test.hpp:235:66: note: expanded from macro 'TRIAL_PROTOCOL_TEST_THROW_EQUAL'
              (#EXCEP, __FILE__, __LINE__, BOOST_CURRENT_FUNCTION);     \
                                                                 ^
/usr/include/boost/core/lightweight_test.hpp:125:13: note: 'throw_failed_impl' declared here
inline void throw_failed_impl(const char* expr, char const * excep, char const * file, int line, char const * function)
            ^
breese commented 3 years ago

Possibly fixed in 38989dd3b2138d3a21d59b8140ec20c9ccd88c57

vinipsmaker commented 3 years ago

Yes, it fixed. Thanks.

I'm having troubles on other test now.

/home/vinipsmaker/Projetos/trial.protocol/test/dynamic/variable_comparison_suite.cpp:490:33: error: no member named 'numeric_limits' in namespace 'std'
        const variable big(std::numeric_limits<signed int>::max());
                           ~~~~~^

Hopefully just a missing include.

I'm using clang 12.0.0.

breese commented 3 years ago

Yes it was: ba3e9898b625da90c40fe92c77575370f4e12d3b

vinipsmaker commented 3 years ago

Okay. I can compile and run the tests again. Thanks.