Moved xauthentication.hpp and xzmq_serializer.hpp to src folder

jupyter-xeus · Apr 12, 2024 · 2c5879d · 2c5879d
1 parent 928a044
commit 2c5879d
Show file tree

Hide file tree

Showing 9 changed files with 20 additions and 19 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -123,7 +123,6 @@ find_package(OpenSSL REQUIRED)
 # ============
 
 set(XEUS_ZMQ_HEADERS
-    ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xauthentication.hpp
     ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xclient_zmq.hpp
     ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xdap_tcp_client.hpp
     ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xdebugger_base.hpp
@@ -132,11 +131,11 @@ set(XEUS_ZMQ_HEADERS
     ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xserver_zmq.hpp
     ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xthread.hpp
     ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xzmq_context.hpp
-    ${XEUS_ZMQ_INCLUDE_DIR}/xeus-zmq/xzmq_serializer.hpp
 )
 
 set(XEUS_ZMQ_SOURCES
     ${XEUS_ZMQ_SOURCE_DIR}/xauthentication.cpp
+    ${XEUS_ZMQ_SOURCE_DIR}/xauthentication.hpp
     ${XEUS_ZMQ_SOURCE_DIR}/xclient_zmq.cpp
     ${XEUS_ZMQ_SOURCE_DIR}/xclient_messenger.hpp
     ${XEUS_ZMQ_SOURCE_DIR}/xclient_messenger.cpp
@@ -181,6 +180,7 @@ set(XEUS_ZMQ_SOURCES
     ${XEUS_ZMQ_SOURCE_DIR}/xzmq_messenger.hpp
     ${XEUS_ZMQ_SOURCE_DIR}/xzmq_messenger.cpp
     ${XEUS_ZMQ_SOURCE_DIR}/xzmq_serializer.cpp
+    ${XEUS_ZMQ_SOURCE_DIR}/xzmq_serializer.hpp
 )
 
 # Targets and link

diff --git a/src/xauthentication.cpp b/src/xauthentication.cpp
@@ -27,7 +27,7 @@
 #endif
 
 #include "xeus/xstring_utils.hpp"
-#include "xeus-zmq/xauthentication.hpp"
+#include "xauthentication.hpp"
 
 namespace xeus
 {

diff --git a/include/xeus-zmq/xauthentication.hpp → src/xauthentication.hpp b/include/xeus-zmq/xauthentication.hpp → src/xauthentication.hpp
@@ -13,11 +13,9 @@
 #include <memory>
 #include <string>
 
-#include "xeus-zmq.hpp"
-
 namespace xeus
 {
-    class XEUS_ZMQ_API xraw_buffer
+    class xraw_buffer
     {
     public:
 
@@ -33,7 +31,7 @@ namespace xeus
         size_t m_size;
     };
 
-    class XEUS_ZMQ_API xauthentication
+    class xauthentication
     {
     public:
 
@@ -74,7 +72,6 @@ namespace xeus
                                  const xraw_buffer& content) const = 0;
     };
 
-    XEUS_ZMQ_API
     std::unique_ptr<xauthentication> make_xauthentication(const std::string& scheme,
                                                           const std::string& key);
 }

diff --git a/src/xclient_zmq_impl.cpp b/src/xclient_zmq_impl.cpp
@@ -9,9 +9,9 @@
 
 #include <iostream>
 
-#include "xeus-zmq/xauthentication.hpp"
+#include "xauthentication.hpp"
 #include "xclient_zmq_impl.hpp"
-#include "xeus-zmq/xzmq_serializer.hpp"
+#include "xzmq_serializer.hpp"
 
 namespace xeus
 {

diff --git a/src/xserver_zmq_default.cpp b/src/xserver_zmq_default.cpp
@@ -11,8 +11,8 @@
 #include <iostream>
 
 #include "xeus/xguid.hpp"
-#include "xeus-zmq/xauthentication.hpp"
-#include "xeus-zmq/xzmq_serializer.hpp"
+#include "xauthentication.hpp"
+#include "xzmq_serializer.hpp"
 #include "xserver_zmq_default.hpp"
 #include "xmiddleware_impl.hpp"
 #include "xpublisher.hpp"

diff --git a/src/xserver_zmq_split.cpp b/src/xserver_zmq_split.cpp
@@ -12,9 +12,9 @@
 
 #include "zmq_addon.hpp"
 #include "xeus/xguid.hpp"
-#include "xeus-zmq/xauthentication.hpp"
 #include "xeus-zmq/xmiddleware.hpp"
-#include "xeus-zmq/xzmq_serializer.hpp"
+#include "xauthentication.hpp"
+#include "xzmq_serializer.hpp"
 #include "xserver_zmq_split.hpp"
 #include "xcontrol.hpp"
 #include "xheartbeat.hpp"

diff --git a/src/xzmq_serializer.cpp b/src/xzmq_serializer.cpp
@@ -7,7 +7,7 @@
 * The full license is in the file LICENSE, distributed with this software. *
 ****************************************************************************/
 
-#include "xeus-zmq/xzmq_serializer.hpp"
+#include "xzmq_serializer.hpp"
 
 namespace xeus
 {

diff --git a/include/xeus-zmq/xzmq_serializer.hpp → src/xzmq_serializer.hpp b/include/xeus-zmq/xzmq_serializer.hpp → src/xzmq_serializer.hpp
@@ -14,12 +14,11 @@
 
 #include "xeus/xmessage.hpp"
 
-#include "xeus-zmq.hpp"
 #include "xauthentication.hpp"
 
 namespace xeus
 {
-    class XEUS_ZMQ_API xzmq_serializer
+    class xzmq_serializer
     {
     public:
 

diff --git a/test/client_ipc.cpp b/test/client_ipc.cpp
@@ -6,7 +6,12 @@
 * The full license is in the file LICENSE, distributed with this software. *
 ****************************************************************************/
 
-#include <iostream>
+// TODO: this client should be reimplemented with
+// the new client framework. Notice that it was not
+// tested in the CI but was here for trying to reproduce
+// some issue with IPC.
+
+/*#include <iostream>
 
 #include "zmq_addon.hpp"
 #include "nlohmann/json.hpp"
@@ -53,5 +58,5 @@ int main(int, char**)
     std::cout << resp.content().dump(4) << std::endl;
 
     return 0;
-}
+}*/