shuaijinchao commented on a change in pull request #6442:
URL: https://github.com/apache/apisix/pull/6442#discussion_r816535374



##########
File path: docs/en/latest/how-to-build.md
##########
@@ -117,20 +117,20 @@ Follow the steps below to install Apache APISIX via the 
source release package.
   make install
   ```
 
-  **Note**: If `make deps` fails with "install `lualdap` failed" with an error 
like `Could not find header file for LDAP` try the solution below.
-
-      Solution: luarocks supports adding custom package filepaths to the 
project. You can set `LDAP_DIR` with `luarocks config` manually, for example 
`luarocks config variables.LDAP_DIR /usr/local/opt/openldap/`. You should 
config as follows:
-      1. Install `openldap` locally using tools including but not limited to 
'brew';
-      2. Locate the 'openldap' local installation directory. If you are macOS 
and use `brew` to install `openldap`, you can find the local installation 
directory with the `brew --prefix openldap` command.
-      3. Manually set the `LDAP_DIR` variable via `luarocks config`, e.g. 
`luarocks config variables.LDAP_DIR /usr/local/opt/openldap/`.
-      4. Of course, you can also choose to change the default configuration 
file of luarocks directly, execute the 'cat ~/.luarocks/config-5.1.lua' 
command, and then add the installation directory of 'openldap' to the file. 
Example as follows:
-      ```lua
-      variables = {
-          LDAP_DIR = "/opt/homebrew/cellar/openldap/2.6.1",
-          LDAP_INCDIR = "/opt/homebrew/cellar/openldap/2.6.1/include",
-      }
-      ```
-      If you have similar problems installing other dependencies, you can 
refer to this solution as well.
+  **Note**: It will definitely be a problem set of dependency failed 
installation, If you fail to install with `make deps` and get an error message 
like `Could not find header file for LDAP`. Similarly, you can use this method 
to solve installation problems such as `Could not find header file for PCRE` or 
others.
+
+  Solution for macOS:
+
+    1. Install `openldap` with `brew install openldap`;
+    2. Locate installation directory with `brew --prefix openldap`;
+    3. Add the path to the project configuration file(choose one of the 
following two methods):
+       1. Solution A: You can set `LDAP_DIR` with `luarocks config` manually, 
for example `luarocks config variables.LDAP_DIR /usr/local/opt/openldap/`;

Review comment:
       The first method is the original general solution. Is it suitable to 
integrate it into the macos system general solution?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to