[devel] gear-rules: allow backslashes to escape any characters

Alexey Gladkov legion на altlinux.ru
Ср Дек 9 18:51:57 MSK 2020


On Wed, Dec 09, 2020 at 05:39:42PM +0300, Dmitry V. Levin wrote:
> On Wed, Dec 09, 2020 at 11:40:15AM +0100, Alexey Gladkov wrote:
> > On Wed, Dec 09, 2020 at 05:10:04AM +0300, Dmitry V. Levin wrote:
> > > On Thu, Nov 26, 2020 at 04:51:50PM +0100, Alexey Gladkov wrote:
> > > > On Thu, Nov 26, 2020 at 04:17:01PM +0100, Alexey Gladkov wrote:
> [...]
> > > > > Если хотите увеличить читабильность, то давайте реализуем многострочные
> > > > > директивы. Тогда можно будет написать:
> > > > > 
> > > > > 	tar: v на version@:.
> > > > > 	diff: v на version@:. . \
> > > > > 		name=@new_dir на -@version на -@release на .patch \
> > > > > 		exclude=.gear
> > > > 
> > > > Быстрый патч, который позволит сделать такое. Нужно проверить не сломаются
> > > > ли какие-нибудь gear-rules.
> > > 
> > > По крайней мере в Сизифе gear-rules, содержащих символ "\", не обнаружено.
> > > 
> > > > Signed-off-by: Alexey Gladkov <legion на altlinux.org>
> > > 
> > > Это хорошо, но в gear-rules(5) сейчас сказано буквально следующее:
> > > "There is no support for line continuation."
> > > 
> > > Так что documentation update и commit message тоже нужны. :)
> > 
> > Так это был набросок, а не патч ))) Ок, сделаю из него настоящий патч.
> 
> Вопрос в том, действительно ли мы хотим эту фичу,
> которая по сути allows backslashes to escape any characters.

C одной стороны, да это сильно расширяет, но с другой стороны можно
вынести valid_string из if'a и проверить все options [1].

[1] http://git.altlinux.org/people/ldv/packages/gear.git?p=gear.git;a=blob;f=gear;h=340321465f695449a28e8b5efcedfec7acbca2ab;hb=HEAD#l586

-- 
Rgrds, legion



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