From 3688b70b780621a9d653796b782cace1ade9cebb Mon Sep 17 00:00:00 2001 From: psakiev Date: Tue, 30 Apr 2024 09:18:34 -0600 Subject: [PATCH] Remove unnecessary mock package --- lib/spack/spack/test/concretize.py | 4 +- .../builtin.mock/packages/zlib-ng/package.py | 97 ------------------- 2 files changed, 2 insertions(+), 99 deletions(-) delete mode 100644 var/spack/repos/builtin.mock/packages/zlib-ng/package.py diff --git a/lib/spack/spack/test/concretize.py b/lib/spack/spack/test/concretize.py index 0fa4d608a02..8e0df757ec1 100644 --- a/lib/spack/spack/test/concretize.py +++ b/lib/spack/spack/test/concretize.py @@ -2817,7 +2817,7 @@ def mock_runtime_dependencies(*args, **kwargs): @pytest.mark.only_clingo("clingo only re-use feature being tested") @pytest.mark.regression("38484") -def test_git_ref_version_can_be_reused(monkeypatch, mock_packages, install_mockery_mutable_config): +def test_git_ref_version_can_be_reused(monkeypatch, install_mockery_mutable_config): # override gcc-runtime dep and make all installs reusable monkeypatch.setattr(spack.solver.asp, "_has_runtime_dependencies", mock_runtime_dependencies) @@ -2832,7 +2832,7 @@ def test_git_ref_version_can_be_reused(monkeypatch, mock_packages, install_mocke @pytest.mark.only_clingo("clingo only re-use feature being tested") def test_reuse_prefers_standard_over_git_versions( - monkeypatch, mock_packages, install_mockery_mutable_config + monkeypatch, install_mockery_mutable_config ): """ order matters in this test. typically re-use would pick the last installed match diff --git a/var/spack/repos/builtin.mock/packages/zlib-ng/package.py b/var/spack/repos/builtin.mock/packages/zlib-ng/package.py deleted file mode 100644 index 903e451347f..00000000000 --- a/var/spack/repos/builtin.mock/packages/zlib-ng/package.py +++ /dev/null @@ -1,97 +0,0 @@ -# Copyright 2013-2024 Lawrence Livermore National Security, LLC and other -# Spack Project Developers. See the top-level COPYRIGHT file for details. -# -# SPDX-License-Identifier: (Apache-2.0 OR MIT) - -from spack.build_systems import autotools, cmake -from spack.package import * - - -class ZlibNg(AutotoolsPackage, CMakePackage): - """ - zlib replacement with optimizations for next generation systems. - -- added to mock repo and deleted patches for re-use and git ref testing - """ - - homepage = "https://github.com/zlib-ng/zlib-ng" - url = "https://github.com/zlib-ng/zlib-ng/archive/2.0.0.tar.gz" - git = "https://github.com/zlib-ng/zlib-ng.git" - - maintainers("haampie") - - license("Zlib") - - version("2.1.6", sha256="a5d504c0d52e2e2721e7e7d86988dec2e290d723ced2307145dedd06aeb6fef2") - version("2.1.5", sha256="3f6576971397b379d4205ae5451ff5a68edf6c103b2f03c4188ed7075fbb5f04") - version("2.1.4", sha256="a0293475e6a44a3f6c045229fe50f69dc0eebc62a42405a51f19d46a5541e77a") - version( - "2.1.3", - sha256="d20e55f89d71991c59f1c5ad1ef944815e5850526c0d9cd8e504eaed5b24491a", - deprecated=True, - ) - version( - "2.1.2", - sha256="383560d6b00697c04e8878e26c0187b480971a8bce90ffd26a5a7b0f7ecf1a33", - deprecated=True, - ) - version("2.0.7", sha256="6c0853bb27738b811f2b4d4af095323c3d5ce36ceed6b50e5f773204fb8f7200") - version("2.0.0", sha256="86993903527d9b12fc543335c19c1d33a93797b3d4d37648b5addae83679ecd8") - - variant("compat", default=True, description="Enable compatibility API") - variant("opt", default=True, description="Enable optimizations") - variant("shared", default=True, description="Build shared library") - variant("pic", default=True, description="Enable position-independent code (PIC)") - - conflicts("+shared~pic") - - variant("new_strategies", default=True, description="Enable new deflate strategies") - - provides("zlib-api", when="+compat") - - # Default to autotools, since cmake would result in circular dependencies if it's not - # reused. - build_system("autotools", "cmake", default="autotools") - - def libs(self): - name = "libz" if self.spec.satisfies("+compat") else "libz-ng" - return find_libraries( - name, root=self.prefix, recursive=True, shared=self.spec.satisfies("+shared") - ) - - def flag_handler(self, name, flags): - if name == "cflags" and self.spec.satisfies("+pic build_system=autotools"): - flags.append(self.compiler.cc_pic_flag) - return (flags, None, None) - - -class AutotoolsBuilder(autotools.AutotoolsBuilder): - @run_before("configure") - def pretend_gcc(self): - # All nice things (PIC flags, symbol versioning) that happen to the compilers that are - # recognized as gcc (%gcc, %clang, %intel, %oneapi) we want for some other compilers too: - if self.spec.compiler.name in ["nvhpc"]: - filter_file(r"^gcc=0$", "gcc=1", join_path(self.configure_directory, "configure")) - - def configure_args(self): - args = [] - if self.spec.satisfies("+compat"): - args.append("--zlib-compat") - if self.spec.satisfies("~opt"): - args.append("--without-optimizations") - if self.spec.satisfies("~shared"): - args.append("--static") - if self.spec.satisfies("~new_strategies"): - args.append("--without-new-strategies") - return args - - -class CMakeBuilder(cmake.CMakeBuilder): - def cmake_args(self): - return [ - self.define_from_variant("ZLIB_COMPAT", "compat"), - self.define_from_variant("WITH_OPTIM", "opt"), - self.define("BUILD_SHARED_LIBS", self.spec.satisfies("+shared")), - self.define_from_variant("CMAKE_POSITION_INDEPENDENT_CODE", "pic"), - self.define_from_variant("WITH_NEW_STRATEGIES", "new_strategies"), - self.define("ZLIB_ENABLE_TESTS", self.pkg.run_tests), - ]