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

Improve Unikraft support in build files #9

Merged

Conversation

razvand
Copy link

@razvand razvand commented Sep 28, 2023

Some parts of the build process are error-prone and do not perfectly fit into the Unikraft build model. Update Makefile and Makefile.unikernel to improve the build process for Unikraft.

Replace error-prone copying of configuration file with the use of a
defconfig file. Add `defconfigs/qemu-x86_64` and use `make defconfig`
option in `Makefile.unikernel`.

Signed-off-by: Razvan Deaconescu <[email protected]>
Move file generation rules (.config, model.h, tokenizer.h) out of
`Makefile.unikernel` and into `Makefile`. This simplifies
`Makefile.unikernel` making it Unikraft-specific. And making it easy to
reuse as part of `make -f Makefile.unikraft ...` rules.

Signed-off-by: Razvan Deaconescu <[email protected]>
@razvand razvand force-pushed the razvand/improve-unikraft-support branch from 91a2d19 to 9d51600 Compare October 8, 2023 14:58
Copy link
Owner

@trholding trholding left a comment

Choose a reason for hiding this comment

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

I am merging this. I will make small changes later as this pull originates before the last update to the makefile.

@trholding trholding merged commit 5f5763d into trholding:master Oct 9, 2023
6 checks passed
@trholding
Copy link
Owner

Thank you!

@razvand razvand deleted the razvand/improve-unikraft-support branch June 11, 2024 17:06
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.

None yet

2 participants