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

[Dmitry V. Levin]

On Tue, Jun 19, 2007 at 03:57:12PM +0300, Kirill A. Shutemov wrote:
> On [Tue, 19.06.2007 12:51], Dmitry V. Levin wrote:
> > 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_arch сейчас qemu в hsh
> qemu_path получается после "$(opt_check_read "$1" "/usr/bin/qemu-$2.static")"

Две переменные -- это перебор и возможная причина конфликта переменных.
Надо придумать способ уместить эту сущность в одну переменную.

> P.S. Мне соит перезаписать историю в git или эти изменения отдельным
> коммитом?

Я ещё не делал merge, поэтому при желании можно перезаписать.


-- 
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/e1071c31/attachment-0001.bin>