[Reproducible-commits] [dpkg] 24/90: Dpkg::Changelog::Entry::Debian: Remove non-functional timezone name support
Jérémy Bobbio
lunar at moszumanska.debian.org
Sat Aug 29 18:26:10 UTC 2015
This is an automated email from the git hooks/post-receive script.
lunar pushed a commit to branch pu/reproducible_builds
in repository dpkg.
commit 9849eff1e45bcd0c341a6ffb8675874a478fad9b
Author: Guillem Jover <guillem at debian.org>
Date: Wed Jul 1 05:10:08 2015 +0200
Dpkg::Changelog::Entry::Debian: Remove non-functional timezone name support
The Debian changelog trailer regex intended to support a timezone name
inside parenthesis, like this:
-- Name <mail at example.org> Sat, 30 May 2015 03:18:43 +0200 (CEST)
is bogus, and it only accepts one character. As in:
-- Name <mail at example.org> Sat, 30 May 2015 03:18:43 +0200 (C)
which while also "valid", as per RFC822, they are pretty much useless
as per RFC5322. The implementation has kept that part unchanged since
its inception in dpkg 1.3.0, 1996-08.
So let's just drop it instead of fixing the regex, given that the
timezone name has never been accepted, many time-parsing functions
ignore it, it is redundant, declared obsolete by RFC5322 and Debian
policy dropped an explicit reference to it due to bug #569174.
---
debian/changelog | 3 +++
scripts/Dpkg/Changelog/Entry/Debian.pm | 2 +-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/debian/changelog b/debian/changelog
index 7e8e6b4..6d6ebea 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,9 @@ dpkg (1.18.2) UNRELEASED; urgency=low
[ Guillem Jover ]
* Fix plural form translations for single plural languages. Closes: #790025
+ * Perl modules:
+ - Remove non-functional timezone name support from
+ Dpkg::Changelog::Entry::Debian.
* Documentation:
- Fix grammar in dpkg-architecture(1).
Thanks to Chris Lamb <lamby at debian.org>. Closes: #787616
diff --git a/scripts/Dpkg/Changelog/Entry/Debian.pm b/scripts/Dpkg/Changelog/Entry/Debian.pm
index f9d8b2f..e4c2193 100644
--- a/scripts/Dpkg/Changelog/Entry/Debian.pm
+++ b/scripts/Dpkg/Changelog/Entry/Debian.pm
@@ -64,7 +64,7 @@ our $regex_header = qr/^(\w$name_chars*) \(([^\(\) \t]+)\)((?:\s+$name_chars+)+)
# The matched content is the maintainer name ($1), its email ($2),
# some blanks ($3) and the timestamp ($4).
-our $regex_trailer = qr/^ \-\- (.*) <(.*)>( ?)((\w+\,\s*)?\d{1,2}\s+\w+\s+\d{4}\s+\d{1,2}:\d\d:\d\d\s+[-+]\d{4}(\s+\([^\\\(\)]\))?)\s*$/o;
+our $regex_trailer = qr/^ \-\- (.*) <(.*)>( ?)((\w+\,\s*)?\d{1,2}\s+\w+\s+\d{4}\s+\d{1,2}:\d\d:\d\d\s+[-+]\d{4})\s*$/o;
## use critic
--
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/reproducible/dpkg.git
More information about the Reproducible-commits
mailing list