Hi Guillem,

> 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 for the patch!

My understanding from the relevant thread is that this TZ name feature didn't 
work properly in the past and that it now cannot be used because dpkg will 
reject the changelog entry as being invalid. That should mean that we won't 
see new changelog entries with the TZ name in them, but are there historical 
changelog entries that include it? Do we know how many that might be?

python-debian's changelog support is used to iterate through the entire 
changelog not just the last changelog entry so removing the ability to parse 
these entries would either generate warnings or raise exceptions depending on 
the strictness specified by the caller. 

A question for you (and my co-maintainers), does it make sense to become more 
strict in what is accepted?

cheers
Stuart


-- 
Stuart Prescott    http://www.nanonanonano.net/   stu...@nanonanonano.net
Debian Developer   http://www.debian.org/         stu...@debian.org
GPG fingerprint    90E2 D2C1 AD14 6A1B 7EBB 891D BBC1 7EBB 1396 F2F7

Reply via email to