[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#695967: [new check] GFDL invariant



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


Reply to: