Cześć!
To mój pierwszy patch więc proszę o dużo uwag co zostało źle
zrobione ;-)

Nie działało mi przeglądanie "skórek" i do tego znalazłem w Changelog w
mplayerze:

"    * skins now reside in a directory named 'skins', not
'Skin'" (gdzieś w 2006 roku....)

no i poprawiłem (czekam na wiadomości co przy okazji zepsułem?) to tak
jak widać w załączniku. 
Pewnie powinienem jeszcze poprawić release, ale nie jestem tego pewien,
więc nie robię.

Czekam na wszelkie uwagi, szczególnie te krytyczne.
-- 
Przemysław Firszt
--- mplayer.spec	2008-01-11 20:10:58.000000000 +0000
+++ mplayer.spec.	2008-01-11 19:46:37.000000000 +0000
@@ -465,7 +465,7 @@
 install -d \
 	$RPM_BUILD_ROOT{%{_bindir},%{_pixmapsdir},%{_sysconfdir}/mplayer} \
 	$RPM_BUILD_ROOT%{_mandir}/{cs,de,es,fr,hu,it,pl,sv,zh_CN,}/man1 \
-	$RPM_BUILD_ROOT%{_datadir}/mplayer/Skin \
+	$RPM_BUILD_ROOT%{_datadir}/mplayer/skins \
 	$RPM_BUILD_ROOT%{_desktopdir}
 
 # default config files
@@ -490,7 +490,7 @@
 ln -sf font-arial-iso-8859-2/font-arial-24-iso-8859-2 $RPM_BUILD_ROOT%{_datadir}/mplayer/font
 
 %if %{with gui}
-ln -s Blue $RPM_BUILD_ROOT%{_datadir}/%{name}/Skin/default
+ln -s Blue $RPM_BUILD_ROOT%{_datadir}/%{name}/skins/default
 install %{SOURCE5} $RPM_BUILD_ROOT%{_desktopdir}
 %endif
 install %{SOURCE8} $RPM_BUILD_ROOT%{_desktopdir}
@@ -528,8 +528,8 @@
 %defattr(644,root,root,755)
 %attr(755,root,root) %{_bindir}/gmplayer*
 %{_desktopdir}/gmplayer.desktop
-%dir %{_datadir}/%{name}/Skin
-%ghost %{_datadir}/%{name}/Skin/default
+%dir %{_datadir}/%{name}/skins
+%ghost %{_datadir}/%{name}/skins/default
 %endif
 
 %if %{with mencoder}
--- mplayer-skins.spec	2008-01-11 20:11:06.000000000 +0000
+++ mplayer-skins.spec.	2008-01-11 19:47:08.000000000 +0000
@@ -137,7 +137,7 @@
 BuildArch:	noarch
 BuildRoot:	%{tmpdir}/%{name}-%{version}-root-%(id -u -n)
 
-%define		_skindir	%{_datadir}/mplayer/Skin
+%define		_skindir	%{_datadir}/mplayer/skins
 
 %description
 Additional skins for MPlayer. Included are skins emulating look of
_______________________________________________
pld-devel-pl mailing list
[email protected]
http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl

Odpowiedź listem elektroniczym