Bug 847697 (mate-backgrounds) - Review Request: mate-backgrounds - Backgrounds for MATE Desktop
Summary: Review Request: mate-backgrounds - Backgrounds for MATE Desktop
Keywords:
Status: CLOSED ERRATA
Alias: mate-backgrounds
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: MATE-DE-tracker
TreeView+ depends on / blocked
 
Reported: 2012-08-13 09:53 UTC by Dan Mashal
Modified: 2012-09-17 22:04 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-22 21:06:16 UTC
rdieter: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Comment 1 Rex Dieter 2012-08-19 00:11:10 UTC
I can review

Comment 2 Rex Dieter 2012-08-19 00:20:57 UTC
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.

Comment 3 Dan Mashal 2012-08-19 02:12:53 UTC
New Package SCM Request
=======================
Package Name: mate-backgrounds
Short Description: Backgrounds for MATE Desktop
Owners: vicodan raveit65 rdieter
Branches: f16 f17 f18

Comment 4 Wolfgang Ulbrich 2012-08-19 08:51:24 UTC
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.

Comment 5 Dan Mashal 2012-08-19 09:02:00 UTC
MATE uses its own background. 

I'm not sure that I want to use Fedora's background. We'll discuss offline.

Comment 6 Rex Dieter 2012-08-19 12:50:26 UTC
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

Comment 7 Wolfgang Ulbrich 2012-08-19 13:29:53 UTC
(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

Comment 8 Rex Dieter 2012-08-19 18:47:53 UTC
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.

Comment 9 Wolfgang Ulbrich 2012-08-19 19:15:40 UTC
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

Comment 10 Rex Dieter 2012-08-19 19:23:28 UTC
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.

Comment 11 Gwyn Ciesla 2012-08-19 20:00:01 UTC
Git done (by process-git-requests).

Comment 12 Fedora Update System 2012-08-20 07:39:15 UTC
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

Comment 13 Fedora Update System 2012-08-20 07:39:25 UTC
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

Comment 14 Fedora Update System 2012-08-20 07:39:37 UTC
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

Comment 15 Fedora Update System 2012-08-20 19:50:04 UTC
mate-backgrounds-1.4.0-1.fc18 has been pushed to the Fedora 18 testing repository.

Comment 16 Fedora Update System 2012-08-22 21:06:16 UTC
mate-backgrounds-1.4.0-1.fc17 has been pushed to the Fedora 17 stable repository.

Comment 17 Fedora Update System 2012-08-28 23:30:04 UTC
mate-backgrounds-1.4.0-1.fc16 has been pushed to the Fedora 16 stable repository.

Comment 18 Fedora Update System 2012-09-17 22:04:20 UTC
mate-backgrounds-1.4.0-1.fc18 has been pushed to the Fedora 18 stable repository.


Note You need to log in before you can comment on or make changes to this bug.