Use docker COPY directice

[glensc]

you can replace this:

# Clone xhgui and install master as there is no suitable tag, then remove useless files
RUN     git clone -b 0.13.0 --depth 1 https://github.com/perftools/xhgui /usr/local/src/xhgui && \
        cd /usr/local/src/xhgui && \
        rm -rf /usr/local/src/xhgui/.git \
               /usr/local/src/xhgui/.scrutinizer.yml \
               /usr/local/src/xhgui/.travis.yml \
               /usr/local/src/xhgui/phpunit.xml \
               /usr/local/src/xhgui/README.md \
               /usr/local/src/xhgui/tests

with:

COPY --from=xhgui/xhgui:0.13.0 /var/www/xhgui /usr/local/src/xhgui

what does it mean "no suitable tag"? have you reported your issue to xhgui issue tracker?

[edyan]

Hi, I'll look at that, and thanks for your comment. It looks like xhgui and the profiler are now two separate packages (look at master + the docker image doesn't contain the external/ directory anymore, even in 0.13). Either I put both on the same image (that we can mount a common volume between apps) or I only install xhgui without profiler, with a doc for each ton integrate the profiler as he wish

About the "no suitable tag" it was when xhgui didn't maintained well its tags ;)

[edyan]

Hi @glensc , I have refactored my images for PHP > 7. I am relying now on the latest stable version of xhgui. They have removed the header.php script, so i have created mine. If you have time to test, I would be glad 👍

[glensc]

I'm not using this project, so I will not validate your outcome.

I see now, that you are using 0.14.0:

• https://github.com/edyan/docker-xhgui/blob/d3f816860b07ffe3634d1c3e7e1999ce5ad14e91/php7.4/Dockerfile#L6

but you should update to at least the latest 0.16, so people don't need to rely on hacks:

• https://github.com/ngyuki-sandbox/php-xhprof-xhgui/issues/1

in other words, this ended up in 0.16.0:

• https://github.com/perftools/xhgui/issues/209