From 3a9a98fd48adbcdfa11f48c65bf2ee883fd0884a Mon Sep 17 00:00:00 2001 From: Ben Darwin Date: Fri, 7 Jun 2024 20:38:45 -0400 Subject: [PATCH 1/3] castxml: unbreak binary by adding llvm to buildInputs Fixes #318055. --- pkgs/by-name/ca/castxml/package.nix | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkgs/by-name/ca/castxml/package.nix b/pkgs/by-name/ca/castxml/package.nix index 684d868be686..1c254f805d79 100644 --- a/pkgs/by-name/ca/castxml/package.nix +++ b/pkgs/by-name/ca/castxml/package.nix @@ -30,7 +30,6 @@ stdenv.mkDerivation (finalAttrs: { nativeBuildInputs = [ cmake - (lib.getDev llvm) ] ++ lib.optionals (withManual || withHTML) [ sphinx @@ -39,6 +38,7 @@ stdenv.mkDerivation (finalAttrs: { buildInputs = [ libffi libxml2 + llvm zlib ] ++ lib.optionals (!stdenv.isDarwin) [ libclang From 068d1888e7d02d1e237bbd1ca94564fecb4637f8 Mon Sep 17 00:00:00 2001 From: Ben Darwin Date: Fri, 7 Jun 2024 21:25:29 -0400 Subject: [PATCH 2/3] castxml: use `testers.testVersion` in `passthru.tests` (in order to prevent CLI regressions such as https://github.com/NixOS/nixpkgs/pull/318055) --- pkgs/by-name/ca/castxml/package.nix | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/pkgs/by-name/ca/castxml/package.nix b/pkgs/by-name/ca/castxml/package.nix index 1c254f805d79..35ba5dbac31a 100644 --- a/pkgs/by-name/ca/castxml/package.nix +++ b/pkgs/by-name/ca/castxml/package.nix @@ -7,6 +7,7 @@ llvmPackages, python3, stdenv, + testers, zlib, # Boolean flags withHTML ? true, @@ -66,6 +67,10 @@ stdenv.mkDerivation (finalAttrs: { runHook postCheck ''; + passthru.tests = testers.testVersion { + package = finalAttrs.finalPackage; + }; + meta = { homepage = "https://github.com/CastXML/CastXML"; description = "C-family Abstract Syntax Tree XML Output"; From fff1d11f45b72e25f6157fee747fd8cfbe647bfd Mon Sep 17 00:00:00 2001 From: Ben Darwin Date: Sat, 8 Jun 2024 00:02:56 -0400 Subject: [PATCH 3/3] castxml: fix CLANG_RESOURCE_DIR; enable tests --- pkgs/by-name/ca/castxml/package.nix | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/pkgs/by-name/ca/castxml/package.nix b/pkgs/by-name/ca/castxml/package.nix index 35ba5dbac31a..ba76cb9a445d 100644 --- a/pkgs/by-name/ca/castxml/package.nix +++ b/pkgs/by-name/ca/castxml/package.nix @@ -46,27 +46,19 @@ stdenv.mkDerivation (finalAttrs: { ]; cmakeFlags = [ - (lib.cmakeOptionType "path" "CLANG_RESOURCE_DIR" "${lib.getDev libclang}") + (lib.cmakeOptionType "path" "CLANG_RESOURCE_DIR" + "${lib.getLib libclang}/lib/clang/${lib.versions.major libclang.version}") + (lib.cmakeBool "SPHINX_HTML" withHTML) (lib.cmakeBool "SPHINX_MAN" withManual) ] ++ lib.optionals stdenv.isDarwin [ (lib.cmakeOptionType "path" "Clang_DIR" "${lib.getDev libclang}/lib/cmake/clang") ]; - # 97% tests passed, 97 tests failed out of 2881 - # mostly because it checks command line and nix append -isystem and all - doCheck = false; + doCheck = true; strictDeps = true; - # -E exclude 4 tests based on names - # see https://github.com/CastXML/CastXML/issues/90 - checkPhase = '' - runHook preCheck - ctest -E 'cmd.cc-(gnu|msvc)-((c-src-c)|(src-cxx))-cmd' - runHook postCheck - ''; - passthru.tests = testers.testVersion { package = finalAttrs.finalPackage; };