Update transport, fix metrics to use shared ptr, and add go for macos…

… build
Quicr · Apr 28, 2024 · e715dee · e715dee
1 parent 1dc747d
commit e715dee
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/cmake.yml b/.github/workflows/cmake.yml
@@ -29,7 +29,7 @@ jobs:
 
     - name: Install Packages (macOS)
       run: |
-        brew install pkg-config llvm
+        brew install pkg-config llvm go
         ln -s "$(brew --prefix llvm)/bin/clang-tidy" "/usr/local/bin/clang-tidy"
         brew install google-benchmark
       if: matrix.os == 'macos-latest'

diff --git a/dependencies/transport b/dependencies/transport
diff --git a/src/metrics_exporter.cpp b/src/metrics_exporter.cpp
@@ -26,8 +26,8 @@ namespace quicr {
     MetricsExporter::MetricsExporter(const cantina::LoggerPointer& logger) :
         logger(std::make_shared<cantina::Logger>("MExport", logger))
     {
-      metrics_conn_samples = std::make_unique<safe_queue<MetricsConnSample>>(MAX_METRICS_SAMPLES_QUEUE);
-      metrics_data_samples = std::make_unique<safe_queue<MetricsDataSample>>(MAX_METRICS_SAMPLES_QUEUE);
+      metrics_conn_samples = std::make_shared<safe_queue<MetricsConnSample>>(MAX_METRICS_SAMPLES_QUEUE);
+      metrics_data_samples = std::make_shared<safe_queue<MetricsDataSample>>(MAX_METRICS_SAMPLES_QUEUE);
     }
 
     MetricsExporter::~MetricsExporter() {