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

docs: update youki build instructions #232

Merged
merged 1 commit into from
Jun 9, 2024
Merged

docs: update youki build instructions #232

merged 1 commit into from
Jun 9, 2024

Conversation

hegerdes
Copy link
Contributor

@hegerdes hegerdes commented Jun 9, 2024

Explanation

Youki changed its build system form make to just.

Both are not really needed to build youki with wasm since you can just run the build script. Updated the instructions and added missing apt packages for wasm like libzstd-dev, curl and git

Related issue

Does not exist yet

What type of PR is this

/kind documentation

Proposed Changes

Update the build instructions

Copy link
Collaborator

alabulei1 commented Jun 9, 2024

Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.


Commit 301d66a5feceedf188ef4a61096ecf7ed60cbbfe

Key Changes:

  1. Updated build instructions for youki to reflect a change from using make to executing a build script.
  2. Added missing apt packages for wasm support, such as libzstd-dev, curl, and git.
  3. Included a command to install the Rust toolchain if not already installed.

Potential Problems:

  1. The instruction to run curl https://sh.rustup.rs -sSf | sudo sh -s -- -y to install the Rust toolchain with sudo privileges may pose security risks. It is generally recommended to avoid running scripts directly from the internet with sudo.
  2. The patch should include information on the potential impacts or risks involved in changing the build system from make to executing a build script for users who are familiar with the previous build process.
  3. The patch doesn't specify the purpose of the changes or any additional context, which could be helpful for reviewers to understand the rationale behind the modifications.

@hydai
Copy link
Member

hydai commented Jun 9, 2024

Hi @hegerdes
Could you please sign your commits with DCO? Ref: https://www.secondstate.io/articles/dco/

Youki changed its build system form make to just.

Both are not really needed to build youki with wasm since you can just run the build script. Updated the instructions and added missing apt packages for wasm like `libzstd-dev`, `curl` and `git`

Signed-off-by: Henrik Gerdes <[email protected]>
@hegerdes
Copy link
Contributor Author

hegerdes commented Jun 9, 2024

Thought of trying out GitHub UI for this small change - gives the source branch a meaningless name and does not sign-off.

Lernend my lesson, local git form now on only with my known config. Is now fixed.

@hydai hydai merged commit 2f6bf43 into WasmEdge:main Jun 9, 2024
6 checks passed
@hydai
Copy link
Member

hydai commented Jun 9, 2024

Thanks

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.

3 participants