https://bugzilla.redhat.com/show_bug.cgi?id=1805870 Parag AN(पराग) <panemade@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #4 from Parag AN(पराग) <panemade@xxxxxxxxx> --- Review: I downloaded manually required srpm file and did below review of this package. This package follows new fonts packaging guidelines. Rpmlint ------- Checking: sil-apparatus-fonts-1.0-1.fc33.noarch.rpm sil-apparatus-fonts-1.0-1.fc33.src.rpm sil-apparatus-fonts.noarch: W: invalid-url URL: https://software.sil.org/ApparatusSIL/ HTTP Error 404: Not Found sil-apparatus-fonts.src: W: invalid-url URL: https://software.sil.org/ApparatusSIL/ HTTP Error 404: Not Found 2 packages and 0 specfiles checked; 0 errors, 2 warnings. Source checksums ---------------- https://scripts.sil.org/cms/scripts/render_download.php?format=file&media_id=AppSIL1.0.zip&filename=ApparatusSIL.zip#/ApparatusSIL.zip : CHECKSUM(SHA256) this package : 0d8e3143b9c46c97916ab2cf85ae2187fe33b9b40859567a5524266dd58581bc CHECKSUM(SHA256) upstream package : 0d8e3143b9c46c97916ab2cf85ae2187fe33b9b40859567a5524266dd58581bc Requires -------- sil-apparatus-fonts (rpmlib, GLIBC filtered): config(sil-apparatus-fonts) fontpackages-filesystem Provides -------- sil-apparatus-fonts: config(sil-apparatus-fonts) font(apparatussil) metainfo() metainfo(org.fedoraproject.sil-apparatus-fonts.metainfo.xml) sil-apparatus-fonts All other things like license, config, metainfo files looks good. APPROVED. -- 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
