Bug 11488 - Не выполнять cron-скрипты в fake'овом окружении
: Не выполнять cron-скрипты в fake'овом окружении
Status: CLOSED FIXED
: Sisyphus
(All bugs in Sisyphus/man)
: unstable
: all Linux
: P2 normal
Assigned To:
:
:
:
:
:
  Show dependency tree
 
Reported: 2007-04-12 18:28 by
Modified: 2008-08-02 04:08 (History)


Attachments
Proposal patch (602 bytes, patch)
2008-01-16 15:00, Slava Semushin
no flags Details | Diff
Новая версия патча (574 bytes, patch)
2008-02-06 14:52, Slava Semushin
no flags Details | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2007-04-12 18:28:44
При установке в hasher ставится неправильно и портит права.
Нужно не запускать скрипт в /etc/cron.daily если установлены переменные
окружения  FAKED_MODE/FAKEROOTKEY.

Проверку предлагаю добавить в сам скрипт.
------- Comment #1 From 2007-04-12 18:30:56 -------
Не может этого быть:
$ rpmquery -p --scripts x86_64/RPMS/man-1.5m2-alt4.x86_64.rpm 
preinstall scriptlet (through /bin/sh):
/usr/sbin/useradd -r -g man -d /var/cache/man -s /dev/null -n cacheman
>/dev/null 2>&1 ||:
------- Comment #2 From 2007-04-15 03:23:32 -------
Я полагаю, что речь идёт о каком-то другом пакете.
------- Comment #3 From 2007-04-18 23:11:29 -------
rpm -qf /etc/cron.weekly/makewhatis
man-1.5m2-alt4

из man.spec:
%triggerpostun -- %name < 1.5m2-alt2
echo -n 'Rebuilding whatis database... '
/etc/cron.weekly/makewhatis
echo done.

В этом скрипте содержится вызов su, со всеми вытекающими отсюда последствиями:
su -l cacheman -s /bin/sh -c /usr/sbin/makewhatis

я полагаю что fakeroot'у от этого плохеет.

Если я не прав, на кого перевешивать? ;)
------- Comment #4 From 2007-04-19 12:02:21 -------
Где ты взял man < 1.5m2-alt2?
------- Comment #5 From 2007-04-19 23:08:09 -------
Гм. Извиняюсь, действительно дело не в этом триггере.
Дело в том что во всех man-pages:
%post -p /etc/cron.daily/makewhatis

и, наверное, проще пофиксить это в самом этом скрипте, чем городить проверки во
всех man-pages.*
------- Comment #6 From 2007-04-21 02:31:25 -------
Ладно, хуже от этого не будет.
------- Comment #7 From 2008-01-16 14:54:57 -------
Так. Давайте я попробую в этом разобраться, чтобы исправить баг.

Во-первых, мне непонятно как и почему портятся права? Что это значит?

Во-вторых, если я правильно понял, то править нужно не только
/etc/cron.daily/makewhatis, но и /etc/cron.weekly/makewhatis. Я прав?

Также я посмотрел мельком доку по fakechroot
(http://fakechroot.alioth.debian.org/), в её мане вижу следующее: "FAKECHROOT
The value is true for fake chroot environment." Если у нас та же версия что и у
апстрима, то имеет смысл использовать именно эту переменную для проверки в
чруте
мы или нет (вместо FAKED_MODE/FAKEROOTKEY.). Да?

В итоге. Я сейчас приложу предлагаемый мной патч. Мне хотелось бы услышать от
вас ответы на свои вопросы и подтверждение/замечания по предлагаемому патчу.

:-)
------- Comment #8 From 2008-01-16 15:00:33 -------
Created an attachment (id=2380) [details]
Proposal patch

Что-то не очень понял как проверить $FAKECHROOT true она или нет. То ли там
строка с текстом то ли просто 1 или 0.

2ldv@: Прошу от'review'ить мой патч и высказаться.
------- Comment #9 From 2008-01-17 08:49:21 -------
Короче, патч мой явно кривой. Может быть достаточно будет только проверки
существования пемеренной? [ -z "$FAKECHROOT" ] ? Если нет, то как лучше
проверить её значение?
------- Comment #10 From 2008-01-29 14:40:19 -------
*ping*
------- Comment #11 From 2008-01-29 18:26:34 -------
(In reply to comment #8)
> Created an attachment (id=2380) [edit] [details]
> Proposal patch
> 
> Что-то не очень понял как проверить $FAKECHROOT true она или нет. То ли там
> строка с текстом то ли просто 1 или 0.

Если проверять, то $FAKEROOTKEY
------- Comment #12 From 2008-02-06 14:52:47 -------
Created an attachment (id=2421) [details]
Новая версия патча

2ldv@: такой патч подойдёт?
------- Comment #13 From 2008-02-06 15:05:52 -------
(In reply to comment #12)
> Created an attachment (id=2421) [edit] [details]
> Новая версия патча
> 
> 2ldv@: такой патч подойдёт?

Проверки можно склеить, но и так будет работать.
------- Comment #14 From 2008-02-13 09:35:26 -------
(In reply to comment #13)
> > 2ldv@: такой патч подойдёт?
> Проверки можно склеить, но и так будет работать.

Спасибо.

// что-то мне комментарии по баге не приходили, оказывается меня почему-то нет в
СС к баге..
------- Comment #15 From 2008-02-27 10:30:11 -------
В Сизиф ушел man-1.6f-alt1, в котором этот баг должен быть исправлен.