-
Notifications
You must be signed in to change notification settings - Fork 30
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 default configuration file #22
Add default configuration file #22
Conversation
f7553e9
to
df1e8e7
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.
This looks good.
Please also add a commit that updates the README.md
file with detailed instructions, similar with the README.md
of app-helloworld
, including the TLDR instructions. Leave out KraftKit.
Corresponding contents from this PR would be moved as part of the README.md
file, and keep a reference to it.
Similar to what we have in other applications, add a default minimal config file that can be used by `make defconfig` to generate a proper full configuration. Also update the Makefile to match the new directory layout. Signed-off-by: Stefan Jumarea <stefanjumarea02@gmail.com>
Update the .gitignore to ignore the .unikraft/ directory, which is the output of the setup stage. Signed-off-by: Stefan Jumarea <stefanjumarea02@gmail.com>
df1e8e7
to
304cc10
Compare
Done @razvand. Since this is an application with a more advanced purpose, and basic users will just use the pre-built elfloader images, I left the running / debugging instructions in the |
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 my comments
Update the `REAMDE.md` file with instructions on how to setup, configure and build the `app-elfloader`, using the already provided minimal configuration file. Signed-off-by: Stefan Jumarea <stefanjumarea02@gmail.com>
304cc10
to
3de89de
Compare
Done @razvand |
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.
Reviewed-by: Razvan Deaconescu razvand@unikraft.io
Approved-by: Razvan Deaconescu razvand@unikraft.io
Update the .gitignore to ignore the .unikraft/ directory, which is the output of the setup stage. Signed-off-by: Stefan Jumarea <stefanjumarea02@gmail.com> Reviewed-by: Razvan Deaconescu <razvand@unikraft.io> Approved-by: Razvan Deaconescu <razvand@unikraft.io> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: #22
Update the `REAMDE.md` file with instructions on how to setup, configure and build the `app-elfloader`, using the already provided minimal configuration file. Signed-off-by: Stefan Jumarea <stefanjumarea02@gmail.com> Reviewed-by: Razvan Deaconescu <razvand@unikraft.io> Approved-by: Razvan Deaconescu <razvand@unikraft.io> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: #22
No description provided.