Skip to content
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

Failure of http extension using installer #1032

Closed
aum-tcm opened this issue Dec 17, 2024 · 3 comments · Fixed by #1033
Closed

Failure of http extension using installer #1032

aum-tcm opened this issue Dec 17, 2024 · 3 comments · Fixed by #1033
Labels
bug Something isn't working

Comments

@aum-tcm
Copy link

aum-tcm commented Dec 17, 2024

Version of install-php-extensions

v.2.7.7

Error description

Hi, first of all, thank you for maintaining this great tool.
I’ve been successfully using it to install the http extension for a while, but I recently encountered an issue where the installation now fails. I wanted to bring this to your attention.

An issue occurs when attempting to install the http extension via the mlocati/docker-php-extension-installer tool. The installation fails during the build process of a dependency, resulting in compilation errors related to the iconv function. Below are the error messages and details.

localconverter.c: In function 'localconverter_iconv_conv':
localconverter.c:602:21: error: passing argument 2 of 'iconv' from incompatible pointer type [-Wincompatible-pointer-types]
  602 |         iconv(ictx, (const char **)NULL, &inleft, &outbuf, &outleft);
      |                     ^
      |                     |
      |                     const char **
In file included from localconverter.c:195:
/usr/include/iconv.h:17:23: note: expected 'char ** restrict' but argument is of type 'const char **'
   17 | size_t iconv(iconv_t, char **__restrict, size_t *__restrict, char **__restrict, size_t *__restrict);
      |                       ^

localconverter.c:607:26: error: passing argument 2 of 'iconv' from incompatible pointer type [-Wincompatible-pointer-types]
  607 |         sz = iconv(ictx, (const char **)&inbuf, &inleft, &to, &outleft);
      |                          ^~~~~~~~~~~~~~~~~~~~~
      |                          |
      |                          const char **
/usr/include/iconv.h:17:23: note: expected 'char ** restrict' but argument is of type 'const char **'
   17 | size_t iconv(iconv_t, char **__restrict, size_t *__restrict, char **__restrict, size_t *__restrict);
      |                       ^

localconverter.c:633:26: error: passing argument 2 of 'iconv' from incompatible pointer type [-Wincompatible-pointer-types]
  633 |         sz = iconv(ictx, (const char **)NULL, &inleft, &to, &outleft);
      |                          ^
      |                          |
      |                          const char **
/usr/include/iconv.h:17:23: note: expected 'char ** restrict' but argument is of type 'const char **'
   17 | size_t iconv(iconv_t, char **__restrict, size_t *__restrict, char **__restrict, size_t *__restrict);
      |                       ^

make[1]: *** [Makefile:800: libidnkit_la-localconverter.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory '/tmp/src/tmp.oEpmPG/idnkit-2.3/lib'
make: *** [Makefile:329: install-recursive] Error 1
error building image: error building stage: failed to execute command: waiting for process to exit: exit status 2

I've tested different versions of the tool (v2.7.6 and v2.7.5) and PHP (8.4) without success

Thank you in advance for your time and support

Docker image

php:8.3-fpm-alpine

Minimal Dockerfile

FROM php:8.3-fpm-alpine

# Basics
RUN apk --update --no-cache add bash nano

# Add easy extension installer
ADD https://github.com/mlocati/docker-php-extension-installer/releases/latest/download/install-php-extensions /usr/local/bin/
RUN chmod +x /usr/local/bin/install-php-extensions

# Install default required extensions
RUN install-php-extensions http
@aum-tcm aum-tcm added the bug Something isn't working label Dec 17, 2024
Copy link

Version of install-php-extensions

latest

@aum-tcm the install-php-extensions version that you specified (latest) is not valid.

Please specify the install-php-extensions version you are using: it's the first line printed by the script.
For example:

install-php-extensions v.X.Y.Z

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 17, 2024
@mlocati mlocati reopened this Dec 17, 2024
@mlocati
Copy link
Owner

mlocati commented Dec 17, 2024

Thanks for the report: I've just published version 2.7.8 which should fix this issue.

@aum-tcm
Copy link
Author

aum-tcm commented Dec 17, 2024

Wow, indeed it works now. Thanks for support !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants