Prośba o dodanie speca

[Adam Gołębiowski]

On Fri, Feb 25, 2005 at 02:35:11PM +0100, Marcin Kałuża wrote:
> zrobiłem speca do programu xhkeys - to coś jak hotkeys,  które jest w cvsie,
>  z tym, że to sie kompiluje i działa, a tamten nie :(.
> Starałem się robić speca zgodnie ze wskazówkami na stronie PLD.

Parę uwag

> # $Revision:$, $Date:$
> Summary:	xhkeys - a tool for assigning various actions to unused keys in X.
> Summary(pl):	xhkeys - narzędzie do przypisywania nieużywanym w X'ach klawiszom różnych akcji
> Name:		xhkeys
> Version:	2.2.1
> Release:	1
> License:	GPL
> 

Po co ta pusta linia?
> Vendor:		Michael Glickman <wmalms w yahoo.com>
> Group:		X11/Applications 
> ######		Unknown group!

Grupa dobra, komentarz do usunięcia

> Source0:	http://www.geocities.com/wmalms/%{name}-%{version}.tar.gz

Brak Source0-md5 (dodajemy przez ./builder -a5)

> URL:		http://wmalms.tripod.com

Stramy się dodawać '/' na końcu

> BuildRequires:	sed
> BuildRequires:	XFree86-devel

Jeśli nie potrzebujemy seda >= 4.0, nie dodajemy tego BR: (jest wymagane
przez rpm-build)

> BuildRoot:	%{tmpdir}/%{name}-%{version}-root-%(id -u -n)
> 
> 

Kolejna nadmiarowa pusta linia

> The package includes configuration utility.
> 
> 
> %description -l pl

I znów

> Pakiet zawiera narzędzie do konfiguracji.
> 
> %define prefix /usr

A to %dfine jest w ogóle nie potrzebne

> %build
> export CFLAGS='-O4 -march=%_target_cpu -mcpu=%_target_cpu'

Absolutnie

> ./configure --prefix=$RPM_BUILD_ROOT/%{_prefix}
> mv Makefile Makefile.orig
> sed -e "s/\$(LDCONFIG)/#\$(LDCONFIG)/g" Makefile.orig > Makefile

Tu wolimy raczej łatkę na Makefile.in (.am)

> %{__make}
> 
> %install
> rm -rf $RPM_BUILD_ROOT
> %{__make} install
> %{__make} install_doc

A to nam napewno zainstaluje w $RPM_BUILD_ROOT ?
> 
> %post -p /sbin/ldconfig
> %postun -p /sbin/ldconfig
> 
> 

Na pewno potrzebujemy ldconfig? I kolejna nadmiarowa linia,

> %files
> %defattr(644,root,root,755)
> %doc XHkeys.sample
> %doc manual.html
> %doc xhkeys.lsm

Lepiej zwinąć to do jednej linii: '%doc XHkeys.sample manual.html'

> %attr(755,root,root) %{_bindir}/xhkeys
> %attr(755,root,root) %{_bindir}/xhkconf
> %{_prefix}/lib/xhkeys/xhkeys_cdaudio.so
> %{_prefix}/lib/xhkeys/xhkeys_mixer.so

Złe, złe, złe! %{_libdir} jezeli juz i brak odpowiednich uprawniń (nie
powinno być %attr(755,root,root) ?)

> 
> 

pusta linia

> %clean
> rm -rf $RPM_BUILD_ROOT

%clean powinno być przed %files



Postaraj się te błędy poprawić.

-- 
http://www.mysza.eu.org/ | Everybody needs someone sure, someone true,
   PLD Linux developer   | Everybody needs some solid rock, I know I do.