Unverified Commit 1ec72988 authored by Sergei Zimmerman's avatar Sergei Zimmerman Committed by Sergei Zimmerman
Browse files

lib2geom: use ctestCheckHook 

- Resolves a todo.
- Needs `dontUseNinjaCheck = true` to not use ninja's checkPhase.
parent 2720a8b3

pkgs/by-name/li/lib2geom/package.nix

+24 −31
Original line number Diff line number Diff line
@@ -10,6 +10,7 @@ 
  cairo,

  double-conversion,

  gtest,

  ctestCheckHook,

  lib,

  inkscape,

  pkgsCross,
@@ -47,6 +48,7 @@ stdenv.mkDerivation (finalAttrs: { 


  nativeCheckInputs = [

    gtest

    ctestCheckHook

  ];



  cmakeFlags = [
@@ -56,10 +58,7 @@ stdenv.mkDerivation (finalAttrs: { 
  ];



  doCheck = stdenv.buildPlatform.canExecute stdenv.hostPlatform;



  # TODO: Update cmake hook to make it simpler to selectively disable cmake tests: #113829

  checkPhase =

    let

  dontUseNinjaCheck = true;

  disabledTests =

    lib.optionals stdenv.hostPlatform.isMusl [

      # Fails due to rounding differences
@@ -85,12 +84,6 @@ stdenv.mkDerivation (finalAttrs: { 
      "bezier-test"

      "ellipse-test"

    ];

    in

    ''

      runHook preCheck

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

      runHook postCheck

    '';



  passthru = {

    tests = {