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

Transient Error when Building Imagick in php 8.3.x on amd64 on alpine #643

Open
en-jschuetze opened this issue Nov 29, 2023 · 16 comments
Open

Comments

@en-jschuetze
Copy link

en-jschuetze commented Nov 29, 2023

A follow up of 582

Building on linux/amd64 the Dockerfile:

FROM alpine:edge
RUN apk add php83 php83-pear php83-openssl php83-sockets
RUN apk add --no-cache binutils build-base openssl-dev autoconf pcre2-dev automake libtool linux-headers imagemagick imagemagick-dev imagemagick-libs php83-dev~=8.3.0 --virtual .build-deps     && MAKEFLAGS="-j $(nproc)" pecl83 install imagick     && strip --strip-all /usr/lib/php83/modules/imagick.so     && echo "extension=imagick" > /etc/php83/conf.d/00_imagick.ini     && apk del --no-network .build-deps     && apk add --no-cache imagemagick imagemagick-libs libgomp

and the error is (not happening every time, but 75% of all runs):

38 10.77 cc -shared  .libs/imagick_file.o .libs/imagick_class.o .libs/imagickdraw_class.o .libs/imagickpixel_class.o .libs/imagickpixeliterator_class.o .libs/imagick_helpers.o .libs/imagick.o .libs/imagickkernel_class.o .libs/shim_im6_to_im7.o  -lgomp -lMagickWand-7.Q16HDRI -lMagickCore-7.Q16HDRI  -Wl,-soname -Wl,imagick.so -o .libs/imagick.so
#38 10.82 creating imagick.la
#38 10.82 (cd .libs && rm -f imagick.la && ln -s ../imagick.la imagick.la)
#38 10.82 /bin/sh /tmp/pear/temp/pear-build-defaultuserhbpaOF/imagick-3.7.0/libtool --mode=install cp ./imagick.la /tmp/pear/temp/pear-build-defaultuserhbpaOF/imagick-3.7.0/modules
#38 10.84 cp ./.libs/imagick.so /tmp/pear/temp/pear-build-defaultuserhbpaOF/imagick-3.7.0/modules/imagick.so
#38 10.84 cp ./.libs/imagick.lai /tmp/pear/temp/pear-build-defaultuserhbpaOF/imagick-3.7.0/modules/imagick.la
#38 10.85 PATH="$PATH:/sbin" ldconfig -n /tmp/pear/temp/pear-build-defaultuserhbpaOF/imagick-3.7.0/modules
#38 10.85 ----------------------------------------------------------------------
#38 10.85 Libraries have been installed in:
#38 10.85    /tmp/pear/temp/pear-build-defaultuserhbpaOF/imagick-3.7.0/modules
#38 10.85 
#38 10.85 If you ever happen to want to link against installed libraries
#38 10.85 in a given directory, LIBDIR, you must either use libtool, and
#38 10.85 specify the full pathname of the library, or use the `-LLIBDIR'
#38 10.85 flag during linking and do at least one of the following:
#38 10.85    - add LIBDIR to the `LD_LIBRARY_PATH' environment variable
#38 10.85      during execution
#38 10.85    - add LIBDIR to the `LD_RUN_PATH' environment variable
#38 10.85      during linking
#38 10.85    - use the `-Wl,--rpath -Wl,LIBDIR' linker flag
#38 10.85 
#38 10.85 See any operating system documentation about shared libraries for
#38 10.85 more information, such as the ld(1) and ld.so(8) manual pages.
#38 10.85 ----------------------------------------------------------------------
#38 10.86 
#38 10.86 Build complete.
#38 10.86 Don't forget to run 'make test'.
#38 10.86 
#38 10.86 running: make INSTALL_ROOT="/tmp/pear/temp/pear-build-defaultuserhbpaOF/install-imagick-3.7.0" install
#38 10.87 Parse /tmp/pear/temp/imagick/ImagickDraw.stub.php to generate /tmp/pear/temp/imagick/ImagickDraw_arginfo.h
#38 10.87 Parse /tmp/pear/temp/imagick/ImagickPixelIterator.stub.php to generate /tmp/pear/temp/imagick/ImagickPixelIterator_arginfo.h
#38 10.87 Parse /tmp/pear/temp/imagick/ImagickPixel.stub.php to generate /tmp/pear/temp/imagick/ImagickPixel_arginfo.h
#38 10.88 Parse /tmp/pear/temp/imagick/Imagick.stub.php to generate /tmp/pear/temp/imagick/Imagick_arginfo.h
#38 10.91 Connecting to github.com (140.82.112.4:443)
#38 11.06 Connecting to codeload.github.com (140.82.113.10:443)
#38 11.23 saving to 'v5.0.0alpha3.tar.gz'
#38 11.28 v5.0.0alpha3.tar.gz  100% |********************************|  157k  0:00:00 ETA
#38 11.28 'v5.0.0alpha3.tar.gz' saved
#38 11.28 PHP-Parser-5.0.0alpha3/
#38 11.28 PHP-Parser-5.0.0alpha3/.php-cs-fixer.dist.php
#38 11.28 PHP-Parser-5.0.0alpha3/LICENSE
[sniiiipppppp]
#38 11.32 PHP-Parser-5.0.0alpha3/lib/PhpParser/PrettyPrinterAbstract.php
#38 11.32 PHP-Parser-5.0.0alpha3/lib/PhpParser/Token.php
#38 11.32 PHP-Parser-5.0.0alpha3/lib/PhpParser/compatibility_tokens.php
#38 11.32 PHP-Parser-5.0.0alpha3/phpstan-baseline.neon
#38 11.32 PHP-Parser-5.0.0alpha3/phpstan.neon.dist
#38 11.34 Saved /tmp/pear/temp/imagick/ImagickPixel_arginfo.h
#38 11.34 Saved /tmp/pear/temp/imagick/ImagickPixelIterator_arginfo.h
#38 11.35 Parse /tmp/pear/temp/imagick/ImagickKernel.stub.php to generate /tmp/pear/temp/imagick/ImagickKernel_arginfo.h
#38 11.35 Saved /tmp/pear/temp/imagick/ImagickDraw_arginfo.h
#38 11.38 In /tmp/pear/temp/imagick/Imagick.stub.php:
#38 11.38 Unterminated preprocessor conditions
#38 11.39 make: *** [Makefile:196: /tmp/pear/temp/imagick/Imagick_arginfo.h] Error 1
#38 11.39 make: *** Waiting for unfinished jobs....
#38 11.41 Saved /tmp/pear/temp/imagick/ImagickKernel_arginfo.h
#38 11.41 Installing header files:          /tmp/pear/temp/pear-build-defaultuserhbpaOF/install-imagick-3.7.0/usr/include/php83/
#38 11.44 ERROR: `make INSTALL_ROOT="/tmp/pear/temp/pear-build-defaultuserhbpaOF/install-imagick-3.7.0" install' failed
#38 ERROR: process "/bin/sh -c apk add --no-cache binutils build-base openssl-dev autoconf pcre2-dev automake libtool linux-headers imagemagick imagemagick-dev imagemagick-libs ${PHP_PACKAGE_BASENAME}-dev~=${PHP_VERSION} --virtual .build-deps     && MAKEFLAGS=\"-j $(nproc)\" pecl83 install imagick     && strip --strip-all /usr/lib/$PHP_PACKAGE_BASENAME/modules/imagick.so     && echo \"extension=imagick\" > /etc/$PHP_PACKAGE_BASENAME/conf.d/00_imagick.ini     && apk del --no-network .build-deps     && apk add --no-cache imagemagick imagemagick-libs libgomp" did not complete successfully: exit code: 1

it does not happen ever on arm architecture.

for reference an example run in a "real" world at https://github.com/Endava/docker-php/actions/runs/7009886813/job/19071577630

@en-jschuetze en-jschuetze changed the title Transient Error when Building Imagick on amd64 on alpine Transient Error when Building Imagick in php 8.3.x on amd64 on alpine Nov 29, 2023
@mryfan
Copy link

mryfan commented Nov 30, 2023

mee to

@K2ouMais
Copy link

K2ouMais commented Nov 30, 2023

It is because of this?

#641

All my pipelines fail sporadic and not every time.

@dantheman2865
Copy link

dantheman2865 commented Nov 30, 2023

If you're looking for a workaround, you can explicitly use PHP 8.2. Here's an example of how I've done that in my workflow.

Until this gets merged, I am using a workaround in my automated builds to explicitly fix the PHP version to 8.2. This is suggested by Composer:

PHP version & extensions

(optimal) create your own build image and install Composer inside it.
Note: Docker 17.05 introduced multi-stage builds, simplifying this enormously:
COPY --from=composer /usr/bin/composer /usr/bin/composer

In my Dockerfile I had:

FROM composer:2.5 as composer_base

I've updated it to copy Composer from their image into a PHP 8.2 image in order to build with that version:

FROM composer:2.5

# TODO: Revert back to image `composer:2.5`
FROM php:8.2-alpine as composer_base
COPY --from=composer /usr/bin/composer /usr/bin/composer

@en-jschuetze
Copy link
Author

@dantheman2865

Can you explain what effect this has on the imagick extension?

@dantheman2865
Copy link

dantheman2865 commented Nov 30, 2023

Can you explain what effect this has on the imagick extension?

@en-jschuetze
I've updated the comment to be more clear. I'm hoping to help folks who need a workaround because their automated builds are broken (as mine were).

@en-jschuetze
Copy link
Author

Ah I see. You avoid php 8.3 :)

@daFish
Copy link

daFish commented Nov 30, 2023

This is the workaround I used the last time:

RUN curl -L -o /tmp/imagick.tar.gz https://github.com/Imagick/imagick/archive/7088edc353f53c4bc644573a79cdcd67a726ae16.tar.gz \
    && tar --strip-components=1 -xf /tmp/imagick.tar.gz \
    && phpize \
    && ./configure \
    && make \
    && make install \
    && echo "extension=imagick.so" > /usr/local/etc/php/conf.d/ext-imagick.ini \
    && rm -rf /tmp/* \

The package is the commit from #641

@fpetrovic
Copy link

What is the status of this one? Is it close to be done?

@0x6D682D7331
Copy link

0x6D682D7331 commented Jan 27, 2024

Same problem, Debian 12 images.
The workaround below worked for us (thx @daFish).

# build
docker build -t localhost:5000/poc-8.3-imagick - <<EOF
FROM php:8.3

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

RUN \
  set -eux; \
  install-php-extensions https://codeload.github.com/Imagick/imagick/tar.gz/7088edc353f53c4bc644573a79cdcd67a726ae16

EOF

# verify
docker run --rm localhost:5000/poc-8.3-imagick php -r 'var_export(phpversion("imagick"));'

@en-jschuetze
Copy link
Author

This is the workaround I used the last time:

RUN curl -L -o /tmp/imagick.tar.gz https://github.com/Imagick/imagick/archive/7088edc353f53c4bc644573a79cdcd67a726ae16.tar.gz \
    && tar --strip-components=1 -xf /tmp/imagick.tar.gz \
    && phpize \
    && ./configure \
    && make \
    && make install \
    && echo "extension=imagick.so" > /usr/local/etc/php/conf.d/ext-imagick.ini \
    && rm -rf /tmp/* \

The package is the commit from #641

This works also if you use 3.7.0 as commit id. So I guess I am fine for my project for now, but it would be awesome to get it fixed in a pecl release, too.

@Danack
Copy link
Collaborator

Danack commented Feb 8, 2024

not happening every time, but 75% of all runs

It is very odd that it passes some of the time.

For the record, this doesn't happen on my computer compiling either inside a docker Ubuntu box, or natively on OSX.

I've possibly fixed it, though would prefer to see it compiling in the Github actions....which appear to be broken for something that is not my fault.

@grEvenX
Copy link

grEvenX commented Mar 4, 2024

@Danack Any updates on this issue. Is there anything we can do to help related to the Github actions you mentioned in the last comment?

carlesbarreda added a commit to carlesbarreda/docker-php that referenced this issue Mar 10, 2024
Transient Error when Building Imagick in php 8.3.x on amd64 on alpine #643

Imagick/imagick#643
@apoca
Copy link

apoca commented Mar 12, 2024

Same issue here, waiting resolution to use PHP 8.3 in production.

@radziejewicz
Copy link

This is the workaround I used the last time:

RUN curl -L -o /tmp/imagick.tar.gz https://github.com/Imagick/imagick/archive/7088edc353f53c4bc644573a79cdcd67a726ae16.tar.gz \
    && tar --strip-components=1 -xf /tmp/imagick.tar.gz \
    && phpize \
    && ./configure \
    && make \
    && make install \
    && echo "extension=imagick.so" > /usr/local/etc/php/conf.d/ext-imagick.ini \
    && rm -rf /tmp/* \

The package is the commit from #641

This solution works, but it is very temporary :D

@vaclavekp
Copy link

I have encountered the same problem, when building multiplatform image on linux + AMD CPU. Build for linux/amd64 was successull, but for linux/arm64/v8 it ended up with this error.

Interestingly, when I tried to build it within container FROM docker:latest, it completed successfully for both platforms.

Is there any progress or indication, where the problem might be?

@vaclavekp
Copy link

OK, so we are waiting for imagick to release new version where this bug is fixed. Until then, there is temporary solution to use unreleased version of imagick.

Source: #640

COPY --from=mlocati/php-extension-installer /usr/bin/install-php-extensions /usr/local/bin/
RUN install-php-extensions imagick/imagick@master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests