Skip to content

Conversation

@Dana-Maria04
Copy link

What’s included

  • Config.uk & Makefile.uk
    Unikraft-specific configuration and build scripts for the Lua unikernel
  • Makefile & setup.sh
    Top-level wrappers to bootstrap the build (make all, make distclean, etc.)
  • helloworld.lua in rootfs/
    A minimal Lua script that prints “Hello from Lua!”
  • README.md
    How to configure, build and run (QEMU/Xen) the Lua unikernel
  • Defconfigs
    • fc.x86_64.json, fc.arm64.json for functional tests
    • xen.x86_64.cfg, xen.arm64.cfg for Xen targets

@razvand razvand self-requested a review May 17, 2025 06:10
@razvand razvand self-assigned this May 17, 2025
Copy link
Contributor

@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.

Use a single commit. Squash the two current commits into a single one.

@Dana-Maria04
Copy link
Author

All feedback applied !

@Dana-Maria04 Dana-Maria04 requested a review from razvand May 17, 2025 12:16
@Dana-Maria04 Dana-Maria04 force-pushed the add-lua-app branch 3 times, most recently from 53372c8 to 2ee68a8 Compare May 17, 2025 14:00
Copy link
Contributor

@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.

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]>.

Copy link
Contributor

@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.

Add instructions on building the filesystem.

Copy link
Contributor

@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.

Add instructions on building the filesystem.

@Dana-Maria04 Dana-Maria04 force-pushed the add-lua-app branch 2 times, most recently from c9cd4c7 to 834751b Compare May 17, 2025 14:40
@Dana-Maria04 Dana-Maria04 requested a review from razvand May 17, 2025 14:41
@Dana-Maria04 Dana-Maria04 force-pushed the add-lua-app branch 5 times, most recently from 9f7ecd4 to c98432c Compare May 17, 2025 15:55
Copy link
Contributor

@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.

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.

Copy link
Contributor

@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.

There is no rootfs/ directory (with the helloworld.lua file inside of it).

@Dana-Maria04 Dana-Maria04 force-pushed the add-lua-app branch 2 times, most recently from e41a383 to ea08759 Compare May 17, 2025 16:42
@Dana-Maria04 Dana-Maria04 requested a review from razvand May 17, 2025 16:42
@Dana-Maria04 Dana-Maria04 changed the title Add Lua 5.4 application to catalog-core Introduce Lua 5.4 application to catalog-core May 17, 2025
Copy link
Contributor

@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.

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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants