Closed dhruvmehtaaa closed 10 months ago
Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.
Key changes:
crun
with WasmEdge support.crun
before replacing it.Potential problems:
git
, autogen.sh
, make
, and sudo
installed. It would be helpful to provide instructions or dependencies for those tools if they are not already installed.Overall, the changes seem to be straightforward and valuable. The patch provides clear instructions for building and installing crun
with WasmEdge support, and includes a warning to backup the existing crun
binary before replacing it.
Hi @juntao Can you please review
Thank you. Can you rebase to remove my commit (the install file) in the PR? Thanks!
@juntao I have rebased to remove your commit
@juntao I think we can merge this PR?
Explanation
Step 3 of https://wasmedge.org/docs/develop/deploy/podman/#prerequisites: It seems like some instructions are missing.
Build and configure crun with WasmEdge support
Next, configure and build a crun binary with WasmEdge support.
Maybe there should be a git clone, similar to https://wasmedge.org/docs/develop/deploy/oci-runtime/crun/#prerequisites
Related issue
Fixes #195 Fixes https://github.com/WasmEdge/WasmEdge/issues/3028
What type of PR is this
/kind documentation
Proposed Changes
Adds the missing git clone in step 3 of https://wasmedge.org/docs/develop/deploy/podman/#prerequisites Also fixed a typo