Skip to content

Commit

Permalink
edencommon: add patch for glog ≥ 0.7.0
Browse files Browse the repository at this point in the history
  • Loading branch information
emilazy committed Jan 6, 2025
1 parent 40bc7c2 commit 05f5a38
Show file tree
Hide file tree
Showing 2 changed files with 35 additions and 4 deletions.
27 changes: 27 additions & 0 deletions pkgs/by-name/ed/edencommon/glog-0.7.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 8d7b3454df..2ce7b5af1a 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -50,8 +50,7 @@
"${CMAKE_CURRENT_SOURCE_DIR}/build/fbcode_builder/CMake"
${CMAKE_MODULE_PATH})

-find_package(Glog MODULE REQUIRED)
-include_directories(${GLOG_INCLUDE_DIR})
+find_package(Glog CONFIG REQUIRED)

find_package(Gflags REQUIRED)
include_directories(${GFLAGS_INCLUDE_DIR})
diff --git a/eden/common/testharness/CMakeLists.txt b/eden/common/testharness/CMakeLists.txt
index bef7421906..f35067efa9 100644
--- a/eden/common/testharness/CMakeLists.txt
+++ b/eden/common/testharness/CMakeLists.txt
@@ -19,7 +19,7 @@
${BOOST_LIBRARIES}
Folly::folly_test_util
${LIBGMOCK_LIBRARIES}
- ${GLOG_LIBRARY}
+ glog::glog
)

target_include_directories(
12 changes: 8 additions & 4 deletions pkgs/by-name/ed/edencommon/package.nix
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,14 @@ stdenv.mkDerivation (finalAttrs: {
hash = "sha256-9JCyXFWglnIuDw5jSSqcnuMfQ2JXMdNwFVyyBccjoag=";
};

patches = lib.optionals (stdenv.hostPlatform.isDarwin && stdenv.hostPlatform.isx86_64) [
# Test discovery timeout is bizarrely flaky on `x86_64-darwin`
./increase-test-discovery-timeout.patch
];
patches =
[
./glog-0.7.patch
]
++ lib.optionals (stdenv.hostPlatform.isDarwin && stdenv.hostPlatform.isx86_64) [
# Test discovery timeout is bizarrely flaky on `x86_64-darwin`
./increase-test-discovery-timeout.patch
];

nativeBuildInputs = [
cmake
Expand Down

0 comments on commit 05f5a38

Please sign in to comment.