info: hdparm.spec

Krzysztof G. Baranowski kgb w manjak.knm.org.pl
Czw, 24 Wrz 1998, 12:30:03 CEST


On Thu, 24 Sep 1998, Wojtek Slusarczyk wrote:
> > %build
> > make CFLAGS=$RPM_OPT_FLAGS
> Tutaj to ja z taka doza niepewnosci do tego podchodze... nie wiem czy
> przypadkiem te flagi nie przykrywaja innych... bo jak tak to moze w
> Makefile'u dolozyc $(RPM_FLAGS) czy jakos tak ... albo
> CFLAGS="$RPM_OPT_FLAGS" make ... sam nie wiem.. 
*blush* ! To w ogole nie dotyka Makefile. Tam jest tak:

--- cut ---
# Makefile for hdparm

all: hdparm

hdparm: hdparm.c /usr/include/linux/hdreg.h
        gcc -Wall -Wstrict-prototypes -O2 -s -o hdparm hdparm.c

install: all hdparm.8
[...]
--- cut ---	

IMO powinno sie zacerowac Makefila.

--- cut ---
# Makefile for hdparm
OPTFLAGS = -O2

all: hdparm

hdparm: hdparm.c /usr/include/linux/hdreg.h
        gcc -Wall -Wstrict-prototypes $(OPTFLAGS) -o hdparm hdparm.c
[...]
--- cut ---

Rationale:
1) Warning moga pozostac tam gdzie byly, bo sa tylko warningami.
2) Zmiana -O2 na $(OPTFLAGS) daje mozliwosc manipulacji tymi
   z poziomu rpma. Na dodatek standardowe rpmowe optflags, tzn.
	   -O2 -m486 -fno-strength-reduce
   niczego tu nie zepsuja.
3) Usunalem '-s' z flag kompilacji, bo stripowanie powinno sie odbywac
   podczas rozsiewania pakietow 'install -s'. Gdybym chcial
   zbudowac sobie    pakiet z przeznaczeniem do debugowana, to
   musialbym cerowac makefile. A tak tylko w specu zmienie sobie OPTFLAGS 
   dodajac -g, wywale -s z %install i po przebudowaniu jestem
   gotowy do debugingu.

Latwo, czysto i przyjemnie. 

Jesli nie uslysze sprzeciwu, to poprawie pakiet i podesle
Ci i kloczkowi mailem na priva (bo jest dosc maly ~20kB).   

> > install -s hdparm $RPM_BUILD_ROOT/sbin/hdparm
> > install hdparm.8 $RPM_BUILD_ROOT/usr/man/man8
> a binarka jest stripowana, bo nie sprawdzalem..?
man install, /-s   :-)

Kris
-- 
Ojciec Hihotnik Trzeci, Szesciotysiecznik, HMS, Jego Wysokość
Prezes Klubu Nieszkodliwych Manjaków, Krzysztof G. Baranowski
a.k.a. KGB.                          <http://www.knm.org.pl/>



Więcej informacji o liście dyskusyjnej pld-devel-pl