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

Compile error in OS X #28

Closed
pbarbashov opened this issue May 27, 2019 · 2 comments
Closed

Compile error in OS X #28

pbarbashov opened this issue May 27, 2019 · 2 comments

Comments

@pbarbashov
Copy link

Mac os 10.14.4 XCode 10.2
clang --version
Apple LLVM version 10.0.1 (clang-1001.0.46.3)
Target: x86_64-apple-darwin18.5.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

[ 58%] Building CXX object CMakeFiles/FizzTool.dir/tool/FizzServerCommand.cpp.o
/Users/pavelbarbasov/Documents/work/cpp/fizz/fizz/tool/FizzServerCommand.cpp:79:49: error: non-virtual member function
marked 'override' hides virtual member function
const SocketAddress& clientAddr) noexcept override;
^
/usr/local/include/folly/io/async/AsyncServerSocket.h:160:18: note: hidden overloaded virtual function
'folly::AsyncServerSocket::AcceptCallback::connectionAccepted' declared here: type mismatch at 1st parameter
('int' vs 'folly::NetworkSocket')
virtual void connectionAccepted(
^
/Users/pavelbarbasov/Documents/work/cpp/fizz/fizz/tool/FizzServerCommand.cpp:635:22: error: variable type
'fizz::tool::(anonymous namespace)::FizzServerAcceptor' is an abstract class
FizzServerAcceptor acceptor(port, serverContext, loop, &evb, sslContext);
^
/usr/local/include/folly/io/async/AsyncServerSocket.h:160:18: note: unimplemented pure virtual method
'connectionAccepted' in 'FizzServerAcceptor'
virtual void connectionAccepted(
^
2 errors generated.
make[2]: *** [CMakeFiles/FizzTool.dir/tool/FizzServerCommand.cpp.o] Error 1
make[1]: *** [CMakeFiles/FizzTool.dir/all] Error 2
make: *** [all] Error 2

@reanimus
Copy link
Contributor

You may need to update folly to ensure it has this commit: facebook/folly@3c6096c

@mingtaoy
Copy link
Contributor

Closing this out, please reopen if this is still an issue.

facebook-github-bot pushed a commit that referenced this issue Mar 14, 2022
Summary:
X-link: facebook/fb303#28

X-link: facebook/fboss#115

X-link: facebook/folly#1736

X-link: facebook/proxygen#403

X-link: facebook/fbthrift#488

This adds a way to pass arguments to the `b2` build tool, used by Boost. This is needed in order to link a getdeps built boost into an relocatable `.so`. The motivating use case is that we need to statically link Boost into a native library used by a python wheel, which must be relocatable. This functionality already exists for CMake-based projects.

Reviewed By: mackorone

Differential Revision: D34796774

fbshipit-source-id: 0d6a9f4703865dc02048b87e77394c44ef646af6
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

3 participants