Alright, I've enabled the default license matcher for our license checker as part of this work: https://issues.apache.org/jira/browse/IGNITE-13574
- Denis On Mon, Oct 12, 2020 at 9:30 PM Denis Magda <[email protected]> wrote: > Instead of using the "addDefaultLicenseMatchers" setting, we developed a > practice of excluding non-Apache licenses from the checklist of the license > checker. A couple of examples: > > - BSD license: > https://github.com/apache/ignite/blob/ignite-13574/parent/pom.xml#L889 > - Another BSD license: > https://github.com/apache/ignite/blob/ignite-13574/parent/pom.xml#L984 > - JetBrains sources: > https://github.com/apache/ignite/blob/ignite-13574/parent/pom.xml#L903 > - the list goes on and on, check all the exclusions of the license > checker: > https://github.com/apache/ignite/blob/ignite-13574/parent/pom.xml#L842 > > Have we ever tried to use the "addDefaultLicenseMatchers" setting instead? > > - > Denis > > > On Mon, Oct 12, 2020 at 3:25 PM Denis Magda <[email protected]> wrote: > >> Igniters, >> >> I was adding a missing license header to some sources that were included >> in our repository and found that the license checker rejects the MIT >> license (it's compliant with Apache 2.0 >> <https://www.apache.org/legal/resolved.html#category-a>). Take these >> files as an example that are protected by the MIT license but rejected by >> the license checker: >> >> - >> >> https://github.com/apache/ignite/blob/ignite-13574/docs/_sass/rouge-base16-solarized.scss >> - >> >> https://github.com/apache/ignite/blob/ignite-13574/docs/_plugins/asciidoctor-extensions.rb >> >> >> All the checks pass if to set the "addDefaultLicenseMatchers" setting of >> the license checker to "true". Presently, that setting is "false": >> https://github.com/apache/ignite/blob/ignite-13574/parent/pom.xml#L806 >> >> Does anybody know why the "addDefaultLicenseMatchers" parameter is set to >> "false"? I would go ahead and change it to "true". >> >> - >> Denis >> >
