Hi James.

On Sun, 10 Apr 2011 23:51:36 -0400, James Vega <james...@debian.org> wrote:
> On Tue, Apr 05, 2011 at 04:19:39AM +0400, Dmitry Kurochkin wrote:
> > The Apache license rexex in licensecheck version 2.10.71 is:
> > 
> >   /under the Apache License, Version ([^ ]+) \(the License\)/
> > 
> > Since the stuff in parenthesis is meant for the following text which may
> > not be present in many customized preambles, it should not be matched for.
> 
> Have you seen instances where that was the problem?

Yes, the polygraph [1] package I am trying to get accepted to Debian.
See discussion on the debian-mentors mailing list [2].

> It seems like the
> more relevant problem is that the regular expression doesn't contain the
> double quotes around "License", like official boilerplate does.
> 

The quotes are missing indeed. But apparently licensecheck does some
preprocessing on the input because the current regex successfully
matches the official boilerplate.

This may not be the most relevant problem, but we have at least one
example when it would help. And since fixing the issue is easy, I
believe it should be done.

Regards,
  Dmitry

[1] 
http://mentors.debian.net/cgi-bin/maintainer-packages?action=details;package=polygraph
[2] http://lists.debian.org/debian-mentors/2011/04/msg00038.html



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to