sendip.spec

[Jakub Bogusz]

On Sat, Jan 22, 2005 at 12:18:29AM +0100, Jan Wideł wrote:
> On Fri, 21 Jan 2005, Artur Makutunowicz wrote:
> >
> > Witam.
> > Pakiet buduje się i działa.
> 
> Dzięki dodane.
> 
> Moje uwagi:
> 
> > License:        GPL
> 
> źle, nie przejżałeś LICENCES:
> License:        GPL v2

"nie przejrzałeś". LICENSE, nie LICENSES.
+README, bo w nim napisane, że obowiązuje wersja z pliku (bez
alternatywy w postaci późniejszej wersji, co się często zdarza
w przypadku (L)GPL).

> > %build
> > %{__make} PREFIX=%{_prefix} LIBDIR=%{_libdir}/sendip
> 
> wystarczyło:
> 
> %{__make} \
>         PREFIX=%{_prefix}

Nie, %{_libdir} to nie to samo co %{_prefix}/lib

> > install -d $RPM_BUILD_ROOT{%{_bindir},%{_mandir}/man1,%{_libdir}/sendip}
> 
> > install *.so $RPM_BUILD_ROOT%{_libdir}/sendip
> > install sendip $RPM_BUILD_ROOT%{_bindir}
> > install sendip.1 $RPM_BUILD_ROOT%{_mandir}/man1
> 
> niepotrzebne, wszytko załatwiał:
> 
> %{__make} install \
>          PREFIX=$RPM_BUILD_ROOT%{_prefix}
+ LIBDIR=...

> > %attr(755,root,root) %{_bindir}/sendip
> 
> Lepiej używać makr, po to są:
> 
> %attr(755,root,root) %{_bindir}/%{name}

Tutaj? Dyskusyjne, z %{name} jest IMO mniej czytelne.
Ale nie ma przyjętego standardu.


PS. nie rozumiem powodu wydzielenia modułów dynamicznych (nazwanie ich
"bibliotekami współdzielonymi" jest naciągane) do podpakietu -libs.
I wywoływania ldconfig w nim.


-- 
Jakub Bogusz    http://cyber.cs.net.pl/~qboosh/