Bug 34756 - gear-changelog перестал ставить точку в конце предложений
Summary: gear-changelog перестал ставить точку в конце предложений
Status: CLOSED WONTFIX
Alias: None
Product: Sisyphus
Classification: Development
Component: gear (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Dmitry V. Levin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-03 15:46 MSK by Mikhail Efremov
Modified: 2023-08-18 13:12 MSK (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Efremov 2018-04-03 15:46:29 MSK
Это регрессия в 2.2.0-alt1.
Comment 1 Alexey Gladkov 2018-04-03 17:28:04 MSK
Это не регрессия, а намеренное изменение. Решение о том ставить ли точку теперь за пользователем.
Comment 2 Mikhail Efremov 2018-04-03 18:48:13 MSK
А по каким причинам не хочется ставить точки в конце предложений? Записи в changelog - это именно предложения, в конце предложений принято ставить точку.
В commit message же - это заголовок, там точка не нужна.
Comment 3 Alexey Gladkov 2018-04-03 21:22:45 MSK
> А по каким причинам не хочется ставить точки в конце предложений?

1. Потому что появились пользователи, которые хотят иметь другое форматирование.
2. Потому что из-за этого пустая строчка становится непустой.
Comment 4 Mikhail Efremov 2018-04-03 21:55:51 MSK
Тогда я думаю это должно быть настраиваемое поведение, очень напрягает расставлять теперь точки вручную.
И ставить точку - быть значением по умолчанию, я думаю это чаще нужно.
Comment 5 Alexey Gladkov 2018-04-03 22:00:36 MSK
(В ответ на комментарий №4)
> Тогда я думаю это должно быть настраиваемое поведение, очень напрягает
> расставлять теперь точки вручную.

Сейчас ставить точку или нет решает пользователь. Раньше она ставилась всегда принудительно и убрать её было нельзя.

> И ставить точку - быть значением по умолчанию, я думаю это чаще нужно.

Это вкусовщина.
Comment 6 Dmitry V. Levin 2018-04-03 22:04:07 MSK
(In reply to comment #5)
> (В ответ на комментарий №4)
> > Тогда я думаю это должно быть настраиваемое поведение, очень напрягает
> > расставлять теперь точки вручную.
> 
> Сейчас ставить точку или нет решает пользователь.

А какие у пользователя есть возможности настройки этого поведения?
Comment 7 Alexey Gladkov 2018-04-04 00:07:09 MSK
(В ответ на комментарий №6)
> А какие у пользователя есть возможности настройки этого поведения?

Добавить в группу:

filter: s/^.*/&./

или поставить точку в уже существующем фильтре, если он есть.
Comment 8 Mikhail Efremov 2018-04-04 02:43:32 MSK
А глобально это как-то можно сделать? Не писать же такие правила для каждого своего gear-репозитория.
Comment 9 Alexey Gladkov 2018-04-04 09:59:17 MSK
(В ответ на комментарий №8)
> А глобально это как-то можно сделать? Не писать же такие правила для каждого
> своего gear-репозитория.

Постойте, а как вы используете эту утилиту ?

Она задумывалась как агригатор, который работает с правилами, иначе gear-changelog ни чем не отпличается "git log --format='- %s (by %an).'". У меня везде, где я её использую есть правила. Поэтому мне не казалось неправильным перенос точки в правила.
Comment 10 Mikhail Efremov 2018-04-04 18:19:23 MSK
(In reply to comment #9)
> (В ответ на комментарий №8)
> > А глобально это как-то можно сделать? Не писать же такие правила для каждого
> > своего gear-репозитория.
> 
> Постойте, а как вы используете эту утилиту ?
> 
> Она задумывалась как агригатор, который работает с правилами, иначе
> gear-changelog ни чем не отпличается "git log --format='- %s (by %an).'". У
> меня везде, где я её использую есть правила. Поэтому мне не казалось
> неправильным перенос точки в правила.

Ну, у меня скрипт использует gear-changelog --no-rules и добавляет в spec этакий полуфабрикат, из которого обычно нужно удалить лишнее. Он далеко не всегда делает то, что мне надо, конечно, но это все же лучше, чем просто git log --format: gear-changelog делает заголовок для %changelog и изменения берет только с последнего тэга.
Я могу свои скрипты написать, но раз есть gear-changelog, то хотелось бы использовать его.
Comment 11 Alexey Gladkov 2018-04-04 18:36:07 MSK
(В ответ на комментарий №10)
> Ну, у меня скрипт использует gear-changelog --no-rules и добавляет в spec
> этакий полуфабрикат, из которого обычно нужно удалить лишнее. Он далеко не
> всегда делает то, что мне надо, конечно, но это все же лучше, чем просто git
> log --format: gear-changelog делает заголовок для %changelog и изменения берет
> только с последнего тэга.

Интересный usecase.

> Я могу свои скрипты написать, но раз есть gear-changelog, то хотелось бы
> использовать его.

А что он должен уметь делать, чтобы вам было удобно ?
Comment 12 Mikhail Efremov 2018-04-04 20:02:15 MSK
(In reply to comment #11)
> А что он должен уметь делать, чтобы вам было удобно ?

Получается, что 2 взаимоисключающих вещи: если я мержу новый апстримный релиз (обычно тэг), то очевидно, мне хочется видеть в changelog только изменения из моего текущего бранча с последней сборки (т.е. с предыдущего тэга), все коммиты же апстрима мне в качестве записей в changelog совсем не нужны. Сейчас они все попадают в changеlog и их приходится вычищать.
Если же я мержу апсримный бранч между релизами и решаю, что это будет отображено в changelog как отдельные патчи из апстрима, то мне удобно иметь там записи для апстримных коммитов. То же самое когда мержится чей-то еще бранч с изменениями.
Т.е. для одного и того же гит-репозитория в разные моменты хочется разного поведения. Хотя если правила будут позволять задать нужное поведение, то можно использовать --rules для разных правил, видимо. Тем более, что такие правила скорее всего будут одинаковые для разных репо и нет никакого смысла их дублировать для каждого.
Comment 13 Alexey Gladkov 2018-04-04 22:37:00 MSK
Либо я не понял необходимого поведения, либо не увидел уведел конкретики. Вы как-то очень туманно (на мой взгляд) описали то, что хочется получить от утилиты.

Кроме того, работа без правил это не то, чего я хотел. С помощью правил я группирую, фильтрую и форматирую записи. Именно поэтому дополнительное форматирование (типа точек в конце) мешает.
Comment 14 Alexey Gladkov 2023-08-18 13:12:44 MSK
Не думаю, что это нужно исправлять.