Hello, as you can see, I fixed this bug without integrating your patch. This is because po4a is only a parser, not a generator of such files. There is nothing we can do to fix the debian changelog files. Instead, we must deal with all variants of such files. Your change would have make po4a applicable to even less existing changelogs...
Anyway, thanks for the report, it's fixed now (hopefully). Bye, Mt. On Tue, Jul 14, 2015 at 05:54:16PM +0200, Guillem Jover wrote: > Package: po4a > Version: 0.46-2 > Severity: wishlist > Tags: patch > > Hi! > > As pointed out in > <https://lists.debian.org/debian-policy/2015/06/msg00034.html>, the > changelog trailer part of the regex to support timezone names is bogus. > And it seems the best thing to do is to remove it instead of fixing it. > For more details either see tha above mail or the commit message on the > attached patch. Beware, that the patch has not been tested! > > Thanks, > Guillem > From f162d6383b800a4f1ce2ae2406e6778df550a476 Mon Sep 17 00:00:00 2001 > From: Guillem Jover <guil...@debian.org> > Date: Tue, 14 Jul 2015 17:32:25 +0200 > Subject: [PATCH] Locale::Po4a::Text: Remove support for broken timezone names > MIME-Version: 1.0 > Content-Type: text/plain; charset=UTF-8 > Content-Transfer-Encoding: 8bit > > The Debian changelog trailer regex intended to support a timezone name > inside parenthesis, like this: > > -- Name <m...@example.org> Sat, 30 May 2015 03:18:43 +0200 (CEST) > > is bogus, and it only accepts one character. As in: > > -- Name <m...@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. It seems the regex was “inherited” from the dpkg > implementation which 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. > > Signed-off-by: Guillem Jover <guil...@debian.org> > --- -- J'aime mon pays mais je crains mon gouvernement. -- Anonyme
signature.asc
Description: PGP signature
