Skip to content

Commit

Permalink
fix macOS build
Browse files Browse the repository at this point in the history
Summary:

macOS build was failing with:

   * missing header includes, added them

macOS test run was failing with:
   * libcpp exception text has changed from "with" to "due to" which was breaking ExceptionTest.cpp, update the expectation to cover both.
   * FileUtilDetail.cpp was only enabling custom temp dir on linux, breaking FileUtilTest.cpp.  Enable custom temp dir for non-windows (i.e. mac and other unix) to fix it.

Test Plan:

CI.  Before, build error on vector
```
CMakeFiles/tdigest_benchmark.dir/folly/stats/test/TDigestBenchmark.cpp.o -MF CMakeFiles/tdigest_benchmark.dir/folly/stats/test/TDigestBenchmark.cpp.o.d -o CMakeFiles/tdigest_benchmark.dir/folly/stats/test/TDigestBenchmark.cpp.o -c /Users/runner/work/folly/folly/folly/stats/test/TDigestBenchmark.cpp
In file included from /Users/runner/work/folly/folly/folly/stats/test/TDigestBenchmark.cpp:17:
/Users/runner/work/folly/folly/folly/stats/DigestBuilder.h:59:25: error: implicit instantiation of undefined template 'std::vector<double>'
    std::vector<double> buffer;
```

then build error on array
```
CMakeFiles/constexpr_math_test.dir/folly/test/ConstexprMathTest.cpp.o -c /Users/runner/work/folly/folly/folly/test/ConstexprMathTest.cpp
/Users/runner/work/folly/folly/folly/test/ConstexprMathTest.cpp:38:29: error: implicit instantiation of undefined template 'std::array<unsigned long, 7>'
    std::array<res_t, size> res{};
```

then test failues

After, works
  • Loading branch information
ahornby committed Oct 1, 2023
1 parent d8bef8b commit 0e2d678
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 4 deletions.
6 changes: 3 additions & 3 deletions folly/detail/FileUtilDetail.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,13 +44,13 @@ std::string getTemporaryFilePathStringWithoutTempDirectory(

std::string getTemporaryFilePathStringWithTemporaryDirectory(
const std::string& temporaryDirectory) {
#if defined(__linux__) && !FOLLY_MOBILE
#if !defined(_WIN32) && !FOLLY_MOBILE
return (temporaryDirectory.back() == '/')
? (temporaryDirectory + std::string{"tempForAtomicWrite.XXXXXX"})
: (temporaryDirectory + std::string{"/tempForAtomicWrite.XXXXXX"});
#else
// The implementation currently, does not support any platform other than
// linux for temporary directory based atomic file writes.
// The implementation currently does not support win32 or mobile
// for temporary directory based atomic file writes.
static_cast<void>(temporaryDirectory);
assert(false);

Expand Down
2 changes: 1 addition & 1 deletion folly/lang/test/ExceptionTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ template <typename Ex>
static std::string message_for_terminate_with(std::string const& what) {
auto const name = folly::pretty_name<Ex>();
std::string const p0 = "terminate called after throwing an instance of";
std::string const p1 = "terminating with uncaught exception of type";
std::string const p1 = "terminating (due to|with) uncaught exception of type";
// clang-format off
return
folly::kIsGlibcxx ? p0 + " '" + name + "'\\s+what\\(\\):\\s+" + what :
Expand Down
1 change: 1 addition & 0 deletions folly/stats/DigestBuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
#pragma once

#include <memory>
#include <vector>

#include <folly/Memory.h>
#include <folly/SpinLock.h>
Expand Down
1 change: 1 addition & 0 deletions folly/test/ConstexprMathTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@

#include <folly/ConstexprMath.h>

#include <array>
#include <cmath>
#include <limits>
#include <type_traits>
Expand Down

0 comments on commit 0e2d678

Please sign in to comment.