https://bugzilla.redhat.com/show_bug.cgi?id=1315495



--- Comment #6 from Parag AN(पराग) <[email protected]> ---
Suggestions:
1) Drop following

Requires:       google-noto-sans-coptic-fonts
Obsoletes:      ibus-coptic < 0.1-3

we don't add requirement on fonts and I see there was no such ibus-coptic
package existed on Fedora

2) Following is now optional and can be removed
rm -rf $RPM_BUILD_ROOT

3) Please use macros as mentioned here ->
https://fedoraproject.org/wiki/Packaging:RPMMacros

/usr/share => %{_datadir}  

4) I see other similar packages are not using %post section which is added here
for this package. Also explained in comment#4 here

Please update the package and provide new SPEC and SRPM link by bumping release
number and adding related changelog entry.

-- 
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
[email protected]
https://lists.fedoraproject.org/admin/lists/[email protected]

Reply via email to