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

Include all licenses in the resulting containers #45

Open
jpkrohling opened this issue Dec 16, 2021 · 8 comments
Open

Include all licenses in the resulting containers #45

jpkrohling opened this issue Dec 16, 2021 · 8 comments
Labels

Comments

@jpkrohling
Copy link
Member

From open-telemetry/opentelemetry-collector#2604

@jpkrohling jpkrohling added enhancement New feature or request good first issue Good for newcomers labels Dec 16, 2021
@bogdandrutu
Copy link
Member

I would generate this file as part of the "builder" and maybe include in the container if we want.

@shree007
Copy link

@jpkrohling i may work

@shree007
Copy link

I was out of station. I started to look into it.

@jpkrohling
Copy link
Member Author

Welcome back!

@shree007
Copy link

@jpkrohling I went throgh the open-telemetry/opentelemetry-collector#2604 So all licences will be copied into third-party folder in build phase and that folder will be copied in docker container, for which i need to write docker insturctions.
right ?

@jpkrohling
Copy link
Member Author

That sounds right.

@shree007
Copy link

shree007 commented Mar 6, 2022

@jpkrohling From open-telemetry/opentelemetry-collector#2604 here,
go-licenses has been used to collect the licenses which is this https://github.com/google/go-licenses.
Its first prerequisites that targeted folder should have .go file,
Earlier when I started to work, There was internal/tools folder which contains .go files, Now I am not seeing any .go files in main branch.

May you guide me little bit here to proceed ?
Thanks

@jpkrohling
Copy link
Member Author

The sources are generated. Run 'make build' and you should end up with the source code for each one of the distributions. This source code isn't checked in, though.

@shree007 shree007 removed their assignment Jul 23, 2023
@dosubot dosubot bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Jul 16, 2024
@dosubot dosubot bot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 23, 2024
@dosubot dosubot bot removed the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Jul 23, 2024
@mx-psi mx-psi reopened this Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants