rrw: SPECS xine-lib.spec

[Jacek Konieczny]

On Tue, Aug 07, 2001 at 02:12:20PM +0200, PLD CVS wrote:
> Module name:	SPECS
> Changes by:	rrw	01/08/07 14:12:20
> 
> Modified files:
> 	.              : xine-lib.spec 
> 
> Log message:
> - conditionalized build for most plugins

Widzę, że pluginy wrzuciłeś do osobnych podpakietów - bardzo ładnie.
Ale mam następujące uwagi:

1. Nie potrzebnie całe definicje podpakietów dałeś do %{?....}. Jeśli
pyste będzie %files dla danego podpakietu, to rpm go nie wygeneruje, 
a tak to spec jest bardzo nieczytelny. Także samo "%files" można
dać bezwarunkowo --- teraz nie widać gdzie zaczynają się pliki kolejnych
podpakietów.

2. Podział na podpakiety warto IMHO robić głównie żeby uniknąć
zależności. A u ciebie główny pakiet zależeć będzie od libX11. 
Na pewno nie da się tego uniknąć?
Ktoś peweni spyta po co? Odpowiedź: dla zasady. Jak ktoś chce filmy
na libaa oglądać nie każmy mu instalować Xów. :-)

3. Jeśli udałoby się uniknąć zależności głównego pakietu od
XFree86-libs, to zmieniłbym mu jeszcze prefix :-)

> - started support for DXR3 (yes, I need it ;), nowhere near to finish
> - added post/postun ldconfig
>