This patch adds dist-tag for sawfish. It's not mandatory, but it seems to be current practice on rpm-based systems. See:
https://fedoraproject.org/wiki/Packaging/DistTag Files-section should not include "locale/*". Correct way to handle them is to use %find_lang, which is already used. From Fedora packaging guidelines: https://fedoraproject.org/wiki/Packaging/Guidelines "Packages that use %{_datadir}/* to grab all the locale files in one line also grab ownership of the locale directories, which is not permitted." File "datadir/apps/ksmserver/windowmanagers/sawfish.desktop" was not packaged, add it. --- sawfish-1.5.1/sawfish.spec.in.orig 2009-09-05 09:56:01.000000000 +0300 +++ sawfish-1.5.1/sawfish.spec.in 2009-09-05 10:24:35.000000000 +0300 @@ -1,7 +1,7 @@ Summary: An extensible window manager for the X Window System. Name: sawfish Version: @version@ -Release: 1 +Release: 1%{?dist} Requires: librep >= @REP_MIN_VER@, rep-gtk >= @REP_GTK_MIN_VER@ License: GPL Group: User Interface/Desktops @@ -67,9 +67,9 @@ %{_bindir}/* %dir %{_datadir}/sawfish %{_datadir}/sawfish/%{version} -%{_datadir}/locale %{_datadir}/gnome/wm-properties/sawfish-wm.desktop %{_datadir}/applications/sawfish.desktop +%{_datadir}/apps/ksmserver/windowmanagers/sawfish.desktop %{_datadir}/xsessions/sawfish.desktop %{_datadir}/pixmaps/sawfish*.png %{_libdir}/pkgconfig/sawfish.pc @@ -78,6 +78,9 @@ %{_infodir}/sawfish* %changelog +* Sat Sep 05 2009 Kim B. Heino <[email protected]> +- add dist-tag, update files list + * Sun Jan 18 2008 Christopher Bratusek <[email protected]> - several fixups
