search

Jasig / maven-notice-plugin

Apache License 2.0
7 stars 16 forks source link

add <includeScopes> and <excludeScopes> configuration #10

Closed fstab closed 7 years ago

fstab commented 7 years ago

The official Apache licensing howto says: "Dependencies which are not included in the distribution MUST NOT be added to LICENSE and NOTICE". Therefore, I think it would be nice to be able to exclude test dependencies, because they are usually not bundled with a distribution.

This PR adds an configuration option like this:

<configuration>
    <includeScopes>
        <includeScopes>compile</includeScopes>
    </includeScopes>
</configuration>

You can either specify <includeScopes> to list only the scopes you want to have included in the NOTICE file, or <excludeScopes> to list the scopes to be omitted from the NOTICE file.

See here for the quote above: http://www.apache.org/dev/licensing-howto.html#bundled-vs-non-bundled

ChristianMurphy commented 7 years ago

This would be something that Licensing Group might be able to weigh in on. /cc @apetro

ChristianMurphy commented 7 years ago

@fstab Could you sign the Jasig/Apereo ICLA? https://www.apereo.org/licensing/agreements/icla

fstab commented 7 years ago

I signed it.

bdemers commented 7 years ago

Sweet! I was just stopping by to request this feature 👍

fstab commented 7 years ago

I opened a topic in the Licensing Group: https://groups.google.com/a/apereo.org/forum/#!topic/licensing-discuss/sG95wiLkpJo