achille-roussel / nanomsgxx

Nanomsg binding for C++11
MIT License
115 stars 35 forks source link

Compile error using socket.h which installed by CMake. #27

Open aYasuharuYamada opened 2 years ago

aYasuharuYamada commented 2 years ago

Hi, I installed nanomsgxx using CMake, and then I compile a file which include nnxx/socket.h but compile error happens.

/usr/include/nnxx/socket.h:232:10: fatal error: 'nnxx/socket.hpp' file not found
#include <nnxx/socket.hpp>
         ^~~~~~~~~~~~~~~~~
1 error generated.

The reason is make install command by CMake does not install socket.hpp. In my check, there are other files not installed, ex: *.hpp, message, etc.

Fix patch is following:

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index a734018..c42654b 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -34,27 +34,33 @@ set(NNXX_SRCS   ./nnxx/error.cpp
                 ./nnxx/tcp.cpp
                 ./nnxx/timeout.cpp)

-set(NNXX_HDRS   ./nanomsg/ext/nnxx_ext.h
-                ./nnxx/bus.h
+set(NNXX_HDRS   ./nnxx/bus.h
                 ./nnxx/chrono.h
                 ./nnxx/error.h
                 ./nnxx/inproc.h
                 ./nnxx/ipc.h
+                ./nnxx/message
                 ./nnxx/message.h
                 ./nnxx/message_control.h
                 ./nnxx/message_istream.h
+                ./nnxx/message_istream.hpp
                 ./nnxx/message_iterator.h
                 ./nnxx/message_ostream.h
+                ./nnxx/message_ostream.hpp
                 ./nnxx/message_streambuf.h
+                ./nnxx/message_streambuf.hpp
                 ./nnxx/nn.h
                 ./nnxx/pair.h
                 ./nnxx/pipeline.h
                 ./nnxx/poll.h
                 ./nnxx/pubsub.h
                 ./nnxx/reqrep.h
+                ./nnxx/socket
                 ./nnxx/socket.h
+                ./nnxx/socket.hpp
                 ./nnxx/survey.h
                 ./nnxx/tcp.h
+                ./nnxx/testing
                 ./nnxx/timeout.h
                 ./nnxx/unittest.h)