From f7918fd8abab5bf7b28195de39b0c22ccfa1cd85 Mon Sep 17 00:00:00 2001 From: "Adam J. Stewart" Date: Sat, 12 Aug 2023 02:08:25 -0500 Subject: Python: remove maintainer (#39384) --- lib/spack/spack/build_systems/python.py | 2 +- lib/spack/spack/test/cmd/maintainers.py | 7 +------ lib/spack/spack/test/directives.py | 4 ++-- var/spack/repos/builtin/packages/python/package.py | 2 +- 4 files changed, 5 insertions(+), 10 deletions(-) diff --git a/lib/spack/spack/build_systems/python.py b/lib/spack/spack/build_systems/python.py index c567cc933c..a84a2811cd 100644 --- a/lib/spack/spack/build_systems/python.py +++ b/lib/spack/spack/build_systems/python.py @@ -30,7 +30,7 @@ from ._checks import BaseBuilder, execute_install_time_tests class PythonExtension(spack.package_base.PackageBase): - maintainers("adamjstewart", "pradyunsg") + maintainers("adamjstewart") @property def import_modules(self): diff --git a/lib/spack/spack/test/cmd/maintainers.py b/lib/spack/spack/test/cmd/maintainers.py index 3a80877c63..5063877a10 100644 --- a/lib/spack/spack/test/cmd/maintainers.py +++ b/lib/spack/spack/test/cmd/maintainers.py @@ -54,12 +54,10 @@ def test_all(mock_packages, capfd): "user3", "py-extension1:", "adamjstewart,", - "pradyunsg,", "user1,", "user2", "py-extension2:", - "adamjstewart,", - "pradyunsg", + "adamjstewart", ] with capfd.disabled(): @@ -74,9 +72,6 @@ def test_all_by_user(mock_packages, capfd): "adamjstewart:", "py-extension1,", "py-extension2", - "pradyunsg:", - "py-extension1,", - "py-extension2", "user0:", "maintainers-3", "user1:", diff --git a/lib/spack/spack/test/directives.py b/lib/spack/spack/test/directives.py index 4ec3e58238..e32ec6ac08 100644 --- a/lib/spack/spack/test/directives.py +++ b/lib/spack/spack/test/directives.py @@ -78,8 +78,8 @@ def test_error_on_anonymous_dependency(config, mock_packages): "package_name,expected_maintainers", [ ("maintainers-1", ["user1", "user2"]), - # Reset from PythonPackage - ("py-extension1", ["adamjstewart", "pradyunsg", "user1", "user2"]), + # Extends PythonPackage + ("py-extension1", ["adamjstewart", "user1", "user2"]), # Extends maintainers-1 ("maintainers-3", ["user0", "user1", "user2", "user3"]), ], diff --git a/var/spack/repos/builtin/packages/python/package.py b/var/spack/repos/builtin/packages/python/package.py index 56ce86b136..ab8e6f310f 100644 --- a/var/spack/repos/builtin/packages/python/package.py +++ b/var/spack/repos/builtin/packages/python/package.py @@ -32,7 +32,7 @@ class Python(Package): list_depth = 1 tags = ["windows"] - maintainers("adamjstewart", "skosukhin", "scheibelp", "pradyunsg") + maintainers("adamjstewart", "skosukhin", "scheibelp") phases = ["configure", "build", "install"] -- cgit v1.2.3-60-g2f50