Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=828444 Shawn Iwinski <shawn.iwinski@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+ --- Comment #4 from Shawn Iwinski <shawn.iwinski@xxxxxxxxx> --- ===== SHOULD items ===== > [!]: Package does not include license text files separate from upstream. I believe this is because of a license update in the actual license file? Has upstream been made aware of the needed update? > [!]: Packages should try to preserve timestamps of original installed files. Please consider adding the "-p" (or "--preserve-timestamps") flag to your install command. Per phpci, after initial import, please add "php-pcre" requirement. No blockers. ===== APPROVED ===== -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=J6WlPyJsNI&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review