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=593915
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |panemade@gmail.com AssignedTo|nobody@fedoraproject.org |panemade@gmail.com Flag| |fedora-review+
--- Comment #1 from Parag AN(पराग) panemade@gmail.com 2010-05-20 06:32:07 EDT ---
Review: + package builds in mock (rawhide i686). koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=2198870 + rpmlint output for SRPM and for RPM. dconf.src: W: spelling-error %description -l en_US backend -> backed, back end, back-end dconf.src: W: no-cleaning-of-buildroot %clean dconf.src: W: no-buildroot-tag dconf.src: W: no-%clean-section dconf.x86_64: W: spelling-error %description -l en_US backend -> backed, back end, back-end ==> false messages.
+ source files match upstream url (sha1sum) 49eafdb8a845e9d618a97a97313489259093e120 dconf-0.3.tar.bz2 49eafdb8a845e9d618a97a97313489259093e120 ../SOURCES/dconf-0.3.tar.bz2 + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + license is open source-compatible. + License text is included in package. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + ldconfig scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Package dconf-0.3-1.fc14.x86_64 => Provides: libdconfsettings.so()(64bit) Requires: libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.4)(64bit) libgio-2.0.so.0()(64bit) libglib-2.0.so.0()(64bit) libgmodule-2.0.so.0()(64bit) libgobject-2.0.so.0()(64bit) libgthread-2.0.so.0()(64bit) libpthread.so.0()(64bit) librt.so.1()(64bit) rtld(GNU_HASH) + Not a GUI application
Suggestions: 1) It will be good to include some documentation say README file which can say what this package provides. Please contact upstream and ask them to add README.
2) update summary as Summary: A simple key-based configuration system
APPROVED.