[devel] [git update] packages/hasher: heads/arm-port
Kirill A. Shutemov
=?iso-8859-1?q?k=2Eshutemov_=CE=C1_velesys=2Ecom?=
Вт Июн 19 13:06:46 MSD 2007
On [Tue, 19.06.2007 12:51], Dmitry V. Levin wrote:
> 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.
Ok. Поправлю.
> + Добавляемый в hsh-mkchroot код
> if [ ! -z "$qemu" ]; then
> install -p -m755 $verbose "$qemu" .host/
> fi
> вызывает 2 вопроса: зачем написано "! -z"
Поправлю на -n.
>и почему имя создаваемого файла
> на выходе зависит от архитектуры? предлагаю изменить на
> [ -z "$qemu_path" ] ||
> install -p -m755 $verbose "$qemu" .host/qemu
Этот скопированный qemu будет использоваться через binfmt. Записей в binfmt
может быть несколько для разных архитектур.
--
Regards, Kirill A. Shutemov
+ Belarus, Minsk
+ Velesys LLC, http://www.velesys.com/
+ ALT Linux Team, http://www.altlinux.com/
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип : application/pgp-signature
Размер : 189 байтов
Описание: Digital signature
Url : <http://lists.altlinux.org/pipermail/devel/attachments/20070619/1bbe1fff/attachment-0001.bin>
Подробная информация о списке рассылки Devel