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