Skip to content

Add FAQ: composer patches hanging on macOS#63

Merged
peterjaap merged 2 commits into
qoliber:mainfrom
Aquive:docs/faq-macos-composer-patches
Apr 7, 2026
Merged

Add FAQ: composer patches hanging on macOS#63
peterjaap merged 2 commits into
qoliber:mainfrom
Aquive:docs/faq-macos-composer-patches

Conversation

@Aquive
Copy link
Copy Markdown
Contributor

@Aquive Aquive commented Apr 3, 2026

Summary

  • Adds a new FAQ entry under "macOS Specific" explaining why composer install hangs when applying patches via cweagans/composer-patches
  • Documents the root cause: BSD patch on macOS prompts interactively when file paths don't resolve, unlike GNU patch on Linux which tries harder to find files
  • Provides the fix: strip vendor/<package>/ prefix from patch file paths

Context

MageBox runs PHP natively on macOS, so composer install uses the macOS BSD patch binary. Docker-based tools (DDEV, Warden) run inside Linux containers where GNU patch is more forgiving with incorrect paths. This is a common gotcha when migrating projects from Docker-based setups to MageBox.

Because MageBox runs natively on macOS, BSD patch is used instead of
GNU patch. BSD patch hangs interactively when patch file paths don't
resolve, which affects projects using cweagans/composer-patches with
vendor-prefixed paths. Documents the cause and fix.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@peterjaap
Copy link
Copy Markdown
Collaborator

Maybe add the hint to switch to the vastly superior vaimo/composer-patches?

@jakwinkler
Copy link
Copy Markdown
Contributor

Agreed, add a hint to use vaimo/composer-patches - which is better ;-)

@Aquive
Copy link
Copy Markdown
Contributor Author

Aquive commented Apr 4, 2026

I didn't know of vaimo/composer-patches. I'll check it out and add the hint. Thanks.

@Aquive
Copy link
Copy Markdown
Contributor Author

Aquive commented Apr 7, 2026

Done @peterjaap

@peterjaap peterjaap merged commit 9d7240e into qoliber:main Apr 7, 2026
9 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.

3 participants