diff --git a/pkgs/applications/misc/ns-usbloader/default.nix b/pkgs/applications/misc/ns-usbloader/default.nix index 8ab52711a6cee..a97d3e6f9f88d 100644 --- a/pkgs/applications/misc/ns-usbloader/default.nix +++ b/pkgs/applications/misc/ns-usbloader/default.nix @@ -37,7 +37,6 @@ maven.buildMavenPackage rec { x86_64-linux = "sha256-vXZAlZOh9pXNF1RL78oQRal5pkXFRKDz/7SP9LibgiA="; aarch64-linux = "sha256-xC+feb41EPi30gBrVR8usanVULI2Pt0knztzNagPQiw="; }; - mvnParameters = "-DskipTests"; nativeBuildInputs = [ copyDesktopItems @@ -46,6 +45,8 @@ maven.buildMavenPackage rec { gvfs ]; + doCheck = false; + # Don't wrap binaries twice. dontWrapGApps = true; diff --git a/pkgs/applications/science/misc/openrefine/default.nix b/pkgs/applications/science/misc/openrefine/default.nix index 6a9d4be6c50b2..b94f23671024b 100644 --- a/pkgs/applications/science/misc/openrefine/default.nix +++ b/pkgs/applications/science/misc/openrefine/default.nix @@ -55,11 +55,13 @@ in maven'.buildMavenPackage { postPatch = '' cp -r ${npmPkg} main/webapp/modules/core/3rdparty ''; - mvnParameters = "-DskipTests=true -pl !packaging"; + mvnParameters = "-pl !packaging"; mvnHash = "sha256-0qsKUMV9M0ZaddR5ust8VikSrsutdxVNNezKqR+F/6M="; nativeBuildInputs = [ makeWrapper ]; + doCheck = false; + installPhase = '' mkdir -p $out/lib/server/target/lib cp -r server/target/lib/* $out/lib/server/target/lib/ diff --git a/pkgs/by-name/co/commafeed/package.nix b/pkgs/by-name/co/commafeed/package.nix index cc203468feead..369a5801ff488 100644 --- a/pkgs/by-name/co/commafeed/package.nix +++ b/pkgs/by-name/co/commafeed/package.nix @@ -56,7 +56,6 @@ maven.buildMavenPackage { "-Dskip.npm" "-Dspotless.check.skip" "-Dmaven.gitcommitid.skip" - "-DskipTests" ]; nativeBuildInputs = [ makeWrapper ]; @@ -71,6 +70,8 @@ maven.buildMavenPackage { runHook postConfigure ''; + doCheck = false; + installPhase = '' runHook preInstall diff --git a/pkgs/by-name/ko/kotlin-interactive-shell/package.nix b/pkgs/by-name/ko/kotlin-interactive-shell/package.nix index 23848a48969e6..752be68d42452 100644 --- a/pkgs/by-name/ko/kotlin-interactive-shell/package.nix +++ b/pkgs/by-name/ko/kotlin-interactive-shell/package.nix @@ -12,10 +12,12 @@ maven.buildMavenPackage rec { }; mvnHash = "sha256-m1o0m0foqJhEzWjC9behBeld5HT08WClcZN2xc3fZrI="; - mvnParameters = "-DskipTests compile"; + mvnParameters = "compile"; nativeBuildInputs = [ makeWrapper ]; + doCheck = false; + installPhase = '' runHook preInstall diff --git a/pkgs/by-name/ma/mariadb-connector-java/package.nix b/pkgs/by-name/ma/mariadb-connector-java/package.nix index d6fed528e1d06..3fa6f7a5186c5 100644 --- a/pkgs/by-name/ma/mariadb-connector-java/package.nix +++ b/pkgs/by-name/ma/mariadb-connector-java/package.nix @@ -17,8 +17,7 @@ maven.buildMavenPackage rec { mvnHash = "sha256-7O+G5HT6mtp12zWL3Gn12KPVUwp3GMaWGvXX6Sg1+6k="; - # Disable tests because they require networking - mvnParameters = "-DskipTests"; + doCheck = false; # Requires networking installPhase = '' runHook preInstall diff --git a/pkgs/by-name/s3/s3proxy/package.nix b/pkgs/by-name/s3/s3proxy/package.nix index b2fd1b03a5ab9..72f18af79cc13 100644 --- a/pkgs/by-name/s3/s3proxy/package.nix +++ b/pkgs/by-name/s3/s3proxy/package.nix @@ -21,7 +21,7 @@ maven.buildMavenPackage { hash = "sha256-GhZPvo8wlXInHwg8rSmpwMMkZVw5SMpnZyKqFUYLbrE="; }; - mvnParameters = lib.optionalString stdenv.isDarwin "-DskipTests"; + doCheck = !stdenv.isDarwin; nativeBuildInputs = [ makeWrapper ]; diff --git a/pkgs/by-name/so/sonarlint-ls/package.nix b/pkgs/by-name/so/sonarlint-ls/package.nix index 8382d3b8b2708..42b8d6098947e 100644 --- a/pkgs/by-name/so/sonarlint-ls/package.nix +++ b/pkgs/by-name/so/sonarlint-ls/package.nix @@ -40,7 +40,7 @@ mavenJdk17.buildMavenPackage rec { # disable node and npm module installation because the need network access # for the tests. - mvnDepsParameters = "-Dskip.installnodenpm=true -Dskip.npm -DskipTests package"; + mvnDepsParameters = "-Dskip.installnodenpm=true -Dskip.npm package"; # disable failing tests which either need network access or are flaky mvnParameters = lib.escapeShellArgs [ @@ -53,6 +53,8 @@ mavenJdk17.buildMavenPackage rec { !JavaMediumTests" ]; + doCheck = false; + installPhase = '' runHook preInstall diff --git a/pkgs/development/tools/build-managers/apache-maven/build-package.nix b/pkgs/development/tools/build-managers/apache-maven/build-package.nix index df5d67cb42e6b..f9ff54696dfe9 100644 --- a/pkgs/development/tools/build-managers/apache-maven/build-package.nix +++ b/pkgs/development/tools/build-managers/apache-maven/build-package.nix @@ -6,6 +6,7 @@ { src , sourceRoot ? null , buildOffline ? false +, doCheck ? true , patches ? [ ] , pname , version @@ -22,6 +23,7 @@ # created to allow using maven packages in the same style as rust let + mvnSkipTests = lib.optionalString (!doCheck) "-DskipTests"; fetchedMavenDeps = stdenv.mkDerivation ({ name = "${pname}-${version}-maven-deps"; inherit src sourceRoot patches; @@ -49,7 +51,7 @@ let mvn dependency:sources -DincludeGroupIds="$group" -DincludeArtifactIds="$artifact" -Dmaven.repo.local=$out/.m2 done '' + lib.optionalString (!buildOffline) '' - mvn package -Dmaven.repo.local=$out/.m2 ${mvnParameters} + mvn package -Dmaven.repo.local=$out/.m2 ${mvnSkipTests} ${mvnParameters} '' + '' runHook postBuild ''; @@ -85,7 +87,7 @@ stdenv.mkDerivation (builtins.removeAttrs args [ "mvnFetchExtraArgs" ] // { runHook preBuild mvnDeps=$(cp -dpR ${fetchedMavenDeps}/.m2 ./ && chmod +w -R .m2 && pwd) - mvn package -o -nsu "-Dmaven.repo.local=$mvnDeps/.m2" ${mvnParameters} + mvn package -o -nsu "-Dmaven.repo.local=$mvnDeps/.m2" ${mvnSkipTests} ${mvnParameters} runHook postBuild ''; diff --git a/pkgs/development/tools/java/java-language-server/default.nix b/pkgs/development/tools/java/java-language-server/default.nix index 0cca4d4ca56ed..744e20fce6110 100644 --- a/pkgs/development/tools/java/java-language-server/default.nix +++ b/pkgs/development/tools/java/java-language-server/default.nix @@ -23,7 +23,6 @@ maven.buildMavenPackage rec { }; mvnFetchExtraArgs.dontConfigure = true; - mvnParameters = "-DskipTests"; mvnHash = "sha256-2uthmSjFQ43N5lgV11DsxuGce+ZptZsmRLTgjDo0M2w="; nativeBuildInputs = [ jdk makeWrapper ]; @@ -39,6 +38,8 @@ maven.buildMavenPackage rec { --compress 2 ''; + doCheck = false; + installPhase = '' runHook preInstall diff --git a/pkgs/games/forge-mtg/default.nix b/pkgs/games/forge-mtg/default.nix index cc5c56977bb98..8ca1675591ee3 100644 --- a/pkgs/games/forge-mtg/default.nix +++ b/pkgs/games/forge-mtg/default.nix @@ -25,10 +25,10 @@ maven.buildMavenPackage { pname = "forge-mtg"; inherit version src patches; - # Tests need a running Xorg. - mvnParameters = "-DskipTests"; mvnHash = "sha256-QK9g0tG75lIhEtf4jW03N32YbD9Fe5iI0JTuqmCTtnE="; + doCheck = false; # Needs a running Xorg + nativeBuildInputs = [ makeWrapper ]; installPhase = '' diff --git a/pkgs/servers/h2/default.nix b/pkgs/servers/h2/default.nix index 84a8b98b6dfc9..297802f59a920 100644 --- a/pkgs/servers/h2/default.nix +++ b/pkgs/servers/h2/default.nix @@ -13,11 +13,13 @@ maven.buildMavenPackage rec { hash = "sha256-pS9jSiuInA0eULPOZK5cjwr9y5KDVY51blhZ9vs4z+g="; }; - mvnParameters = "-f h2/pom.xml -DskipTests"; + mvnParameters = "-f h2/pom.xml"; mvnHash = "sha256-hUzE4F+RNCAfoY836pjrivf04xqN4m9SkiLXhmVzZRA="; nativeBuildInputs = [ makeWrapper ]; + doCheck = false; + installPhase = '' mkdir -p $out/share/java install -Dm644 h2/target/h2-${version}.jar $out/share/java