Unverified Commit 091041ce authored by Sergei Zimmerman's avatar Sergei Zimmerman
Browse files

pdal: migrate to ctestCheckHook

parent 6534a099

pkgs/by-name/pd/pdal/package.nix

+5 −8
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ 
  lib,

  stdenv,

  callPackage,

  ctestCheckHook,

  fetchFromGitHub,

  testers,
@@ -92,6 +93,9 @@ stdenv.mkDerivation (finalAttrs: { 
  ];



  doCheck = true;

  # tests are flaky and they seem to fail less often when they don't run in

  # parallel

  enableParallelChecking = false;



  disabledTests = [

    # Tests failing due to TileDB library implementation, disabled also
@@ -116,16 +120,9 @@ stdenv.mkDerivation (finalAttrs: { 


  nativeCheckInputs = [

    gdal # gdalinfo

    ctestCheckHook

  ];



  checkPhase = ''

    runHook preCheck

    # tests are flaky and they seem to fail less often when they don't run in

    # parallel

    ctest -j 1 --output-on-failure -E '^${lib.concatStringsSep "|" finalAttrs.disabledTests}$'

    runHook postCheck

  '';



  postInstall = ''

    patchShebangs --update --build $out/bin/pdal-config

  '';