Skip to content

Cmake reorder files2

Shervin Nourbakhsh requested to merge cmake_reorder_files2 into master

Changes:

  • reordered files of the C++ API as follows:
    • cameo-api-cpp/src/cameo/ -> cameo-api-cpp/src/
    • public headers in cameo-api-cpp/include
    • fixed the include paths removing the relative paths and letting CMake to provide the correct list of include directories where to find the headers
  • removed protobuf from CMake configuration
  • added a draft of clang-format configuration: at some point we should enforce its usage before any push in order to make sure that the code is written by everyone respecting a given set of rules
    • benefits:
      • avoid changes in spaces and indent that would create artificially large diffs
      • the entire code has the same look
  • added Doxygen config file for generating the documentation page
  • added a README file with the list of dependencies and instructions to compile and install
  • added a gitlab CI YML file for automatic compilation checks, currently only Ubuntu being used
  • Added a CONTRIBUTING.md file with the instructions to propagate changes related to the C++ API from this repository to the dedicated C++ API repository using the "subtree" strategy.

List of installed files:

Install the project...
-- Install configuration: ""
-- Installing: /tmp/devel/lib64/cmake/cameoConfigVersion.cmake
-- Installing: /tmp/devel/include/cameo/include
-- Installing: /tmp/devel/include/cameo/include/ConnectionTimeout.h
-- Installing: /tmp/devel/include/cameo/include/RequesterCreationException.h
-- Installing: /tmp/devel/include/cameo/include/cameo.h
-- Installing: /tmp/devel/include/cameo/include/Services.h
-- Installing: /tmp/devel/include/cameo/include/RemoteException.h
-- Installing: /tmp/devel/include/cameo/include/ConcurrentQueue.h
-- Installing: /tmp/devel/include/cameo/include/EventStreamSocket.h
-- Installing: /tmp/devel/include/cameo/include/Response.h
-- Installing: /tmp/devel/include/cameo/include/Event.h
-- Installing: /tmp/devel/include/cameo/include/ConnectionChecker.h
-- Installing: /tmp/devel/include/cameo/include/EventListener.h
-- Installing: /tmp/devel/include/cameo/include/SubscriberCreationException.h
-- Installing: /tmp/devel/include/cameo/include/UnmanagedApplicationException.h
-- Installing: /tmp/devel/include/cameo/include/OutputStreamSocket.h
-- Installing: /tmp/devel/include/cameo/include/InvalidArgumentException.h
-- Installing: /tmp/devel/include/cameo/include/PublisherCreationException.h
-- Installing: /tmp/devel/include/cameo/include/Server.h
-- Installing: /tmp/devel/include/cameo/include/SocketException.h
-- Installing: /tmp/devel/include/cameo/include/Serializer.h
-- Installing: /tmp/devel/include/cameo/include/TimeCondition.h
-- Installing: /tmp/devel/include/cameo/include/Application.h
-- Installing: /tmp/devel/include/cameo/include/ResponderCreationException.h
-- Installing: /tmp/devel/lib64/libcameo.so.0.3.0
-- Installing: /tmp/devel/lib64/libcameo.so.0
-- Installing: /tmp/devel/lib64/libcameo.so
-- Installing: /tmp/devel/lib64/cmake/cameoConfig.cmake
-- Installing: /tmp/devel/lib64/cmake/cameoConfig-noconfig.cmake

This merge request addresses the following issues:

Propagation of changes from this repository to the one dedicated to the C++ API

The main issue is that fake commits are created by the process. A string "(split#)" is prepended to the commit message to recognize them. They should be ignored when looking at the history.

Edited by Shervin Nourbakhsh

Merge request reports