Apply patch on root folder

[tomasnagy]

Is there any way to apply a patch the root of the project, not specifically targeting a package.
Currently I couldn't any documentation about this.
Is this supported in any form or way?

[cweagans]

No, there's not. Presumably, those files are committed to your repository. Why not just change them directly?

[erikhansen]

@cweagans I too would love to be able to apply patches at the root of a project. My use case for this is as follows: I work on a lot of Magento 2 projects. Magento 2 projects use Composer to install its packages. Unlike applications like Drupal (which seems to use a single Composer package for the entire application) Magento 2 has separate composer packages for its dozens of packages.

There are many situations where a patch would need to be applied to multiple Magento 2 packages simultaneously. With the current state of this plugin, the patch would need to be split between multiple .patch files, which would be very clunky. It would be great to be able to apply a patch to multiple packages simultaneously, and it seems like the simplest way to enable this would be to allow patches to be applied to the root of the project. Although allowing patches to be applied to the vendor directory would also work, although it seems less elegant.

I'm a contributor to the capistrano-magento2 Gem and we're currently discussing the best way to patch Magento when deploying it to stage/production environments. In my opinion, using this plugin would be better than building a specific patching mechanism into the capistrano-magento2 Gem. We would just need root-patching support in order for the compser-patches plugin to fully satisfy our requirements.

[cweagans]

If you want to code something up to show what this would look like, I'd take a look. I can't promise it'll go in as-is, but I'm certainly not opposed to looking at it.

[cweagans]

This will be a part of 2.0, FWIW.

[erikhansen]

@cweagans Fantastic news! Any idea of a timeline for 2.0?

[cweagans]

When it's done :) I have less than 1 page in my notebook for remaining tasks. This week is crazy, but I have Monday off, so I should be able to take a big bite out of it. I may put that list up on GH later today.

[erikhansen]

@cweagans Completely understood. I appreciate you putting so much of your personal time into this very useful plugin.

Once 2.0.0 is released, I expect even more Magento 2 developers will start using this plugin as it solves a very real problem with Magento 2 development and deployment.

[Vedrillan]

+1

How is the 2.x going ? :)

Trying to apply Magento patches but does not work out of the box if the patch applies to several package at the same time.

[erikhansen]

@Vedrillan While it's a bit of work, it is possible to split a Magento patch into multiple files so that it can be applied to each of the relevant Composer packages. Someone on my team recently did this with a patch supplied from Magento.

[Vedrillan]

Yep indeed it is still possible, chopping down the patches to target only 1 package at a time, it is also necessary to adapt the files path, but if we can avoid this fastidious part thanks to the 2.x, I'd be happier !

[hostep]

@erikhansen & @Vedrillan: I fail to see how this can be solved, well for Magento2 patches I mean.

One Magento2 patch file can contain fixes for multiple modules at once. Without any mapping to say which part of the file should be applied to which module, there is no way this is ever going to work.
Next to that, the patches always use paths like app/code/Magento/SomeModule/SomeDirectory/SomeFile.php but in the vendor/magento/module-somemodule directory, we only have SomeDirectory/SomeFile.php. So that's already something else which needs manual labour.

I can't see how this can be automated, unless I'm missing something obvious here?
The only thing I can think of is searching through composers autoloader files and try to find matches in there, but that is very specific to what type of project which is being used I guess.

So yeah, we'll have to manually keep splitting the patches into multiple patch files for each submodule and change the paths in the patch files I'm afraid, but I'm already used to it by now:

Example

sigh ;)

[erikhansen]

@hostep You are correct that if you generate a patch directly from a Github commit or PR, you would have to manually change the paths to Composer-based paths. However this could still be done in the context of a single file and applied using a “root patching” method and in my mind would be preferable because it would keep a patch contained within a single file.

Also, for Magento Enterprise Merchants that get patches directly from Magento, those patches are in a “Composer friendly” format that can be applied directly to the root of a Magento install.

Another limitation I’ve run into with composer-patches is that it doesn’t appear possible to apply two separate patches to the same Composer package. There’s probably an issue about that somewhere on this repo.

[cweagans]

Another limitation I’ve run into with composer-patches is that it doesn’t appear possible to apply two separate patches to the same Composer package.

There is no such limitation in this plugin. If you're encountering errors, it's likely that the patches conflict with each other, so you'll have to manually apply them both and generate a combined patch to get around the conflicts.

[erikhansen]

@cweagans Ok, good to know. Thanks for letting me know.