Description : DeviceKit is a framework for enumerating devices. http://people.freedesktop.org/~david/gdu-dkd-review/DeviceKit.spec http://people.freedesktop.org/~david/gdu-dkd-review/DeviceKit-002-0.git20080720.fc10.src.rpm
See also bug 456033 and bug 456034.
Hmm, rpmlint is busted, so no rpmlint check for now... Looking over the spec file informally, I notice 2 things: 1) you probably need to require udev for /etc/udev/rules.d ownership 2) the handling of %doc seems obscure. Whats the purpose of that ?
rpmlint output: [mclasen@golem ~]$ rpmlint DeviceKit-002-0.git20080720.fc10.i386.rpm DeviceKit-devel-002-0.git20080720.fc10.i386.rpm DeviceKit-debuginfo-002-0.git20080720.fc10.i386.rpm DeviceKit.i386: W: non-conffile-in-etc /etc/dbus-1/system.d/org.freedesktop.DeviceKit.conf DeviceKit.i386: W: non-conffile-in-etc /etc/udev/rules.d/98-devkit.rules 3 packages and 0 specfiles checked; 0 errors, 2 warnings. I'd double-check, but I believe it is common practise to treat neither dbus conf files nor udev rules as conf files.
package name: ok spec file name: ok packaging guidelines: - you should probably straighten the %doc stuff - Source: points to a nonexisting location, should just make it a filename and add a comment explaining that it is a git snapshot license: ok license field: ok license file: ok spec language: ok spec legibility: ok upstream sources: ok, see above for Source: tag problem ExcludeArch: n/a BuildRequires: ok locale handling: ok ldconfig: ok relocatable: n/a directory ownership: - need to Requires: udev for /etc/udev/rules.d duplicate files: ok permissions: ok %clean: ok macro use: consistent content: permissible large docs: n/a doc content: ok headers: ok static libs: n/a pc files: ok shared libs: ok devel deps: ok libtool archives: ok gui apps: n/a file/directory ownership: - /usr/share/dbus-1/interfaces should be owned by dbus %install: ok utf8 filenames: ok
(In reply to comment #3) > I'd double-check, but I believe it is common practise to treat neither dbus conf > files nor udev rules as conf files. Sure, of course these are not configuration files - rpmlint and/or the guidelines are just busted.
(In reply to comment #2) > 1) you probably need to require udev for /etc/udev/rules.d ownership Should do that anyway. Fixed locally. > 2) the handling of %doc seems obscure. Whats the purpose of that ? The fact that some files are not in the root directory of the tarball. It's not particulary obscure. Suggestions on how to fix that welcome.
(In reply to comment #4) > - Source: points to a nonexisting location, should just make it a filename Fixed. > and add a comment explaining that it is a git snapshot It's evident from the Release tag that it's a git snapshot. > - /usr/share/dbus-1/interfaces should be owned by dbus Yeah, that's a D-Bus problem. Thanks for the review.
(In reply to comment #6) > (In reply to comment #2) > > 2) the handling of %doc seems obscure. Whats the purpose of that ? > > The fact that some files are not in the root directory of the tarball. It's not > particulary obscure. Suggestions on how to fix that welcome. Just %doc README AUTHORS NEWS COPYING HACKING doc/TODO seemed to do what I wanted. I've uploaded new SPEC and SRPMS with all fixes at the same location - does this look OK? Thanks.
Yes, looks good now. Even though the source url guidelines really want you put a comment there explaining how to go from git url to tarball....
New Package CVS Request ======================= Package Name: DeviceKit Short Description: Device Enumeration Framework Owners: davidz Branches: InitialCC: Cvsextras Commits: yes
cvs done.