-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[libusb] Fix undefined symbols on osx #29393
Conversation
xiaozhuai
commented
Feb 2, 2023
- Fix undefined symbols on osx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have modified or added at least one vcpkg.json where you should check the license
field.
If you feel able to do so, please consider replacing the deprecated license identifiers in the following files:
ports/libusb/vcpkg.json
(has deprecated licenseLGPL-2.1
)
Deprecated and non deprecated license identifiers can be found here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!
PRs must add only one version and must not modify any published versions
When making any changes to a library, the version or port-version in vcpkg.json
or CONTROL
must be modified.
error: checked-in files for libusb have changed but the version was not updated
version: 1.0.26#3
old SHA: 01d788329687e36ad42b813027dfbc8912bf2b12
new SHA: 540652cc1251aa3894206c20296f6119da1ccea4
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
"description": "a cross-platform library to access USB devices", | ||
"homepage": "https://github.com/libusb/libusb", | ||
"license": "LGPL-2.1", | ||
"license": "LGPL-2.1-or-later", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is supported by:
Thanks for the fix! |