-
Notifications
You must be signed in to change notification settings - Fork 107
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
0.4.9 fatal error 'ProcessBuilder not found' #142
Comments
I'm getting this error too, have tried running |
Seems the pr #140 actually broke this package completely. The complete symfony/process signature changed. There won't be an easy way to cover Symfony 4 without breaking backward compatibility. PHP Fatal error: Uncaught Error: Class 'Symfony\Component\Process\ProcessBuilder' not found in /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/ProcessBuilder/ProcessBuilderFactory.php:69
Stack trace:
#0 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Adapter/VersionProbe/ZipVersionProbe.php(70): Alchemy\Zippy\ProcessBuilder\ProcessBuilderFactory->create()
#1 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Adapter/AbstractAdapter.php(146): Alchemy\Zippy\Adapter\VersionProbe\ZipVersionProbe->getStatus()
#2 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Zippy.php(156): Alchemy\Zippy\Adapter\AbstractAdapter->isSupported()
#3 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Zippy.php(67): Alchemy\Zippy\Zippy->getAdapterFor('zip')
#4 /Users/luceos/Sites/flarum/workbench/backup/src/Console/BackupCommand.php(69): Alchemy\Zippy\Zippy->create('/Users/luceos/S...', Array)
#5 /Users/luceos/Sites/flarum/workbench/backup/src/Console/BackupCommand.php(32): Flagrow\Backup\Console\BackupCommand->prepareArchive()
#6 [intern in /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/ProcessBuilder/ProcessBuilderFactory.php on line 69 Fixing that, causes: PHP Fatal error: Uncaught Error: Call to undefined method Symfony\Component\Process\Process::add() in /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Adapter/VersionProbe/ZipVersionProbe.php:71
Stack trace:
#0 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Adapter/AbstractAdapter.php(146): Alchemy\Zippy\Adapter\VersionProbe\ZipVersionProbe->getStatus()
#1 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Zippy.php(156): Alchemy\Zippy\Adapter\AbstractAdapter->isSupported()
#2 /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Zippy.php(67): Alchemy\Zippy\Zippy->getAdapterFor('zip')
#3 /Users/luceos/Sites/flarum/workbench/backup/src/Console/BackupCommand.php(69): Alchemy\Zippy\Zippy->create('/Users/luceos/S...', Array)
#4 /Users/luceos/Sites/flarum/workbench/backup/src/Console/BackupCommand.php(32): Flagrow\Backup\Console\BackupCommand->prepareArchive()
#5 [internal function]: Flagrow\Backup\Console\BackupCommand->handle(Object(Flagrow\Backup\Registry))
#6 /Users/luceos/Sites/flarum/vendor/illuminate/container/Bound in /Users/luceos/Sites/flarum/vendor/alchemy/zippy/src/Adapter/VersionProbe/ZipVersionProbe.php on line 71 |
Having the same issue and error here Updating broke all of my zippy code... |
ProcessBuilder was deprecated in symfony/component 3.4.0 and removed in symfony/component 4.0 |
Error when updating zippy alchemy-fr/Zippy#142
0.4.9 fatal error 'ProcessBuilder not found'
Someone has information about the progress of this issue, I still have the problem but I saw that a PR was open to solve this problem, will it soon be merge? |
Is this library just broken and dead? |
ProcessBuilder is not available in symfony/process 4. Example of fixing this case in another project - https://github.com/pact-foundation/pact-php/pull/61/files |
@luceos Are you a maintainer of this package? |
As I had to upgrade Symfony and the maintainers of the project stopped to "maintain", I had no choice other than creating a nasty patch, that allows to use the zipper by ignoring the "ZipVersionProbe" functionality. I don´t know what the ZipVersionProbe class it is meant to do as I can't afford to spend more time with researching. With me it works. Even with larger files 200mb+ Maybe this is a small help to somebody who runs into the same issue. |
I'm not. And I don't remember using this in my apps anymore. If you need this, I highly suggest forking and adding a link to the fork here. |
v0.4.8
still works fine, but upgrading to0.4.9
and then running this exact code gives me a fatal error.Here's how I'm using Zippy:
And here's the error:
The text was updated successfully, but these errors were encountered: