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.

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

Proposal: Add an option to include all licenses in one file #701

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

maxlazio
Copy link
Contributor

@maxlazio maxlazio commented Aug 8, 2016

Description

We had multiple users requesting one file which contains all the generated licenses for their internal approving processes.

This PR is more of a proposal to see if this use case would be acceptable for more users, the code is written to demonstrate how this might look.

If license_compiled_output true, the main LICENSE file would contain the actual text from each individual license, otherwise the behaviour remains the same.

My questions would be: Would something like this be useful to more omnibus users and the maintainers? If yes, how could this be improved to be accepted? If no, thanks for reading :)


/cc @chef/omnibus-maintainers <- This ensures the Omnibus Maintainers team are notified to review this PR.

@maxlazio maxlazio changed the title Proposal: Add an an option to include all licenses in one file Proposal: Add an option to include all licenses in one file Aug 8, 2016
@tas50 tas50 removed the Signed CLA label Aug 10, 2018
@brodock
Copy link
Contributor

brodock commented Jul 30, 2020

@chef/omnibus-maintainers is there any intention of accepting this? we are maintaining this change in our internal fork

@tas50
Copy link
Contributor

tas50 commented Jul 31, 2020

@maxlazio @brodock This is actually the direction I'd like to take our builds. Any chance you can rebase this so I can give it some local testing?

@lamont-granquist
Copy link
Contributor

obviously needs a rebase and needs fixing, but i'm cool with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants