From cce8dc4da87a934644712158b97242960a8d138e Mon Sep 17 00:00:00 2001
From: Mark Reynolds <mreynolds@redhat.com>
Date: Apr 30 2024 18:26:55 +0000
Subject: Issue 9579 - Remove bash_completions_dir for RHEL


RHEL 9 does not support the bach_completions_dir macro, but it is still
needed for Fedora builds

Fixes: https://pagure.io/freeipa/issue/9579

Signed-off-by: Mark Reynolds <mreynolds@redhat.com>
Reviewed-By: Francisco Trivino <ftrivino@redhat.com>

---

diff --git a/freeipa.spec.in b/freeipa.spec.in
index b23c780..2a220b4 100755
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -1082,12 +1082,14 @@ rm -f %{buildroot}%{_usr}/share/ipa/ui/images/product-name.png
 %endif
 # RHEL spec file only: END
 
-# Register CLI tools for bash completion
+%if 0%{?fedora} >= 38
+# Register CLI tools for bash completion (fedora only)
 for clitool in ipa-migrate
 do
     register-python-argcomplete "${clitool}" > "${clitool}"
     install -p -m 0644 -D -t '%{buildroot}%{bash_completions_dir}' "${clitool}"
 done
+%endif
 
 %find_lang %{gettext_domain}
 
@@ -1420,7 +1422,9 @@ fi
 %{_sbindir}/ipa-cert-fix
 %{_sbindir}/ipa-acme-manage
 %{_sbindir}/ipa-migrate
+%if 0%{?fedora} >= 38
 %{bash_completions_dir}/ipa-migrate
+%endif
 %{_libexecdir}/certmonger/dogtag-ipa-ca-renew-agent-submit
 %{_libexecdir}/certmonger/ipa-server-guard
 %dir %{_libexecdir}/ipa