Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

kraft.yaml: Update configuration to use Musl #5

Closed
wants to merge 2 commits into from

Conversation

razvand
Copy link
Contributor

@razvand razvand commented Nov 24, 2022

Update configuration to use Musl, instead of newlib and pthread-embedded

Copy link

@mariasfiraiala mariasfiraiala left a comment

Choose a reason for hiding this comment

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

Looks good to me, however, I believe that also changing the unikraft version and adding ARM as an architecture would make sense.

@razvand
Copy link
Contributor Author

razvand commented Nov 28, 2022

Looks good to me, however, I believe that also changing the unikraft version and adding ARM as an architecture would make sense.

Thanks, @mariasfiraiala . Fixed.

@razvand razvand added the enhancement New feature or request label Nov 28, 2022
Copy link

@mariasfiraiala mariasfiraiala left a comment

Choose a reason for hiding this comment

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

Works, thanks @razvand !

Reviewed-by: Maria Sfiraiala maria.sfiraiala@gmail.com

Add `arm64` architecture and `kvm` platform as target.

Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Replace `newlibc` and `pthread-embedded` libraries with `musl`.

Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Copy link
Member

@nderjung nderjung left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks

Approved-by: Alexander Jung alex@unikraft.io

unikraft-bot pushed a commit that referenced this pull request Nov 29, 2022
Replace `newlibc` and `pthread-embedded` libraries with `musl`.

Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
Reviewed-by: Maria Sfiraiala <maria.sfiraiala@gmail.com>
Approved-by: Alexander Jung <alex@unikraft.io>
Tested-by: Unikraft CI <monkey@unikraft.io>
GitHub-Closes: #5
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ci/merged enhancement New feature or request
Projects
Status: Done
Status: Done!
Development

Successfully merging this pull request may close these issues.

None yet

4 participants