[devel] [PATCH 1/3] Introduce nodiff directive for gear-rules

Vladimir D. Seleznev vseleznv на altlinux.org
Чт Ноя 26 17:54:39 MSK 2020


On Thu, Nov 26, 2020 at 03:49:27PM +0100, Alexey Gladkov wrote:
> On Thu, Nov 26, 2020 at 05:16:34PM +0300, Vladimir D. Seleznev wrote:
> > The directive specifies a glob pattern that define files that should be
> > excluded from diff generation.
> 
> Мои комментарии относятся только к технической части. Относительно
> именования директив и нужности директивы вообще должен Дима высказаться.
> 
> Лично мне не нравится, что опция для директивы diff сделана директивой.
> Это выглядит хаком. У вас аргумент nodiff не может иметь пробелы. В этом
> случае несложно сделать `diff: exclude=glob_pattern`.

Я просто исходил из логики, что в подавляющей большинстве случаев там,
где это уместно использовать, rules будет выглядеть так:

	tar: v на version@:.
	diff: v на version@:. .
	nodiff: .gear

В этом плане

	tar: v на version@:.
	diff: v на version@:. exclude=.gear

по-моему выглядит менее красиво. Но я могу подготовить и такой вариант.

-- 
   WBR,
   Vladimir D. Seleznev


Подробная информация о списке рассылки Devel