[devel] eciadsl-usermode-0.12-alt1

[Kirill Maslinsky]

On Mon, Jan 21, 2008 at 12:39:32AM +0300, Alexey Tourbin wrote:
> >  ecasound-devel-2.4.5-alt1	Requires(rpmlib)	rpmlib(PayloadFilesHavePrefix) <= 4.0-1
> > +eciadsl-usermode-0.12-alt1	Requires	/usr/bin/eciadsl-ctrlui
> >  eciadsl-usermode-0.12-alt1	Requires	SysVinit
> 
> Этот файл не запаковывается при сборке пакета.
> 
> + /usr/lib/rpm/check-files
> + exit 0
> warning: Installed (but unpackaged) file(s) found:
>     /usr/bin/eciadsl-ctrlui
> Wrote: /usr/src/RPM/SRPMS/eciadsl-usermode-0.12-alt1.src.rpm
> Wrote: /usr/src/RPM/RPMS/x86_64/eciadsl-usermode-0.12-alt1.x86_64.rpm

Спасибо, это я уже заметил и исправил, просто не обновлял еще пакет.

> Он используется в скрипте /usr/bin/eciadsl-stop:
> 
> $ rpmpeek eciadsl-usermode-0.12-alt1.i586.rpm cat -n ./usr/bin/eciadsl-stop |grep -C5 eciadsl-ctrlui
>     11  #
>     12  
>     13  echo -e "EciAdsl: shutting down..."
>     14  ps ax | grep "eciadsl-pppoeci" | grep -v grep > /dev/null 2>&1
>     15  if [ $? -eq 0 ]; then
>     16          eciadsl-ctrlui --disconnect
>     17          sleep 1
>     18          ps ax | grep "pppd" | grep -v grep > /dev/null 2>&1
>     19          if [ $? -eq 0 ]; then
>     20          killall pppd
>     21              if [ $? -eq 0 ]; then
> $
> 
> Вообще этот скрипт довольно подозрительный -- вместо 'ps ax'
> и 'killall pppd' лучше использовать start-stop-daemon.

Боюсь, что там все скрипты весьма подозрительные. 
Посмотрю, может быть попробую что-то улучшить к следующей сборке пакета. 
Спасибо за замечания. 

--
Kirill Maslinsky
ALT Linux Team