Spec URL: http://vicodan.fedorapeople.org/matespec/mate-backgrounds.spec SRPM URL: http://vicodan.fedorapeople.org/materpms/srpms/mate-backgrounds-1.4.0-1.fc17.src.rpm Description: Backgrounds for MATE Desktop
I can review
naming: ok sources: ok $ md5sum *.xz 225a2ce7672c2a236c3b7d01da95af5a mate-backgrounds-1.4.0.tar.xz licensing 1 SHOULD remove BuildRequires: mate-conf-devel mate-corba-devel these aren't needed (builds ok for me without these) macros: mostly ok, except for: 2. SHOULD not use undefined macro po_package, change %find_lang %{po_package} --all-name ... %files -f %{po_package}.lang to %find_lang %{name} --all-name ... %files -f %{name}.lang 3. SHOULD: dir /usr/share/pixmaps/backgrounds seems unowned. mind either owning this here too, or consider if it's possible to move /usr/share/pixmaps/backgrounds/mate to /usr/share/backgrounds/mate to be closer to how gnome does things? (fwiw, kde uses /usr/share/wallpapers) scriptlets: ok (n/a) fairly good and simple, APPROVED. please address items 1-3 prior to doing any builds.
New Package SCM Request ======================= Package Name: mate-backgrounds Short Description: Backgrounds for MATE Desktop Owners: vicodan raveit65 rdieter Branches: f16 f17 f18
missing requrires: Requires: desktop-backgrounds-compat and for f17 if you really want to build for F16/f17 Requires: beefy-miracle-backgrounds-single Requires: beefy-miracle-backgrounds-gnome For f18 you need the new background packages. Your package install the backgrounds in %{_datadir}/pixmaps/backgrounds/mate/ but fedora use in general a own background path %{_datadir}/backgrounds/mate/ IMO you have to fix this in a patch, i can send you this patch.
MATE uses its own background. I'm not sure that I want to use Fedora's background. We'll discuss offline.
I too don't understand the need for the dependency changes in Comment #4, please explain and provide the justification for doing so. except for %{_datadir}/pixmaps/backgrounds/mate/, which is something I mentioned in the review, item 3
(In reply to comment #6) > I too don't understand the need for the dependency changes in Comment #4, > please explain and provide the justification for doing so. > > except for %{_datadir}/pixmaps/backgrounds/mate/, which is something I > mentioned in the review, item 3 @ Requires: desktop-backgrounds-compat root@mother rave]# rpm -ql desktop-backgrounds-compat /usr/share/backgrounds/default.png /usr/share/backgrounds/images/default-16_10.png /usr/share/backgrounds/images/default-5_4.png /usr/share/backgrounds/images/default.png Without this require we have no /usr/share/backgrounds/default.png I think nobody wants that. For get this working we need also the other requires, because 'default.png' is a symlink to current fedora bckground image. for f16 [root@mother rave]# ls -l /usr/share/backgrounds/default.png lrwxrwxrwx 1 root root 40 8. Feb 2012 /usr/share/backgrounds/default.png -> verne/default/standard/verne.png
the handling of desktop-backgrounds-compat would best be handled elsewhere, particularly when/if you offer a default configuration that actually uses these. For example, for kde, we provide a kde-settings package that does that.
Which background image Mate-Desktop use in the end is handled by a mateconf key from libmate. I can add those requires in libmate , np
like I said, my suggestion would be to put branding/configuration defaults elsewhere, preferably in a separate package. one advantage in doing so is that it makes it easier for others to customize and change that way, but I'll leave the ultimate decision and implementation up to you mate maintainers.
Git done (by process-git-requests).
mate-backgrounds-1.4.0-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-backgrounds-1.4.0-1.fc17
mate-backgrounds-1.4.0-1.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/mate-backgrounds-1.4.0-1.fc18
mate-backgrounds-1.4.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/mate-backgrounds-1.4.0-1.fc16
mate-backgrounds-1.4.0-1.fc18 has been pushed to the Fedora 18 testing repository.
mate-backgrounds-1.4.0-1.fc17 has been pushed to the Fedora 17 stable repository.
mate-backgrounds-1.4.0-1.fc16 has been pushed to the Fedora 16 stable repository.
mate-backgrounds-1.4.0-1.fc18 has been pushed to the Fedora 18 stable repository.