From f743dfce93ea6a4c78aee92831fd2e3b63efe335 Mon Sep 17 00:00:00 2001
From: Martin Krizek <martin.krizek@gmail.com>
Date: Tue, 4 Nov 2025 16:19:15 +0100
Subject: [PATCH] basic.py: add proper deprecations (#86064)

* basic.py: add proper deprecations

* fix sanity
---
 .../fragments/basic-py-deprecations.yml       |  4 +++
 lib/ansible/module_utils/basic.py             | 32 +++++++++----------
 2 files changed, 19 insertions(+), 17 deletions(-)
 create mode 100644 changelogs/fragments/basic-py-deprecations.yml

diff --git a/changelogs/fragments/basic-py-deprecations.yml b/changelogs/fragments/basic-py-deprecations.yml
new file mode 100644
index 00000000000..e7b82485c72
--- /dev/null
+++ b/changelogs/fragments/basic-py-deprecations.yml
@@ -0,0 +1,4 @@
+deprecated_features:
+  - The ``get_platfrom()`` function from ``ansible.module_utils.basic`` is deprecated and will be removed in ansible-core 2.24. Use ``platform.system()`` from the Python standard library instead.
+  - The ``load_platform_subclass()`` function from ``ansible.module_utils.basic`` is deprecated and will be removed in ansible-core 2.24. Use ``get_platform_subclass()`` from ``ansible.module_utils.common.sys_info`` instead.
+  - The ``get_all_subclasses()`` function from ``ansible.module_utils.basic`` is deprecated and will be removed in ansible-core 2.24. Use ``get_all_subclasses()`` from ``ansible.module_utils.common._utils`` instead.
diff --git a/lib/ansible/module_utils/basic.py b/lib/ansible/module_utils/basic.py
index 1b849cb769e..ae3e23d8af2 100644
--- a/lib/ansible/module_utils/basic.py
+++ b/lib/ansible/module_utils/basic.py
@@ -212,42 +212,40 @@ USERS_RE = re.compile(r'^[ugo]+$')
 PERMS_RE = re.compile(r'^[rwxXstugo]*$')
 
 
-#
-# Deprecated functions
-#
-
 def get_platform():
     """
-    **Deprecated** Use :py:func:`platform.system` directly.
-
     :returns: Name of the platform the module is running on in a native string
 
     Returns a native string that labels the platform ("Linux", "Solaris", etc). Currently, this is
     the result of calling :py:func:`platform.system`.
     """
+    deprecate(
+        msg="The `get_platfrom()` function from `ansible.module_utils.basic` is deprecated.",
+        version="2.24",
+        help_text="Use `platform.system()` from the Python standard library instead.",
+    )
     return platform.system()
 
-# End deprecated functions
-
-
-#
-# Compat shims
-#
 
 def load_platform_subclass(cls, *args, **kwargs):
-    """**Deprecated**: Use ansible.module_utils.common.sys_info.get_platform_subclass instead"""
+    deprecate(
+        msg="The `load_platform_subclass()` function from `ansible.module_utils.basic` is deprecated.",
+        version="2.24",
+        help_text="Use `get_platform_subclass()` from `ansible.module_utils.common.sys_info` instead.",
+    )
     platform_cls = get_platform_subclass(cls)
     return super(cls, platform_cls).__new__(platform_cls)
 
 
 def get_all_subclasses(cls):
-    """**Deprecated**: Use ansible.module_utils.common._utils.get_all_subclasses instead"""
+    deprecate(
+        msg="The `get_all_subclasses()` function from `ansible.module_utils.basic` is deprecated.",
+        version="2.24",
+        help_text="Use `get_all_subclasses()` from `ansible.module_utils.common._utils` instead.",
+    )
     return list(_get_all_subclasses(cls))
 
 
-# End compat shims
-
-
 def heuristic_log_sanitize(data, no_log_values=None):
     """ Remove strings that look like passwords from log messages """
     # Currently filters: