Fedora Merge Review: xorg-x11-xbitmaps http://cvs.fedora.redhat.com/viewcvs/devel/xorg-x11-xbitmaps/ Initial Owner: sandmann
$ rpmlint xorg-x11-xbitmaps.spec xorg-x11-xbitmaps-1.1.0-1.fc13.src.rpm noarch/xorg-x11-xbitmaps-1.1.0-1.fc13.noarch.rpm xorg-x11-xbitmaps.spec:17: W: unversioned-explicit-provides xbitmaps xorg-x11-xbitmaps.spec:18: W: unversioned-explicit-provides xbitmaps-devel xorg-x11-xbitmaps.src:17: W: unversioned-explicit-provides xbitmaps xorg-x11-xbitmaps.src:18: W: unversioned-explicit-provides xbitmaps-devel xorg-x11-xbitmaps.noarch: W: no-documentation xorg-x11-xbitmaps.noarch: W: devel-file-in-non-devel-package /usr/share/pkgconfig/xbitmaps.pc 2 packages and 1 specfiles checked; 0 errors, 6 warnings. - There is ChangeLog and COPYING for %doc. - Isn't %configure and --libdir a bit hacky? You assume, that the latest used --libdir is used in the configure. Don't know if that's always the case. Hope so ;) - Could the provides xbitmaps deleted, if bitmaps requires xorg-x11-xbitmaps and not xbitmaps anymore? AFAIK this is the only programm which uses 'xbitmaps' $ repoquery --whatrequires xbitmaps bitmap-0:1.0.3-6.fc12.x86_64 - parallel make is not really needed, because nothing is build in make, but could you use 'make %{?_smp_mflags}' anyway? - from https://fedoraproject.org/wiki/Packaging/Guidelines#Pkgconfig_Files requires: pkgconfig is missing Rest looks ok.
Hmm, just wondering, which package owns /usr/include/X11 $ yum provides /usr/include/X11 says: xorg-x11-proto-devel and xorg-x11-xtrans-devel But this package should require a programm, for owning /usr/include/X11, but those two don't make sense...
CC'ing Adam Jackson
(In reply to comment #1) > - Isn't %configure and --libdir a bit hacky? > You assume, that the latest used --libdir is used in the configure. Don't > know > if that's always the case. Hope so ;) It is. Fixed the doc stuff and smp_mflags in F15. The rest is minor enough to not need fixing; if a provenpackager wants to clean that up, feel free.
where is the official review?
I will leave this package to be reviewed by Thomas Spura as he has done some initial review.
Now all the 3 packages(xorg-x11-xbitmaps, xorg-x11-proto-devel and xorg-x11-xtrans-devel) own /usr/include/X11 Done some cleanup commits already. APPROVED.
hah! howcome i missed this package to set the flag fedora-review+ ?