ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Paul Wolneykien <manowar@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] gear-changelog: group commits by branch
Date: Thu, 14 Jul 2011 00:34:31 +0400
Message-ID: <4E1E0157.1090203@altlinux.org> (raw)
In-Reply-To: <4E1DF09D.7000906@altlinux.ru>

13.07.2011 23:23, Alexey Gladkov пишет:
> 13.07.2011 20:24, Paul Wolneykien wrote:
>>   А давайте группировать коммиты и по бранчам тоже. Очень удобно, когда
>> каждый бранч развивается в каком-то определённом направлении.
> 
> Не понял зачем такая группировка ?

  Насколько я понимаю, использование директивы `diff' в .gear/rules
предполагает, что каждый бранч заводится для определённого вида
изменений, преследующих определённую цель. Логично группировать
информацию об изменениях, объединённых одной задачей. Кроме того,
различные бранчи могут представлять собой просто различные части
проекта, разнесённые по отдельным директориям.

> 
> Судя по вашей реализации директива branch аналогична regexp по
> назначению, но при этом regexp имеет больший приоритет.

  Использование regexp предполагает, что формат сообщений оговорён и что
все его придерживаются. На правтике это далеко не всегде возможно.
  Насчёт приоритета, думаю, правильно: если человек явно в сообщении
указал, что изменения касаются какого-то определённого объекта, то
следует верить ему.

> 
> Также ваш способ фильтрации (последний ханк) по branch очень не
> эффективна.

  Да, я знаю. Буду рад более эффективной реализации.

> 


  reply	other threads:[~2011-07-13 20:34 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-13 16:24 Paul Wolneykien
2011-07-13 19:23 ` Alexey Gladkov
2011-07-13 20:34   ` Paul Wolneykien [this message]
2011-07-13 21:32     ` Alexey Gladkov
2011-07-13 21:45       ` Paul Wolneykien

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E1E0157.1090203@altlinux.org \
    --to=manowar@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git