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 c++ helloworld example application #4

Merged
merged 1 commit into from
Jan 3, 2023

Conversation

huyhoang8398
Copy link

add c++ helloworld example programs

@razvand
Copy link
Collaborator

razvand commented May 15, 2022

Please use Add ... as part of your commit message.

@huyhoang8398 huyhoang8398 changed the title c++ helloworld example application Add c++ helloworld example application May 15, 2022
lang/c++/Makefile Outdated Show resolved Hide resolved
lang/c++/Makefile Outdated Show resolved Hide resolved
lang/c++/Makefile Show resolved Hide resolved
lang/c++/Makefile Outdated Show resolved Hide resolved
Copy link
Collaborator

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Thanks, @huyhoang8398 . I made some comments.

Apart from those, please:

  • Add the helloworld executable file to the commit.
  • Remove the C folder from the commit.

lang/c++/Makefile Show resolved Hide resolved
lang/c++/Makefile Outdated Show resolved Hide resolved
lang/c++/Makefile Outdated Show resolved Hide resolved
lang/c++/Makefile Outdated Show resolved Hide resolved
lang/c++/helloworld.cpp Outdated Show resolved Hide resolved
lang/c++/helloworld.cpp Outdated Show resolved Hide resolved
Add C++ helloworld application as static PIE: add source code, binary
executable and Makefile.

Signed-off-by: Do Hoang <huyhoang8398@gmail.com>
Co-authored-by: Razvan Deaconescu <razvand@unikraft.io
Copy link
Collaborator

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Thanks, @huyhoang8398.

Reviewed-by: Razvan Deaconescu razvand@unikraft.io
Approved-by: Razvan Deaconescu razvand@unikraft.io

@razvand razvand merged commit 2beed15 into unikraft:master Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants