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

[nasa/cryptolib#379] Updated to gcc-14 for MC/DC #381

Merged
merged 10 commits into from
Jan 7, 2025

Conversation

Donnie-Ice
Copy link
Contributor

Updated for CAP MC/DC

@Donnie-Ice Donnie-Ice self-assigned this Jan 3, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.97%. Comparing base (4567cee) to head (14ee573).
Report is 40 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #381      +/-   ##
==========================================
- Coverage   76.53%   75.97%   -0.57%     
==========================================
  Files          81       81              
  Lines       22041    22263     +222     
  Branches     1777     1807      +30     
==========================================
+ Hits        16870    16914      +44     
- Misses       4531     4707     +176     
- Partials      640      642       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Donnie-Ice Donnie-Ice changed the title [nasa/cryptolib#379] Updated to gcc-14 for MC/DC WIP: [nasa/cryptolib#379] Updated to gcc-14 for MC/DC Jan 3, 2025
@Donnie-Ice Donnie-Ice changed the title WIP: [nasa/cryptolib#379] Updated to gcc-14 for MC/DC [nasa/cryptolib#379] Updated to gcc-14 for MC/DC Jan 3, 2025
@Donnie-Ice
Copy link
Contributor Author

Donnie-Ice commented Jan 3, 2025

Had to restore functionality to CI. The docker file is written for gcc/gcov-14 to work with MCDC. It is being used by CodeQL and [#258], but the build.yml is still using the old docker file in dev.

@@ -17,19 +17,21 @@
#

ARG WOLFSSL_VERSION=5.6.0-stable
FROM ubuntu:jammy-20240530 AS cl0
FROM ubuntu:latest AS cl0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we lock this to whatever the latest actual release is?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

@Donnie-Ice Donnie-Ice merged commit 1e8ab55 into dev Jan 7, 2025
12 checks passed
@Donnie-Ice Donnie-Ice deleted the 379-update-cryptolib-dockerfile branch January 7, 2025 16:13
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.

4 participants