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

Update CBMC starter kit #59

Merged
merged 4 commits into from
Jul 14, 2022
Merged

Update CBMC starter kit #59

merged 4 commits into from
Jul 14, 2022

Conversation

angelonakos
Copy link
Contributor

@angelonakos angelonakos commented Jul 13, 2022

Issue #, if available:

Description of changes:
This PR:

  • changes the SPDX identifier in CBMC proof Makefile from Apache-2.0 to MIT-0
  • introduces updates to the files related to the CBMC starter kit (this was accomplished by running cbmc-starter-kit-update).
  • removes 2 .git submodules (this was accomplished by running cbmc-starter-kit-update --remove-starter-kit-submodule --remove-litani-submodule). symlinks to files present in the CBMC starter-kit submodule are no longer present Additionally, a user will have to install litani (instructions here) in order to run CBMC proofs locally. litani is present in the CI system that executes the CBMC proofs.
  • updates the README to include a section on CBMC.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

archigup
archigup previously approved these changes Jul 13, 2022
paulbartell
paulbartell previously approved these changes Jul 14, 2022
@angelonakos angelonakos dismissed stale reviews from paulbartell and archigup via 78a8230 July 14, 2022 12:54
@angelonakos
Copy link
Contributor Author

The latest force-push concerned solely the SPDX identifier change.

@paulbartell paulbartell merged commit c16275a into aws:main Jul 14, 2022
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.

3 participants