[devel] Introduce gear-rules nodiff directive

[devel] Introduce gear-rules nodiff directive

[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

[devel] [PATCH] Introduce nodiff directive for gear-rules

[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

Re: [devel] Introduce gear-rules nodiff directive

[Антон Мидюков]

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>

Re: [devel] Introduce gear-rules nodiff directive

[Sergey V Turchin]

On Tuesday, 10 November 2020 07:27:31 MSK Антон Мидюков wrote:

[...]
> Скажите, пожалуйста, что нам мешает игнорировать директорию .gear
> всегда? Причём не только при генерации патча, но и упаковки типа:
> 
> tar: .
У меня какие-то сборки сломаются, но я не против исправить. Будет красивее.
 
> У части пакетов и так всё нужное для сборки в .gear, так что в них
> ничего делать не придётся.

-- 
Regards, Sergey.

Re: [devel] Introduce gear-rules nodiff directive

[Anton Farygin]

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, так что в них 
> ничего делать не придётся.
>
Кстати да, Антон, спасибо. Это решение мне казалось радикальным, но оно 
выглядит правильным.

Re: [devel] Introduce gear-rules nodiff directive

[Anton Farygin]

С такой схемой не получится сделать разный список 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

Re: [devel] Introduce gear-rules nodiff directive

[Andrey Savchenko]

[-- 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 --]

Re: [devel] Introduce gear-rules nodiff directive

[Vitaly Lipatov]

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

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

[Alexey Gladkov]

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

Re: [devel] Introduce gear-rules nodiff directive

[Arseny Maslennikov]

[-- 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 --]

Re: [devel] Introduce gear-rules nodiff directive

[Sergey V Turchin]

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.

Re: [devel] Introduce gear-rules nodiff directive

[Dmitry V. Levin]

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

Re: [devel] Introduce gear-rules nodiff directive

[Dmitry V. Levin]

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

Re: [devel] Introduce gear-rules nodiff directive

[Sergey V Turchin]

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.

Re: [devel] Introduce gear-rules nodiff directive

[Vladimir D. Seleznev]

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

Re: [devel] Introduce gear-rules nodiff directive

[Vladimir D. Seleznev]

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

Re: [devel] Introduce gear-rules nodiff directive

[Vladimir D. Seleznev]

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

Re: [devel] Introduce gear-rules nodiff directive

[Andrey Savchenko]

[-- 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 --]

Re: [devel] Introduce gear-rules nodiff directive

[Vladimir D. Seleznev]

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

Re: [devel] Introduce gear-rules nodiff directive

[Andrey Savchenko]

[-- 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 --]

Re: [devel] Introduce gear-rules nodiff directive

[Alexey V. Vissarionov]

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

Re: [devel] gear-rules: new directive name

[Arseny Maslennikov]

[-- 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 --]

Re: [devel] Introduce gear-rules nodiff directive

[Michael Shigorin]

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

Re: [devel] Introduce gear-rules nodiff directive

[Anton Farygin]

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
>

Re: [devel] Introduce gear-rules nodiff directive

[Anton V. Boyarshinov]

В Tue, 10 Nov 2020 17:47:13 +0300
"Dmitry V. Levin" <ldv@altlinux.org> пишет:

> > Скажите, пожалуйста, что нам мешает игнорировать директорию .gear 
> > всегда? Причём не только при генерации патча, но и упаковки типа:  
> 
> Обратная совместимость.  Все тэги должны, сделанные в прошлом, должны
> обрабатываться так же, как и раньше.

Ну э... Я сейчас страшное предложу, но... Можно ввести новое поведение
только для тэгов, которые созданы после определённой даты.

Re: [devel] Introduce gear-rules nodiff directive

[Sergey V Turchin]

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.

Re: [devel] Introduce gear-rules nodiff directive

[Anton Farygin]

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
>>> всегда? Причём не только при генерации патча, но и упаковки типа:
>> Обратная совместимость.  Все тэги должны, сделанные в прошлом, должны
>> обрабатываться так же, как и раньше.
> Ну э... Я сейчас страшное предложу, но... Можно ввести новое поведение
> только для тэгов, которые созданы после определённой даты.
Почему страшное ? класс!