SOURCES: alsa-udev.sh, alsa-udev.rules - script moved from /etc/ud...

[Jakub Bogusz]

On Mon, Oct 29, 2007 at 10:12:59PM +0100, Kamil Dziedzic wrote:
> Dnia poniedziałek 29 październik 2007, Jakub Bogusz napisał:
> > > Index: SOURCES/alsa-udev.rules
> > > diff -u SOURCES/alsa-udev.rules:1.1 SOURCES/alsa-udev.rules:1.2
> > > --- SOURCES/alsa-udev.rules:1.1	Wed Dec 21 21:25:46 2005
> > > +++ SOURCES/alsa-udev.rules	Mon Oct 29 14:46:33 2007
> > > @@ -1 +1 @@
> > > -SUBSYSTEM=="sound", KERNEL=="controlC*", RUN+="/etc/udev/scripts/alsa.sh
> > > %n" +SUBSYSTEM=="sound", KERNEL=="controlC*", RUN+="/lib/udev/alsa-udev
> > > %n"
> >
> >                                                   ^^^^^
> >
> > Ta ścieżka nie wszędzie jest poprawna.
> 
> W jakim sensie? Samo lib jest poprawiane w trakcie budowania:

Nawet nie zauważyłem - ale i tak jest źle.

> install -D %{SOURCE0} $RPM_BUILD_ROOT/etc/udev/rules.d/alsa.rules
> sed -i -e 's#/lib/udev/#/%{_lib}/udev/#g' %{SOURCE0}
> 
> Ale teraz widzę, że mi się pochrzaniło i w złym miejscu... powinno być raczej 
> w odwrotnej kolejności:
> sed -i -e 's#/lib/udev/#/%{_lib}/udev/#g' %{SOURCE0}
> install -D %{SOURCE0} $RPM_BUILD_ROOT/etc/udev/rules.d/alsa.rules

Nie, budowanie pakietu NIE MOŻE modyfikować SOURCES.

> Hmmm... a w sumie to chyba powinno być tak:
> install -D %{SOURCE0} $RPM_BUILD_ROOT/etc/udev/rules.d/alsa.rules
> sed -i -e 's#/lib/udev/#/%{_lib}/udev/#g' 
> $RPM_BUILD_ROOT/etc/udev/rules.d/alsa.rules

Lepiej. Albo w potoku, w czasie tworzenia pliku:

install -d $RPM_BUILD_ROOT/etc/udev/rules.d
sed -e 's#/lib/udev/#/%{_lib}/udev/#g' %{SOURCE0} >$RPM_BUILD_ROOT/etc/udev/rules.d/alsa.rules


-- 
Jakub Bogusz    http://qboosh.pl/