Proposing licensecheck change

James McCoy jamessan at debian.org
Mon Jul 27 06:47:27 UTC 2015


On Tue, Jul 21, 2015 at 07:31:07PM +0200, Dominique Dumont wrote:
> Currently, licensecheck -r uses find to scan a directory and accepts files 
> based on their suffix (i.e. accepts .c .h .cxx ...)
> 
> This list of suffixes is a big regexp that must be updated regurlarly. Still 
> some files are missed like config.guess.
> 
> Maintaining this regexp is not efficient.

True.  The current mechanism is tedious and either requires intervention
from the user are requesting updates to the regexp.

> I'd like to propose to use 'file' command to decide whether to scan a file or 
> not. All files of mime type 'text/*' and 'application/xml' would be scanned.
> (Note that file is already used to find the charset of each scanned file.)

If I understand correctly, you're proposing something like the attached,
untested patch.  Keep the -c switch so users can still filter by
extension if they like, but by default find all files and filter based
on file type.

> This way, I think that licensecheck would miss less source files.

Sounds good to me.

Cheers,
-- 
James
GPG Key: 4096R/331BA3DB 2011-12-05 James McCoy <jamessan at debian.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: licensecheck-text.diff
Type: text/x-diff
Size: 1609 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20150727/eff7856c/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 949 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20150727/eff7856c/attachment.sig>


More information about the devscripts-devel mailing list