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

Update the source for building crun #224

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

CaptainVincent
Copy link
Member

The nnpreload and plugin loading mechanisms are already in the upstream.

Explanation

Replace demo crun source.

Related issue

WasmEdge/WasmEdge#3217

What type of PR is this

Documentation

Proposed Changes

Update repo link

The nnpreload and plugin loading mechanisms are already in the upstream.

Signed-off-by: vincent <vincent@secondstate.io>
Copy link
Collaborator

alabulei1 commented Apr 3, 2024

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


Commit 873abc5623e0c4e9080eb75c479fdd5ac4cc29c0

Key Changes:

  1. Updates were made to the source for building crun - specifically, the nnpreload and plugin loading mechanisms are already in the upstream.
  2. The patch modified the instructions for building crun, specifying building with wasmedge enabled only, rather than with both wasmedge and plugin systems enabled.
  3. The patch made minor adjustments in the documentation for setting up the container runtime involving crun, wasmedge, and the plugin system.

Potential Problems:

  1. The patch changes the build process for crun to only enable wasmedge, without mentioning the importance or necessity of enabling the plugin system. This could potentially lead to inconsistency or issues if the plugin system is required for certain functionality.
  2. The patch includes some inconsistencies in the naming conventions used in the code snippets, which could lead to confusion or errors during implementation.
  3. The patch does not provide detailed information on why the decision was made to update the build process for crun in this specific way, potentially missing important context for reviewers or users.

@hydai hydai merged commit bb20170 into WasmEdge:main Apr 3, 2024
6 checks passed
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