[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