SPECS: em8300.spec (HEAD) [blues]

[Blues]

On Thu, 5 Sep 2002, Robert R. Wal wrote:
> > > Module name:	SPECS
> > > Changes by:	blues	02/09/05 19:21:22
> > > Modified files:
> > > 	em8300.spec
> > > Log message:
> > > - few more fixes - added static package, changes in groups - STBR now
> > > Index: em8300.spec
> > > ===================================================================
> > Dopiero teraz zobaczyłem, że w tym pakiecie jest moduł... :-/
> > To tak nie może być.
> > 1. Moduł powinien być w osobnym module, żebym nie musiał, kiedy chcę 
> > mplayera zrobić instalować moduły - to niedopuszczalne.
> Nie musisz instalować modułu, nie ma sztywnych zależności mplayer
> requires -> em8300, bo mplayer robi tylko ioctl() po tym device.

Ale, żeby zbudować muszę zainstalować ten moduł.

> > 2. Powinno być w jednym przebiegu budowane UP i SMP moduły, ale jak 
> > poprawisz punkt pierwszy to będzie poprawione i 2 :)
> > 3. Podpakiet -lib powinien być wydzielony - żeby też nie trzeba było 
> > demona instalować.
> Nie ma demona, potrzebne jest tylko ładowanie mikrokodu do karty.

Jak zwał tak zwał. Powinno być oddzielone.

> > Zauważ, że w obecnej postaci BR: em8300-devel nie powinien się znaleźć w 
> > mplayerze, bo powoduje duże problemy.
> Devel jest devel, mplayer potrzebuje tylko nagłówków do kompilacji.

To popraw requires.

> Słuchaj. Nie umiem tego zrobić w/g Twojego widzimisię. Uważasz że
> powinno trzymać się jakiejś ideefixe, to zrób tak, dasz dobry
> przykład.

To nie jest moje widzimisie, ale ogolne standardy, jakie możesz obejrzeć 
we wszystkich pakietach. Chcesz wzór? Zobacz cokolwiek - np. capsel, ians.
Dlaczego ja tego nie zrobię? Bo mi się nie chce i mam lepsze zajęcia. 
Jeżeli chcesz, żeby ten pakiet był w PLD i mplayer był z nim budowany to 
go popraw, bo nie wydaje mi się, że kloczek będzie chciał go w aktualnej 
postaci na builderze zainstalować. em8300 nawet na ftp-ie się nie pojawi.

> Na razie w imię jakiejś swojej idei wyrzuciłeś działający fragment kodu.

Może i działający, ale jeżeli chcesz żeby to było w zasobach 
dystrybucyjnych to zrób to porządnie.

> > Niniejszym to komentuje do czasu jak  nie poprawisz pkt.1 i 3.
> A ja natychmiast po dorwaniu się do kompa na którym mam dostęp do CVS-u
> to odkomentuje i przywrócę do postaci działającej.
> Nie podoba Ci się? To popraw żeby Ci się podobało, ale nie psuj.
> Primum non nocere.

Właśnie - to ty szkodzisz.

O, nie, nie nie... Nawet nie próbuj.

Ja nie chcę na moim builderze mieć żadnego badziewnego modułu. Zresztą 
jeszcze źle zapakowanego. Podziel to ładnie, a nikt nie powie ci złego 
słowa. Aktualnie pakiet się nie nadaje do włączenia w zasoby 
dystrybucyjne.
Wyobraź sobie moją wściekłość, kiedy zobaczyłem, jak mi wyrzuca, złe 
symbole po depmod. A w %files nie ma ani słowa o żadnym module, dopiero po 
_bardzo_ bliskim przyjrzeniu się....

-- 
---------------------------------
pozdr.  Paweł Gołaszewski        
---------------------------------
CPU not found - software emulation...