[devel] strange scripts

[devel] strange scripts

[Stanislav Ievlev]

[-- Attachment #1: Type: text/plain, Size: 4576 bytes --]

Привет всем!

В attach'е приведены ошибочные на мой взгляд скрипты в пакетах.

Очень возможно я где-то не прав, обязательно сообщите мне об этом
иначе это всё пойдет в BTS с разными severity ;)

Что у меня вызывало подозрение:
- добавление служебного пользователя с оболочкой
- "эмуляция" альтернатив
- неиспользование стандартных макросов
- проведение исправления в конфигурационных файлах, принадлежащим другим
  пакетам
- просто явные опечатки
- слишком длинные скрипты

Стандартная ошибка: переопределение в пакете %prefix приводит к порче
макросов и соотв. скриптов.
Например /usr/bin/update-menus превращается /usr/games/bin/update-menus
или /usr/X11R6/update-menus ;)

Вот список пакетов в которых обнаружены подозрительные скрипты:
FirebirdCS-1.0.3.972-alt2.i586.rpm
FirebirdSS-1.0.3.972-alt2.i586.rpm
WMRack-1.2-alt1.i586.rpm
XFree86-libs-4.3.0-alt13.i586.rpm
XFree86-4.3.0-alt13.i586.rpm
XFree86-cyr_rfx-fonts-cp1251-1.1-ipl6mdk.noarch.rpm
XFree86-cyr_rfx-fonts-iso10646-0400-1.1-alt3.noarch.rpm
XFree86-cyr_rfx-fonts-iso8859-5-1.1-ipl3mdk.noarch.rpm
XFree86-cyr_rfx-fonts-koi8-r-1.1-alt7.noarch.rpm
XFree86-cyr_rfx-fonts-koi8-u-1.1-alt8.noarch.rpm
Zope-ZUtils-2.6.4-alt1.i586.rpm
abiword-2.0.0-alt2.i586.rpm
aegis-4.16-alt2.i586.rpm
amanda-common-2.4.4-alt4.1.i586.rpm
amavisd-new-20030616-alt8.1.noarch.rpm
bforce-xinetd-0.22.8-alt4.i586.rpm
binkd-0.9.5-alt1.i586.rpm
bootparamd-0.17-ipl4mdk.i586.rpm
bugzilla-2.16.3-alt1.noarch.rpm
bugzilla-ru-2.16.1.1.0-alt1.noarch.rpm
crashecho-0.2.14-alt1.i586.rpm
cyrus-imapd-2.2.3-alt1.2.i586.rpm
design-bootsplash-2.2-alt3.Master.i586.rpm
design-bootsplash-Compact-2.2.9-alt2.i586.rpm
design-bootsplash-Junior-2.2-alt2.i586.rpm
design-bootsplash-iRu-2.3-alt2.i586.rpm
design-graphics-Compact-3.1.0.1-alt2.noarch.rpm
design-graphics-iRu-3.1.0-alt5.1.noarch.rpm
design-graphics-sisyphus-3.1.1-alt1.noarch.rpm
djvu-plugin-3.5.13-alt1.cvs.i586.rpm
emacs-manual-ru-20.7-alt1.noarch.rpm
emacs-preview-latex-0.8-alt1.noarch.rpm
exim-heavy-4.30-alt3.i586.rpm
exim-info-4.30-alt3.i586.rpm
exim-ldap-4.30-alt3.i586.rpm
exim-light-4.30-alt3.i586.rpm
exim-monitor-4.30-alt3.i586.rpm
exim-mysql-4.30-alt3.i586.rpm
exim-pdf-doc-4.30-alt3.i586.rpm
exim-pgsql-4.30-alt3.i586.rpm
floppy-0.12-alt2.i586.rpm
freefont-fonts-ttf-0.030422-alt4.noarch.rpm
ftn-2.2-alt4.noarch.rpm
gtklp-0.9k-alt3.i586.rpm
isdn4net-1.4.6-alt3.noarch.rpm
jackd-0.94.0-alt1.i586.rpm
jadetex-3.13-alt1.noarch.rpm
kdegames-atlantik-3.2.1-alt1.i586.rpm
kdegames-kasteroids-3.2.1-alt1.i586.rpm
kdegames-katomic-3.2.1-alt1.i586.rpm
kdegames-kbackgammon-3.2.1-alt1.i586.rpm
kdegames-kbattleship-3.2.1-alt1.i586.rpm
kdegames-kblackbox-3.2.1-alt1.i586.rpm
kdegames-kbounce-3.2.1-alt1.i586.rpm
kdegames-kenolaba-3.2.1-alt1.i586.rpm
degames-kenolaba-3.2.1-alt1.i586.rpm
kdegames-kfouleggs-3.2.1-alt1.i586.rpm
kdegames-klickety-3.2.1-alt1.i586.rpm
kdegames-klines-3.2.1-alt1.i586.rpm
kdegames-kmahjongg-3.2.1-alt1.i586.rpm
kdegames-kmines-3.2.1-alt1.i586.rpm
kdegames-kolf-3.2.1-alt1.i586.rpm
kdegames-konquest-3.2.1-alt1.i586.rpm
kdegames-kpat-3.2.1-alt1.i586.rpm
kdegames-kpoker-3.2.1-alt1.i586.rpm
kdegames-kreversi-3.2.1-alt1.i586.rpm
kdegames-ksame-3.2.1-alt1.i586.rpm
kdegames-kshisen-3.2.1-alt1.i586.rpm
kdegames-ksirtet-3.2.1-alt1.i586.rpm
kdegames-ksmiletris-3.2.1-alt1.i586.rpm
kdegames-ksnake-3.2.1-alt1.i586.rpm
kdegames-ksokoban-3.2.1-alt1.i586.rpm
kdegames-kspaceduel-3.2.1-alt1.i586.rpm
kdegames-ktron-3.2.1-alt1.i586.rpm
kdegames-ktuberling-3.2.1-alt1.i586.rpm
kdegames-kwin4-3.2.1-alt1.i586.rpm
kdegames-libs-3.2.1-alt1.i586.rpm
kdegames-lskat-3.2.1-alt1.i586.rpm
latex2html-2002-alt5.noarch.rpm
ldetect-lst-0.2.23-alt1.noarch.rpm
libfreetds-0.61-alt2.i586.rpm
lyx-1.3.4-alt1.i586.rpm
mailgraph-1.2-alt1.i586.rpm
net-scripts-0.4.3-alt1.i586.rpm
identd-3.0.16-alt1.i586.rpm
recode-3.6-alt2.i586.rpm
sleezeball-0.6-alt5.i586.rpm
slib-2d6-alt1.noarch.rpm
snort-2.1.0-alt1.i586.rpm
snort-bloat-2.1.0-alt1.i586.rpm
snort-contrib-2.1.0-alt1.i586.rpm
snort-mysql+flexresp-2.1.0-alt1.i586.rpm
snort-mysql-2.1.0-alt1.i586.rpm
snort-odbc+flexresp-2.1.0-alt1.i586.rpm
snort-odbc-2.1.0-alt1.i586.rpm
snort-plain+flexresp-2.1.0-alt1.i586.rpm
snort-postgresql+flexresp-2.1.0-alt1.i586.rpm
snort-postgresql-2.1.0-alt1.i586.rpm
snownews-1.5.1-alt1.i586.rpm
subversion-server-common-1.0.1-alt1.i586.rpm
timidity-eaw-patches-1.2-alt1.noarch.rpm
timidity-instruments-1.0-ipl14mdk.noarch.rpm
tv-fonts-1.1-alt1.i586.rpm
util-vserver-0.26-alt1.i586.rpm
w3c-validator-0.6.1-alt1.noarch.rpm
wmnd-0.4.7-alt1.i586.rpm
wwwoffle-2.8b-alt1.i586.rpm
xawtv-vdr-3.90-alt2.i586.rpm

--
Стас.


[-- Attachment #2: strange-scripts.bz2 --]
[-- Type: application/x-bzip2, Size: 7015 bytes --]

Re: [devel] strange scripts

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 663 bytes --]

* Stanislav Ievlev <inger@> [040421 14:09]:
> bforce-xinetd-0.22.8-alt4.i586.rpm
> binkd-0.9.5-alt1.i586.rpm
> crashecho-0.2.14-alt1.i586.rpm
Тут не исправление, а создание логов с правильными правами
(кстати, в qico то же самое, а в списке его нет ;-).
Псевдопользователь ftn не умеет создавать файлы в /var/log/ftn.

control(8) тут поможет?

> ftn-2.2-alt4.noarch.rpm
(добавление пользователя с оболочкой, зачем?)
Дурной вопрос - system(3) и crond(1) будут работать для
пользователя без шелла?  Если да, то нет проблем.

-- 
Regards, Sir Raorn.
-------------------
Список недостатков не обсуждать, а исправлять надо ;-)
		-- rider in devel@

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] strange scripts

[Stanislav Ievlev]

On Wed, Apr 21, 2004 at 02:25:53PM +0400, Alexey I. Froloff wrote:
> * Stanislav Ievlev <inger@> [040421 14:09]:
> > bforce-xinetd-0.22.8-alt4.i586.rpm
> > binkd-0.9.5-alt1.i586.rpm
> > crashecho-0.2.14-alt1.i586.rpm
> Тут не исправление, а создание логов с правильными правами
> (кстати, в qico то же самое, а в списке его нет ;-).
Я не стал однотипные вещи дублировать ;)
> Псевдопользователь ftn не умеет создавать файлы в /var/log/ftn.
Все необходимые права можно задать прямо в spec-файле на каталог
/var/log/ftn.
> 
> control(8) тут поможет?
> 
> > ftn-2.2-alt4.noarch.rpm
> (добавление пользователя с оболочкой, зачем?)
> Дурной вопрос - system(3) и crond(1) будут работать для
> пользователя без шелла?  Если да, то нет проблем.
Надо разобраться по подробнее.
Что там за cronjob? Как он добавляется? От чьего имени исполняется?

> 
> -- 
> Regards, Sir Raorn.
> -------------------
> Список недостатков не обсуждать, а исправлять надо ;-)
> 		-- rider in devel@



> _______________________________________________
> Devel mailing list
> Devel@altlinux.ru
> http://lists.altlinux.ru/mailman/listinfo/devel

Re: [devel] strange scripts

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 842 bytes --]

* Stanislav Ievlev <inger@> [040421 15:33]:
> > Псевдопользователь ftn не умеет создавать файлы в /var/log/ftn.
> Все необходимые права можно задать прямо в spec-файле на каталог
> /var/log/ftn.
Помнится, ldv на этот счёт очень возражал.  non-root owned и/или
group-writable.

> > > ftn-2.2-alt4.noarch.rpm
> > (добавление пользователя с оболочкой, зачем?)
> > Дурной вопрос - system(3) и crond(1) будут работать для
> > пользователя без шелла?  Если да, то нет проблем.
> Надо разобраться по подробнее.
> Что там за cronjob? Как он добавляется? От чьего имени исполняется?
cronjob - разные шеллскрипты, которые запускают кучу ftn барахла.
Добавляется руками (crontab -u ftn -e), исполняются от имени ftn.

-- 
Regards, Sir Raorn.
-------------------
[...] любители ругаться не читают документацию :-)
		-- aen in devel@

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] strange scripts

[Sviataslau Svirydau]

Stanislav Ievlev wrote:

>Привет всем!
>
>В attach'е приведены ошибочные на мой взгляд скрипты в пакетах.
>
>Очень возможно я где-то не прав, обязательно сообщите мне об этом
>иначе это всё пойдет в BTS с разными severity ;)
>
>...
>subversion-server-common-1.0.1-alt1.i586.rpm
>  
>
------------ (это наверное стоит перенести в триггер)
subversion-server-common-1.0.1-alt1.i586.rpm
preinstall scriptlet (through /bin/sh):
/usr/sbin/groupadd -r -f subversion
/usr/sbin/useradd -r -g subversion -d /var/lib/subversion -s /dev/null 
-c 'Subversion' subversion >/dev/null 2>
&1 ||:
if [ $1 -gt 1 ]; then
        /usr/sbin/usermod -d /var/lib/subversion subversion
fi
--------------

Где можно посмотреть пример как это правильно делать?
(подобный скрипт тоже делался по примеру из другого пакета :))

-- 
Regards,
Sviatoslav Sviridov <Sviataslau_Svirydau at epam.com>
phone:  +375 17 210 1662, ext. 1302
mobile: +375 29 641 4214
-- 
The sendmail configuration file is one of those files that looks like someone
beat their head on the keyboard.  After working with it... I can see why!
		-- Harry Skelton

Re: [devel] strange scripts

[Dmitry Vukolov]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wednesday 21 April 2004 13:35, Stanislav Ievlev wrote:
> Привет всем!
>
> В attach'е приведены ошибочные на мой взгляд скрипты в пакетах.
>
> Очень возможно я где-то не прав, обязательно сообщите мне об этом
> иначе это всё пойдет в BTS с разными severity ;)
>
> Что у меня вызывало подозрение:
<skipped>
> - проведение исправления в конфигурационных файлах, принадлежащим
> другим пакетам

А можно немного подробнее об этом пункте? Как в таких случаях поступать, 
например, с не указанным в списке libsane-hpoj? Отправлять 
пользователей самостоятельно пакет доустанавливать? Или к данному 
пакету вышеуказанный пункт не относится?

Спасибо.

- -- 
Dmitry Vukolov
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFAhqUC71tprB1tyF0RAlE2AKCJtNCNqp/INQQBmpn/sLBFgg0NLACfbY77
s7iFlJ1O2EWr9Pvs1f4BaAE=
=3RgS
-----END PGP SIGNATURE-----

Re: [devel] strange scripts

[Alex Ott]

Hello

>>>>> "SI" == Stanislav Ievlev writes:
 SI> emacs-preview-latex-0.8-alt1.noarch.rpm

там зовется texhash, для которого я что-то не нашел макроса. ткните меня
носом, если я не прав

-- 
With best wishes, Alex Ott
-------------------------------
Jet Infosystems, Moscow, Russia    mailto: ottalex@narod.ru
http://xtalk.msk.su/~ott/          ICQ #22005116

Re: [devel] strange scripts

[Vitaly Lipatov]

On Wednesday 21 April 2004 22:57, Alex Ott wrote:
> Hello
>
> >>>>> "SI" == Stanislav Ievlev writes:
>
>  SI> emacs-preview-latex-0.8-alt1.noarch.rpm
>
> там зовется texhash, для которого я что-то не нашел макроса.
> ткните меня носом, если я не прав
Я вот такую конструкцию вставил:
[ -x /usr/bin/texhash ] && /usr/bin/texhash 2>/dev/null ||:
-- 
Lav
Виталий Липатов
Санкт-Петербург
GNU! ALT Linux Team! LaTeX! LyX!

Re: [devel] strange scripts

[Vitaly Lipatov]

On Wednesday 21 April 2004 20:44, Dmitry Vukolov wrote:
> А можно немного подробнее об этом пункте? Как в таких случаях
> поступать, например, с не указанным в списке libsane-hpoj?
> Отправлять пользователей самостоятельно пакет доустанавливать?
> Или к данному пакету вышеуказанный пункт не относится?
Я думаю, что можно перечислить hpoj в dll.conf пакета sane, даже 
если на ваш пакет и нет жёсткой зависимости. Sane же корректно 
обрабатывает такой случай...

-- 
Lav
Виталий Липатов
Санкт-Петербург
GNU! ALT Linux Team! LaTeX! LyX!

Re: [devel] strange scripts

[Stanislav Ievlev]

On Wed, Apr 21, 2004 at 03:18:26PM +0300, Sviataslau Svirydau wrote:
> Stanislav Ievlev wrote:
> 
> >Привет всем!
> >
> >В attach'е приведены ошибочные на мой взгляд скрипты в пакетах.
> >
> >Очень возможно я где-то не прав, обязательно сообщите мне об этом
> >иначе это всё пойдет в BTS с разными severity ;)
> >
> >...
> >subversion-server-common-1.0.1-alt1.i586.rpm
> > 
> >
> ------------ (это наверное стоит перенести в триггер)
> subversion-server-common-1.0.1-alt1.i586.rpm
> preinstall scriptlet (through /bin/sh):
> /usr/sbin/groupadd -r -f subversion
> /usr/sbin/useradd -r -g subversion -d /var/lib/subversion -s /dev/null 
> -c 'Subversion' subversion >/dev/null 2>
> &1 ||:
> if [ $1 -gt 1 ]; then
>        /usr/sbin/usermod -d /var/lib/subversion subversion
> fi
> --------------
> 
> Где можно посмотреть пример как это правильно делать?
> (подобный скрипт тоже делался по примеру из другого пакета :))
У нас много всяких примеров разной степени правильности ;)
Ну в принципе все тоже самое, только надо оформить это как %trigger
Лучшее описание тригерров которое я когда-либо видел в самом rpm ;)
> 
> -- 
> Regards,
> Sviatoslav Sviridov <Sviataslau_Svirydau at epam.com>
> phone:  +375 17 210 1662, ext. 1302
> mobile: +375 29 641 4214
> -- 
> The sendmail configuration file is one of those files that looks like 
> someone
> beat their head on the keyboard.  After working with it... I can see why!
> 		-- Harry Skelton
> 

> _______________________________________________
> Devel mailing list
> Devel@altlinux.ru
> http://lists.altlinux.ru/mailman/listinfo/devel

Re: [devel] strange scripts

[Stanislav Ievlev]

On Wed, Apr 21, 2004 at 08:44:46PM +0400, Dmitry Vukolov wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On Wednesday 21 April 2004 13:35, Stanislav Ievlev wrote:
> > Привет всем!
> >
> > В attach'е приведены ошибочные на мой взгляд скрипты в пакетах.
> >
> > Очень возможно я где-то не прав, обязательно сообщите мне об этом
> > иначе это всё пойдет в BTS с разными severity ;)
> >
> > Что у меня вызывало подозрение:
> <skipped>
> > - проведение исправления в конфигурационных файлах, принадлежащим
> > другим пакетам
> 
> А можно немного подробнее об этом пункте? Как в таких случаях поступать, 
> например, с не указанным в списке libsane-hpoj? Отправлять 
> пользователей самостоятельно пакет доустанавливать? Или к данному 
> пакету вышеуказанный пункт не относится?
его и нет в списке.
Тут чаще всего проблема в тех пакетах которые эти конфигурационные файлы
предоставляют. Проблема их плохого дизайна. Поэтому большинство таких
пакетов не критиковались.

> 
> Спасибо.
> 
> - -- 
> Dmitry Vukolov
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.2.4 (GNU/Linux)
> 
> iD8DBQFAhqUC71tprB1tyF0RAlE2AKCJtNCNqp/INQQBmpn/sLBFgg0NLACfbY77
> s7iFlJ1O2EWr9Pvs1f4BaAE=
> =3RgS
> -----END PGP SIGNATURE-----

> _______________________________________________
> Devel mailing list
> Devel@altlinux.ru
> http://lists.altlinux.ru/mailman/listinfo/devel

Re: [devel] strange scripts

[Stanislav Ievlev]

On Wed, Apr 21, 2004 at 03:42:19PM +0400, Alexey I. Froloff wrote:
> * Stanislav Ievlev <inger@> [040421 15:33]:
> > > Псевдопользователь ftn не умеет создавать файлы в /var/log/ftn.
> > Все необходимые права можно задать прямо в spec-файле на каталог
> > /var/log/ftn.
> Помнится, ldv на этот счёт очень возражал.  non-root owned и/или
> group-writable.
> 
> > > > ftn-2.2-alt4.noarch.rpm
> > > (добавление пользователя с оболочкой, зачем?)
> > > Дурной вопрос - system(3) и crond(1) будут работать для
> > > пользователя без шелла?  Если да, то нет проблем.
> > Надо разобраться по подробнее.
> > Что там за cronjob? Как он добавляется? От чьего имени исполняется?
> cronjob - разные шеллскрипты, которые запускают кучу ftn барахла.
> Добавляется руками (crontab -u ftn -e), исполняются от имени ftn.
Мда, тут сложно понять, что лучше. Пусть остаётся.

> 
> -- 
> Regards, Sir Raorn.
> -------------------
> [...] любители ругаться не читают документацию :-)
> 		-- aen in devel@



> _______________________________________________
> Devel mailing list
> Devel@altlinux.ru
> http://lists.altlinux.ru/mailman/listinfo/devel

Re: [devel] strange scripts

[Stanislav Ievlev]

On Wed, Apr 21, 2004 at 11:52:44PM +0400, Vitaly Lipatov wrote:
> On Wednesday 21 April 2004 22:57, Alex Ott wrote:
> > Hello
> >
> > >>>>> "SI" == Stanislav Ievlev writes:
> >
> >  SI> emacs-preview-latex-0.8-alt1.noarch.rpm
> >
> > там зовется texhash, для которого я что-то не нашел макроса.
> > ткните меня носом, если я не прав
> Я вот такую конструкцию вставил:
> [ -x /usr/bin/texhash ] && /usr/bin/texhash 2>/dev/null ||:
И она более правильная чем:
texhash

> -- 
> Lav
> Виталий Липатов
> Санкт-Петербург
> GNU! ALT Linux Team! LaTeX! LyX!
> _______________________________________________
> Devel mailing list
> Devel@altlinux.ru
> http://lists.altlinux.ru/mailman/listinfo/devel

Re: [devel] strange scripts

[Alex Ott]

>On Wed, Apr 21, 2004 at 11:52:44PM +0400, Vitaly Lipatov wrote:
>> On Wednesday 21 April 2004 22:57, Alex Ott wrote:
>> > Hello
>> >
>> > >>>>> "SI" == Stanislav Ievlev writes:
>> >
>> >  SI> emacs-preview-latex-0.8-alt1.noarch.rpm
>> >
>> > там зовется texhash, для которого я что-то не нашел макроса.
>> > ткните меня носом, если я не прав
>> Я вот такую конструкцию вставил:
>> [ -x /usr/bin/texhash ] && /usr/bin/texhash 2>/dev/null ||:
>И она более правильная чем:
>texhash
>
ок - понял, я буду заливать в понедельник новую версию preview-latex, так что сам поправлю

[devel] Re: strange scripts

[Michael Shigorin]

On Thu, Apr 22, 2004 at 12:03:55PM +0400, Stanislav Ievlev wrote:
> > > там зовется texhash, для которого я что-то не нашел макроса.
> > > ткните меня носом, если я не прав
> > Я вот такую конструкцию вставил:
> > [ -x /usr/bin/texhash ] && /usr/bin/texhash 2>/dev/null ||:
> И она более правильная чем: texhash

А не потянет зависимость?

TEXHASH=/usr/bin/texhash
#...
[ ! -x "$TEXHASH" ] || "$TEXHASH" 2>/dev/null ||:

-- так их отцеплять?  Или в спеке (=>BuildReq, не Req) --> OK?

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[devel] Re: strange scripts

[Michael Shigorin]

[-- Attachment #1: Type: text/plain, Size: 956 bytes --]

On Thu, Apr 22, 2004 at 11:59:12AM +0400, Stanislav Ievlev wrote:
> > /usr/sbin/groupadd -r -f subversion
> > Где можно посмотреть пример как это правильно делать?
> > (подобный скрипт тоже делался по примеру из другого пакета :))
> У нас много всяких примеров разной степени правильности ;)

И это плохо :(

> Ну в принципе все тоже самое, только надо оформить это как %trigger
> Лучшее описание тригерров которое я когда-либо видел в самом rpm ;)

Надо оформить этот %trigger макросом и не изобретать каждый раз.

Особенно в связи с тем, что ошибки приводят к болезненному
загромождению базы "лечащими" триггерами и от этого вроде как не
избавиться без переименования пакета (если вообще возможно) :-(

Собственно, это можно воспринимать как согласие с тем, что
пользователи живут в пакетах, а не setup?  Обсуждение этой темы
уже дважды закончилось ничем.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[devel] Re: strange scripts

[Michael Shigorin]

On Wed, Apr 21, 2004 at 01:35:26PM +0400, Stanislav Ievlev wrote:
> Что у меня вызывало подозрение:
> - добавление служебного пользователя с оболочкой

А что делать.  У нас вроде как практикуется privsep в разных
объемах и проявлениях, причем некоторым системным пользователям
/dev/null или /bin/true действительно не хватает.

Позволю себе напомнить, что мы честно пытались поднять этот
вопрос минимум два раза -- и так и не пришли к нонсен...
концес... консенсусу, во.

> - неиспользование стандартных макросов

Это не грех.  Особенно если они добавляются молча и без
автоматического линтера. (правильно, разбаловали :-)

> - проведение исправления в конфигурационных файлах,
>   принадлежащим другим пакетам


> - просто явные опечатки
> - слишком длинные скрипты
> 
> Стандартная ошибка: переопределение в пакете %prefix приводит к
> порче макросов и соотв. скриптов.  Например
> /usr/bin/update-menus превращается /usr/games/bin/update-menus
> или /usr/X11R6/update-menus ;)

А не проблема ли это макросов?  Иначе надо объявить %prefix
неизменяемым (что вроде как противоречит его изначальному
смыслу?) и добавить проверку в sisyphus_check.

> mailgraph-1.2-alt1.i586.rpm

Отдаю в хорошие руки, в процессе.

> ------------------(не используются post/preun_service)

А... я-то думал, там основная проблема -- другое:

> preinstall scriptlet (through /bin/sh):
> /usr/sbin/groupadd -r -f mailgraph 2>/dev/null ||:
> /usr/sbin/useradd -g mailgraph -G adm \
> 	-c 'Mailgraph the Postfix Logfile Analyzer' \
> 	-d /var/lib/mailgraph -s /dev/null -r mailgraph \
> 	2>/dev/null ||:

и

> /sbin/chkconfig --add mailgraph
> if [ -f /etc/httpd/conf/httpd.conf ] && \
> 	! grep -q "^Include.*/mailgraph.conf" /etc/httpd/conf/httpd.conf; then
> 	echo "Include /etc/httpd/conf/mailgraph.conf" >> /etc/httpd/conf/httpd.conf
> 	/etc/rc.d/init.d/httpd condrestart 1>&2
> fi

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/