aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorSamuel Henrique <samueloph@debian.org>2022-08-01 18:49:19 +0100
committerGuido Günther <agx@sigxcpu.org>2023-02-24 18:59:47 +0100
commit11728ec3158e315697adb0384b7cedbc0333e1f7 (patch)
treebf1311f374eeca7e2e869ea1f31459bdfaa65e9a
parentcfaf0e9b5e8b7e57c9dcd99a019451cb69a74b46 (diff)
rpm/policy: Add missing dash before version in RPM changelog header
As stated in the documentation at https://rpm-packaging-guide.github.io/#working-with-spec-files "... Follow this format for the first line: * Day-of-Week Month Day Year Name Surname <email> - Version-Release ..."
-rw-r--r--gbp/rpm/policy.py2
-rw-r--r--tests/30_test_rpm_changelog.py26
2 files changed, 14 insertions, 14 deletions
diff --git a/gbp/rpm/policy.py b/gbp/rpm/policy.py
index a2155e20..59989bb8 100644
--- a/gbp/rpm/policy.py
+++ b/gbp/rpm/policy.py
@@ -85,7 +85,7 @@ class RpmPkgPolicy(PkgPolicy):
body_name_re = r'\[(?P<name>.*)\]'
# Changelog header format (when writing out changelog)
- header_format = "* %(time)s %(name)s <%(email)s> %(revision)s"
+ header_format = "* %(time)s %(name)s <%(email)s> - %(revision)s"
header_time_format = "%a %b %d %Y"
header_rev_format = "%(version)s"
diff --git a/tests/30_test_rpm_changelog.py b/tests/30_test_rpm_changelog.py
index 85142a41..2a0d068d 100644
--- a/tests/30_test_rpm_changelog.py
+++ b/tests/30_test_rpm_changelog.py
@@ -34,7 +34,7 @@ class TestChangelogHeader(object):
time = datetime(2014, 1, 29, 12, 13, 14)
header = _ChangelogHeader(RpmPkgPolicy, time, name="John Doe",
email="user@host.com", revision="1")
- eq_(str(header), "* Wed Jan 29 2014 John Doe <user@host.com> 1\n")
+ eq_(str(header), "* Wed Jan 29 2014 John Doe <user@host.com> - 1\n")
def test_str_format_err(self):
"""Test missing properties"""
@@ -79,7 +79,7 @@ class TestChangelogSection(object):
def test_str_format(self):
"""Basic test"""
section = self.default_sect
- eq_(str(section), "* Wed Jan 29 2014 J. D. <u@h> 1\n- my change\n\n")
+ eq_(str(section), "* Wed Jan 29 2014 J. D. <u@h> - 1\n- my change\n\n")
def test_append_entry(self):
"""Test add_entry() method"""
@@ -87,7 +87,7 @@ class TestChangelogSection(object):
entry = _ChangelogEntry(RpmPkgPolicy, author="",
text="- another\n change")
new_entry = section.append_entry(entry)
- eq_(str(section), "* Wed Jan 29 2014 J. D. <u@h> 1\n- my change\n"
+ eq_(str(section), "* Wed Jan 29 2014 J. D. <u@h> - 1\n- my change\n"
"- another\n change\n\n")
eq_(new_entry, section.entries[-1])
@@ -96,25 +96,25 @@ class TestChangelogSection(object):
section = self.default_sect
time = datetime(2014, 1, 30)
section.set_header(time=time, name="Jane", email="u@h", revision="1.1")
- eq_(str(section), "* Thu Jan 30 2014 Jane <u@h> 1.1\n- my change\n\n")
+ eq_(str(section), "* Thu Jan 30 2014 Jane <u@h> - 1.1\n- my change\n\n")
class TestChangelogParser(object):
"""Test the default changelog parser"""
cl_default_style = """\
-* Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.3-1
+* Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.3-1
- Version bump
- Drop foo.patch
-* Tue Jan 28 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.2
+* Tue Jan 28 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.2
- Update to 0.2
-* Mon Jan 27 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.1
+* Mon Jan 27 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.1
- Initial version
"""
cl_with_authors = """\
-* Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.3-1
+* Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.3-1
[Markus Lehtonen]
- Version bump
[John Doe]
@@ -122,17 +122,17 @@ class TestChangelogParser(object):
"""
# Invalid timestamp / name
cl_broken_header_1 = """\
-* Wed Jan 29 2014Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.3-1
+* Wed Jan 29 2014Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.3-1
- Version bump
"""
# Whitespace before the asterisk in the header
cl_broken_header_2 = """\
- * Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.3-1
+ * Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.3-1
- Version bump
"""
# Invalid timestamp
cl_broken_header_3 = """\
-* Wed Jan 32 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.3-1
+* Wed Jan 32 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.3-1
- Version bump
"""
# Missing email
@@ -143,7 +143,7 @@ class TestChangelogParser(object):
# Garbage before section header
cl_broken_header_5 = """\
---garbage---
-* Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> 0.3-1
+* Wed Jan 29 2014 Markus Lehtonen <markus.lehtonen@linux.intel.com> - 0.3-1
- Version bump
"""
@@ -222,5 +222,5 @@ class TestChangelog(object):
time = datetime(2014, 1, 30)
new_section = changelog.add_section(time=time, name="Jane Doe",
email="j@doe.com", revision="1.2")
- eq_(str(changelog), "* Thu Jan 30 2014 Jane Doe <j@doe.com> 1.2\n\n")
+ eq_(str(changelog), "* Thu Jan 30 2014 Jane Doe <j@doe.com> - 1.2\n\n")
eq_(new_section, changelog.sections[0])