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

Sync MQTT files from C-SDK for commit e60683b #1

Merged
merged 4 commits into from
Aug 27, 2020

Conversation

aggarw13
Copy link
Contributor

@aggarw13 aggarw13 commented Aug 26, 2020

Copy all files under aws-iot-device-sdk-embedded-C/tree/development/libraries/standard/mqtt folder for commit ID e60683b

Note: Subsequent PRs will bring in third-party tool dependencies, re-add LICENSE/README/CONTRIBUTING files, re-structure folder, rename files and remove integration tests, and provide functioning CMake, doc and CBMC targets.

Diff with commit on C-SDK repository:
The only difference between mqtt folder in C-SDK and this PR is the presence of the CBMC REAMDE files. These files are symlinked in C-SDK, and these symlinks will be enabled in a subsequent PR that brings CBMC third-party dependency

╭─ubuntu@ip-172-31-24-209 ~/Repos/coreMQTT  ‹sync/c-sdk-commit-d36047b› 
╰─➤  diff -r ../aws-iot-device-sdk-embedded-C/libraries/standard/mqtt/ .
Only in .: .git
diff: ./cbmc/include/README.md: No such file or directory
diff: ./cbmc/proofs/Makefile.common: No such file or directory
diff: ./cbmc/proofs/README.md: No such file or directory
diff: ./cbmc/proofs/prepare.py: No such file or directory
diff: ./cbmc/sources/README.md: No such file or directory
diff: ./cbmc/stubs/README.md: No such file or directory

@@ -1,4 +0,0 @@
## Code of Conduct
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to delete these files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will re-add these files in a separate PR with the correct content.
The purpose of this PR is to only have the master branch contain the same code as that under libraries/standard/mqtt from C-SDK

@aggarw13 aggarw13 changed the title Sync MQTT files from C-SDK for commit d36047 Sync MQTT files from C-SDK for commit e60683b Aug 27, 2020
@aggarw13 aggarw13 merged commit e0cc0f8 into master Aug 27, 2020
@aggarw13 aggarw13 deleted the sync/c-sdk-commit-d36047b branch August 27, 2020 17:34
AniruddhaKanhere pushed a commit to AniruddhaKanhere/coreMQTT that referenced this pull request Oct 22, 2024
Add opaque struct and functions to serialize it.
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