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=772504 --- Comment #2 from Lubomir Rintel <lkundrak@xxxxx> 2012-01-09 04:34:57 EST --- Thank you for your suggestions. Here are the updated packages: SPEC: http://v3.sk/~lkundrak/SPECS/btkbdd.spec SRPM: http://v3.sk/~lkundrak/SRPMS/btkbdd-1.1-2.el6.src.rpm (In reply to comment #1) > I think the package summary does not describe the package very well. The > description gave me a completely different view. What view did it give you? I can't really think of a better description and would appreciate help here. > Defattr is no longer necessary. If you're not going for EPEL 5, you can drop > the clean section, buildroot definition and the rm in the install section. I choose not to break compatibility with el5 rebuilds for packages that rebuild cleanly. As it gets more obsolete, I'll remove those parts. > Please use the name macro, as you did in Source. Looking at your manpage, you > forgot a "d" in %{_localstatedir}/lib/btkbd. The manpage was actually wrong (the example udev rules wrote to /var/lib/btkbd). However, it makes more sense with two "d"s, so I adjusted that appropriately. > There's a macro called sharedstatedir, by the way, that is equivalent to > %{_localstatedir}/lib. This would break el5 builds: $ rpm --eval %_sharedstatedir /usr/com > The FSF address is outdated. Changed that to <http://fsf.org>, which seems to be used in GPLv3. > Why do you want to own %{_sysconfdir}/udev/rules.d? That was a mistake. Removed it. > Why do you require pod2man explicitly? I choose to :) It is not intuitively obvious (to me) that it is part of perl, nor that it won't change in future or that perl will stay in build group forever. Though all of that is unlikely, I like it to be specified explicitly. -- 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