Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632056 Rex Dieter <rdieter@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|14 |rawhide Flag|fedora-review? |fedora-review+ --- Comment #4 from Rex Dieter <rdieter@xxxxxxxxxxxx> 2011-05-20 11:31:09 EDT --- $ rpmlint *.rpm x86_64/*.rpm libsocialweb-qt.src: W: invalid-url Source0: libsocialweb-qt-0.1.2.tar.bz2 libsocialweb-qt.x86_64: E: incorrect-fsf-address /usr/share/doc/libsocialweb-qt-0.1.2/COPYING libsocialweb-qt-debuginfo.x86_64: W: hidden-file-or-dir /usr/src/debug/libsocialweb-qt-0.1.2/.moc libsocialweb-qt-debuginfo.x86_64: W: hidden-file-or-dir /usr/src/debug/libsocialweb-qt-0.1.2/.moc libsocialweb-qt-devel.x86_64: W: no-documentation 4 packages and 0 specfiles checked; 1 errors, 4 warnings. mostly harmless Naming: ok License: ok Sources: meh, though hard to verify as-is. :-/ Builds/installs: ok, though 1. SHOULD: use versioned BuildRequires: rest-devel >= 0.7 2. SHOULD: move sed -i 's|LIBDIR = .*$|LIBDIR = %{_libdir}|' libsocialweb-qt.pro to %prep section, instead of %build macros: ok scriptlets: ok I won't consider those SHOULD items blockers, but please do address them prior to doing any official builds. APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review