From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham autolearn_force=no version=3.4.1 Date: Thu, 26 Nov 2020 16:17:01 +0100 From: Alexey Gladkov To: ALT Linux Team development discussions Message-ID: <20201126151701.zm627zsqorxzd52z@example.org> References: <20201126141636.2742206-1-vseleznv@altlinux.org> <20201126144927.h36zzrisholvca3s@example.org> <20201126145439.GA2742643@portlab> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20201126145439.GA2742643@portlab> Subject: Re: [devel] [PATCH 1/3] Introduce nodiff directive for gear-rules X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 26 Nov 2020 15:17:06 -0000 Archived-At: List-Archive: List-Post: 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