macOS build fix

.github/workflows/macos.yaml

@@ -16,23 +16,35 @@ jobs:
       - name: Sync repository
         uses: actions/checkout@v2
 
+      - name: Update PATH
+        run: echo "PATH=$PATH:/opt/homebrew/bin:/opt/homebrew/sbin" >> $GITHUB_ENV
+
+      - name: Install dependencies
+        run: |
+          source $GITHUB_ENV
+          brew update
+          brew install mosquitto zeromq
+        shell: bash
+
       - name: Cache Qt
         id: cache-qt
-        uses: actions/cache@v1  # not v2!
+        uses: actions/cache@v4  # updated to v4
         with:
           path: '${{ github.workspace }}/qt_installation/'
           key: ${{ runner.os }}-QtCache
 
       - name: Install Qt
-        uses: jurplel/install-qt-action@v2.13.0
+        uses: jurplel/install-qt-action@v4.1.0  # updated to v4.1.0
         with:
           version: '5.15.2'
           host: 'mac'
           dir: '${{ github.workspace }}/qt_installation/'
           cached: ${{ steps.cache-qt.outputs.cache-hit }}
+          aqtversion: '==3.1.18'  # updated to 3.1.18
+          py7zrversion: '==0.22.0'
 
       - name: Build Plotjuggler
         shell: pwsh
-        run: >
-          cmake -B build -DCMAKE_INSTALL_PREFIX=install PlotJuggler;
-          cmake --build build --target install
+        run: |
+          cmake -B build -DCMAKE_INSTALL_PREFIX=install PlotJuggler
+          cmake --build build --target install

COMPILE.md

@@ -93,8 +93,10 @@ brew install cmake qt@5 protobuf mosquitto zeromq zstd
 If a newer version of qt is installed, you may need to temporarily link to qt5
 
 ```shell
-brew link qt@5 --override
-# brew link qt --override # Run once you are done building to restore the original linking
+brew link qt@5 --overwrite
+#In case needed and still qt@5 was not found by cmake you can do:
+brew unlink qt@5 && brew link --force qt@5
+# brew link qt --overwrite # Run once you are done building to restore the original linking
 ```
 
 Add CMake into your env-vars to be detected by cmake

cmake/FindZeroMQ.cmake

@@ -5,13 +5,9 @@
 # ZeroMQ_LIBRARIES - The libraries needed to use ZeroMQ
 # ZeroMQ_DEFINITIONS - Compiler switches required for using ZeroMQ
 
-find_path ( ZeroMQ_INCLUDE_DIR zmq.h )
-find_library ( ZeroMQ_LIBRARY NAMES zmq )
+find_package(PkgConfig REQUIRED)
+pkg_check_modules(ZMQ REQUIRED libzmq)
 
-set ( ZeroMQ_LIBRARIES ${ZeroMQ_LIBRARY} )
-set ( ZeroMQ_INCLUDE_DIRS ${ZeroMQ_INCLUDE_DIR} )
-
-include ( FindPackageHandleStandardArgs )
-# handle the QUIETLY and REQUIRED arguments and set ZeroMQ_FOUND to TRUE
-# if all listed variables are TRUE
-find_package_handle_standard_args ( ZeroMQ DEFAULT_MSG ZeroMQ_LIBRARY ZeroMQ_INCLUDE_DIR )
+set(ZeroMQ_FOUND TRUE)
+set(ZeroMQ_INCLUDE_DIRS ${ZMQ_INCLUDE_DIRS})
+set(ZeroMQ_LIBRARIES ${ZMQ_LIBRARIES})

plotjuggler_plugins/DataStreamZMQ/CMakeLists.txt

@@ -1,33 +1,54 @@
+# Check for vcpkg, conan, or manual build environments
 if(BUILDING_WITH_VCPKG)
     message(STATUS "Finding ZeroMQ with vcpkg")
 elseif(BUILDING_WITH_CONAN)
     message(STATUS "Finding ZeroMQ with conan")
 else()
     message(STATUS "Finding ZeroMQ without package managers")
-    set(ZeroMQ_LIBS ${ZeroMQ_LIBRARIES})
+
+    # Find ZeroMQ using PkgConfig
+    find_package(PkgConfig REQUIRED)
+    pkg_check_modules(ZMQ REQUIRED libzmq)
+
+    # Set the ZeroMQ libraries and include directories for manual configuration
+    set(ZeroMQ_LIBRARIES ${ZMQ_LIBRARIES})
+    set(ZeroMQ_INCLUDE_DIRS ${ZMQ_INCLUDE_DIRS})
+    set(ZeroMQ_LIBRARY_DIRS /opt/homebrew/lib)  # Add this line
+
+    # Add the library path explicitly
+    link_directories(${ZeroMQ_LIBRARY_DIRS})  # Add this line
 endif()
 
+# Find ZeroMQ library
 find_package(ZeroMQ QUIET)
 
+# Check if ZeroMQ was found
 if(ZeroMQ_FOUND)
     message(STATUS "[ZeroMQ] found")
 
+    # Add QT definitions if needed
     add_definitions(${QT_DEFINITIONS})
     add_definitions(-DQT_PLUGIN)
 
-    QT5_WRAP_UI ( UI_SRC  datastream_zmq.ui  )
+    # Wrap the UI file for Qt
+    QT5_WRAP_UI(UI_SRC datastream_zmq.ui)
 
-    add_library(DataStreamZMQ SHARED datastream_zmq.cpp ${UI_SRC}  )
+    # Add the DataStreamZMQ library
+    add_library(DataStreamZMQ SHARED datastream_zmq.cpp ${UI_SRC})
 
+    # Link Qt5Widgets and the plotjuggler_base target to DataStreamZMQ
     target_link_libraries(DataStreamZMQ ${Qt5Widgets_LIBRARIES} plotjuggler_base)
 
+    # Handle different library linking for vcpkg, conan, or manual builds
     if(BUILDING_WITH_VCPKG OR BUILDING_WITH_CONAN)
         target_link_libraries(DataStreamZMQ libzmq-static)
     else()
-        target_link_libraries(DataStreamZMQ ${ZeroMQ_LIBRARIES})
+        target_include_directories(DataStreamZMQ PRIVATE ${ZeroMQ_INCLUDE_DIRS})
+        target_link_libraries(DataStreamZMQ zmq)  # Changed this to just 'zmq'
     endif()
 
-    install(TARGETS DataStreamZMQ DESTINATION ${PJ_PLUGIN_INSTALL_DIRECTORY}  )
+    # Install the plugin
+    install(TARGETS DataStreamZMQ DESTINATION ${PJ_PLUGIN_INSTALL_DIRECTORY})
 else()
-    message("[ZeroMQ] not found. Skipping plugin DataStreamZMQ.")
+    message(STATUS "[ZeroMQ] not found. Skipping plugin DataStreamZMQ.")
 endif()

plotjuggler_plugins/ParserProtobuf/error_collectors.cpp

@@ -23,6 +23,15 @@ void FileErrorCollector::AddWarning(const std::string& filename, int line, int,
   qDebug() << msg;
 }
 
+void FileErrorCollector::RecordError(absl::string_view filename, int line, int column,
+                                     absl::string_view message) {
+    QString errorMessage = QString("Error in file: %1 at line: %2, column: %3: %4")
+                               .arg(QString::fromStdString(std::string(filename)))
+                               .arg(line)
+                               .arg(column)
+                               .arg(QString::fromStdString(std::string(message)));
+    _errors << errorMessage;
+}
 void IoErrorCollector::AddError(int line, google::protobuf::io::ColumnNumber,
                                 const std::string& message)
 {

plotjuggler_plugins/ParserProtobuf/error_collectors.h

@@ -10,10 +10,10 @@ class IoErrorCollector : public google::protobuf::io::ErrorCollector
 {
 public:
   void AddError(int line, google::protobuf::io::ColumnNumber column,
-                const std::string& message) override;
+                const std::string& message) ;
 
   void AddWarning(int line, google::protobuf::io::ColumnNumber column,
-                  const std::string& message) override;
+                  const std::string& message);
 
   const QStringList& errors()
   {
@@ -27,19 +27,25 @@ class IoErrorCollector : public google::protobuf::io::ErrorCollector
 class FileErrorCollector : public google::protobuf::compiler::MultiFileErrorCollector
 {
 public:
-  void AddError(const std::string& filename, int line, int,
-                const std::string& message) override;
-
-  void AddWarning(const std::string& filename, int line, int,
-                  const std::string& message) override;
-
-  const QStringList& errors()
-  {
-    return _errors;
-  }
-
+    void AddError(const std::string& filename, int line, int column,
+                  const std::string& message);
+
+    void AddWarning(const std::string& filename, int line, int column,
+                    const std::string& message);
+    void RecordError(absl::string_view filename, int line, int column,
+                     absl::string_view message) override;
+
+    const QStringList& errors() const {
+        return _errors;
+    }
+
+    // Accessor for the collected warnings
+    const QStringList& warnings() const {
+        return _warnings;
+    }
 private:
-  QStringList _errors;
+    QStringList _errors;
+    QStringList _warnings;
 };
 
 #endif  // ERROR_COLLECTORS_H