Misc fixes for oss (facebookresearch#134)

Summary: Pull Request resolved: facebookresearch#134 X-link: facebookresearch/vrs#168 - fmt::join has been moved in the fmt library from one header to another. We need to include both location to be sure to build in all context and in open source with older and newer versions of fmt. - make sure the vrs hash is current - remove some fb-only includes Differential Revision: D66797878
georges-berenger · Dec 5, 2024 · 7ac4a69 · 7ac4a69
1 parent baaef19
commit 7ac4a69
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/csrc/reader/MultiVRSReader.cpp b/csrc/reader/MultiVRSReader.cpp
@@ -23,6 +23,7 @@
 #include <string>
 
 #include <fmt/format.h>
+#include <fmt/ranges.h>
 
 #include <pybind11/attr.h>
 #include <pybind11/cast.h>

diff --git a/csrc/vrs b/csrc/vrs
diff --git a/csrc/writer/VRSWriter.cpp b/csrc/writer/VRSWriter.cpp
@@ -35,10 +35,6 @@
 #define DEFAULT_LOG_CHANNEL "VRSWriter"
 #include <logging/Log.h>
 
-#include <vrs/gaia/GaiaClient.h>
-#include <vrs/gaia/GaiaUploader.h>
-#include <vrs/gaia/UploadMetadata.h>
-
 #include "../VrsBindings.h"
 #include "../utils/PyUtils.h"
 #include "StreamFactory.h"