https://bugzilla.redhat.com/show_bug.cgi?id=1360482 Remi Collet <fedora@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? --- Comment #15 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> --- Issues: ======= [!]: If the package is under multiple licenses, the licensing breakdown must be documented in the spec. => Need documentation (if possible... ) [!]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). => hidden-file-or-dir => .tx and .bower.json can very probably be removed => wrong-script-interpreter => I don't care (especially for PHP as this allow to use SCL) but some files can probably be dropped (relative to upstream build stuff) /usr/share/nextcloud/l10n/rm-old.sh /usr/share/nextcloud/apps/gallery/build/xdebug_install.sh /usr/share/nextcloud/apps/gallery/build/after_failure.sh /usr/share/nextcloud/apps/gallery/build/documentation/docpublisher.sh About %dir %attr(0755,apache,apache) According to https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions should be 2775 -- 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