Re: lintian: r307 - in trunk: debian lib
On Thu, Jul 01, 2004 at 08:06:03PM +0200, Jeroen van Wolffelaar wrote:
> On Thu, Jul 01, 2004 at 06:51:48PM +0200, djpig@debian.org wrote:
> > Author: djpig
> > Date: 2004-07-01 18:51:47 +0200 (Thu, 01 Jul 2004)
> > New Revision: 307
> >
> > Modified:
> > trunk/debian/changelog
> > trunk/lib/Util.pm
> > Log:
> > a space after the colon is not mandatory in dpkg control files
> >
> >
> > Modified: trunk/debian/changelog
> > ===================================================================
> > --- trunk/debian/changelog 2004-07-01 16:45:46 UTC (rev 306)
> > +++ trunk/debian/changelog 2004-07-01 16:51:47 UTC (rev 307)
> > @@ -109,6 +109,8 @@
> > * lib/Util.pm:
> > + [FL] dpkg supports comments in control files now, handle them
> > correctly
> > + + [FL] a space after the colon is not mandatory in dpkg control
> > + files
> >
> > * testset/filenames:
> > + [JvW] Fix two lintian warnings in lintian itself by touching files only
> >
> > Modified: trunk/lib/Util.pm
> > ===================================================================
> > --- trunk/lib/Util.pm 2004-07-01 16:45:46 UTC (rev 306)
> > +++ trunk/lib/Util.pm 2004-07-01 16:51:47 UTC (rev 307)
> > @@ -92,7 +92,7 @@
> > $last_tag = $tag;
> > }
> > # new field?
> > - elsif (m/^(\S+): (.*)$/o) {
> > + elsif (m/^(\S+):(.*)$/o) {
>
> Erm, shouldn't there be some extra \s* here? To kill the whitespace
> before the value that is being put into $2?
Heh, didn't read last commit... nvm... seems that even drunk I spot
these kind of issues :-/
--Jeroen
--
Jeroen van Wolffelaar
Jeroen@wolffelaar.nl (also for Jabber & MSN; ICQ: 33944357)
http://Jeroen.A-Eskwadraat.nl
Reply to: