ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Vladimir D. Seleznev" <vseleznv@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] Introduce gear-rules nodiff directive
Date: Tue, 10 Nov 2020 19:22:15 +0300
Message-ID: <20201110162215.GD2300502@portlab> (raw)
In-Reply-To: <20201110191527.647b4ae577a616e070ca59b6@altlinux.org>

On Tue, Nov 10, 2020 at 07:15:27PM +0300, Andrey Savchenko wrote:
> On Tue, 10 Nov 2020 18:51:11 +0300 Vladimir D. Seleznev wrote:
> > On Tue, Nov 10, 2020 at 03:46:14PM +0300, Arseny Maslennikov wrote:
> > > On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > > > Hi, devel@!
> > > > 
> > > > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > > > предложить немного другое решение: вместо предлагаемой дополнительной
> > > > опции exclude= использовать директиву nodiff, в котором перечислены
> > > > шаблоны путей, которые следует исключить при генерации патча. Пример
> > > > того, как при этом может выглядеть .gear/rules:
> > > > 
> > > > tar: v@version@:.
> > > > diff: v@version@:. .
> > > > nodiff: .gear/** *.spec
> > > 
> > > Имя совершенно не к месту, создаётся ощущение, что новая директива
> > > как-то отменяет некоторые действия директивы diff. На самом же деле она
> > > задаёт список исключаемых из диффа шаблонов путей.
> > > 
> > > Может, назвать diff-exclude? Или diffignore, если дефисы вдруг нельзя.
> > > Эти слова, по крайней мере, лучше ассоциируются с путями.
> > 
> > Пока оно не applied, предлагайте варианты названия.
> 
> Мне нравится вариант diffignore. 

diffexclude мне кажется более точно отражающим смысл. diffingore ничуть
не лучше nodiff по смыслу, а символов больше.

-- 
   WBR,
   Vladimir D. Seleznev


  reply	other threads:[~2020-11-10 16:22 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-10  3:16 Vladimir D. Seleznev
2020-11-10  3:16 ` [devel] [PATCH] Introduce nodiff directive for gear-rules Vladimir D. Seleznev
2020-11-10 10:47   ` Alexey Gladkov
2020-11-10  4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
2020-11-10  7:53   ` Sergey V Turchin
2020-11-10  8:01   ` Anton Farygin
2020-11-10  9:02   ` Andrey Savchenko
2020-11-10 10:24     ` Vitaly Lipatov
2020-11-10 13:07       ` Sergey V Turchin
2020-11-10 14:47   ` Dmitry V. Levin
2020-11-11  7:16     ` Anton V. Boyarshinov
2020-11-11  8:14       ` Anton Farygin
2020-11-10  8:03 ` Anton Farygin
2020-11-10 15:20   ` Vladimir D. Seleznev
2020-11-11  6:25     ` Anton Farygin
2020-11-10 12:46 ` Arseny Maslennikov
2020-11-10 15:51   ` Vladimir D. Seleznev
2020-11-10 16:15     ` Andrey Savchenko
2020-11-10 16:22       ` Vladimir D. Seleznev [this message]
2020-11-10 17:00         ` Andrey Savchenko
2020-11-10 18:26         ` [devel] gear-rules: new directive name Arseny Maslennikov
2020-11-10 18:47         ` [devel] Introduce gear-rules nodiff directive Michael Shigorin
2020-11-11  8:07           ` Sergey V Turchin
2020-11-10 17:30     ` Alexey V. Vissarionov
2020-11-10 14:48 ` Dmitry V. Levin
2020-11-10 14:54   ` Sergey V Turchin
2020-11-10 15:29   ` Vladimir D. Seleznev

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=20201110162215.GD2300502@portlab \
    --to=vseleznv@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