docs: sync install-dependencies.md & polish it #3467
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: spacewander <[email protected]>
Signed-off-by: spacewander <[email protected]>
Yiyiyimu
approved these changes
Feb 1, 2021
membphis
approved these changes
Feb 1, 2021
doc/zh-cn/install-dependencies.md
Outdated
| @@ -54,7 +52,10 @@ sudo yum install yum-utils | |||
| sudo yum-config-manager --add-repo https://openresty.org/package/centos/openresty.repo | |||
|
|
|||
| # 安装 OpenResty 和 编译工具 | |||
| sudo yum install -y openresty curl git gcc luarocks lua-devel | |||
| sudo yum install -y openresty curl git gcc lua-devel openresty-openssl-devel | |||
There was a problem hiding this comment.
can we use the luajit of openresty? if yes, we do not need to install the lua-devel.
Signed-off-by: spacewander <[email protected]>
|
When I follow the installation steps and install luarocks with script, I got the following error: Environment: Ubuntu 18.04, on Windows (WSL2) |
|
The |
Signed-off-by: spacewander <[email protected]>
spacewander
force-pushed
the
instal-dependencies
branch
from
c03cbc8 to
af99e57
Compare
tokers
reviewed
Feb 1, 2021
| - By default Apache APISIX runs with LuaJIT of OpenResty 1.19 (priority) or Lua 5.1. If you run in to an issue `luajit: lj_asm_x86.h:2819: asm_loop_fixup: Assertion '((intptr_t)target & 15) == 0' failed`, it is caused by to the compatibility of OpenResty version. OpenResty 1.19 is recommended, please take a look at this installation step script [linux-install-openresty](../utils/linux-install-openresty.sh). | ||
| - If it is OpenResty 1.19, APISIX will use OpenResty's built-in LuaJIT to run `bin/apisix`; otherwise it will use Lua 5.1. If you encounter `luajit: lj_asm_x86.h:2819: asm_loop_ fixup: Assertion '((intptr_t)target & 15) == 0' failed`, this is a problem with the low version of OpenResty's built-in LuaJIT under certain compilation conditions. | ||
|
|
||
| - - On some platforms, installing LuaRocks via the package manager will cause Lua to be upgraded to Lua 5.3, so we recommend installing LuaRocks via source code. if you install OpenResty and its OpenSSL develop library (openresty-openssl-devel for rpm and openresty-openssl-dev for deb) via the official repository, then [we provide a script for automatic installation](. /... /utils/linux-install-luarocks.sh). If you compile OpenResty yourself, you can refer to the above script and change the path in it. If you don't specify the OpenSSL library path when you compile, you don't need to configure the OpenSSL variables in LuaRocks, because the system's OpenSSL is used by default. If the OpenSSL library is specified at compile time, then you need to ensure that LuaRocks' OpenSSL configuration is consistent with OpenResty's. |
There was a problem hiding this comment.
We may also give some tips about how to configuring the OpenSSL configuration for LuaRocks.
There was a problem hiding this comment.
Read the script is enough. Should not repeat yourself.
Signed-off-by: spacewander <[email protected]>
spacewander
force-pushed
the
instal-dependencies
branch
from
40fdde4 to
871ecd1
Compare
membphis
approved these changes
Feb 2, 2021
spacewander
added a commit
that referenced
this pull request
Feb 3, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: spacewander [email protected]
What this PR does / why we need it:
Pre-submission checklist: