Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support loading mutliple licenseConfigFiles #538

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Support loading mutliple licenseConfigFiles #538

wants to merge 10 commits into from

Conversation

tisoft
Copy link

@tisoft tisoft commented Dec 11, 2023

This PR contains several changes. If you prefer, I can also make individual PRs for each.

Main features:

  • allow explicit include/exclusion of Modules in the aggregate mojo
    • not all out modules are client deliverables, we include only those in the license aggrgation
  • allow to load licenseConfigFile from URLs and classpath
    • we provide a company wide config file in a JAR, that can be loaded from classpath
  • allow to load multiple licenseConfigFiles
    • we use this to have a company wide config file, that can be augmented with project specific files

Additionally I enabled this to build on jitpack, for an easy possibility to test unreleased versions and I fixed the maven wrapper binary.

tisoft and others added 10 commits December 11, 2023 15:52
@tisoft
align .gitattributes with .editorconfig
11e29a1
@tisoft
skip ITs on jitpack
75160e4
@tisoft
update version and groupId so the plugin descriptor is correct
93a998e
@tisoft
Convert mac file endings for CDATA
a5b87c2
@tisoft
Added license.excludedModules
8f70e4d 
Exclude modules from processing.
Comma separated list of relative module paths. Exclusions are not recursive.
@tisoft
Mark .jar as binary
9cb2729
@tisoft
Added license.includedModules
6558b49 
Include modules for processing.
Comma separated list of relative module paths. Inclusions are not recursive.
Empty value includes all.
@tisoft
use resource manager to load licensesConfigFile
a7619a7 
* supports loading licensesConfigFile from file/url/classpath
@tisoft
Allow a list of licensesConfigFiles
d7bb266 
* licensesConfigFile is still supported
@tisoft
Updated maven wrapper
423b1fb
@slawekjaranowski
Copy link
Member

Thanks for proposition.

  • First I prefer to deploy snapshot version to oss.sonatype.org - I can do it

Pleas a separate PR for each feature.

@slawekjaranowski
Copy link
Member

Deploy will be added in: #539

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tisoft @slawekjaranowski