Bug 1227983 - python3-Markdown must be required for all DC
Summary: python3-Markdown must be required for all DC
Status: IN_PROGRESS
Alias: None
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: Samba (show other bugs)
Version: Slowroll
Hardware: Other Other
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: Samuel Cabrero
QA Contact: The 'Opening Windows to a Wider World' guys
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-07-17 07:35 UTC by Peter Varkoly
Modified: 2024-07-17 11:13 UTC (History)
2 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Varkoly 2024-07-17 07:35:42 UTC 
samba-tool reuires python3-Markdown only if it was built as DC with MIT kerberos.
This is bad. python3-Markdown is required even it was built with Heimdal kerberos:

Index: samba.spec
===================================================================
--- samba.spec  (revision 11810e8fe55875da766cbbe350b4df85)
+++ samba.spec  (working copy)
@@ -548,7 +548,7 @@
 Requires:       samba = %{version}
 Requires:       samba-ldb-ldap = %{version}
 Requires:       samba-python3 = %{version}
-%if %{with_mit_dc}
+%if %{with_dc}
 Requires:       python3-Markdown
 %endif
Comment 1 Samuel Cabrero 2024-07-17 11:13:16 UTC 
(In reply to Peter Varkoly from comment #0)
> samba-tool reuires python3-Markdown only if it was built as DC with MIT
> kerberos.
> This is bad. python3-Markdown is required even it was built with Heimdal
> kerberos:
> 
> Index: samba.spec
> ===================================================================
> --- samba.spec  (revision 11810e8fe55875da766cbbe350b4df85)
> +++ samba.spec  (working copy)
> @@ -548,7 +548,7 @@
>  Requires:       samba = %{version}
>  Requires:       samba-ldb-ldap = %{version}
>  Requires:       samba-python3 = %{version}
> -%if %{with_mit_dc}
> +%if %{with_dc}
>  Requires:       python3-Markdown
>  %endif

Thanks, I will import the patch.