search

dmgerman / ninka

a license identification tool for Source Code
http://ninka.turingmachine.org
GNU General Public License v2.0
103 stars 30 forks source link

Fix tokenizer regex for MPLv2.0 #22

Closed wyhfrank closed 8 years ago

wyhfrank commented 9 years ago

This fixes the bug described in #21, which is, MPLv2.0 cannot be recognized.

However, in order to solve the problem caused by source files using https instead of http in the URL part in the license header (detailed in the later part of #21), we might need to change all the matching rule from http<colon> to https?<colon>.

dmgerman commented 9 years ago

On Mon, Sep 21, 2015 at 1:04 AM, wyhfrank notifications@github.com wrote:

However, in order to solve the problem caused by source files using https instead of http in the URL part in the license header (detailed in the later part of #21 https://github.com/dmgerman/ninka/issues/21), we might need to change all the matching rule from http to https?.

that sounds good. I think it is the way to do it. Thanks for the work.

--dmg

--dmg

Daniel M. German http://turingmachine.org

bufferoverflow commented 8 years ago

@dmgerman can you merge this?