* Re: [devel] [git update] packages/giter-factory: heads/master
2007-09-02 17:21 ` [devel] [git update] packages/giter-factory: heads/master Slava Semushin
@ 2007-09-02 18:33 ` Alexey Tourbin
0 siblings, 0 replies; 2+ messages in thread
From: Alexey Tourbin @ 2007-09-02 18:33 UTC (permalink / raw)
To: ALT Devel discussion list
[-- Attachment #1: Type: text/plain, Size: 1674 bytes --]
On Mon, Sep 03, 2007 at 12:21:36AM +0700, Slava Semushin wrote:
> 2007/8/31, Alexey M. Tourbin <at@altlinux.org>:
> > Update of /people/at/packages/giter-factory.git
> [...]
> > +Fatal()
> > +{
> > + echo "$*" >&2
> > + exit 1
> > +}
> > +
> > +Progress()
> > +{
> > + echo "$*" >>out/progress
> > + > out/log
> > +}
> [...]
>
> Предлагаю использовать printf, вместо echo. (Так, на всякий случай..)
Довольно мелочное замечание при непонимании сути идеи.
Однако сделаю пояснение. У команды echo есть такая особенность, что она
может брать опциональные опции -n, -e и -E, а также (вероятно) их
комбинации -ne, -nE, -en, -En, -een, -EEn, -eEn и т.п.
А может и не брать.
Поэтому в случае untrusted data, т.е. если мы печатаем нечто, что пришло
откуда-то ещё, и нет априорных ограничений на это что-то, то есть риск,
что аргумент echo будет истолкован как одна большая опция (особенно если
он начинается с минуса; типа echo -eeeeeeeeeeeeee).
Но вообще эта предосторожность хорошо граничит с чувством вкуса.
Если нет оснований думать, что данные "слишком плохие", тогда и нет
оснований избегать использования echo.
В данном случае все вызовы функций Fatal и Progress начинаются с
константной строки.
Суть же идеи совсем в другом. Идея в том, что попытка сборки пакета
происходит с определённой гранулярностью. А гранулярность нужна для
того, чтобы обнаружить одно простое обстоятельство: ПРИ СБОРКЕ ПАКЕТА
ИЗМЕНИЛОСЬ СОДЕРЖИМОЕ БИЛДРУТА, И ОН ПЕРЕСТАЛ СОБИРАТЬСЯ.
Соответственно, имеется причинно-следственная связь между изменением
билдрута (и в чем оно состоит) и невозможностью сборки пакета.
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread