Include project headers relative to top-level src directory

CMakeLists.txt

@@ -1,28 +1,16 @@
 cmake_minimum_required(VERSION 3.8)
-  if(COMMAND cmake_policy)
-    cmake_policy(SET CMP0003 NEW)
-  endif(COMMAND cmake_policy)
+if(COMMAND cmake_policy)
+  cmake_policy(SET CMP0003 NEW)
+endif(COMMAND cmake_policy)
   
 project(blah2)
 include(CMakePrintHelpers)
+include(CTest)
 
 SET(CMAKE_CXX_STANDARD 20)
 SET(CMAKE_CXX_STANDARD_REQUIRED ON)
 SET(CMAKE_CXX_EXTENSIONS OFF)
-
-SET (PROJECT_ROOT "${PROJECT_SOURCE_DIR}")
 SET (CMAKE_RUNTIME_OUTPUT_DIRECTORY "${PROJECT_ROOT}/bin")
-SET (PROJECT_TEST_DIR "${PROJECT_SOURCE_DIR}/test")
-SET (PROJECT_SOURCE_DIR "${PROJECT_SOURCE_DIR}/src")
-SET (PROJECT_BINARY_DIR "${PROJECT_ROOT}/bin")
-SET (PROJECT_BINARY_TEST_DIR "${PROJECT_ROOT}/bin/test")
-SET (PROJECT_BINARY_TEST_UNIT_DIR "${PROJECT_ROOT}/bin/test/unit")
-SET (PROJECT_LIB_DIR "${PROJECT_ROOT}/lib")
-MESSAGE ("Source path: ${PROJECT_SOURCE_DIR}")
-MESSAGE ("Binary path: ${PROJECT_BINARY_DIR}")
-MESSAGE ("Binary test path: ${PROJECT_BINARY_TEST_DIR}")
-MESSAGE ("Lib path: ${PROJECT_LIB_DIR}")
-MESSAGE ("CMAKE_TOOLCHAIN_FILE=${CMAKE_TOOLCHAIN_FILE}")
 
 find_package(Threads REQUIRED)
 find_package(asio CONFIG REQUIRED)
@@ -32,7 +20,16 @@ find_package(httplib REQUIRED)
 find_package(Armadillo REQUIRED)
 find_package(Catch2 CONFIG REQUIRED)
 
-# find_package(fftw3 REQUIRED)
+# TODO: When release CI is finished, don't use these dirs, install target should go to prod
+SET (PROJECT_ROOT "${PROJECT_SOURCE_DIR}")
+SET (CMAKE_RUNTIME_OUTPUT_DIRECTORY "${PROJECT_ROOT}/bin")
+SET (PROJECT_BINARY_TEST_DIR "${PROJECT_ROOT}/bin/test")
+
+MESSAGE ("Binary path: ${PROJECT_BINARY_DIR}")
+MESSAGE ("Binary test path: ${PROJECT_BINARY_TEST_DIR}")
+
+# Include from top-level src dir
+include_directories(src)
 
 # TODO: Create FindSdrplay.cmake for this
 add_library(sdrplay /usr/local/include/sdrplay_api.h)
@@ -68,35 +65,27 @@ target_link_libraries(blah2 PRIVATE
   fftw3
   fftw3_threads
   sdrplay
-  )
-
-include_directories("${PROJECT_SOURCE_DIR}/capture/")
-include_directories("${PROJECT_SOURCE_DIR}/capture/rspduo/")
-include_directories("${PROJECT_SOURCE_DIR}/process/ambiguity/")
-include_directories("${PROJECT_SOURCE_DIR}/process/clutter/")
-include_directories("${PROJECT_SOURCE_DIR}/process/detection/")
-include_directories("${PROJECT_SOURCE_DIR}/process/tracker/")
-include_directories("${PROJECT_SOURCE_DIR}/process/spectrum/")
-include_directories("${PROJECT_SOURCE_DIR}/process/meta/")
-include_directories("${PROJECT_SOURCE_DIR}/data/")
-include_directories("${PROJECT_SOURCE_DIR}/data/meta/")
+)
 
 # unit tests
 add_executable(testAmbiguity
-  ${PROJECT_TEST_DIR}/unit/process/ambiguity/TestAmbiguity.cpp
-  ${PROJECT_SOURCE_DIR}/data/IqData.cpp
-  ${PROJECT_SOURCE_DIR}/data/Map.cpp
-  ${PROJECT_SOURCE_DIR}/process/ambiguity/Ambiguity.cpp
-  ${PROJECT_SOURCE_DIR}/process/meta/HammingNumber.cpp)
+  test/unit/process/ambiguity/TestAmbiguity.cpp
+  src/data/IqData.cpp
+  src/data/Map.cpp
+  src/process/ambiguity/Ambiguity.cpp
+  src/process/meta/HammingNumber.cpp)
 target_link_libraries(testAmbiguity PRIVATE Catch2::Catch2WithMain fftw3 fftw3_threads)
 set_target_properties(testAmbiguity PROPERTIES 
   RUNTIME_OUTPUT_DIRECTORY "${PROJECT_BINARY_TEST_UNIT_DIR}")
 
 add_executable(testTracker
-  ${PROJECT_TEST_DIR}/unit/process/tracker/TestTracker.cpp
-  ${PROJECT_SOURCE_DIR}/data/Detection.cpp
-  ${PROJECT_SOURCE_DIR}/data/Track.cpp
-  ${PROJECT_SOURCE_DIR}/process/tracker/Tracker.cpp)
+  test/unit/process/tracker/TestTracker.cpp
+  src/data/Detection.cpp
+  src/data/Track.cpp
+  src/process/tracker/Tracker.cpp)
 target_link_libraries(testTracker PRIVATE Catch2::Catch2WithMain)
 set_target_properties(testTracker PROPERTIES 
   RUNTIME_OUTPUT_DIRECTORY "${PROJECT_BINARY_TEST_UNIT_DIR}")
+
+add_test(NAME testAmbiguity COMMAND testAmbiguity)
+add_test(NAME testTracker COMMAND testTracker)

src/blah2.cpp

@@ -2,23 +2,24 @@
 /// @brief A real-time radar.
 /// @author 30hours
 
+#include "capture/Capture.h"
+#include "data/IqData.h"
+#include "data/Map.h"
+#include "data/Detection.h"
+#include "data/meta/Timing.h"
+#include "data/Track.h"
+#include "process/ambiguity/Ambiguity.h"
+#include "process/clutter/WienerHopf.h"
+#include "process/detection/CfarDetector1D.h"
+#include "process/detection/Centroid.h"
+#include "process/detection/Interpolate.h"
+#include "process/spectrum/SpectrumAnalyser.h"
+#include "process/tracker/Tracker.h"
+
 #include <ryml/ryml.hpp>
 #include <ryml/ryml_std.hpp> // optional header, provided for std:: interop
 #include <c4/format.hpp> // needed for the examples below
 #include <asio.hpp>
-#include <Capture.h>
-#include <Ambiguity.h>
-#include <WienerHopf.h>
-#include <CfarDetector1D.h>
-#include <Tracker.h>
-#include <IqData.h>
-#include <Map.h>
-#include <Detection.h>
-#include <Track.h>
-#include <Centroid.h>
-#include <Interpolate.h>
-#include <Timing.h>
-#include <SpectrumAnalyser.h>
 #include <sys/types.h>
 #include <getopt.h>
 #include <string>

src/capture/Capture.cpp

@@ -1,5 +1,5 @@
 #include "Capture.h"
-#include "RspDuo.h"
+#include "rspduo/RspDuo.h"
 #include <iostream>
 #include <thread>
 #include <httplib.h>

src/capture/Capture.h

@@ -7,7 +7,7 @@
 #define CAPTURE_H
 
 #include <string>
-#include <IqData.h>
+#include "data/IqData.h"
 
 class Capture
 {

src/capture/rspduo/RspDuo.cpp

@@ -1,4 +1,5 @@
-#include <RspDuo.h>
+#include "RspDuo.h"
+
 #include <stdbool.h>
 #include <stdint.h>
 #include <stdio.h>

src/capture/rspduo/RspDuo.h

@@ -20,9 +20,10 @@
 #define RSPDUO_H
 
 #include "sdrplay_api.h"
+#include "data/IqData.h"
+
 #include <stdint.h>
 #include <string>
-#include <IqData.h>
 
 #define BUFFER_SIZE_NR 1024
 

src/data/Track.h

@@ -15,7 +15,8 @@
 #ifndef TRACK_H
 #define TRACK_H
 
-#include <Detection.h>
+#include "data/Detection.h"
+
 #include <stdint.h>
 #include <vector>
 #include <string>

src/process/ambiguity/Ambiguity.h

@@ -8,9 +8,9 @@
 
 #pragma once
 
-#include <IqData.h>
-#include <Map.h>
-#include <HammingNumber.h>
+#include "data/IqData.h"
+#include "data/Map.h"
+#include "process/meta/HammingNumber.h"
 #include <stdint.h>
 #include <fftw3.h>
 #include <memory>

src/process/clutter/WienerHopf.h

@@ -9,7 +9,7 @@
 #ifndef WIENERHOPF_H
 #define WIENERHOPF_H
 
-#include <IqData.h>
+#include "data/IqData.h"
 #include <stdint.h>
 #include <fftw3.h>
 #include <armadillo>

src/process/detection/Centroid.h

@@ -7,7 +7,7 @@
 #ifndef CENTROID_H
 #define CENTROID_H
 
-#include <Detection.h>
+#include "data/Detection.h"
 #include <stdint.h>
 
 class Centroid

src/process/detection/CfarDetector1D.cpp

@@ -1,5 +1,6 @@
 #include "CfarDetector1D.h"
-#include "Map.h"
+#include "data/Map.h"
+
 #include <iostream>
 #include <vector>
 #include <cmath>

src/process/detection/CfarDetector1D.h

@@ -8,8 +8,8 @@
 #ifndef CFARDETECTOR1D_H
 #define CFARDETECTOR1D_H
 
-#include <Map.h>
-#include <Detection.h>
+#include "data/Map.h"
+#include "data/Detection.h"
 #include <stdint.h>
 #include <complex>
 

src/process/detection/Interpolate.h

@@ -11,8 +11,8 @@
 #ifndef INTERPOLATE_H
 #define INTERPOLATE_H
 
-#include <Map.h>
-#include <Detection.h>
+#include "data/Map.h"
+#include "data/Detection.h"
 
 class Interpolate
 {

src/process/spectrum/SpectrumAnalyser.h

@@ -9,7 +9,7 @@
 #ifndef SPECTRUMANALYSER_H
 #define SPECTRUMANALYSER_H
 
-#include <IqData.h>
+#include "data/IqData.h"
 #include <stdint.h>
 #include <fftw3.h>
 

src/process/tracker/Tracker.h

@@ -10,8 +10,8 @@
 #ifndef TRACKER_H
 #define TRACKER_H
 
-#include <Detection.h>
-#include <Track.h>
+#include "data/Detection.h"
+#include "data/Track.h"
 #include <stdint.h>
 
 class Tracker

test/unit/process/ambiguity/TestAmbiguity.cpp

@@ -9,7 +9,7 @@
 #include <catch2/matchers/catch_matchers_floating_point.hpp>
 #include <catch2/generators/catch_generators.hpp>
 
-#include "Ambiguity.h"
+#include "process/ambiguity/Ambiguity.h"
 #include <random>
 #include <iostream>
 

test/unit/process/tracker/TestTracker.cpp

@@ -4,9 +4,10 @@
 
 #include <catch2/catch_test_macros.hpp>
 
-#include "Detection.h"
-#include "Tracker.h"
-#include "Track.h"
+#include "data/Detection.h"
+#include "data/Track.h"
+#include "process/tracker/Tracker.h"
+
 #include <string>
 #include <vector>
 #include <random>