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

Pure PHP implementation of Patch (avoid a dependency from the Operating System) #262

Closed
marclaporte opened this issue Mar 21, 2019 · 3 comments

Comments

@marclaporte
Copy link

As a spin off #93 (comment) , copying the comment below:

About "Research: Pure PHP implementation of patch?" -> The patch utility is not always available and thus, we'd like this to be handled by cweagans/composer-patches (directly or indirectly)

Would this work?
https://github.com/FMaz008/UnifiedDiffPatcher

Are you aware of any other suitable libs? Anything already on Packagist.org?

Thanks!

@marclaporte marclaporte changed the title Pure PHP implementation of Patch Pure PHP implementation of Patch (avoid a dependency from the Operating System) Mar 21, 2019
@marclaporte
Copy link
Author

@cweagans I would love to know your thoughts on this. Thanks!

@cweagans
Copy link
Owner

i thought about it a bit (did you and i talk about it on IRC a few years ago?). I'm not sure this should go into composer-patches itself, but my intent is to make it possible to extend the methods by which a patch can be applied to a particular package in 2.x (which is currently vaporware). Composer has a way for plugins to extend other plugins, so it should be a relatively straightforward thing to do this.

Most people just use patch or git though.

@cweagans
Copy link
Owner

cweagans commented Feb 7, 2023

main supports pluggable patchers now, so this can be done outside of the main plugin. I'm also kicking around the idea of using https://justine.lol/ape.html to compile GNU patch and distribute it either with the plugin or with a separate plugin that extends this one.

@cweagans cweagans closed this as completed Feb 7, 2023
@cweagans cweagans mentioned this issue Feb 7, 2023
31 tasks
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

No branches or pull requests

2 participants