-
Notifications
You must be signed in to change notification settings - Fork 284
Upgrade catch version to 2.13.6 #6018
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
Upgrade catch version to 2.13.6 #6018
Conversation
7afabfc to
8a5ec47
Compare
|
Could you please upgrade to v2.13.5 and mark the commit as |
|
@tautschnig I will look into this, yes. |
8a5ec47 to
12a5eb6
Compare
Downloaded through https://github.com/catchorg/Catch2/releases/download/v2.13.6/catch.hpp The reason for the upgrade is a problem identified when attempting to build CBMC using CMAKE under a m1-aarch64 environment. Fixes diffblue#5840.
5bf9a5b to
f381498
Compare
A lot of the comparisons we did previously seemed unsound - they compared integers of different signs. Catch didn't like this much, forcing us to change sign comparisons to be explicit.
72e8106 to
38a3ea0
Compare
|
Hi @tautschnig, can you take a look at the last commit of this PR please? As a sanity check. |
Codecov Report
@@ Coverage Diff @@
## develop #6018 +/- ##
===========================================
+ Coverage 67.40% 75.11% +7.70%
===========================================
Files 1157 1450 +293
Lines 95236 159541 +64305
===========================================
+ Hits 64197 119832 +55635
- Misses 31039 39709 +8670
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
The rationale behind the upgrading of the version of catch is to overcome a build issue on
m1/aarch64when building undercmake.The exact error during the build is the following, without this change is: