On Wed, 27 Apr 2022 09:52:07 +1000, Ben Finney wrote: > The matching is incorrectly case-sensitive. It should accept valid > variations such as ‘VCS-Git’ and ‘VCS-Browser’, but instead it > crashes: > > Invalid field given (VCS_Git) at /usr/share/perl5/Debian/Control.pm line > 122. > > The matching should be case-insensitive, understanding ‘vcs-git’ and > ‘VCS-Git’ and ‘Vcs-Git’ and ‘vcs-Git’ to all be the same field name.
Hm, interesting bug report :) First, I wanted to ask "Why?" but then I found Debian Policy 5.1: Field names are not case-sensitive, but it is usual to capitalize the field names using mixed case as shown below. Then I found t/Control.t which tests exactly this issue. And then I found the following d/changelog entry for 0.95: [ Alex Muntada ] * Debian::Control::Stanza: accept case-insensitive field names in new() as required by Debian Policy while retaining the canonical accessors. Thanks to Ben Finney for the bug report. (Closes: #860023) And #860023 from 5 years ago is also interesting :) But yeah, it's not only a déjà-vu, apparently we need to take a look at this part of the code again … Cheers, gregor -- .''`. https://info.comodo.priv.at -- Debian Developer https://www.debian.org : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D 85FA BB3A 6801 8649 AA06 `. `' Member VIBE!AT & SPI Inc. -- Supporter Free Software Foundation Europe `-
signature.asc
Description: Digital Signature