jp-gouin commented on a change in pull request #3894:
URL: https://github.com/apache/apisix/pull/3894#discussion_r725601851
##########
File path: .github/workflows/cli.yml
##########
@@ -65,7 +65,7 @@ jobs:
key: ${{ runner.os }}-${{ env.cache-name }}-${{ matrix.job_name
}}-${{ hashFiles('rockspec/apisix-master-0.rockspec') }}
- name: Linux Get dependencies
- run: sudo apt install -y cpanminus build-essential libncurses5-dev
libreadline-dev libssl-dev perl libpcre3 libpcre3-dev
+ run: sudo apt install -y cpanminus build-essential libncurses5-dev
libreadline-dev libssl-dev perl libpcre3 libpcre3-dev libldap2-dev
Review comment:
I needed to add `libldap2-dev` at this level otherwise
linux_apisix_current_luarocks_in_customed_nginx in CLI test is failed.
I don't know why this test is not using the utils/linux-install-openresty.sh
file to install openresty and dependencies
--
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]
