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 <bmas...@redhat.com>
Signed-off-by: Herton R. Krzesinski <her...@redhat.com>

---
 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                    |   4 +++-
 26 files changed, 7 insertions(+), 33 deletions(-)
--
_______________________________________________
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to