ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Aleksey Avdeev <solo@solin.spb.ru>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] I: nut-2.2.2
Date: Tue, 10 Feb 2009 19:35:33 +0300
Message-ID: <4991ACD5.9010407@solin.spb.ru> (raw)
In-Reply-To: <20090210152125.GD28187@wo.int.altlinux.org>

[-- Attachment #1: Type: text/plain, Size: 2230 bytes --]

Dmitry V. Levin пишет:
> On Tue, Feb 10, 2009 at 05:58:40PM +0300, Aleksey Avdeev wrote:
>> Dmitry V. Levin пишет:
>> ...
>>> Очень сложно и очень неудобно.  Нет, это 
>>> ужасно.
>>   Есть более удобный вариант?
> 
> Просто поменять имя релиза и выполнить add_changelog существенно проще.

1. Имя релиза в бранче у нас жёстко связано с релизом в сизифе, и может 
быть вычислено автоматически. Так, зачем его вводить вручную, рискуя 
нарваться на очепятку (которую можно и вообще не заметить)?

2. В %changelog нужно ещё добавить комментарий. Хотя бы стандартный 
Backports to <>.

   Да, оба эти пункта способна реализовать утилита...

> Я слышал, что есть утилита, которая делает это ещё проще.

   Но утилита не учитывает ситуации, когда в зависимости от бранча нужно 
включать/отключать части спека -- для этого всё равно придётся вводить 
управляемый руками флаг. Так почему не повесить на этот флаг и генерацию 
правильного релиза? Это позволяет сильно сократить поле возможных ошибок 
(за счёт ограничения степеней свободы, т. к. несколько нечётких 
регуляторов сведены в один, имеющий фиксированный набор положений).

   Да, можно отказаться от лишних (связанных с разными бранчами) условий 
в спеке. Можно:

а) разнести бэкпорты по разным бранчам;

б) убрать ветвления в спеках.

   Но тогда потеряем возможность варьировать сборку в отрыве от 
репозитария. Сейчас у меня это возможно: для любого srpm (независимо от 
бранча под который он был собран) целевой дистрибутив можно задать через 
параметры командной строки rpmbuild`а: пакет соберётся с под заданный 
бранч с нужным релизом (да, приэтом он может отличаться от того что уже 
есть в changelog`е).

> 
> Разводить на каждое изменение в spec-файле по бранчу -- извините, это
> злоупотребление git'ом.

1. Это здорова помогает отделять мух от котлет (технические части спека 
от содержательны).

2. Хорошо страхует от потери изменений.

   Да, количество бранчей можно сократить, сделав упор на cherry-pick и 
rebase. Но тогда будет страдать поддерживаемость кода...

PS: Да, код получился довольно объёмным. И я пока не придумал, как его 
разместить по внешним макросам.

-- 

С уважением. Алексей.



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 552 bytes --]

      reply	other threads:[~2009-02-10 16:35 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-09 22:51 Aleksey Avdeev
2009-02-09 23:09 ` Dmitry V. Levin
2009-02-09 23:40   ` Aleksey Avdeev
2009-02-10  7:24     ` Eugene Prokopiev
2009-02-10  8:10       ` Aleksey Avdeev
2009-02-10 14:46         ` Dmitry V. Levin
2009-02-10 14:58           ` Aleksey Avdeev
2009-02-10 15:21             ` Dmitry V. Levin
2009-02-10 16:35               ` Aleksey Avdeev [this message]

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=4991ACD5.9010407@solin.spb.ru \
    --to=solo@solin.spb.ru \
    --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