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

can't build cocaine v0.12 with blackhole master #134

Closed
minaevmike opened this issue Jul 13, 2016 · 5 comments
Closed

can't build cocaine v0.12 with blackhole master #134

minaevmike opened this issue Jul 13, 2016 · 5 comments

Comments

@minaevmike
Copy link
Contributor

g++ --version
g++ (Ubuntu 5.4.0-6ubuntu1~16.04.1) 5.4.0 20160609
In file included from /root/prog/cocaine-core/src/service/logging.cpp:26:0:
/usr/include/blackhole/builder.hpp:68:34: error: declaration of ‘blackhole::v1::builder<blackhole::v1::handler::blocking_t> blackhole::v1::experimental::partial_builder<blackhole::v1::handler::blocking_t, Parent>::builder’ [-fpermissive]
     builder<handler::blocking_t> builder;
                                  ^
In file included from /usr/include/blackhole/builder.hpp:14:0,
                 from /root/prog/cocaine-core/src/service/logging.cpp:26:
/usr/include/blackhole/handler/blocking.hpp:14:7: error: changes meaning of ‘builder’ from ‘class blackhole::v1::builder<blackhole::v1::handler::blocking_t>’ [-fpermissive]
 class builder<handler::blocking_t> {
       ^
In file included from /root/prog/cocaine-core/src/service/logging.cpp:26:0:
/usr/include/blackhole/builder.hpp:100:28: error: declaration of ‘blackhole::v1::builder<blackhole::v1::formatter::string_t> blackhole::v1::experimental::partial_builder<blackhole::v1::formatter::string_t, Parent>::builder’ [-fpermissive]
     builder<internal_type> builder;
                            ^
In file included from /usr/include/blackhole/builder.hpp:12:0,
                 from /root/prog/cocaine-core/src/service/logging.cpp:26:
/usr/include/blackhole/formatter/string.hpp:103:7: error: changes meaning of ‘builder’ from ‘class blackhole::v1::builder<blackhole::v1::formatter::string_t>’ [-fpermissive]
 class builder<formatter::string_t> {
       ^
In file included from /root/prog/cocaine-core/src/service/logging.cpp:26:0:
/usr/include/blackhole/builder.hpp:128:28: error: declaration of ‘blackhole::v1::builder<blackhole::v1::sink::console_t> blackhole::v1::experimental::partial_builder<blackhole::v1::sink::console_t, Parent>::builder’ [-fpermissive]
     builder<internal_type> builder;
                            ^
In file included from /usr/include/blackhole/builder.hpp:13:0,
                 from /root/prog/cocaine-core/src/service/logging.cpp:26:
/usr/include/blackhole/sink/console.hpp:29:7: error: changes meaning of ‘builder’ from ‘class blackhole::v1::builder<blackhole::v1::sink::console_t>’ [-fpermissive]
 class builder<sink::console_t> {
       ^
/root/prog/cocaine-core/src/service/logging.cpp: In lambda function:
/root/prog/cocaine-core/src/service/logging.cpp:79:29: error: ‘configured’ is not a member of ‘blackhole::v1::registry_t’
             auto registry = blackhole::registry_t::configured();
                             ^
/root/prog/cocaine-core/src/service/logging.cpp:80:54: error: expected primary-expression before ‘>’ token
             registry.add<blackhole::formatter::json_t>();
                                                      ^
/root/prog/cocaine-core/src/service/logging.cpp:80:56: error: expected primary-expression before ‘)’ token
             registry.add<blackhole::formatter::json_t>();
                                                        ^
/root/prog/cocaine-core/src/service/logging.cpp:81:49: error: expected primary-expression before ‘>’ token
             registry.add<blackhole::sink::file_t>();
                                                 ^
/root/prog/cocaine-core/src/service/logging.cpp:81:51: error: expected primary-expression before ‘)’ token
             registry.add<blackhole::sink::file_t>();
                                                   ^
/root/prog/cocaine-core/src/service/logging.cpp:82:56: error: expected primary-expression before ‘>’ token
             registry.add<blackhole::sink::socket::tcp_t>();
                                                        ^
/root/prog/cocaine-core/src/service/logging.cpp:82:58: error: expected primary-expression before ‘)’ token
             registry.add<blackhole::sink::socket::tcp_t>();
                                                          ^
/root/prog/cocaine-core/src/service/logging.cpp:83:56: error: expected primary-expression before ‘>’ token
             registry.add<blackhole::sink::socket::udp_t>();
                                                        ^
/root/prog/cocaine-core/src/service/logging.cpp:83:58: error: expected primary-expression before ‘)’ token
             registry.add<blackhole::sink::socket::udp_t>();
                                                          ^
/root/prog/cocaine-core/src/service/logging.cpp:88:66: error: expected primary-expression before ‘>’ token
             auto log = registry.builder<blackhole::config::json_t>(stream)
                                                                  ^
/root/prog/cocaine-core/src/service/logging.cpp:89:18: error: ‘std::stringstream {aka class std::__cxx11::basic_stringstream<char>}’ has no member named ‘build’
                 .build(backend);
                  ^
CMakeFiles/cocaine-core.dir/build.make:590: recipe for target 'CMakeFiles/cocaine-core.dir/src/service/logging.cpp.o' failed
make[2]: *** [CMakeFiles/cocaine-core.dir/src/service/logging.cpp.o] Error 1
CMakeFiles/Makefile2:104: recipe for target 'CMakeFiles/cocaine-core.dir/all' failed
make[1]: *** [CMakeFiles/cocaine-core.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2
@3Hren
Copy link
Owner

3Hren commented Jul 13, 2016

I think it's the Cocaine issue, I just grepped out master and that's the result:

esafronov@5:~/code/cocaine|⇒  grep -r '#include <blackhole/builder.hpp>' .
esafronov@5:~/code/cocaine|

What version do you use?

Also note that everything in blackhole::experimental namespace are, well, experimental, which means that is may be broken at any time.

@minaevmike
Copy link
Contributor Author

@3Hren
Copy link
Owner

3Hren commented Jul 13, 2016

Oh, seems like we didn't bump master branch for a long time. Please take v0.12.8 or v0.12.9

@minaevmike
Copy link
Contributor Author

v0.12.8 have same problem https://github.com/3Hren/cocaine-core/blob/v0.12.8/src/service/logging.cpp#L26 , but v0.12.9 builds fine.
Thank you for help.
By the way, what cocaine-12 do you use in production? Your fork or from cocaine/cocaine-core?

@3Hren
Copy link
Owner

3Hren commented Jul 13, 2016

Our fork.

I'll fix that meaning-change-variable-name in Blackhole.

@3Hren 3Hren closed this as completed in ac0a579 Jul 18, 2016
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