Skip to content

Commit

Permalink
Fix compilation warning in ring library
Browse files Browse the repository at this point in the history
The following warning is being fixed:

warning: unused imports: `PKCS1`, `PSS`
  --> ring/src/rsa/padding.rs:21:13
   |
21 |     pkcs1::{PKCS1, RSA_PKCS1_SHA256, RSA_PKCS1_SHA384, RSA_PKCS1_SHA512},
   |             ^^^^^
22 |     pss::{PSS, RSA_PSS_SHA256, RSA_PSS_SHA384, RSA_PSS_SHA512},
   |           ^^^
   |
   = note: `#[warn(unused_imports)]` on by default

That warning was already fixed with this patch:
briansmith/ring@c4742e0

Once ring library version will be bumped up, this patch could be dropped.

Signed-off-by: Zbigniew Lukwinski <[email protected]>
  • Loading branch information
zlukwins authored and jyao1 committed Dec 19, 2024
1 parent db2b7ca commit fac5664
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions sh_script/pre-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ patch-ring() {
pushd external/ring
git reset --hard 464d367252354418a2c17feb806876d4d89a8508
git clean -xdf

# apply the patch to get rid of unused import warning during compilation
# https://github.com/briansmith/ring/commit/c4742e0cae849f08ff410a817c5266af41670b3d
git cherry-pick c4742e0cae849f08ff410a817c5266af41670b3d

case "$TARGET_OPTION" in
"x86_64-unknown-none")
git apply ../patches/ring/0001-Support-x86_64-unknown-none-target.patch
Expand Down

0 comments on commit fac5664

Please sign in to comment.