You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/09/07 07:42:31 UTC

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7827: feat(saml-auth): add saml-auth plugin

tzssangglass commented on code in PR #7827:
URL: https://github.com/apache/apisix/pull/7827#discussion_r964495640


##########
utils/install-dependencies.sh:
##########
@@ -84,13 +84,14 @@ function install_dependencies_with_apt() {
     sudo apt-get update
 
     # install OpenResty and some compilation tools
-    sudo apt-get install -y git openresty curl openresty-openssl111-dev make gcc libpcre3 libpcre3-dev libldap2-dev unzip
+    sudo apt-get install -y git openresty curl openresty-openssl111-dev make gcc libpcre3 libpcre3-dev libldap2-dev unzip libxml2-dev libxslt-dev
 }
 
 # Install dependencies on mac osx
 function install_dependencies_on_mac_osx() {
     # install OpenResty, etcd and some compilation tools
-    brew install openresty/brew/openresty luarocks lua@5.1 wget curl git pcre openldap
+    brew install openresty/brew/openresty luarocks lua@5.1 wget curl git pcre openldap libxml2 libxslt
+:qa

Review Comment:
   typo?



-- 
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: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org