Bug#687452: devscripts: fails to recognize licenses in (fixed-form) Fortran code

Benjamin Drung bdrung at debian.org
Wed Sep 19 21:27:58 UTC 2012


Am Mittwoch, den 19.09.2012, 19:28 +0200 schrieb Francesco Poli:
> Moving the comment removal statement up to the top of the block is
> crucial: it allows the removal of "[cC] " at the beginning of each line
> of text
> 
>   $content =~ s/^[cC] //gm;
> 
> *before* the following statement removes all tabs and all newline
> characters:
> 
>   $content =~ tr/\t\r\n/ /;

Thanks. Now it makes sense (to remove the Fortran comments _before_
removing the newline characters).

> > Having the comment on a separate line is fine. I have applied your
> > previous patch and added the test case. Thanks for your contribution.
> 
> That's really great news!   :-)
> Thanks to you for dealing with this bug so quickly and effectively.

You're welcome.

Side note: The work to add a test suite pays off.

-- 
Benjamin Drung
Debian & Ubuntu Developer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20120919/6cf9d09e/attachment.pgp>


More information about the devscripts-devel mailing list