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

Ensure correct dependencies license information is included #49

Open
Nemo157 opened this issue Dec 6, 2018 · 0 comments
Open

Ensure correct dependencies license information is included #49

Nemo157 opened this issue Dec 6, 2018 · 0 comments

Comments

@Nemo157
Copy link
Owner

Nemo157 commented Dec 6, 2018

#38 added a subcommand to print cargo-lichking's dependencies licenses using the new bundle printing functionality. Unfortunately this is very easy to have incorrect, and even with the test added in #48 will be incorrect if someone builds without using the lockfile (which includes via cargo install).

One way to do this would be to provide the main functionality via a library, which can then be used as part of the build script to generate this file for the current build. That will probably double the already quite long build time though....

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

No branches or pull requests

1 participant