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

Fix ring workaround version check #234

Merged

Conversation

wpikulik
Copy link
Contributor

@wpikulik wpikulik commented Oct 31, 2023

With this fix I'm trying to fix a problem with workaround for ring trait.
Before it showed me this Warning:
[DEBUG] unable to apply workaround 'ring' to 'ring 0.17.5': version 0.17.5 is not covered, please file a PR to add it
and at the end I got an error:

 error: failed to satisfy license requirements <<< custom part
    ┌─ /Users/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ring-0.17.5/Cargo.toml:170:13
    │
170 │ license = "(GPL-1.0-or-later AND OpenSSL AND OpenSSL OR BSD-3-Clause) AND (ISC) AND (ISC AND MIT AND OpenSSL AND OpenSSL AND ISC) AND (ISC AND NOASSERTION) AND (MIT) AND (NOASSERTION) AND (NOASSERTION AND OpenSSL) AND (OpenSSL) AND (OpenSSL AND OpenSSL OR (BSD-3-Clause OR (GPL-1.0-or-later OR GPL-2.0-only))) AND (OpenSSL AND OpenSSL OR (BSD-3-Clause OR GPL-1.0-or-later)) AND (OpenSSL AND OpenSSL OR BSD-3-Clause)"

@Jake-Shadle Jake-Shadle changed the title bug: fix ring workaroung version check Fix ring workaround version check Oct 31, 2023
Copy link
Member

@Jake-Shadle Jake-Shadle left a comment

Choose a reason for hiding this comment

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

Thanks!

@Jake-Shadle Jake-Shadle merged commit e55f940 into EmbarkStudios:main Oct 31, 2023
1 check passed
@wpikulik wpikulik deleted the fix-ring-workaround-version branch October 31, 2023 14:17
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

2 participants