-
Notifications
You must be signed in to change notification settings - Fork 16
Add configuration and scripts #16
Add configuration and scripts #16
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good with me besides that one small comment. Thanks @razvand
For building and running everything for `x86_64`, follow the steps below: | ||
|
||
```console | ||
git clone https://github.com/unikraft/app-helloworld-cpp helloworld-cpp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to separate the common commands (the ones with cloning the repos) from the one specific to platform / architectures.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see your point. But, currently, this is how we did for all repositories. I would keep it like this, and, if we think it's a good idea, we will update it all over the place.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, fair point. Then let's merge like it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good now.
Waiting for #13 to be merged first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the comments.
@StefanJum, fixed. |
f5431fa
to
17d935c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good, I'll add the tag after #13 is merged and this is rebased.
Ignore output files and directories. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Add configuration files for Make-based platform / architecture combinations. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Add basic Makefile for Make-based setup. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Explicitly enable depending libraries. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Add information on using KraftKit and on using the Make-based build system. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
17d935c
to
b48dd6a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved-by: Stefan Jumarea stefanjumarea02@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good with rebase. Thanks, @razvand!
Reviewed-by: Radu Nichita radunichita99@gmail.com
Add configuration files for Make-based platform / architecture combinations. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io> Reviewed-by: Radu Nichita <radunichita99@gmail.com> Approved-by: Stefan Jumarea <stefanjumarea02@gmail.com> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: #16
Add basic Makefile for Make-based setup. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io> Reviewed-by: Radu Nichita <radunichita99@gmail.com> Approved-by: Stefan Jumarea <stefanjumarea02@gmail.com> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: #16
Explicitly enable depending libraries. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io> Reviewed-by: Radu Nichita <radunichita99@gmail.com> Approved-by: Stefan Jumarea <stefanjumarea02@gmail.com> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: #16
Add information on using KraftKit and on using the Make-based build system. Signed-off-by: Razvan Deaconescu <razvand@unikraft.io> Reviewed-by: Radu Nichita <radunichita99@gmail.com> Approved-by: Stefan Jumarea <stefanjumarea02@gmail.com> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: #16
Add Make-based setup files.