From 989c077e64aa0a3b95da61699eff96fd8849ca8f Mon Sep 17 00:00:00 2001
From: "Jason R. Coombs" <jaraco@jaraco.com>
Date: Sat, 26 Nov 2022 08:28:49 -0500
Subject: [PATCH] gh-97966: Restore prior expectation that uname_result._fields
 and ._asdict would include the processor. (gh-98343) (cherry picked from
 commit dc063a25d29840d863b15c86fdab15b4a1894c73)

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>
---
 Lib/platform.py                                           | 6 ++++--
 Lib/test/test_platform.py                                 | 8 ++++++++
 .../Library/2022-10-16-18-52-00.gh-issue-97966.humlhz.rst | 2 ++
 3 files changed, 14 insertions(+), 2 deletions(-)
 create mode 100644 Misc/NEWS.d/next/Library/2022-10-16-18-52-00.gh-issue-97966.humlhz.rst

diff --git a/Lib/platform.py b/Lib/platform.py
index e32f9c11cdbf16..6a820c90a1ae42 100755
--- a/Lib/platform.py
+++ b/Lib/platform.py
@@ -776,6 +776,8 @@ class uname_result(
     except when needed.
     """
 
+    _fields = ('system', 'node', 'release', 'version', 'machine', 'processor')
+
     @functools.cached_property
     def processor(self):
         return _unknown_as_blank(_Processor.get())
@@ -789,7 +791,7 @@ def __iter__(self):
     @classmethod
     def _make(cls, iterable):
         # override factory to affect length check
-        num_fields = len(cls._fields)
+        num_fields = len(cls._fields) - 1
         result = cls.__new__(cls, *iterable)
         if len(result) != num_fields + 1:
             msg = f'Expected {num_fields} arguments, got {len(result)}'
@@ -803,7 +805,7 @@ def __len__(self):
         return len(tuple(iter(self)))
 
     def __reduce__(self):
-        return uname_result, tuple(self)[:len(self._fields)]
+        return uname_result, tuple(self)[:len(self._fields) - 1]
 
 
 _uname_cache = None
diff --git a/Lib/test/test_platform.py b/Lib/test/test_platform.py
index 1a688775f46309..2d41166644ab74 100644
--- a/Lib/test/test_platform.py
+++ b/Lib/test/test_platform.py
@@ -268,6 +268,14 @@ def test_uname_slices(self):
         self.assertEqual(res[:], expected)
         self.assertEqual(res[:5], expected[:5])
 
+    def test_uname_fields(self):
+        self.assertIn('processor', platform.uname()._fields)
+
+    def test_uname_asdict(self):
+        res = platform.uname()._asdict()
+        self.assertEqual(len(res), 6)
+        self.assertIn('processor', res)
+
     @unittest.skipIf(sys.platform in ['win32', 'OpenVMS'], "uname -p not used")
     def test_uname_processor(self):
         """
diff --git a/Misc/NEWS.d/next/Library/2022-10-16-18-52-00.gh-issue-97966.humlhz.rst b/Misc/NEWS.d/next/Library/2022-10-16-18-52-00.gh-issue-97966.humlhz.rst
new file mode 100644
index 00000000000000..b725465ae4f0ef
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2022-10-16-18-52-00.gh-issue-97966.humlhz.rst
@@ -0,0 +1,2 @@
+On ``uname_result``, restored expectation that ``_fields`` and ``_asdict``
+would include all six properties including ``processor``.