-
Notifications
You must be signed in to change notification settings - Fork 34
Introduce Lua 5.4 application to catalog-core #60
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
base: main
Are you sure you want to change the base?
Conversation
razvand
left a comment
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.
Use a single commit. Squash the two current commits into a single one.
|
All feedback applied ! |
53372c8 to
2ee68a8
Compare
razvand
left a comment
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.
Update your author and signed-off-by name to be properly written, i.e. use Dana Caruntu <[email protected]> instead of Dana Caruntu <[email protected]>.
razvand
left a comment
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.
Add instructions on building the filesystem.
razvand
left a comment
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.
Add instructions on building the filesystem.
c9cd4c7 to
834751b
Compare
9f7ecd4 to
c98432c
Compare
razvand
left a comment
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.
Update the author (
git config --global user.name 'Dana Caruntu'
git config --global user.email '[email protected]'
git commit --amend --reset-author
You have two signed-off lines. Leave only one.
razvand
left a comment
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.
There is no rootfs/ directory (with the helloworld.lua file inside of it).
e41a383 to
ea08759
Compare
razvand
left a comment
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.
Update your commit message to not mention the .gitignore update in the message description. Do add a reference to the .gitignore file being added. Take inspiration from the commit message introducing python3-hello. See here.
Add Lua 5.4 application Add `lua/` directory: - `Config.uk`: application configuration - `Makefile`: for building the application - `Makefile.uk`: empty placeholder required by the build system - `README.md`: instructions - `setup.sh`: script to initialize and populate the root filesystem - `rootfs/helloworld.lua`: the Lua Hello program - `fc.x86_64.json` / `fc.arm64.json`: Firecracker configurations for x86_64 and ARM64 - `xen.x86_64.cfg` / `xen.arm64.cfg`: Xen configurations for x86_64 and ARM64 - `.gitignore`: ignore generated files Signed-off-by: Dana Caruntu <[email protected]>
What’s included
Unikraft-specific configuration and build scripts for the Lua unikernel
Top-level wrappers to bootstrap the build (
make all,make distclean, etc.)A minimal Lua script that prints “Hello from Lua!”
How to configure, build and run (QEMU/Xen) the Lua unikernel
fc.x86_64.json,fc.arm64.jsonfor functional testsxen.x86_64.cfg,xen.arm64.cfgfor Xen targets