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

Add missing cleanup #7798

Merged
merged 5 commits into from
Aug 9, 2023
Merged

Conversation

tautschnig
Copy link
Collaborator

@tautschnig tautschnig commented Jul 7, 2023

make clean did not actually remove all files.

  • Each commit message has a non-empty body, explaining why the change was made.
  • n/a Methods or procedures I have added are documented, following the guidelines provided in CODING_STANDARD.md.
  • n/a The feature or user visible behaviour I have added or modified has been documented in the User Guide in doc/cprover-manual/
  • Regression or unit tests are included, or existing tests cover the modified code (in this case I have detailed which ones those are in the commit message).
  • n/a My commit message includes data points confirming performance improvements (if claimed).
  • My PR is restricted to a single feature or bugfix.
  • n/a White-space or formatting changes outside the feature-related changed lines are in commits of their own.

@codecov
Copy link

codecov bot commented Jul 8, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (eb20424) 78.68% compared to head (da99314) 78.68%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7798      +/-   ##
===========================================
- Coverage    78.68%   78.68%   -0.01%     
===========================================
  Files         1699     1699              
  Lines       194172   194172              
===========================================
- Hits        152791   152790       -1     
- Misses       41381    41382       +1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@tautschnig tautschnig force-pushed the cleanup/cleaning branch 8 times, most recently from 91bdd2a to 569c28e Compare July 17, 2023 20:59
@tautschnig tautschnig force-pushed the cleanup/cleaning branch 6 times, most recently from c59a15a to 2a1444f Compare July 18, 2023 18:35
Ensure both build infrastructures remain in sync.
These tests should also be run when using Makefile-based builds, and
Makefiles are the only way to perform cleanup.
Ensure that both CMake and Make run the same set of tests, and
facilitate cleanup after running tests (for either build system).
make clean did not actually remove all files. Includes disabling an
export-symex-ready-goto test with --xml-ui for check.py will wrap the
`""` argument and make the test produce an actual goto binary in a file
named `""`.
Make sure "clean" actually does a full cleanup.
@tautschnig tautschnig merged commit 1e8a703 into diffblue:develop Aug 9, 2023
34 of 35 checks passed
@tautschnig tautschnig deleted the cleanup/cleaning branch August 9, 2023 08:10
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.

7 participants