https://bugzilla.redhat.com/show_bug.cgi?id=2010718
Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review+
--- Comment #31 from Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl --- %description → wrap to <=80 columns. Maybe after "de-identify" add "(remove personal information)" or something. The term might not be obvious to most people.
+ package name is OK + latest version + license is acceptable for Fedora (GPLv3) + license is specified correctly + BR/R/P look OK + %check is present and passes (though very minimalistic…) + build and installs and runs OK
rpmlint: alizams.src:34: W: unversioned-explicit-provides bundled(vectormath) alizams.src:35: W: unversioned-explicit-provides bundled(colorspace) This is OK: in principle the guidelines say that a version should be provided, but figuring this out is onerous and often the exact version cannot be determined. And when the bundled code has been modified, as is the case here, no number would make sense. alizams.src: W: spelling-error Summary(en_US) Aliza -> Alissa, Alisa, Eliza alizams.src: W: spelling-error %description -l en_US de -> DE, ed, d alizams.x86_64: W: spelling-error Summary(en_US) Aliza -> Alissa, Alisa, Eliza alizams.x86_64: W: spelling-error %description -l en_US de -> DE, ed, d 4 packages and 0 specfiles checked; 0 errors, 6 warnings. All good.
Package is APPROVED.