Skip to content

CMake: Add CMakeLists.txt #284

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Nov 24, 2020
Merged

CMake: Add CMakeLists.txt #284

merged 3 commits into from
Nov 24, 2020

Conversation

rajkan01
Copy link
Contributor

@rajkan01 rajkan01 commented Nov 16, 2020 •
edited
Loading

  • Added CMakeLists.txt file to TLS examples so that it can build via CMake
  • Added .travis.yml config to pick this example for Travis CI build
  • Update mbed-os.lib to latest master to pick CMake changes

@0xc0170 @hugueskamba

@rajkan01 rajkan01 force-pushed the add_cmakelists branch 2 times, most recently from 74e2a2f to 2d46855 Compare November 16, 2020 11:49
hugueskamba
hugueskamba reviewed Nov 16, 2020
View reviewed changes
authcrypt/CMakeLists.txt
target_sources(${APP_TARGET}
PRIVATE
authcrypt.cpp
main.cpp
Copy link
Contributor

Choose a reason for hiding this comment

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

Please fix indentation.

tls-client/CMakeLists.txt
target_sources(${APP_TARGET}
PRIVATE
HelloHttpsClient.cpp
main.cpp
Copy link
Contributor

Choose a reason for hiding this comment

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

Please fix indentation.

@rajkan01 rajkan01 requested a review from hugueskamba November 16, 2020 12:50
@rajkan01 rajkan01 force-pushed the add_cmakelists branch 2 times, most recently from edb8610 to 5a38ad7 Compare November 20, 2020 15:14
@rajkan01
Copy link
Contributor Author

@Patater Could you review this PR for adding CMake support to tls example

@Patater Patater merged commit 24cfbb6 into ARMmbed:development Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Patater Patater Patater approved these changes

@0xc0170 0xc0170 0xc0170 approved these changes

@hugueskamba hugueskamba hugueskamba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rajkan01 @Patater @0xc0170 @hugueskamba