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 > > 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 или эти изменения отдельным коммитом? -- Regards, Kirill A. Shutemov + Belarus, Minsk + Velesys LLC, http://www.velesys.com/ + ALT Linux Team, http://www.altlinux.com/