Bug#872432: licensecheck patch for "additional ignores"

[Jonas Smedegaard]

Control: retitle -1 licensecheck: should support adding (nor replacing) ignores
Control: block -1 by 960695

[ resposting with 7bit headers to please Debian MTAs ]

Quoting IOhannes m zmölnig (2017-10-06 10:29:21)
> here's a smallish patch that allows to specify a regex for 
> *additionally* ignored files when doing a licensecheck, as requested 
> in #872432.
> 
> this is kept separately from the original "--ignore|-i" flag to not 
> break any existing uses of licensecheck. also i think that "-x" is 
> easier to remember than "--install^Wignore" :-)

Thanks (and sorry for several years of silence!).

I dislike adding more options for same functionality, and I dislike that 
the option --ignore is a regex.  Instead, I consider adding new option 
"--usage" to change behaviour and defaults of other options, treating 
current behaviours as --usage=legacy.  See bug#960695 for details.

This is current "licensecheck --help":

 -i STR --ignore STR        regular expression of files to skip
                            (default value: some backup and VCS files)


How do you think about this future "licensecheck --help":

 -i STR --ignore STR        files to skip;
                            with usage=legacy a regular expression
                            replacing previous or default value,
                            with any other usage a comma-separated list 
                            adding onto previous and default values
                            except an empty value which clears the list
                            (default value: some backup and VCS files)


 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: signature
URL: <http://alioth-lists.debian.net/pipermail/pkg-perl-maintainers/attachments/20200515/342325eb/attachment.sig>