Merge pull request #80004 from peterhoeg/f/crun

crun: run tests and show the proper version number

pkgs/applications/virtualization/crun/default.nix

@@ -1,6 +1,36 @@
-{ stdenv, lib, fetchFromGitHub, autoreconfHook, go-md2man, pkgconfig
+{ stdenv
-, libcap, libseccomp, python3, systemd, yajl }:
+, lib
+, fetchFromGitHub
+, autoreconfHook
+, go-md2man
+, pkgconfig
+, libcap
+, libseccomp
+, python3
+, systemd
+, yajl
+}:
 
+let
+  # these tests require additional permissions
+  disabledTests = [
+    "test_capabilities.py"
+    "test_cwd.py"
+    "test_detach.py"
+    "test_exec.py"
+    "test_hooks.py"
+    "test_hostname.py"
+    "test_paths.py"
+    "test_pid.py"
+    "test_pid_file.py"
+    "test_preserve_fds.py"
+    "test_start.py"
+    "test_uid_gid.py"
+    "test_update.py"
+    "tests_libcrun_utils"
+  ];
+
+in
 stdenv.mkDerivation rec {
   pname = "crun";
   version = "0.12.2.1";
@@ -19,16 +49,18 @@ stdenv.mkDerivation rec {
 
   enableParallelBuilding = true;
 
-  preBuild = ''
+  # we need this before autoreconfHook does its thing in order to initialize
-    cat > git-version.h <<EOF
+  # config.h with the correct values
-    #ifndef GIT_VERSION
+  postPatch = ''
-    # define GIT_VERSION "nixpkgs-${version}"
+    echo ${version} > .tarball-version
-    #endif
+    echo '#define GIT_VERSION "${src.rev}"' > git-version.h
-    EOF
+
+    ${lib.concatMapStringsSep "\n" (e:
+      "substituteInPlace Makefile.am --replace 'tests/${e}' ''"
+    ) disabledTests}
   '';
 
-  # the tests require additional permissions
+  doCheck = true;
-  doCheck = false;
 
   meta = with lib; {
     description = "A fast and lightweight fully featured OCI runtime and C library for running containers";