Silent Composer install not working with patches

[mdjdev]

Hi,

the patching gets stuck in my setup on composer i. I'm not sure if it's a problem with my patch or a config which I didn't set correctly.

The command composer -vv i gives me a hint what the problem is:

> post-package-install: cweagans\Composer\Patches->postInstall
  - Applying patches for jackalope/jackalope-doctrine-dbal
    patches/jackalope-jackalope-doctrine-dbal-src-jackalope-transport-doctrinedbal-repositoryschema-php.patch (Set `path` length to 255 in table `phpcr_nodes`)
patch '-p1' --no-backup-if-mismatch -d '/app/vendor/jackalope/jackalope-doctrine-dbal' < '/app/patches/jackalope-jackalope-doctrine-dbal-src-jackalope-transport-doctrinedbal-repositoryschema-php.patch'
patching file 'src/Jackalope/Transport/DoctrineDBAL/RepositorySchema.php'
Patch creates file that already exists!  Assume -R? [y] 

Unreversed (or previously applied) patch detected!  Ignore -R? [y] 

This is my patch:

--- /dev/null
+++ ../src/Jackalope/Transport/DoctrineDBAL/RepositorySchema.php
@@ -96,7 +96,7 @@
         $nodes = $this->createTable('phpcr_nodes');
         $nodes->addColumn('id', 'integer', ['autoincrement' => true]);
-        $nodes->addColumn('path', 'string', ['length' => $this->getMaxIndexLength()]);
+        $nodes->addColumn('path', 'string', ['length' => null]);
         $nodes->addColumn('parent', 'string', ['length' => $this->getMaxIndexLength()]);
         $nodes->addColumn('local_name', 'string', ['length' => $this->getMaxIndexLength()]);
         $nodes->addColumn('namespace', 'string', ['length' => $this->getMaxIndexLength()]);

How can I avoid these prompts?

[cweagans]

I think your patch is wrong. Your patch is specifically saying that it's creating a new file (--- /dev/null at the top), but the patch itself is modifying an existing file. That's really strange. How did you generate this patch?

[mdjdev]

Thanks for the hint. I used the tool linked below to generate the patch and will have a look to see if i used it incorrectly.

https://github.com/symplify/vendor-patches

[cweagans]

Sounds good -- closing this out for now, but we can re-open if needed.

[xerc]

@cweagans ref. https://www.man7.org/linux/man-pages/man1/patch.1.html

When a patch does not apply, patch usually checks if the patch
looks like it has been applied already by trying to reverse-
apply the first hunk. The --forward option prevents that.

=> patch -p0 --forward --no-backup-if-mismatch -d '/[..]' < '/[..].patch'

[cweagans]

We don't use 'patch' anymore. Everything uses git now.