[devel] [git update] packages/hasher: heads/arm-port

Dmitry V. Levin =?iso-8859-1?q?ldv_=CE=C1_altlinux=2Eorg?=
Вт Июн 19 12:52:07 MSD 2007


Hi,

On Sun, Jun 10, 2007 at 06:49:17PM +0400, Kirill A. Shutemov wrote:
> Update of /people/kas/packages/hasher.git
> 
> New head `arm-port' is available with the following commits:
> commit d8f48255dc1595796553bd6dde83ee3c62273de5
> Author: Kirill A. Shutemov <k.shutemov на gmail>
> Date:   Sat May 5 16:51:25 2007 +0259
> 
>     qemu support

Я бы предпочёл более развёрнутый комментарий, сопровождающий коммит.
Степень развёрнутости можно оценить по моим комментариям в этом проекте.
По сути изменений, у меня есть несколько легко поправимых замечаний:

+ Имя переменной qemu слишком коротко для глобальной переменной
конфигурации и потому слишком многозначно; предлагаю изменить на
qemu_path.
+ Глобальная переменная qemu не объявлена в hsh-sh-functions; см. напр.
объявление переменной no_stuff.
+ Добавляемый в hsh-mkchroot код
if [ ! -z "$qemu" ]; then
	install -p -m755 $verbose "$qemu" .host/
fi
вызывает 2 вопроса: зачем написано "! -z" и почему имя создаваемого файла
на выходе зависит от архитектуры? предлагаю изменить на
[ -z "$qemu_path" ] ||
	install -p -m755 $verbose "$qemu" .host/qemu


-- 
ldv
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип     : application/pgp-signature
Размер  : 189 байтов
Описание: =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Url     : <http://lists.altlinux.org/pipermail/devel/attachments/20070619/b3ad67e0/attachment-0001.bin>


Подробная информация о списке рассылки Devel