More fixes to make the code compatible with ancient absl.

substrait-io · Jul 8, 2023 · 6d19bcc · 6d19bcc
1 parent 5d86e64
commit 6d19bcc
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/src/substrait/common/CMakeLists.txt b/src/substrait/common/CMakeLists.txt
@@ -1,7 +1,5 @@
 # SPDX-License-Identifier: Apache-2.0
 
-find_package(fmt)
-
 add_library(substrait_common Exceptions.cpp)
 
 target_link_libraries(substrait_common fmt::fmt-header-only)

diff --git a/src/substrait/textplan/parser/SubstraitPlanRelationVisitor.cpp b/src/substrait/textplan/parser/SubstraitPlanRelationVisitor.cpp
@@ -790,7 +790,7 @@ std::any SubstraitPlanRelationVisitor::visitExpressionFunctionUse(
  expr.mutable_scalar_function()->set_function_reference(funcReference);
  for (const auto& exp : ctx->expression()) {
  if (endsWith(exp->getText(), "_enum")) {
- auto str = absl::StripSuffix(exp->getText(), "_enum");
+ std::string str = absl::StripSuffix(exp->getText(), "_enum");
  expr.mutable_scalar_function()->add_arguments()->set_enum_(str);
  continue;
  }