https://bugzilla.redhat.com/show_bug.cgi?id=2160771
--- Comment #5 from Yaakov Selkowitz yselkowi@redhat.com --- (In reply to Neal Gompa from comment #4)
Initial spec review:
Source0: %{url}/archive/%{commit}/Flatseal-%{commit}.tar.gz
This has tags, so use those instead: "%{url}/archive/v%{version}/Flatseal-%{version}.tar.gz"
The packaging guidelines say otherwise, since git tags are not immutable:
https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/#_git_ta...
desktop-file-validate %{buildroot}%{_datadir}/applications/%{app_id}.desktop
appstream-util validate-relax --nonet %{buildroot}%{_datadir}/appdata/%{app_id}.appdata.xml
These should go into %check for clarity.
The guidelines allow either:
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_desktop_file_ins...
https://docs.fedoraproject.org/en-US/packaging-guidelines/AppData/#_app_data...