-
Notifications
You must be signed in to change notification settings - Fork 326
configure option disable-memcached-sasl not working #276
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
Comments
Please try php7 branch again. |
Seems that issue is not resolved still. Can be reproduced with following FROM php:7.0.12-fpm-alpine
RUN apk add --update --no-cache \
libmemcached-libs zlib \
ca-certificates \
&& update-ca-certificates \
&& apk add --no-cache --virtual .tools-deps \
git autoconf g++ libtool make \
&& apk add --no-cache --virtual .build-deps \
libmemcached-dev \
zlib-dev \
&& docker-php-source extract \
&& git clone --branch php7 https://github.com/php-memcached-dev/php-memcached \
/usr/src/php/ext/memcached/ \
&& docker-php-ext-configure \
memcached --disable-memcached-sasl \
&& docker-php-ext-install \
memcached Results with following error on step
|
At least the error moved from inside the php-memcached headers to the libmemcached headers. I'll take a look if there are flags that should be set to skip the sasl.h include there, too. |
I don't need this option anymore (for now), since I managed to configure it with the sasl support, but I created the issue so it would be helpful for someone else. |
Install cyrus-sasl-dev can solve this problem. |
@Tairy yeap. But the point of this issue in |
my dockerfile is:
build is ok! but when i test:
error exits!
anyone can take is? |
This is not fixable. When libmemcached is compiled with SASL support, it writes out its header files in such a way that it always requires the sasl.h header even if you disable SASL support in php-memcached. |
The options has effect on configure, but on the build it requests sasl.h anyway.
Btw, I'm using official docker fpm alpine (php:7.0.11-fpm-alpine), and to be able to install the extension (php7 branch), I had to install
cyrus-sasl-dev
.It was hard to find, because there are thousands of packages with sasl in the name and php docs call it "libsasl2".
The text was updated successfully, but these errors were encountered: