Bug 31851 - add exclude to diff options
: add exclude to diff options
Status: NEW
: Sisyphus
(All bugs in Sisyphus/gear)
: unstable
: all Linux
: P3 normal
Assigned To:
:
:
:
:
:
  Show dependency tree
 
Reported: 2016-03-02 11:53 by
Modified: 2018-07-21 01:07 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2016-03-02 11:53:15
Было бы неплохо добавить к diff в gear-rules возможность указывать exclude
каталогам.

Мне это нужно для того, что бы в патч не попадал каталог .gear
например:
diff: v@version@:. . exclude=.gear
------- Comment #1 From 2018-07-20 08:08:28 -------
Сейчас готовлю релиз с этой возможностью, чтобы предложить (одобрить).

Можно посмотреть сейчас в git.altlinux.org/people/imz/packages/gear.git (ветка
alt). Пусть только коммиты с тестами не пугают. По сути они, конечно, работают
и написаны для большинства случаев. Но остаётся привести в лучший вид. Мне не
понравилось, что мы с grenka@ размножили файлы тестов, сейчас хочу
дедуплицировать код тестов.

Будет полезно не только для diff, но правил типа tar. Пример:

diff: v@version@:. . -- . :!.gear

tar: v@version@:. -- . :!.gear

Используется синтаксис (с магией), понимаемый командами Git с некоторый пор:
git cmd ... -- paths..

Из-за ошибки в Git приходится писать не как в примерах (с точкой), а со
звёздочкой (только для правил типа tar, а не diff; т.е. ошибка есть в работе
git archive, но не git diff):

tar: v@version@:. -- * :!.gear

Приветствуется ознакомление заинтересованных.
------- Comment #2 From 2018-07-20 10:04:15 -------
(В ответ на комментарий №1)
> Мне не
> понравилось, что мы с grenka@ размножили файлы тестов, сейчас хочу
> дедуплицировать код тестов.

Залей так. Когда разберёмся с кодом тестов, обновим релиз, в чём проблема?

"Что сделано не вовремя, сделано понапрасну" (с) Френсис Бэкон
------- Comment #3 From 2018-07-20 10:12:49 -------
(In reply to comment #2)
> (В ответ на комментарий №1)
> > Мне не
> > понравилось, что мы с grenka@ размножили файлы тестов, сейчас хочу
> > дедуплицировать код тестов.
> 
> Залей так. Когда разберёмся с кодом тестов, обновим релиз, в чём проблема?

Просто этот пакет кто-то из acl будет смотреть и одобрять.

Я уже этим сообщением предложил посмотреть. И в тот момент, когда кто-то будет
готов одобрить, можно будет отправить текущее состояние.

Пока что переписываю дополнительные тесты с помощью дополнительных функций в
том  же файле (вызывающих основную), без копирования файла. Частично это
сделано, можно увидеть в моих опубликованных коммитах.
------- Comment #4 From 2018-07-20 16:06:57 -------
Можно будет попробовать в task 210568
------- Comment #5 From 2018-07-20 16:19:34 -------
(In reply to comment #1)
> Сейчас готовлю релиз с этой возможностью, чтобы предложить (одобрить).
> 
> Можно посмотреть сейчас в git.altlinux.org/people/imz/packages/gear.git (ветка
> alt).

Там специально commit messages в другом стиле выполнены?
Некоторые по ширине даже у меня на экране не помещаются.
------- Comment #6 From 2018-07-20 16:29:22 -------
(In reply to comment #5)
> (In reply to comment #1)
> > Сейчас готовлю релиз с этой возможностью, чтобы предложить (одобрить).
> > 
> > Можно посмотреть сейчас в git.altlinux.org/people/imz/packages/gear.git (ветка
> > alt).
> 
> Там специально commit messages в другом стиле выполнены?
> Некоторые по ширине даже у меня на экране не помещаются.

Нет, я не обращал внимание. Посмотрю.

Пользуясь случаем, хочу рассказать, что на i586 произошла ошибка в задании
210568:

gear-update: /usr/src/tmp/run.lcG0esBBa/.git/.work/initial-import.cpio.zst:
unrecognized source type: 
[FAIL] (gear_update_subdir_sub_cpio_zst_cwd)
------- Comment #7 From 2018-07-20 16:44:59 -------
(In reply to comment #6)
> Пользуясь случаем, хочу рассказать, что на i586 произошла ошибка в задании
> 210568:
> 
> gear-update: /usr/src/tmp/run.lcG0esBBa/.git/.work/initial-import.cpio.zst:
> unrecognized source type: 
> [FAIL] (gear_update_subdir_sub_cpio_zst_cwd)

http://git.altlinux.org/beehive/logs/Sisyphus-i586/archive/2018/0720/success/gear-2.2.0-alt1
http://git.altlinux.org/beehive/logs/Sisyphus-i586/archive/2018/0719/error/gear-2.2.0-alt1.zst
http://git.altlinux.org/beehive/logs/Sisyphus-i586/archive/2018/0717/success/gear-2.2.0-alt1.zst
...
http://git.altlinux.org/beehive/logs/Sisyphus-i586/archive/2018/0701/success/gear-2.2.0-alt1.zst
------- Comment #8 From 2018-07-20 16:56:14 -------
(In reply to comment #5)
> (In reply to comment #1)
> > Сейчас готовлю релиз с этой возможностью, чтобы предложить (одобрить).
> > 
> > Можно посмотреть сейчас в git.altlinux.org/people/imz/packages/gear.git (ветка
> > alt).
> 
> Там специально commit messages в другом стиле выполнены?
> Некоторые по ширине даже у меня на экране не помещаются.

Переписал. Ещё дописываю обновление документации. Есть ли возражения против
всего остального?

(Пока с этим копался, появилась мысль, что тесты можно было бы покороче
записать, с меньшми повторениями, в т.ч. старые, но я не стремлюсь переписать
их к ближайшему релизу.)
------- Comment #9 From 2018-07-20 20:25:46 -------
В коммите "Amend all remaining gear-rules-archive-*.test with tests for magic
paths" очень странное выравнивание: добавляются строки, начинающиеся с 8 табов.

Ну и вообще читабельность тестов снизилась с "нечитабельно" до "совсем
нечитабельно". :)
------- Comment #10 From 2018-07-21 01:07:31 -------
(In reply to comment #9)
> В коммите "Amend all remaining gear-rules-archive-*.test with tests for magic
> paths" очень странное выравнивание: добавляются строки, начинающиеся с 8 табов.

Поправил, и отправил на сборку. Это уже все изменения с моей стороны на данный
момент, больше ничего добавлять перед предложеннымм релизом не планирую по
своей задумке. Учту замечания.

> Ну и вообще читабельность тестов снизилась с "нечитабельно" до "совсем
> нечитабельно". :)

Я это тоже вижу, но быстро без переосмысления и перестркутурирования не могу
поменять сейчас.