docker-library / docker-library/php

Possibly rework `docker-php-source` to account for patches

Open
#1,555 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Dockerfile
Stars
4k
Forks
2k
Avg merge
5d 21h
Merged PRs (30d)
1

Description

We added a couple backports to 8.1 from PHP upstream changes to get it to compile on Alpine 3.21 in #1552. We should consider keeping them around so that they can be applied with docker-php-source extract in case the patches we backport also apply to extensions we don't compile in (or users that recompile the already built-in extensions).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the current docker-php-source implementation and the PHP upstream backports introduced in #1552. Trace how docker-php-source extract handles source and patches, then clarify whether those backports should be retained and applied to uncompiled or rebuilt extensions. Done means the behavior and validation cases are agreed, including the Alpine 3.21 compilation scenario.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, php
Domain
build-system, devops
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.