This time using group and $1 for submatching On Sat, Dec 15, 2012 at 11:37 PM, Bastien ROUCARIES <roucaries.bastien@gmail.com> wrote: > Sorry forget a bug > > On Sat, Dec 15, 2012 at 11:36 PM, Bastien ROUCARIES > <roucaries.bastien@gmail.com> wrote: >> Corrected and with testsuite >> >> On Sat, Dec 15, 2012 at 10:35 PM, Bastien ROUCARIES >> <roucaries.bastien@gmail.com> wrote: >>> control: tags -1 patch >>> >>> On Sat, Dec 15, 2012 at 11:55 AM, Jakub Wilk <jwilk@debian.org> wrote: >>>> Thanks for the patch. >>>> >>>> * bastien ROUCARIES <roucaries.bastien@gmail.com>, 2012-12-14, 23:56: >>>>> >>>>> + # GFDL state machine >>>>> + if($line =~ m/GNU\s+Free\s+Documentation\s+License/) { >>>>> + $foundGFDL = 1; >>>>> + } >>>>> + if($foundGFDL) { >>>>> + # match in two part in order to not match italic html >>>>> license example >>>>> + if($line =~ m/with\s+the\s+Invariant\s+Sections\s+being/i) >>>>> { >>>>> + if($line !~ m/list\s+their\s+titles/i) { >>>>> + tag 'license-problem-gfdl-invariants', $name; >>>>> + } >>>>> + } >>>>> + if($line =~ m/the\s+Front-Cover\s+Texts\s+being/i) { >>>>> + if($line !~ >>>>> m/the\s+Front-Cover\s+Texts\s+being\s+(?:<var>)?\s+list/) { >>>>> + tag 'license-problem-gfdl-invariants', $name; >>>>> + } >>>>> + } >>>>> + } >>>> >>>> >>>> There are also Back-Cover Texts, which are as bad as Front-Cover Texts. >>>> >>>> AFAICS, the cover text check will trigger for the GFDL license text itself, >>>> which contains: >>>> with the Invariant Sections being @var{list their titles}, with >>>> or: >>>> with the Invariant Sections being LIST THEIR TITLES, with >>>> or: >>>> with the Invariant Sections being <list their titles>, with >>>> (depending on the file format). >>>> >>>> The "with the Invariant Sections being" and "the Front-Cover Texts being" >>>> phrases could be spread across two lines. Please take this into account. >>> >>> I have something better here . >>> >>> Bastien >>> >>>> -- >>>> Jakub Wilk
Attachment:
0001-Improve-cruft-license-check-by-using-a-sliding-windo.patch
Description: Binary data