https://bugzilla.redhat.com/show_bug.cgi?id=1838027 --- Comment #1 from Miro Hrončok <mhroncok@xxxxxxxxxx> --- spec sanity: 1) %global srcname zuul -> consider dropping this, I don't see it used anywhere in the spec 2) 0001-Remove-another-shebang-and-remove-useless-exec-bits.patch -> is the mode thing supported by regular patch, or is %autosetup -S git needed for this? 3) %package webui etc. -> should they all (except maybe doc) have: Requires: %{name} = %{?epoch:%{epoch}:}%{version}-%{release} ? 4) Requires: git -> is git-core not good enough? 5) various manual Requires on python3-... -> is the dependency generator not working? I see you rm requirements.txt, why is that? 6) # Fix non compliant shebangs ... -> the complex fins+sed can be replaced with a single pathfix.py call (see /usr/bin/pathifx.py, also macronized as %py3_shebang_fix in rawhide, backports in progress) 7) %{python3_sitelib}/zuul-*.egg-info -> consider adding trailing slash to assert this is a directory -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
