Spec URL: http://vicodan.fedorapeople.org/matespec/mate-window-manager.spec SRPM URL: http://vicodan.fedorapeople.org/materpms/srpms/mate-window-manager-1.4.0-1.fc17.src.rpm Description: MATE Desktop window manager
MUST add mateconf schema scriptlets SHOULD remove needless ldconfig -devel scriptlet SHOULD remove from -devel Requires: %{name}%{?_isa} = %{version}-%{release} (the dep on -libs is enough)
Fixed. Spec URL: http://vicodan.fedorapeople.org/matespec/mate-window-manager.spec SRPM URL: http://vicodan.fedorapeople.org/materpms/srpms/mate-window-manager-1.4.0-2.fc17.src.rpm Description: MATE Desktop window manager
naming: ok sources: ok b111a2ea36b6fea4603d63e4dc252092 mate-window-manager-1.4.0.tar.xz licensing: NOT ok 1. according to licensecheck, all sources are GPL (v2 or later) (with incorrect FSF address) so MUST use License: GPLv2+ scriptlets: ok 2. -devel SHOULD drop Requires: %{name}%{?_isa} = %{version}-%{release} depending on -libs is enough 3. main pkg MUST Requires: %{name}-libs%{?_isa} = %{version}-%{release} macros: not ok 4. MUST not use undefined %{po_package} macro, either define it, or just use %{name} in place of it 5. MUST own theme parent dirs, instead of: %{_datadir}/themes/ClearlooksRe/metacity-1/ %{_datadir}/themes/Dopple-Left/metacity-1/ %{_datadir}/themes/Dopple/metacity-1/ %{_datadir}/themes/DustBlue/metacity-1/ %{_datadir}/themes/Spidey-Left/metacity-1 %{_datadir}/themes/Spidey/metacity-1/ %{_datadir}/themes/Splint-Left/metacity-1/ %{_datadir}/themes/Splint/metacity-1/ %{_datadir}/themes/WinMe/metacity-1/ %{_datadir}/themes/eOS/metacity-1/ use %{_datadir}/themes/ClearlooksRe/ %{_datadir}/themes/Dopple-Left/ %{_datadir}/themes/Dopple/ %{_datadir}/themes/DustBlue/ %{_datadir}/themes/Spidey-Left/ %{_datadir}/themes/Spidey/ %{_datadir}/themes/Splint-Left/ %{_datadir}/themes/Splint/ %{_datadir}/themes/WinMe/ %{_datadir}/themes/eOS/ or even just %{_datadir}/themes/*
Fixed. Spec URL: http://vicodan.fedorapeople.org/matespec/mate-window-manager.spec SRPM URL: http://vicodan.fedorapeople.org/materpms/srpms/mate-window-manager-1.4.0-3.fc17.src.rpm Description: MATE Desktop window manager
missing Requires(pre): mate-conf Requires(post): mate-conf Requires(preun): mate-conf please add prior to doing any builds. APPROVED.
added to spec and uploaded.
New Package SCM Request ======================= Package Name: mate-window-manger Short Description: MATE Desktop window manager Owners: vicodan rdieter Branches: f16 f17 f18
Git done (by process-git-requests). Corrected name.
mate-window-manager-1.4.0-3.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/mate-window-manager-1.4.0-3.fc18
mate-window-manager-1.4.0-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-window-manager-1.4.0-3.fc17
Like the other mate package, this review missed importants bits : - desktop file usage X-mate seems incorrect, see #847604 - the license should also be more explicit, see #847604 and #847419 ( especially since this would have shown there is gtk code bundled in the tarball, it seems since a few years ).
%changelog * Mon Aug 27 2012 Rex Dieter <rdieter> 1.4.0-4 - main pkg Requires: %%name-libs - drop needless icon scriptlets - s|MATE|X-MATE| .desktop Categories on < f18 only - License: GPLv2+ though your comment about bundled code is ominous, details?
There is code coming from gtk, src/ui/metaaccellabel.c but that's a upstream issue. I do not know how much did the code evolved, and if that matter. I guess this should be discussed with gtk and mate upstream ?
It does say at the top of that file: /* Marco hacked-up GtkAccelLabel */ which certainly implies that it was done on purpose (for some reason).
mate-window-manager-1.4.0-3.fc18 has been pushed to the Fedora 18 testing repository.
mate-window-manager-1.4.0-3.fc17 has been pushed to the Fedora 17 stable repository.
mate-window-manager-1.4.1-1.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/mate-window-manager-1.4.1-1.fc18
mate-window-manager-1.4.1-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mate-window-manager-1.4.1-1.fc17
mate-window-manager-1.4.0-3.fc18 has been pushed to the Fedora 18 stable repository.