Skip to content

Conversation

@JonoYang
Copy link
Member

This is a new builtin plugin that compiles msitools and packs the binaries into a locationprovider plugin. This is used for aboutcode-org/scancode-toolkit#2559 and related to aboutcode-org/scancode-toolkit#2565

JonoYang added 2 commits June 18, 2021 17:25
Signed-off-by: Jono Yang <jyang@nexb.com>
 * Remove dist and build directories Signed-off-by: Jono Yang <jyang@nexb.com>
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks . See a few comments inline

JonoYang added 2 commits June 20, 2021 17:26
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
@JonoYang JonoYang force-pushed the packagedcode-msitools branch from d347bab to 8139daf Compare June 21, 2021 06:03
@JonoYang
Copy link
Member Author

@pombredanne I've addressed your comments. Quick question about license: I noticed that builtins/packagedcode_msitools-linux/src/packagedcode_msitools/bin/msidiff and builtins/packagedcode_msitools-linux/src/packagedcode_msitools/bin/msidump have the gpl-2.0-plus license header. The library msitools is licensed under lgpl-2.1-plus. Would the proper license expression for packagedcode_msitools be apache-2.0 AND lgpl-2.1-plus AND gpl-2.0-plus?

Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See a few nit inline. LGTM otherwise

 * Update build scripts * Add license files and update license expression for the plugin Signed-off-by: Jono Yang <jyang@nexb.com>
@JonoYang JonoYang merged commit 5ca42a4 into main Jul 6, 2021
@JonoYang JonoYang deleted the packagedcode-msitools branch July 6, 2021 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants