From 88fe3b088dacd5ade792932d956149a723dd626a Mon Sep 17 00:00:00 2001
From: Petr Stodulka <pstodulk@redhat.com>
Date: Tue, 27 Jul 2021 11:37:25 +0200
Subject: [PATCH] Fix the python workaround for IPU 8 -> 9

Some packages already use the --keep-foreign option for alternatives
in their scriptlets, some not. For these that handles alretnatives
correctly, we need to drop such a files from the alternatives command
executed in the actor.
---
 .../actors/pythonthreetmpworkaround/actor.py  | 24 +++++++++++++------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/repos/system_upgrade/el8toel9/actors/pythonthreetmpworkaround/actor.py b/repos/system_upgrade/el8toel9/actors/pythonthreetmpworkaround/actor.py
index 5e04a3ddeb..6908fe3ea8 100644
--- a/repos/system_upgrade/el8toel9/actors/pythonthreetmpworkaround/actor.py
+++ b/repos/system_upgrade/el8toel9/actors/pythonthreetmpworkaround/actor.py
@@ -30,15 +30,25 @@ def process(self):
 
         cmd = [
             'alternatives', '--install', '/usr/bin/python3', 'python3', '/usr/bin/python3.9', '1000000',
-            '--slave', '/usr/share/man/man1/python3.1.gz', 'python3-man', '/usr/share/man/man1/python3.9.1.gz',
-            '--slave', '/usr/bin/pip3', 'pip3', '/usr/bin/pip3.9',
-            '--slave', '/usr/bin/pip-3', 'pip-3', '/usr/bin/pip-3.9',
-            '--slave', '/usr/bin/easy_install-3', 'easy_install-3', '/usr/bin/easy_install-3.9',
-            '--slave', '/usr/bin/pydoc3', 'pydoc3', '/usr/bin/pydoc3.9',
-            '--slave', '/usr/bin/pydoc-3', 'pydoc-3', '/usr/bin/pydoc3.9',
-            '--slave', '/usr/bin/pyvenv-3', 'pyvenv-3', '/usr/bin/pyvenv-3.9',
         ]
 
+        sub_cmds = [
+            ['--slave', '/usr/share/man/man1/python3.1.gz', 'python3-man', '/usr/share/man/man1/python3.9.1.gz'],
+            ['--slave', '/usr/bin/pip3', 'pip3', '/usr/bin/pip3.9'],
+            ['--slave', '/usr/bin/pip-3', 'pip-3', '/usr/bin/pip-3.9'],
+            ['--slave', '/usr/bin/easy_install-3', 'easy_install-3', '/usr/bin/easy_install-3.9'],
+            ['--slave', '/usr/bin/pydoc3', 'pydoc3', '/usr/bin/pydoc3.9'],
+            ['--slave', '/usr/bin/pydoc-3', 'pydoc-3', '/usr/bin/pydoc3.9'],
+            ['--slave', '/usr/bin/pyvenv-3', 'pyvenv-3', '/usr/bin/pyvenv-3.9'],
+        ]
+
+        for sub_cmd in sub_cmds:
+            if os.path.exists(sub_cmd[1]):
+                # some rhel 9 packages are already updated and handle
+                # alternatives correctly using the --keep-foreign option
+                continue
+            cmd += sub_cmd
+
         try:
             run(cmd)
         except CalledProcessError as exc: