Re: [devel] [git update] packages/giter-factory: heads/master

Re: [devel] [git update] packages/giter-factory: heads/master

[Slava Semushin]

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. (Так, на всякий случай..)


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [git update] packages/giter-factory: heads/master

[Alexey Tourbin]

[-- 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 --]