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

php: Simplify the end-user installation with a helper script #3

Merged
merged 1 commit into from
May 16, 2024

Conversation

TimWolla
Copy link
Collaborator

Besides making the instructions simpler, this also improves the robustness of the Docker image, because we can adjust filenames and directory structure as necessary.

The Debian Dockerfile has been synchronized with the Alpine one, embedded the SHA-256 hash of the regular downloads, instead of going through apt, as this simplifies the placement in the top-level /tideways/ directory without unecessary duplication or complicated cleanup.

The updated instructions are now as follows:

Debian:

COPY --from=ghcr.io/tideways/php:latest /tideways/ /tideways/
RUN docker-php-ext-enable --ini-name tideways.ini $(php /tideways/get-ext-path.php)

Alpine:

COPY --from=ghcr.io/tideways/php:alpine /tideways/ /tideways/
RUN docker-php-ext-enable --ini-name tideways.ini $(php /tideways/get-ext-path.php)

The previous instructions are no longer valid, but they have not yet been documented.

Besides making the instructions simpler, this also improves the robustness of
the Docker image, because we can adjust filenames and directory structure as
necessary.

The Debian Dockerfile has been synchronized with the Alpine one, embedded the
SHA-256 hash of the regular downloads, instead of going through `apt`, as this
simplifies the placement in the top-level `/tideways/` directory without
unecessary duplication or complicated cleanup.

The updated instructions are now as follows:

Debian:

    COPY --from=ghcr.io/tideways/php:latest /tideways/ /tideways/
    RUN docker-php-ext-enable --ini-name tideways.ini $(php /tideways/get-ext-path.php)

Alpine:

    COPY --from=ghcr.io/tideways/php:alpine /tideways/ /tideways/
    RUN docker-php-ext-enable --ini-name tideways.ini $(php /tideways/get-ext-path.php)

The previous instructions are no longer valid, but they have not yet been
documented.
@TimWolla TimWolla requested a review from edorian May 16, 2024 14:09
@TimWolla TimWolla merged commit 93f85de into main May 16, 2024
3 checks passed
@TimWolla TimWolla deleted the php-simplify branch May 16, 2024 14:13
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

Successfully merging this pull request may close these issues.

2 participants