Unverified Commit dd5343d5 authored by Sandro Jäckel's avatar Sandro Jäckel Committed by GitHub
Browse files

Merge pull request #290250 from lukegb/rapidjson-gtest

parents 999dc2b6 5cafc420
Loading
Loading
Loading
Loading
+8 −1
Original line number Diff line number Diff line
@@ -43,7 +43,14 @@ stdenv.mkDerivation rec {
  # for tests, adding gtest to checkInputs does not work
  # https://github.com/NixOS/nixpkgs/pull/212200
  buildInputs = [ gtest ];
  cmakeFlags = [ "-DGTEST_SOURCE_DIR=${gtest.dev}/include" ];
  cmakeFlags = [
    "-DGTEST_SOURCE_DIR=${gtest.dev}/include"

    # Build rapidjson with std=c++17 so gtest 1.13.0+ works
    # https://github.com/NixOS/nixpkgs/pull/282245#issuecomment-1951796902
    "-DRAPIDJSON_BUILD_CXX11=OFF"
    "-DRAPIDJSON_BUILD_CXX17=ON"
  ];

  nativeCheckInputs = [ valgrind ];
  doCheck = !stdenv.hostPlatform.isStatic && !stdenv.isDarwin;