[OS-BUILD PATCHv2 0/2] redhat: create an empty changelog file when changing its name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Herton R. Krzesinski on gitlab.com
Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2838

Bugzilla: INTERNAL
Upstream Status: RHEL only

Up until now, if we rename the spec package name or bump the rhel major
or minor versions, we used to copy the previous changelog file if it
exists into the new one. However, this makes the changelog grow
indefinitely since a stream start, and it starts to get very big. We
do not need to keep the entire history at each new changelog file,
because eg. in a stream life, we always keep the older changelog files
in the git tree.

Thus, instead of copying the old changelog, just create a new empty
file. This also makes sure we trim the rpm changelog at each new minor
release, since it's used as the source of the rpm package changelog
content. And that's another intention with this change: to avoid having
the rpm changelog contents to grow too big, as reported by Brian Masney.

There was also a case where if we are starting a stream or changing the
package name, and there is no old changelog file available, we copied
a kernel.changelog* if it existed. But looks like this case we should
also do the same and trim it to start fresh, the old changelog file can
be kept if wanted in the tree for reference and we don't necessarily
need to copy all entries from the old changelog.

With this there is some simplification as well and we can remove the
CHANGELOG_PREV_EXT variable.

Reported-by: Brian Masney <bmasney@xxxxxxxxxx>
Signed-off-by: Herton R. Krzesinski <herton@xxxxxxxxxx>

---
 redhat/self-test/data/centos-2585cf9dfaad.el7  |   1 -
 redhat/self-test/data/centos-2585cf9dfaad.fc25 |   1 -
 redhat/self-test/data/centos-78e36f3b0dae.el7  |   1 -
 redhat/self-test/data/centos-78e36f3b0dae.fc25 |   1 -
 redhat/self-test/data/centos-df0cc57e057f.el7  |   1 -
 redhat/self-test/data/centos-df0cc57e057f.fc25 |   1 -
 redhat/self-test/data/centos-fce15c45d3fb.el7  |   1 -
 redhat/self-test/data/centos-fce15c45d3fb.fc25 |   1 -
 redhat/self-test/data/fedora-2585cf9dfaad.el7  |   1 -
 redhat/self-test/data/fedora-2585cf9dfaad.fc25 |   1 -
 redhat/self-test/data/fedora-78e36f3b0dae.el7  |   1 -
 redhat/self-test/data/fedora-78e36f3b0dae.fc25 |   1 -
 redhat/self-test/data/fedora-df0cc57e057f.el7  |   1 -
 redhat/self-test/data/fedora-df0cc57e057f.fc25 |   1 -
 redhat/self-test/data/fedora-fce15c45d3fb.el7  |   1 -
 redhat/self-test/data/fedora-fce15c45d3fb.fc25 |   1 -
 redhat/self-test/data/rhel-2585cf9dfaad.el7    |   1 -
 redhat/self-test/data/rhel-2585cf9dfaad.fc25   |   1 -
 redhat/self-test/data/rhel-78e36f3b0dae.el7    |   1 -
 redhat/self-test/data/rhel-78e36f3b0dae.fc25   |   1 -
 redhat/self-test/data/rhel-df0cc57e057f.el7    |   1 -
 redhat/self-test/data/rhel-df0cc57e057f.fc25   |   1 -
 redhat/self-test/data/rhel-fce15c45d3fb.el7    |   1 -
 redhat/self-test/data/rhel-fce15c45d3fb.fc25   |   1 -
 redhat/Makefile                                |  12 ++++--------
 redhat/kernel.spec.template                    |   3 ++-
 26 files changed, 6 insertions(+), 33 deletions(-)
--
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux