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

[Alexey Gladkov <legion@altlinux.ru>]

On Thu, Nov 26, 2020 at 05:54:39PM +0300, Vladimir D. Seleznev wrote:
> 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

У тебя, кстати, для такого варианта нет проверки, что указан nodiff, но
нет директивы diff так как это явная ошибка.

> В этом плане
> 
> 	tar: v@version@:.
> 	diff: v@version@:. exclude=.gear
> 
> по-моему выглядит менее красиво. Но я могу подготовить и такой вариант.

Для меня это выглядит более понятно так как первый вариант выглядит
понятно только если знать, что nodiff это опция директивы diff.

Если хотите увеличить читабильность, то давайте реализуем многострочные
директивы. Тогда можно будет написать:

	tar: v@version@:.
	diff: v@version@:. . \
		name=@new_dir@-@version@-@release@.patch \
		exclude=.gear

-- 
Rgrds, legion