* [devel] [PATCH] Introduce nodiff directive for gear-rules
2020-11-10 3:16 [devel] Introduce gear-rules nodiff directive Vladimir D. Seleznev
@ 2020-11-10 3:16 ` Vladimir D. Seleznev
2020-11-10 10:47 ` Alexey Gladkov
2020-11-10 4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
` (3 subsequent siblings)
4 siblings, 1 reply; 27+ messages in thread
From: Vladimir D. Seleznev @ 2020-11-10 3:16 UTC (permalink / raw)
To: devel; +Cc: legion, Vladimir D. Seleznev
The directive specifies whitespace-separated list of glob patterns that
define files that should be excluded from diff generation.
Signed-off-by: Vladimir D. Seleznev <vseleznv@altlinux.org>
---
gear | 24 ++++++++++++++++++------
gear-rules.5.in | 18 ++++++++++++++++++
2 files changed, 36 insertions(+), 6 deletions(-)
diff --git a/gear b/gear
index 6282c92..0589b06 100755
--- a/gear
+++ b/gear
@@ -402,13 +402,14 @@ write_git_diff()
{
local optional="$1" && shift
local cmd="$1" && shift
+ local nodiff_patterns="$1" && shift
local old_tree="$1" && shift
local old_dir="$1" && shift
local new_tree="$1" && shift
local new_dir="$1" && shift
local name="${1##*/}" && shift
- local old_id new_id
+ local old_id new_id nodiff=
old_id="$(traverse_tree "$old_tree" "$old_dir" "$optional")" ||
{
@@ -422,8 +423,13 @@ write_git_diff()
exit 1
}
+ for pattern in $nodiff_patterns; do
+ [ -n "$pattern" ] ||continue
+ nodiff="$nodiff $(printf ":^%s" "$pattern")"
+ done
+
git diff-tree --patch-with-stat --text --no-renames --no-ext-diff \
- "$old_id" "$new_id" >"$outdir/$name"
+ "$old_id" "$new_id" $nodiff >"$outdir/$name"
verbose "Extracted diff: $name"
compress_file "$cmd" "$name" ''
@@ -436,6 +442,7 @@ make_difference()
rules_error 'No old_dir or new_dir specified'
local optional="$1" && shift
local cmd="$1" && shift
+ local nodiff_patterns="$1" && shift
local old_dir="$1" && shift
local new_dir="$1" && shift
@@ -456,7 +463,7 @@ make_difference()
diff_new_tree="$(resolve_tree_base_name "$diff_new_tree")" ||
rules_error "Invalid new tree: $diff_new_tree"
- write_git_diff "$optional" "$cmd" "$diff_old_tree" "$old_dir" "$diff_new_tree" "$new_dir" "$diff_name"
+ write_git_diff "$optional" "$cmd" "$nodiff_patterns" "$diff_old_tree" "$old_dir" "$diff_new_tree" "$new_dir" "$diff_name"
}
extract_pattern()
@@ -564,7 +571,7 @@ parse_rules()
exclude_pattern_list=
lineno=0
- local cmd options
+ local cmd options nodiff_patterns=
while read -r cmd options; do
lineno="$((lineno+1))"
if [ "$cmd" = 'exclude:' ]; then
@@ -572,6 +579,11 @@ parse_rules()
rules_error "Invalid exclude pattern specified: $options"
exclude_pattern_list="$exclude_pattern_list $options"
fi
+ if [ "$cmd" = 'nodiff:' ]; then
+ [ -z "$(printf %s "$options" |tr -d '[:alnum:]_.?*-/ ')" ] ||
+ rules_error "Invalid nodiff pattern specified: $options"
+ nodiff_patterns="$nodiff_patterns $options"
+ fi
done <"$workdir/rules"
lineno=0
@@ -601,7 +613,7 @@ parse_rules()
eval "set -- $opts"
case "$cmd" in
- spec|specsubst|exclude|tags)
+ spec|specsubst|exclude|tags|nodiff)
continue
;;
tar|tar.gz|tar.bz2|tar.lzma|tar.xz|tar.zst)
@@ -613,7 +625,7 @@ parse_rules()
rules_error 'Failed to make archive'
;;
diff|diff.gz|diff.bz2|diff.lzma|diff.xz|diff.zst)
- make_difference "$optional" "$cmd" "$@" ||
+ make_difference "$optional" "$cmd" "$nodiff_patterns" "$@" ||
rules_error 'Failed to make diff'
;;
copy|gzip|bzip2|lzma|xz|zstd)
diff --git a/gear-rules.5.in b/gear-rules.5.in
index 5469199..c1b6793 100644
--- a/gear-rules.5.in
+++ b/gear-rules.5.in
@@ -452,6 +452,12 @@ or
.I new_tree_path
specify non-default base trees, these trees still must be valid.
.TP
+.BI "nodiff: " glob_pattern...
+Specify whitespace-separated list of glob patterns that define files that
+should be excluded from diff generation. Affects all directives in the
+rule file irrespective of the order. Specifying a glob pattern which does not
+match any files in the tree is not an error.
+.TP
.BI "specsubst: " variables...
Specify whitespace-separated list of variables that should be substituted
in specfile. Variable names may contain letters, digits and underscore
@@ -541,6 +547,18 @@ However, if a prerelease version is packaged, parts like
\*(lqpreN\*(rq or \*(lqrcN\*(rq should not be included in the package
version, therefore the rule file will need some modifications for such
versions.
+.PP
+You can also want to exclude
+.BR .gear
+subdirectory and spec file from diff generation:
+.RS 4
+.PP
+.ft CW
+.nf
+nodiff: .gear/** *.spec
+.fi
+.RE
+.PP
.
.SS "Archive with unmodified sources and separate patch files"
If you prefer to maintain patch files for local modifications instead
--
2.25.4
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] [PATCH] Introduce nodiff directive for gear-rules
2020-11-10 3:16 ` [devel] [PATCH] Introduce nodiff directive for gear-rules Vladimir D. Seleznev
@ 2020-11-10 10:47 ` Alexey Gladkov
0 siblings, 0 replies; 27+ messages in thread
From: Alexey Gladkov @ 2020-11-10 10:47 UTC (permalink / raw)
To: Vladimir D. Seleznev; +Cc: devel
On Tue, Nov 10, 2020 at 06:16:09AM +0300, Vladimir D. Seleznev wrote:
> The directive specifies whitespace-separated list of glob patterns that
> define files that should be excluded from diff generation.
>
> Signed-off-by: Vladimir D. Seleznev <vseleznv@altlinux.org>
> ---
> gear | 24 ++++++++++++++++++------
> gear-rules.5.in | 18 ++++++++++++++++++
> 2 files changed, 36 insertions(+), 6 deletions(-)
Добавь тесты пожалуйста.
> diff --git a/gear b/gear
> index 6282c92..0589b06 100755
> --- a/gear
> +++ b/gear
> @@ -402,13 +402,14 @@ write_git_diff()
> {
> local optional="$1" && shift
> local cmd="$1" && shift
> + local nodiff_patterns="$1" && shift
> local old_tree="$1" && shift
> local old_dir="$1" && shift
> local new_tree="$1" && shift
> local new_dir="$1" && shift
> local name="${1##*/}" && shift
>
> - local old_id new_id
> + local old_id new_id nodiff=
>
> old_id="$(traverse_tree "$old_tree" "$old_dir" "$optional")" ||
> {
> @@ -422,8 +423,13 @@ write_git_diff()
> exit 1
> }
>
> + for pattern in $nodiff_patterns; do
В parse_rules ты разрешил пробел. Значит в nodiff_patterns может быть
".gear/[Rr ]ules" и тогда pattern будет равна ".gear/[Rr" и "]ules".
Тебе проще будет запретить проблел в паттерне.
> + [ -n "$pattern" ] ||continue
> + nodiff="$nodiff $(printf ":^%s" "$pattern")"
> + done
> +
> git diff-tree --patch-with-stat --text --no-renames --no-ext-diff \
> - "$old_id" "$new_id" >"$outdir/$name"
> + "$old_id" "$new_id" $nodiff >"$outdir/$name"
> verbose "Extracted diff: $name"
>
> compress_file "$cmd" "$name" ''
> @@ -436,6 +442,7 @@ make_difference()
> rules_error 'No old_dir or new_dir specified'
> local optional="$1" && shift
> local cmd="$1" && shift
> + local nodiff_patterns="$1" && shift
> local old_dir="$1" && shift
> local new_dir="$1" && shift
>
> @@ -456,7 +463,7 @@ make_difference()
> diff_new_tree="$(resolve_tree_base_name "$diff_new_tree")" ||
> rules_error "Invalid new tree: $diff_new_tree"
>
> - write_git_diff "$optional" "$cmd" "$diff_old_tree" "$old_dir" "$diff_new_tree" "$new_dir" "$diff_name"
> + write_git_diff "$optional" "$cmd" "$nodiff_patterns" "$diff_old_tree" "$old_dir" "$diff_new_tree" "$new_dir" "$diff_name"
> }
>
> extract_pattern()
> @@ -564,7 +571,7 @@ parse_rules()
> exclude_pattern_list=
> lineno=0
>
> - local cmd options
> + local cmd options nodiff_patterns=
> while read -r cmd options; do
> lineno="$((lineno+1))"
> if [ "$cmd" = 'exclude:' ]; then
> @@ -572,6 +579,11 @@ parse_rules()
> rules_error "Invalid exclude pattern specified: $options"
> exclude_pattern_list="$exclude_pattern_list $options"
> fi
> + if [ "$cmd" = 'nodiff:' ]; then
> + [ -z "$(printf %s "$options" |tr -d '[:alnum:]_.?*-/ ')" ] ||
Ты забыл разрешить '[', ']' для указания диапазона и '!' для исключения
диапазона.
> + rules_error "Invalid nodiff pattern specified: $options"
> + nodiff_patterns="$nodiff_patterns $options"
> + fi
> done <"$workdir/rules"
>
> lineno=0
> @@ -601,7 +613,7 @@ parse_rules()
> eval "set -- $opts"
>
> case "$cmd" in
> - spec|specsubst|exclude|tags)
> + spec|specsubst|exclude|tags|nodiff)
> continue
> ;;
> tar|tar.gz|tar.bz2|tar.lzma|tar.xz|tar.zst)
> @@ -613,7 +625,7 @@ parse_rules()
> rules_error 'Failed to make archive'
> ;;
> diff|diff.gz|diff.bz2|diff.lzma|diff.xz|diff.zst)
> - make_difference "$optional" "$cmd" "$@" ||
> + make_difference "$optional" "$cmd" "$nodiff_patterns" "$@" ||
> rules_error 'Failed to make diff'
> ;;
> copy|gzip|bzip2|lzma|xz|zstd)
> diff --git a/gear-rules.5.in b/gear-rules.5.in
> index 5469199..c1b6793 100644
> --- a/gear-rules.5.in
> +++ b/gear-rules.5.in
> @@ -452,6 +452,12 @@ or
> .I new_tree_path
> specify non-default base trees, these trees still must be valid.
> .TP
> +.BI "nodiff: " glob_pattern...
> +Specify whitespace-separated list of glob patterns that define files that
> +should be excluded from diff generation. Affects all directives in the
> +rule file irrespective of the order. Specifying a glob pattern which does not
> +match any files in the tree is not an error.
> +.TP
> .BI "specsubst: " variables...
> Specify whitespace-separated list of variables that should be substituted
> in specfile. Variable names may contain letters, digits and underscore
> @@ -541,6 +547,18 @@ However, if a prerelease version is packaged, parts like
> \*(lqpreN\*(rq or \*(lqrcN\*(rq should not be included in the package
> version, therefore the rule file will need some modifications for such
> versions.
> +.PP
> +You can also want to exclude
> +.BR .gear
> +subdirectory and spec file from diff generation:
> +.RS 4
> +.PP
> +.ft CW
> +.nf
> +nodiff: .gear/** *.spec
> +.fi
> +.RE
> +.PP
> .
> .SS "Archive with unmodified sources and separate patch files"
> If you prefer to maintain patch files for local modifications instead
> --
> 2.25.4
>
--
Rgrds, legion
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 3:16 [devel] Introduce gear-rules nodiff directive Vladimir D. Seleznev
2020-11-10 3:16 ` [devel] [PATCH] Introduce nodiff directive for gear-rules Vladimir D. Seleznev
@ 2020-11-10 4:27 ` Антон Мидюков
2020-11-10 7:53 ` Sergey V Turchin
` (3 more replies)
2020-11-10 8:03 ` Anton Farygin
` (2 subsequent siblings)
4 siblings, 4 replies; 27+ messages in thread
From: Антон Мидюков @ 2020-11-10 4:27 UTC (permalink / raw)
To: devel
10.11.2020 10:16, Vladimir D. Seleznev пишет:
> Hi, devel@!
>
> По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> предложить немного другое решение: вместо предлагаемой дополнительной
> опции exclude= использовать директиву nodiff, в котором перечислены
> шаблоны путей, которые следует исключить при генерации патча. Пример
> того, как при этом может выглядеть .gear/rules:
>
> tar: v@version@:.
> diff: v@version@:. .
> nodiff: .gear/** *.spec
>
> [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
всегда? Причём не только при генерации патча, но и упаковки типа:
tar: .
У части пакетов и так всё нужное для сборки в .gear, так что в них
ничего делать не придётся.
--
С уважением, Антон Мидюков <antohami@altlinux.org>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
@ 2020-11-10 7:53 ` Sergey V Turchin
2020-11-10 8:01 ` Anton Farygin
` (2 subsequent siblings)
3 siblings, 0 replies; 27+ messages in thread
From: Sergey V Turchin @ 2020-11-10 7:53 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tuesday, 10 November 2020 07:27:31 MSK Антон Мидюков wrote:
[...]
> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
> всегда? Причём не только при генерации патча, но и упаковки типа:
>
> tar: .
У меня какие-то сборки сломаются, но я не против исправить. Будет красивее.
> У части пакетов и так всё нужное для сборки в .gear, так что в них
> ничего делать не придётся.
--
Regards, Sergey.
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
2020-11-10 7:53 ` Sergey V Turchin
@ 2020-11-10 8:01 ` Anton Farygin
2020-11-10 9:02 ` Andrey Savchenko
2020-11-10 14:47 ` Dmitry V. Levin
3 siblings, 0 replies; 27+ messages in thread
From: Anton Farygin @ 2020-11-10 8:01 UTC (permalink / raw)
To: devel
On 10.11.2020 07:27, Антон Мидюков wrote:
> 10.11.2020 10:16, Vladimir D. Seleznev пишет:
>> Hi, devel@!
>>
>> По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
>> предложить немного другое решение: вместо предлагаемой дополнительной
>> опции exclude= использовать директиву nodiff, в котором перечислены
>> шаблоны путей, которые следует исключить при генерации патча. Пример
>> того, как при этом может выглядеть .gear/rules:
>>
>> tar: v@version@:.
>> diff: v@version@:. .
>> nodiff: .gear/** *.spec
>>
>> [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
>> [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
>
> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
> всегда? Причём не только при генерации патча, но и упаковки типа:
>
> tar: .
>
> У части пакетов и так всё нужное для сборки в .gear, так что в них
> ничего делать не придётся.
>
Кстати да, Антон, спасибо. Это решение мне казалось радикальным, но оно
выглядит правильным.
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
2020-11-10 7:53 ` Sergey V Turchin
2020-11-10 8:01 ` Anton Farygin
@ 2020-11-10 9:02 ` Andrey Savchenko
2020-11-10 10:24 ` Vitaly Lipatov
2020-11-10 14:47 ` Dmitry V. Levin
3 siblings, 1 reply; 27+ messages in thread
From: Andrey Savchenko @ 2020-11-10 9:02 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 1785 bytes --]
On Tue, 10 Nov 2020 11:27:31 +0700 Антон Мидюков wrote:
> 10.11.2020 10:16, Vladimir D. Seleznev пишет:
> > Hi, devel@!
> >
> > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > предложить немного другое решение: вместо предлагаемой дополнительной
> > опции exclude= использовать директиву nodiff, в котором перечислены
> > шаблоны путей, которые следует исключить при генерации патча. Пример
> > того, как при этом может выглядеть .gear/rules:
> >
> > tar: v@version@:.
> > diff: v@version@:. .
> > nodiff: .gear/** *.spec
> >
> > [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> > [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
>
> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
> всегда? Причём не только при генерации патча, но и упаковки типа:
>
> tar: .
>
> У части пакетов и так всё нужное для сборки в .gear, так что в них
> ничего делать не придётся.
Мешает то, что сейчас можно класть файлы в .gear и не писать
отдельные правила gear для копирования патчей или иных файлов.
А с предложенным нововведением будет больше никому не нужного
мусора.
Best regards,
Andrew Savchenko
[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 9:02 ` Andrey Savchenko
@ 2020-11-10 10:24 ` Vitaly Lipatov
2020-11-10 13:07 ` Sergey V Turchin
0 siblings, 1 reply; 27+ messages in thread
From: Vitaly Lipatov @ 2020-11-10 10:24 UTC (permalink / raw)
To: ALT Linux Team development discussions
Andrey Savchenko писал 10.11.20 12:02:
> On Tue, 10 Nov 2020 11:27:31 +0700 Антон Мидюков wrote:
...
>> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
>> всегда? Причём не только при генерации патча, но и упаковки типа:
+++1
>>
>> tar: .
>>
>> У части пакетов и так всё нужное для сборки в .gear, так что в них
>> ничего делать не придётся.
>
> Мешает то, что сейчас можно класть файлы в .gear и не писать
> отдельные правила gear для копирования патчей или иных файлов.
> А с предложенным нововведением будет больше никому не нужного
> мусора.
Так почти никто не делает, но запаковать отдельно нужный каталог ведь
несложно.
--
С уважением,
Виталий Липатов,
ALT Linux Team
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 10:24 ` Vitaly Lipatov
@ 2020-11-10 13:07 ` Sergey V Turchin
0 siblings, 0 replies; 27+ messages in thread
From: Sergey V Turchin @ 2020-11-10 13:07 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tuesday, 10 November 2020 13:24:52 MSK Vitaly Lipatov wrote:
> Andrey Savchenko писал 10.11.20 12:02:
> > On Tue, 10 Nov 2020 11:27:31 +0700 Антон Мидюков wrote:
> ...
>
> >> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
>
> >> всегда? Причём не только при генерации патча, но и упаковки типа:
> +++1
>
> >> tar: .
> >>
> >> У части пакетов и так всё нужное для сборки в .gear, так что в них
> >> ничего делать не придётся.
> >
> > Мешает то, что сейчас можно класть файлы в .gear и не писать
> > отдельные правила gear для копирования патчей или иных файлов.
> > А с предложенным нововведением будет больше никому не нужного
> > мусора.
>
> Так почти никто не делает, но запаковать отдельно нужный каталог ведь
> несложно.
Я что только там не делаю, но, как писал выше, готов поправить, т.к. так
тарбол будет гораздо сильнее похож содержимым на оригинал. А автоудаление
.gear будет стимулом сделать корректнее. Можно опцию сделать неумолчательную
для сохранения .gear в тарболе.
--
Regards, Sergey.
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
` (2 preceding siblings ...)
2020-11-10 9:02 ` Andrey Savchenko
@ 2020-11-10 14:47 ` Dmitry V. Levin
2020-11-11 7:16 ` Anton V. Boyarshinov
3 siblings, 1 reply; 27+ messages in thread
From: Dmitry V. Levin @ 2020-11-10 14:47 UTC (permalink / raw)
To: ALT Devel discussion list
On Tue, Nov 10, 2020 at 11:27:31AM +0700, Антон Мидюков wrote:
> 10.11.2020 10:16, Vladimir D. Seleznev пишет:
> > Hi, devel@!
> >
> > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > предложить немного другое решение: вместо предлагаемой дополнительной
> > опции exclude= использовать директиву nodiff, в котором перечислены
> > шаблоны путей, которые следует исключить при генерации патча. Пример
> > того, как при этом может выглядеть .gear/rules:
> >
> > tar: v@version@:.
> > diff: v@version@:. .
> > nodiff: .gear/** *.spec
> >
> > [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> > [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
>
> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
> всегда? Причём не только при генерации патча, но и упаковки типа:
Обратная совместимость. Все тэги должны, сделанные в прошлом, должны
обрабатываться так же, как и раньше.
--
ldv
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 14:47 ` Dmitry V. Levin
@ 2020-11-11 7:16 ` Anton V. Boyarshinov
2020-11-11 8:14 ` Anton Farygin
0 siblings, 1 reply; 27+ messages in thread
From: Anton V. Boyarshinov @ 2020-11-11 7:16 UTC (permalink / raw)
To: Dmitry V. Levin; +Cc: ALT Linux Team development discussions
В Tue, 10 Nov 2020 17:47:13 +0300
"Dmitry V. Levin" <ldv@altlinux.org> пишет:
> > Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
> > всегда? Причём не только при генерации патча, но и упаковки типа:
>
> Обратная совместимость. Все тэги должны, сделанные в прошлом, должны
> обрабатываться так же, как и раньше.
Ну э... Я сейчас страшное предложу, но... Можно ввести новое поведение
только для тэгов, которые созданы после определённой даты.
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-11 7:16 ` Anton V. Boyarshinov
@ 2020-11-11 8:14 ` Anton Farygin
0 siblings, 0 replies; 27+ messages in thread
From: Anton Farygin @ 2020-11-11 8:14 UTC (permalink / raw)
To: devel
On 11.11.2020 10:16, Anton V. Boyarshinov wrote:
> В Tue, 10 Nov 2020 17:47:13 +0300
> "Dmitry V. Levin" <ldv@altlinux.org> пишет:
>
>>> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
>>> всегда? Причём не только при генерации патча, но и упаковки типа:
>> Обратная совместимость. Все тэги должны, сделанные в прошлом, должны
>> обрабатываться так же, как и раньше.
> Ну э... Я сейчас страшное предложу, но... Можно ввести новое поведение
> только для тэгов, которые созданы после определённой даты.
Почему страшное ? класс!
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 3:16 [devel] Introduce gear-rules nodiff directive Vladimir D. Seleznev
2020-11-10 3:16 ` [devel] [PATCH] Introduce nodiff directive for gear-rules Vladimir D. Seleznev
2020-11-10 4:27 ` [devel] Introduce gear-rules nodiff directive Антон Мидюков
@ 2020-11-10 8:03 ` Anton Farygin
2020-11-10 15:20 ` Vladimir D. Seleznev
2020-11-10 12:46 ` Arseny Maslennikov
2020-11-10 14:48 ` Dmitry V. Levin
4 siblings, 1 reply; 27+ messages in thread
From: Anton Farygin @ 2020-11-10 8:03 UTC (permalink / raw)
To: devel
С такой схемой не получится сделать разный список exclude для разных diff.
Эта опция больше похожа на exclude-path:
и она должна оказывать влияние не только на diff но и на tar (как
написал написал Антон).
On 10.11.2020 06:16, Vladimir D. Seleznev wrote:
> Hi, devel@!
>
> По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> предложить немного другое решение: вместо предлагаемой дополнительной
> опции exclude= использовать директиву nodiff, в котором перечислены
> шаблоны путей, которые следует исключить при генерации патча. Пример
> того, как при этом может выглядеть .gear/rules:
>
> tar: v@version@:.
> diff: v@version@:. .
> nodiff: .gear/** *.spec
>
> [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 8:03 ` Anton Farygin
@ 2020-11-10 15:20 ` Vladimir D. Seleznev
2020-11-11 6:25 ` Anton Farygin
0 siblings, 1 reply; 27+ messages in thread
From: Vladimir D. Seleznev @ 2020-11-10 15:20 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tue, Nov 10, 2020 at 11:03:13AM +0300, Anton Farygin wrote:
> С такой схемой не получится сделать разный список exclude для разных diff.
А это востребовано? По моим наблюдениям, если в gear-репозитории
несколько diff'ов, то из них, как правило, либо не надо ничего
исключать, либо исключать одно и то же. Хотя ничего не мешает помимо
этой директивы сделать ещё и отдельно опцию для diff.
> Эта опция больше похожа на exclude-path:
> и она должна оказывать влияние не только на diff но и на tar (как
> написал написал Антон).
Так уже можно делать с помощью gitattributes(5):
http://git.altlinux.org/gears/l/lxc.git?p=lxc.git;a=blob;f=.gitattributes;h=eff18b68348d39b69a72ca5f3a5f25d3cf8d82ae;hb=1fb13a49bb5e0d983aeb66ddee61f9ff143e54a6
--
WBR,
Vladimir D. Seleznev
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 15:20 ` Vladimir D. Seleznev
@ 2020-11-11 6:25 ` Anton Farygin
0 siblings, 0 replies; 27+ messages in thread
From: Anton Farygin @ 2020-11-11 6:25 UTC (permalink / raw)
To: devel
On 10.11.2020 18:20, Vladimir D. Seleznev wrote:
> On Tue, Nov 10, 2020 at 11:03:13AM +0300, Anton Farygin wrote:
>> С такой схемой не получится сделать разный список exclude для разных diff.
> А это востребовано? По моим наблюдениям, если в gear-репозитории
> несколько diff'ов, то из них, как правило, либо не надо ничего
> исключать, либо исключать одно и то же. Хотя ничего не мешает помимо
> этой директивы сделать ещё и отдельно опцию для diff.
Ну непонятно зачем отказываться от этой фичи. Выглядит более стройным
решением.
>
>> Эта опция больше похожа на exclude-path:
>> и она должна оказывать влияние не только на diff но и на tar (как
>> написал написал Антон).
> Так уже можно делать с помощью gitattributes(5):
>
> http://git.altlinux.org/gears/l/lxc.git?p=lxc.git;a=blob;f=.gitattributes;h=eff18b68348d39b69a72ca5f3a5f25d3cf8d82ae;hb=1fb13a49bb5e0d983aeb66ddee61f9ff143e54a6
>
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 3:16 [devel] Introduce gear-rules nodiff directive Vladimir D. Seleznev
` (2 preceding siblings ...)
2020-11-10 8:03 ` Anton Farygin
@ 2020-11-10 12:46 ` Arseny Maslennikov
2020-11-10 15:51 ` Vladimir D. Seleznev
2020-11-10 14:48 ` Dmitry V. Levin
4 siblings, 1 reply; 27+ messages in thread
From: Arseny Maslennikov @ 2020-11-10 12:46 UTC (permalink / raw)
To: ALT Linux Team development discussions; +Cc: legion
[-- Attachment #1: Type: text/plain, Size: 1313 bytes --]
On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> Hi, devel@!
>
> По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> предложить немного другое решение: вместо предлагаемой дополнительной
> опции exclude= использовать директиву nodiff, в котором перечислены
> шаблоны путей, которые следует исключить при генерации патча. Пример
> того, как при этом может выглядеть .gear/rules:
>
> tar: v@version@:.
> diff: v@version@:. .
> nodiff: .gear/** *.spec
Имя совершенно не к месту, создаётся ощущение, что новая директива
как-то отменяет некоторые действия директивы diff. На самом же деле она
задаёт список исключаемых из диффа шаблонов путей.
Может, назвать diff-exclude? Или diffignore, если дефисы вдруг нельзя.
Эти слова, по крайней мере, лучше ассоциируются с путями.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 12:46 ` Arseny Maslennikov
@ 2020-11-10 15:51 ` Vladimir D. Seleznev
2020-11-10 16:15 ` Andrey Savchenko
2020-11-10 17:30 ` Alexey V. Vissarionov
0 siblings, 2 replies; 27+ messages in thread
From: Vladimir D. Seleznev @ 2020-11-10 15:51 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tue, Nov 10, 2020 at 03:46:14PM +0300, Arseny Maslennikov wrote:
> On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > Hi, devel@!
> >
> > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > предложить немного другое решение: вместо предлагаемой дополнительной
> > опции exclude= использовать директиву nodiff, в котором перечислены
> > шаблоны путей, которые следует исключить при генерации патча. Пример
> > того, как при этом может выглядеть .gear/rules:
> >
> > tar: v@version@:.
> > diff: v@version@:. .
> > nodiff: .gear/** *.spec
>
> Имя совершенно не к месту, создаётся ощущение, что новая директива
> как-то отменяет некоторые действия директивы diff. На самом же деле она
> задаёт список исключаемых из диффа шаблонов путей.
>
> Может, назвать diff-exclude? Или diffignore, если дефисы вдруг нельзя.
> Эти слова, по крайней мере, лучше ассоциируются с путями.
Пока оно не applied, предлагайте варианты названия.
--
WBR,
Vladimir D. Seleznev
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 15:51 ` Vladimir D. Seleznev
@ 2020-11-10 16:15 ` Andrey Savchenko
2020-11-10 16:22 ` Vladimir D. Seleznev
2020-11-10 17:30 ` Alexey V. Vissarionov
1 sibling, 1 reply; 27+ messages in thread
From: Andrey Savchenko @ 2020-11-10 16:15 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 1709 bytes --]
On Tue, 10 Nov 2020 18:51:11 +0300 Vladimir D. Seleznev wrote:
> On Tue, Nov 10, 2020 at 03:46:14PM +0300, Arseny Maslennikov wrote:
> > On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > > Hi, devel@!
> > >
> > > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > > предложить немного другое решение: вместо предлагаемой дополнительной
> > > опции exclude= использовать директиву nodiff, в котором перечислены
> > > шаблоны путей, которые следует исключить при генерации патча. Пример
> > > того, как при этом может выглядеть .gear/rules:
> > >
> > > tar: v@version@:.
> > > diff: v@version@:. .
> > > nodiff: .gear/** *.spec
> >
> > Имя совершенно не к месту, создаётся ощущение, что новая директива
> > как-то отменяет некоторые действия директивы diff. На самом же деле она
> > задаёт список исключаемых из диффа шаблонов путей.
> >
> > Может, назвать diff-exclude? Или diffignore, если дефисы вдруг нельзя.
> > Эти слова, по крайней мере, лучше ассоциируются с путями.
>
> Пока оно не applied, предлагайте варианты названия.
Мне нравится вариант diffignore.
Best regards,
Andrew Savchenko
[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 16:15 ` Andrey Savchenko
@ 2020-11-10 16:22 ` Vladimir D. Seleznev
2020-11-10 17:00 ` Andrey Savchenko
` (2 more replies)
0 siblings, 3 replies; 27+ messages in thread
From: Vladimir D. Seleznev @ 2020-11-10 16:22 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tue, Nov 10, 2020 at 07:15:27PM +0300, Andrey Savchenko wrote:
> On Tue, 10 Nov 2020 18:51:11 +0300 Vladimir D. Seleznev wrote:
> > On Tue, Nov 10, 2020 at 03:46:14PM +0300, Arseny Maslennikov wrote:
> > > On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > > > Hi, devel@!
> > > >
> > > > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > > > предложить немного другое решение: вместо предлагаемой дополнительной
> > > > опции exclude= использовать директиву nodiff, в котором перечислены
> > > > шаблоны путей, которые следует исключить при генерации патча. Пример
> > > > того, как при этом может выглядеть .gear/rules:
> > > >
> > > > tar: v@version@:.
> > > > diff: v@version@:. .
> > > > nodiff: .gear/** *.spec
> > >
> > > Имя совершенно не к месту, создаётся ощущение, что новая директива
> > > как-то отменяет некоторые действия директивы diff. На самом же деле она
> > > задаёт список исключаемых из диффа шаблонов путей.
> > >
> > > Может, назвать diff-exclude? Или diffignore, если дефисы вдруг нельзя.
> > > Эти слова, по крайней мере, лучше ассоциируются с путями.
> >
> > Пока оно не applied, предлагайте варианты названия.
>
> Мне нравится вариант diffignore.
diffexclude мне кажется более точно отражающим смысл. diffingore ничуть
не лучше nodiff по смыслу, а символов больше.
--
WBR,
Vladimir D. Seleznev
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 16:22 ` Vladimir D. Seleznev
@ 2020-11-10 17:00 ` Andrey Savchenko
2020-11-10 18:26 ` [devel] gear-rules: new directive name Arseny Maslennikov
2020-11-10 18:47 ` [devel] Introduce gear-rules nodiff directive Michael Shigorin
2 siblings, 0 replies; 27+ messages in thread
From: Andrey Savchenko @ 2020-11-10 17:00 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 2244 bytes --]
On Tue, 10 Nov 2020 19:22:15 +0300 Vladimir D. Seleznev wrote:
> On Tue, Nov 10, 2020 at 07:15:27PM +0300, Andrey Savchenko wrote:
> > On Tue, 10 Nov 2020 18:51:11 +0300 Vladimir D. Seleznev wrote:
> > > On Tue, Nov 10, 2020 at 03:46:14PM +0300, Arseny Maslennikov wrote:
> > > > On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > > > > Hi, devel@!
> > > > >
> > > > > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > > > > предложить немного другое решение: вместо предлагаемой дополнительной
> > > > > опции exclude= использовать директиву nodiff, в котором перечислены
> > > > > шаблоны путей, которые следует исключить при генерации патча. Пример
> > > > > того, как при этом может выглядеть .gear/rules:
> > > > >
> > > > > tar: v@version@:.
> > > > > diff: v@version@:. .
> > > > > nodiff: .gear/** *.spec
> > > >
> > > > Имя совершенно не к месту, создаётся ощущение, что новая директива
> > > > как-то отменяет некоторые действия директивы diff. На самом же деле она
> > > > задаёт список исключаемых из диффа шаблонов путей.
> > > >
> > > > Может, назвать diff-exclude? Или diffignore, если дефисы вдруг нельзя.
> > > > Эти слова, по крайней мере, лучше ассоциируются с путями.
> > >
> > > Пока оно не applied, предлагайте варианты названия.
> >
> > Мне нравится вариант diffignore.
>
> diffexclude мне кажется более точно отражающим смысл. diffingore ничуть
> не лучше nodiff по смыслу, а символов больше.
У меня была аналогия с .gitignore. Здесь то же самое, но с diff.
Best regards,
Andrew Savchenko
[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] gear-rules: new directive name
2020-11-10 16:22 ` Vladimir D. Seleznev
2020-11-10 17:00 ` Andrey Savchenko
@ 2020-11-10 18:26 ` Arseny Maslennikov
2020-11-10 18:47 ` [devel] Introduce gear-rules nodiff directive Michael Shigorin
2 siblings, 0 replies; 27+ messages in thread
From: Arseny Maslennikov @ 2020-11-10 18:26 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 946 bytes --]
On Tue, Nov 10, 2020 at 07:22:15PM +0300, Vladimir D. Seleznev wrote:
> On Tue, Nov 10, 2020 at 07:15:27PM +0300, Andrey Savchenko wrote:
> > On Tue, 10 Nov 2020 18:51:11 +0300 Vladimir D. Seleznev wrote:
> > > Пока оно не applied, предлагайте варианты названия.
> >
> > Мне нравится вариант diffignore.
>
> diffexclude мне кажется более точно отражающим смысл. diffignore ничуть
> не лучше nodiff по смыслу, а символов больше.
Да оба отражают. :)
Вот два предложения:
"Из диффа _исключают_ некоторые пути."
"diff(1) при составлении диффа _не обращает внимания_ на некоторые пути."
Предложения означают одно и тоже, но второе чуть подробнее.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 16:22 ` Vladimir D. Seleznev
2020-11-10 17:00 ` Andrey Savchenko
2020-11-10 18:26 ` [devel] gear-rules: new directive name Arseny Maslennikov
@ 2020-11-10 18:47 ` Michael Shigorin
2020-11-11 8:07 ` Sergey V Turchin
2 siblings, 1 reply; 27+ messages in thread
From: Michael Shigorin @ 2020-11-10 18:47 UTC (permalink / raw)
To: devel
On Tue, Nov 10, 2020 at 07:22:15PM +0300, Vladimir D. Seleznev wrote:
> > > Пока оно не applied, предлагайте варианты названия.
> > Мне нравится вариант diffignore.
> diffexclude мне кажется более точно отражающим смысл.
> diffingore ничуть не лучше nodiff по смыслу, а символов больше.
indiff? :)
--
---- WBR, Michael Shigorin / http://altlinux.org
------ http://opennet.ru / http://anna-news.info
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 18:47 ` [devel] Introduce gear-rules nodiff directive Michael Shigorin
@ 2020-11-11 8:07 ` Sergey V Turchin
0 siblings, 0 replies; 27+ messages in thread
From: Sergey V Turchin @ 2020-11-11 8:07 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tuesday, 10 November 2020 21:47:34 MSK Michael Shigorin wrote:
> On Tue, Nov 10, 2020 at 07:22:15PM +0300, Vladimir D. Seleznev wrote:
> > > > Пока оно не applied, предлагайте варианты названия.
> > >
> > > Мне нравится вариант diffignore.
> >
> > diffexclude мне кажется более точно отражающим смысл.
> > diffingore ничуть не лучше nodiff по смыслу, а символов больше.
>
> indiff? :)
offdiff :-)
--
Regards, Sergey.
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 15:51 ` Vladimir D. Seleznev
2020-11-10 16:15 ` Andrey Savchenko
@ 2020-11-10 17:30 ` Alexey V. Vissarionov
1 sibling, 0 replies; 27+ messages in thread
From: Alexey V. Vissarionov @ 2020-11-10 17:30 UTC (permalink / raw)
To: ALT Linux Team development discussions
On 2020-11-10 18:51:11 +0300, Vladimir D. Seleznev wrote:
>>> путей, которые следует исключить при генерации патча.
>>> Пример того, как при этом может выглядеть .gear/rules:
>>> tar: v@version@:.
>>> diff: v@version@:. .
>>> nodiff: .gear/** *.spec
>> Имя совершенно не к месту, создаётся ощущение, что новая
>> директива как-то отменяет некоторые действия директивы
>> diff. На самом же деле она задаёт список исключаемых из
>> диффа шаблонов путей.
>> Может, назвать diff-exclude? Или diffignore, если дефисы
>> вдруг нельзя. Эти слова, по крайней мере, лучше
> ассоциируются с путями.
> Пока оно не applied, предлагайте варианты названия.
Я придерживаюсь такого правила: если есть несколько вариантов
названия опции, то нужно написать для каждого из них фрагмент
мануала, и где описание этимологии (происхождения) названия
окажется наиболее внятным - именно этот вариант использовать.
diffexclude
This optional parameter may be used to exclude some pathnames
from patch generation.
diffignore
This optional parameter may be used to ignore changes in some
pathnames while generating patches.
Здесь мне больше нравится вариант diffexclude: он и короче, и
более точно описывает происходящее - в перечисленные файлы мы
вообще смотреть не желаем, независимо от того, менялись ли они.
Да, метод взят (хотя и творчески переработан) из махровейшего
энтерпрайза. Но так как это действительно сильно упрощает жизнь,
я подобными способами пользуюсь. И да, во множественном числе:
их значительно больше одного вышеупомянутого.
--
Alexey V. Vissarionov
gremlin ПРИ altlinux ТЧК org; +vii-cmiii-ccxxix-lxxix-xlii
GPG: 0D92F19E1C0DC36E27F61A29CD17E2B43D879005 @ hkp://keys.gnupg.net
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 3:16 [devel] Introduce gear-rules nodiff directive Vladimir D. Seleznev
` (3 preceding siblings ...)
2020-11-10 12:46 ` Arseny Maslennikov
@ 2020-11-10 14:48 ` Dmitry V. Levin
2020-11-10 14:54 ` Sergey V Turchin
2020-11-10 15:29 ` Vladimir D. Seleznev
4 siblings, 2 replies; 27+ messages in thread
From: Dmitry V. Levin @ 2020-11-10 14:48 UTC (permalink / raw)
To: ALT Devel discussion list
On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> Hi, devel@!
>
> По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> предложить немного другое решение: вместо предлагаемой дополнительной
> опции exclude= использовать директиву nodiff, в котором перечислены
> шаблоны путей, которые следует исключить при генерации патча. Пример
> того, как при этом может выглядеть .gear/rules:
>
> tar: v@version@:.
> diff: v@version@:. .
> nodiff: .gear/** *.spec
>
> [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
А чем это лучше, чем предлагаемая дополнительная опция exclude?
--
ldv
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 14:48 ` Dmitry V. Levin
@ 2020-11-10 14:54 ` Sergey V Turchin
2020-11-10 15:29 ` Vladimir D. Seleznev
1 sibling, 0 replies; 27+ messages in thread
From: Sergey V Turchin @ 2020-11-10 14:54 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tuesday, 10 November 2020 17:48:30 MSK Dmitry V wrote:
> On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > Hi, devel@!
> >
> > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > предложить немного другое решение: вместо предлагаемой дополнительной
> > опции exclude= использовать директиву nodiff, в котором перечислены
> > шаблоны путей, которые следует исключить при генерации патча. Пример
> > того, как при этом может выглядеть .gear/rules:
> >
> > tar: v@version@:.
> > diff: v@version@:. .
> > nodiff: .gear/** *.spec
> >
> > [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> > [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
>
> А чем это лучше, чем предлагаемая дополнительная опция exclude?
Сложнее и в реализации и в использовании. Уже и баг нашли и ограничение в
пользовании.
--
Regards, Sergey.
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [devel] Introduce gear-rules nodiff directive
2020-11-10 14:48 ` Dmitry V. Levin
2020-11-10 14:54 ` Sergey V Turchin
@ 2020-11-10 15:29 ` Vladimir D. Seleznev
1 sibling, 0 replies; 27+ messages in thread
From: Vladimir D. Seleznev @ 2020-11-10 15:29 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Tue, Nov 10, 2020 at 05:48:30PM +0300, Dmitry V. Levin wrote:
> On Tue, Nov 10, 2020 at 06:16:07AM +0300, Vladimir D. Seleznev wrote:
> > Hi, devel@!
> >
> > По мотивам недавних обсуждений в рассылке [1] и бегрепорту [2] решил
> > предложить немного другое решение: вместо предлагаемой дополнительной
> > опции exclude= использовать директиву nodiff, в котором перечислены
> > шаблоны путей, которые следует исключить при генерации патча. Пример
> > того, как при этом может выглядеть .gear/rules:
> >
> > tar: v@version@:.
> > diff: v@version@:. .
> > nodiff: .gear/** *.spec
> >
> > [1] https://lists.altlinux.org/pipermail/devel/2020-November/212375.html
> > [2] https://bugzilla.altlinux.org/show_bug.cgi?id=31851
>
> А чем это лучше, чем предлагаемая дополнительная опция exclude?
gear-rules, как мне кажется, будет выглядеть аккуратнее. Хотя можно
дополнительно реализовать опцию exclude.
--
WBR,
Vladimir D. Seleznev
^ permalink raw reply [flat|nested] 27+ messages in thread