Skip to content

Replace wget with curl in all Dockerfiles to fix Alpine busybox compatibility#23

Merged
joehoyle merged 2 commits into
masterfrom
copilot/fix-22
Aug 20, 2025
Merged

Replace wget with curl in all Dockerfiles to fix Alpine busybox compatibility#23
joehoyle merged 2 commits into
masterfrom
copilot/fix-22

Conversation

Copilot AI commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

This PR replaces all wget commands with curl equivalents across all Dockerfiles to resolve compatibility issues with busybox implementations of wget in Alpine Linux.

Problem

The busybox version of wget in some Alpine Linux versions doesn't work reliably, causing Docker builds to fail when downloading WordPress core and PHPUnit test suite archives.

Solution

  1. Added curl package to the apk add commands in all Dockerfiles
  2. Replaced wget commands with functionally equivalent curl commands:
    • wget -nv -O /tmp/file.tar.gz URLcurl -fsSL -o /tmp/file.tar.gz URL

curl flags used:

  • -f: Fail silently on HTTP errors
  • -s: Silent mode (equivalent to wget's -nv)
  • -S: Show errors even in silent mode
  • -L: Follow redirects
  • -o: Output to file (equivalent to wget's -O)

Files affected:

  • Dockerfile.php8.0
  • Dockerfile.php8.1
  • Dockerfile.php8.2
  • Dockerfile.php8.3

Testing

Successfully built and tested Docker images for PHP 8.1 and 8.3 to verify the curl replacements work correctly for downloading both WordPress core and wp-phpunit archives.

Fixes #22.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: joehoyle <161683+joehoyle@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace wget with curl Replace wget with curl in all Dockerfiles to fix Alpine busybox compatibility Aug 20, 2025
Copilot AI requested a review from joehoyle August 20, 2025 15:54
@joehoyle joehoyle marked this pull request as ready for review August 20, 2025 16:02
@joehoyle joehoyle merged commit ea44506 into master Aug 20, 2025
72 checks passed
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.

Replace wget with curl

2 participants