https://bugzilla.redhat.com/show_bug.cgi?id=1129429 Vladimir Stackov <amigo.elite@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |amigo.elite@xxxxxxxxx --- Comment #2 from Vladimir Stackov <amigo.elite@xxxxxxxxx> --- Greetings, a few comments on this review request: 1. Your packages (seafile and ccnet) both introduces build time requirements (cross-package BuildRequires) so you should setup boostraping properly: https://fedoraproject.org/wiki/Packaging:Guidelines#Bootstrapping 2. As you are packaging snapshot, you should use another name for the package: https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/SourceURL#Github 3. You should consider using %makeinstall macros instead of make install DESTDIR=%{buildroot}: # rpm -E %makeinstall 4. You should consider enabling hardened builds: https://fedoraproject.org/wiki/Packaging:Guidelines#PIE 5. You should use %license for license information instead of %doc. Please note that this is informal review. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review