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 installation description for wasmedge_rustls_plugin #176

Merged

Conversation

moznion
Copy link
Contributor

@moznion moznion commented Oct 20, 2023

Explanation

Starting with WasmEdge v0.13.4, the wasmedge_rustls_plugin has been bundled within the WasmEdge core (ref: https://github.com/WasmEdge/WasmEdge/releases/tag/0.13.4). Therefore, this patch modifies the installation instructions for rustls_plugin to indicate that it should be retrieved from the WasmEdge/WasmEdge repository, rather than from the second-state/wasmedge_rustls_plugin.

Related issue

This might be related to #108

What type of PR is this

/kind documentation

Starting with WasmEdge v0.13.4, the `wasmedge_rustls_plugin` has been bundled within
the WasmEdge core (ref: https://github.com/WasmEdge/WasmEdge/releases/tag/0.13.4).
Therefore, this patch modifies the installation instructions for rustls_plugin to
indicate that it should be retrieved from the `WasmEdge/WasmEdge` repository,
rather than from the `second-state/wasmedge_rustls_plugin`.

Signed-off-by: moznion <[email protected]>
Copy link
Collaborator

alabulei1 commented Oct 20, 2023

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


Overall, the patch successfully updates the installation description for wasmedge_rustls_plugin and fixes an incorrect file name in the installation command. No other potential problems were identified.

Details

Commit 903f58db89dc96efd4863e2e1cf6a4217b54b971

Key changes in the patch:

  • The installation instructions for the wasmedge_rustls_plugin have been updated.
  • The plugin is now bundled within the WasmEdge core starting from version 0.13.4.
  • The download link for the plugin has been changed to point to the WasmEdge/WasmEdge repository.
  • The file names for the downloaded plugin have been modified.
  • The copy commands for installing the plugin have been adjusted accordingly.

Potential problems:

  • None of the changes in this patch seem to introduce any potential problems.

Overall, the patch updates the installation instructions for the wasmedge_rustls_plugin to reflect the new bundled version. It also updates the download link and file names accordingly.

Commit ec4fe3861f74ecd7f3b3e1120eb0d176dea7c60f

Key changes in the patch:

  • Updated the installation description for wasmedge_rustls_plugin.
  • Changed the file name from libwasmedge_rustls to libwasmedge_rustls.so in the sudo cp command.

Potential problems:

  • It seems that the file name in the installation description was incorrect (libwasmedge_rustls) before the patch. The patch fixes this by changing it to the correct file name (libwasmedge_rustls.so).
  • No other potential problems were identified in the patch.

@adithyaakrishna
Copy link
Contributor

Looks good to me, @hydai Could you please review this PR? :)

docs/start/install.md Outdated Show resolved Hide resolved
@moznion
Copy link
Contributor Author

moznion commented Oct 22, 2023

@hydai Thank you for pointing out. I corrected them at ec4fe38.

@moznion moznion requested a review from hydai October 22, 2023 05:35
Copy link
Member

@hydai hydai left a comment

Choose a reason for hiding this comment

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

Thanks

@hydai hydai merged commit a4dd7ba into WasmEdge:main Oct 22, 2023
6 checks passed
@moznion moznion deleted the correct_wasmedge_rustls_plugin_installation branch October 22, 2023 16:41
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.

4 participants