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.

Sign up for GitHub

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

Jump to bottom

Remove unused libs #94

Merged
merged 3 commits into from
Oct 17, 2024
Merged

Remove unused libs #94

merged 3 commits into from
Oct 17, 2024

Conversation

vo-nil
Copy link
Contributor

@vo-nil vo-nil commented Oct 16, 2024

closes #93

@vo-nil vo-nil self-assigned this Oct 16, 2024
@vo-nil vo-nil force-pushed the 93-remove-unused-libs branch from a6ebd09 to 4080e79 Compare October 16, 2024 15:13
@vo-nil vo-nil requested a review from martun October 16, 2024 15:13
@vo-nil vo-nil marked this pull request as ready for review October 16, 2024 15:14
@vo-nil vo-nil force-pushed the 93-remove-unused-libs branch from eb49cf6 to 8f7526d Compare October 17, 2024 08:57
@github-actions GitHub Actions
Copy link

Clang Test Results

  151 files   -  7    151 suites   - 7   10m 48s ⏱️ -5s
3 182 tests  - 49  3 176 ✅  - 49  6 💤 ±0  0 ❌ ±0 
3 252 runs   - 49  3 246 ✅  - 49  6 💤 ±0  0 ❌ ±0 

Results for commit 8f7526d. ± Comparison against base commit 0ae4dc1.

This pull request removes 53 and adds 4 tests. Note that renamed tests count towards both. 
bls_serialization ‑ g1_serialization_test
bls_serialization ‑ g2_serialization_test
bls_signature_public_interface_tests ‑ bls_aug_mss
bls_signature_public_interface_tests ‑ bls_basic_mps
bls_signature_public_interface_tests ‑ bls_basic_mss
bls_signature_public_interface_tests ‑ bls_pop_mss
chacha_test_suite.chacha_single_range_encrypt ‑ _0
chacha_test_suite.chacha_single_range_encrypt ‑ _1
chacha_test_suite.chacha_single_range_encrypt ‑ _2
chacha_test_suite.chacha_single_range_encrypt ‑ _3
…

lpc_math_polynomial_suite ‑ lpc_basic_skipping_layers_test
lpc_math_polynomial_suite ‑ lpc_dfs_basic_test
lpc_params_test_suite ‑ lpc_batches_num_3_test
lpc_params_test_suite ‑ lpc_different_hash_types_test

@github-actions GitHub Actions
Copy link

Gcc Test Results

  151 files   -  7    151 suites   - 7   12m 6s ⏱️ -6s
3 176 tests  - 49  3 170 ✅  - 49  6 💤 ±0  0 ❌ ±0 
3 345 runs   - 49  3 339 ✅  - 49  6 💤 ±0  0 ❌ ±0 

Results for commit 8f7526d. ± Comparison against base commit 0ae4dc1.

This pull request removes 53 and adds 4 tests. Note that renamed tests count towards both. 
bls_serialization ‑ g1_serialization_test
bls_serialization ‑ g2_serialization_test
bls_signature_public_interface_tests ‑ bls_aug_mss
bls_signature_public_interface_tests ‑ bls_basic_mps
bls_signature_public_interface_tests ‑ bls_basic_mss
bls_signature_public_interface_tests ‑ bls_pop_mss
chacha_test_suite.chacha_single_range_encrypt ‑ _0
chacha_test_suite.chacha_single_range_encrypt ‑ _1
chacha_test_suite.chacha_single_range_encrypt ‑ _2
chacha_test_suite.chacha_single_range_encrypt ‑ _3
…

lpc_math_polynomial_suite ‑ lpc_basic_skipping_layers_test
lpc_math_polynomial_suite ‑ lpc_dfs_basic_test
lpc_params_test_suite ‑ lpc_batches_num_3_test
lpc_params_test_suite ‑ lpc_different_hash_types_test

@vo-nil vo-nil merged commit a07d5a3 into master Oct 17, 2024
10 checks passed
@vo-nil vo-nil deleted the 93-remove-unused-libs branch October 17, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martun martun martun approved these changes

Assignees

@vo-nil vo-nil

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove unused libraries from crypto3
2 participants
@vo-nil @martun