devel@ where you _can_ ask
 help / color / mirror / Atom feed
* [newbies] changelog in spec and commit message
@ 2025-09-25 11:33 Alexander Lubyagin
  2025-09-25 11:43 ` Anton Farygin
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Alexander Lubyagin @ 2025-09-25 11:33 UTC (permalink / raw)
  To: devel-newbies

Добрый день.

В каких случаях запись 'commit message' (git commit -m) дублирует 
многострочную запись в spec's changelog?

В каких случаях они различаются? Есть ли рекомендации?

--



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [newbies] changelog in spec and commit message
  2025-09-25 11:33 [newbies] changelog in spec and commit message Alexander Lubyagin
@ 2025-09-25 11:43 ` Anton Farygin
  2025-09-25 11:44   ` Anton Farygin
  2025-09-25 12:12 ` Paul Wolneykien
  2025-09-25 21:24 ` Leonid Krivoshein
  2 siblings, 1 reply; 5+ messages in thread
From: Anton Farygin @ 2025-09-25 11:43 UTC (permalink / raw)
  To: devel-newbies

On 9/25/25 14:33, Alexander Lubyagin wrote:
> Добрый день.
>
> В каких случаях запись 'commit message' (git commit -m) дублирует 
> многострочную запись в spec's changelog?
>
> В каких случаях они различаются? Есть ли рекомендации?


Изменения в rpm spec обычно фиксируются через команду gear-commit -a, 
эта команда автоматически в commit message добавит записи из changelog

Другие изменения в пакете лучше фиксировать отдельными коммитами, 
оформленными по обычным рекомендациям к написанию commit message на 
английскком языке:
-------------------------

Короткий заголовок в повелительном наклонении (до 50 символов)

Более развернутое объяснение изменений:
- Что изменено
- Почему это важно
- Ссылки: Fixes #NNN
-------------------------


Важно фиксировать не только то, что изменилось, но и зачем было сделано 
это изменение.

пример (выдуман):
-----------------------

|Addsystemd hardening options tokeepalived.service Enabled additional 
security directives: 
-ProtectSystem=full-ProtectHome=true-ProtectKernelModules=true-RestrictRealtime=trueThese 
options reduce the attack surface bylimiting access tosystemfiles, 
userhomes andkernel interfaces. Fixes: ALTBUG#55878 ------------------|




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [newbies] changelog in spec and commit message
  2025-09-25 11:43 ` Anton Farygin
@ 2025-09-25 11:44   ` Anton Farygin
  0 siblings, 0 replies; 5+ messages in thread
From: Anton Farygin @ 2025-09-25 11:44 UTC (permalink / raw)
  To: devel-newbies

On 9/25/25 14:43, Anton Farygin wrote:
> On 9/25/25 14:33, Alexander Lubyagin wrote:
>> Добрый день.
>>
>> В каких случаях запись 'commit message' (git commit -m) дублирует 
>> многострочную запись в spec's changelog?
>>
>> В каких случаях они различаются? Есть ли рекомендации?
>
>
> Изменения в rpm spec обычно фиксируются через команду gear-commit -a, 
> эта команда автоматически в commit message добавит записи из changelog
>
> Другие изменения в пакете лучше фиксировать отдельными коммитами, 
> оформленными по обычным рекомендациям к написанию commit message на 
> английскком языке:
> -------------------------
>
> Короткий заголовок в повелительном наклонении (до 50 символов)
>
> Более развернутое объяснение изменений:
> - Что изменено
> - Почему это важно
> - Ссылки: Fixes #NNN
> -------------------------
>
>
> Важно фиксировать не только то, что изменилось, но и зачем было 
> сделано это изменение.
>
> пример (выдуман):
> ----------------------- 

Почему-то в примере слетело форматирование:


Add systemd hardening options to keepalived.service


Enabled additional security directives:
- ProtectSystem=full
- ProtectHome=true
- ProtectKernelModules=true
- RestrictRealtime=true

These options reduce the attack surface by limiting access to
system files, user homes and kernel interfaces.
Fixes: ALT#55878




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [newbies] changelog in spec and commit message
  2025-09-25 11:33 [newbies] changelog in spec and commit message Alexander Lubyagin
  2025-09-25 11:43 ` Anton Farygin
@ 2025-09-25 12:12 ` Paul Wolneykien
  2025-09-25 21:24 ` Leonid Krivoshein
  2 siblings, 0 replies; 5+ messages in thread
From: Paul Wolneykien @ 2025-09-25 12:12 UTC (permalink / raw)
  To: devel-newbies

В Thu, 25 Sep 2025 14:33:04 +0300
Alexander Lubyagin <lubyagin@yandex.ru> пишет:

> Добрый день.
> 
> В каких случаях запись 'commit message' (git commit -m) дублирует 
> многострочную запись в spec's changelog?
> 
> В каких случаях они различаются? Есть ли рекомендации?

  Основная рекомендация по моей версии такая. Работая в git вы
комментируете свои изменения для других мэйнтейнеров (и для себя).
Проделав эту работу (или попутно) в %changelog пакета вы перечисляете
важные для пользователя изменения, переводя их на понятный пользователю
язык. Таким образом, главное в том, что целевая аудитория у git и
%changelog разная. Всегда нужно думать о том, для кого вы пишете
сообщение.

  К примеру, в git следует написать о том, что была убрана
неиспользуемая функция или забытая переменная в коде, а также о том,
что были исправлены предупреждения (warnings) во время компиляции.
Однако, поскольку для пользователя ровно ничего от этого не изменилось,
в %changelog об этом писать не следует.

  А вот если вы добавили новую переменную в конфигурационный файл, то
написать об этом в %changelog нужно обязательно. Правда относится это
только к вашим собственным изменениям: дублировать release notes из
апстрима в %changelog не нужно (он раздуется тогда до гигантских
размеров). Поэтому при тривиальном обновлении пишем просто "New
version: такая-то" или подобную запись (не регламентировано). Однако,
если сборка новой версии исправляет известную ошибку -- об этом всё
же стоит написать отдельно (с указанием номера ошибки).

  Бывают ситуации, когда в %changelog и написать особо нечего, а хотя
бы строчку написать нужно -- без этого никак. Поэтому бывают записи
наподобие "Recompiled with new libXXYY", которые для пользователя,
в общем-то, не несут никакой полезной информации.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [newbies] changelog in spec and commit message
  2025-09-25 11:33 [newbies] changelog in spec and commit message Alexander Lubyagin
  2025-09-25 11:43 ` Anton Farygin
  2025-09-25 12:12 ` Paul Wolneykien
@ 2025-09-25 21:24 ` Leonid Krivoshein
  2 siblings, 0 replies; 5+ messages in thread
From: Leonid Krivoshein @ 2025-09-25 21:24 UTC (permalink / raw)
  To: devel-newbies

Добрый день!


On 9/25/25 14:33, Alexander Lubyagin wrote:
> В каких случаях запись 'commit message' (git commit -m) дублирует 
> многострочную запись в spec's changelog?
>
> В каких случаях они различаются? Есть ли рекомендации?
>

Это разные сущности, их вообще не стоит связывать и сравнивать.

Commit message делается по каждому изменению в git'е для других 
разработчиков. Заголовок сообщения отвечает на вопрос, что делает 
коммит, а тело сообщения -- на вопрос, зачем он это делает. Для многих 
коммитов, если это и так очевидно, тело сообщения можно опустить. Ширина 
теста в обоих случаях должна быть ограничена, см. в книге 
https://git-scm.com/book/ru/v2 (раздел "Правила создания коммитов").

Changelog в SPEC'ах делается для пользователей, а не для разработчиков. 
В нём не нужно отражать все изменения в исходниках. В нём отражаются 
наиболее важные изменения новой сборки пакета. По его написанию есть 
своё руководство: 
https://www.altlinux.org/Руководство_по_написанию_changelog

Если я, как разработчик, одним коммитом в git/исходники вношу важное 
изменение в будущую сборку, только тогда сообщения в changelog и commit 
message совпадут, такое встречается, но не так часто.


-- 
WBR, Leonid Krivoshein.



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-09-25 21:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-25 11:33 [newbies] changelog in spec and commit message Alexander Lubyagin
2025-09-25 11:43 ` Anton Farygin
2025-09-25 11:44   ` Anton Farygin
2025-09-25 12:12 ` Paul Wolneykien
2025-09-25 21:24 ` Leonid Krivoshein

devel@ where you _can_ ask

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \
		devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com
	public-inbox-index devel-newbies

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git