graze / parallel-process

:running: Run multiple processes simultaneously
MIT License
103 stars 23 forks source link

Package is not getting installed with PHP version 8.1.5 #30

Open ARY112 opened 2 years ago

ARY112 commented 2 years ago

We are facing issue using graze/parallel-process.It was working fine till php version 7.2.After upgrading PHP to 8.1.5 it is not getting installed. Request you to provide some help.This package is very crucial for our project.

This is the error during composer install

graze/parallel-process[0.1, ..., 0.8.1] require php ^5.6 | ^7.0 -> your php version (8.1.5) does not satisfy that requirement. graze/parallel-process 0.8.1 requires php ^5.5 | ^7.0 -> your php version (8.1.5) does not satisfy that requirement. Root composer.json requires graze/parallel-process * -> satisfiable by graze/console-diff-renderer[0.1, ..., 0.8.1].

Below is the composer.json { "name": "pimcore/skeleton", "type": "project", "license": "GPL-3.0-or-later", "config": { "optimize-autoloader": true, "sort-packages": true, "allow-plugins": { "wikimedia/composer-merge-plugin": true } }, "require": { "php": "8.1.5", "aws/aws-sdk-php": "^3.108", "elasticsearch/elasticsearch": "^8.2", "graze/console-diff-renderer": "^0.6.2", "graze/parallel-process": "^0.8.1", "guzzlehttp/ringphp": "~1.1.1", "guzzlehttp/streams": "~3.0.0", "intervention/image": "^2.5", "league/csv": "^9.2", "ongr/elasticsearch-dsl": ">= 7.0", "phpoffice/phpspreadsheet": "", "pimcore/pimcore": "^10.4", "react/promise": "~2.7.1", "simplesamlphp/saml2": "~3.4.1", "wikimedia/composer-merge-plugin": "", "box/spout": "~3.0", "symfony/monolog-bundle": "3.5.0" }, "conflict": { "monolog/monolog": ">=2" }, "require-dev": { "ext-mysqli": "*", "cache/integration-tests": "^0.16.0", "codeception/codeception": "~2.4.5" }, "autoload": { "psr-4": { "App\": "src/", "Pimcore\Model\DataObject\": "var/classes/DataObject", "Pimcore\Model\Object\": "var/classes/Object", "Website\": "legacy/website/lib" } }, "scripts": { "post-create-project-cmd": "Pimcore\Composer::postCreateProject", "post-install-cmd": [ "Pimcore\Composer::postInstall", "@pimcore-scripts" ], "post-update-cmd": [ "Pimcore\Composer::postUpdate", "@pimcore-scripts", "Pimcore\Composer::executeMigrationsUp", "@pimcore-scripts", "AppBundle\EventListener\Migration::executeMigrationsUp" ], "pimcore-scripts": [ "Pimcore\Composer::clearCache", "Pimcore\Composer::installAssets" ] }, "extra": { "symfony-app-dir": "app", "symfony-bin-dir": "bin", "symfony-var-dir": "var", "symfony-web-dir": "public", "symfony-tests-dir": "tests", "symfony-assets-install": "relative", "merge-plugin": { "include": [ "composer.local.json" ], "recurse": true, "replace": true, "merge-dev": true, "merge-extra": false, "merge-extra-deep": false, "merge-scripts": false } } }

ARY112 commented 2 years ago

Symfony version is 5.4

ARY112 commented 2 years ago

@nadsella , R u looking into this...Also same issue i have updated for diff-renderer..Both are dependent packages

nadsella commented 2 years ago

@ARY112 what happens if you update the PHP requirement to be ^8.1.5?

ARY112 commented 2 years ago

@nadsella As suggested i tried with PHP requirement ^8.1.5.Below is the composer error

Error Problem 1

Composer.json below: { "name": "pimcore/skeleton", "type": "project", "license": "GPL-3.0-or-later", "config": { "optimize-autoloader": true, "sort-packages": true, "allow-plugins": { "wikimedia/composer-merge-plugin": true } }, "require": { "php": "^8.1.5", "aws/aws-sdk-php": "~3.2.6", "box/spout": "~3.0", "elasticsearch/elasticsearch": "^7.2", "graze/console-diff-renderer": "^0.6.2", "graze/parallel-process": "^0.8.1", "guzzlehttp/ringphp": "~1.1.1", "guzzlehttp/streams": "~3.0.0", "intervention/image": "2.6", "league/csv": "^9.2", "ongr/elasticsearch-dsl": "^7.2", "phpoffice/phpspreadsheet": "^1.23", "pimcore/pimcore": "^10.4", "react/promise": "~2.7.1", "simplesamlphp/saml2": "~3.4.1", "symfony/doctrine-messenger": "^5.3.0", "symfony/monolog-bundle": "^3.5", "wikimedia/composer-merge-plugin": "" }, "conflict": { "monolog/monolog": "^2.6.0" }, "require-dev": { "cache/integration-tests": "", "codeception/codeception": "^4.1.12" }, "autoload": { "psr-4": { "App\": ["src/"], "Pimcore\Model\DataObject\": "var/classes/DataObject", "Pimcore\Model\Object\": "var/classes/Object", "Website\": "legacy/website/lib" } }, "scripts": { "post-create-project-cmd": "Pimcore\Composer::postCreateProject", "post-install-cmd": [ "Pimcore\Composer::postInstall", "@pimcore-scripts" ], "post-update-cmd": [ "Pimcore\Composer::postUpdate", "@pimcore-scripts", "Pimcore\Composer::executeMigrationsUp", "@pimcore-scripts" ], "pimcore-scripts": [ "Pimcore\Composer::clearCache", "Pimcore\Composer::installAssets" ] }, "extra": { "symfony-app-dir": "src", "symfony-bin-dir": "bin", "symfony-var-dir": "var", "symfony-web-dir": "public", "symfony-tests-dir": "tests", "symfony-assets-install": "relative", "merge-plugin": { "include": [ "composer.local.json" ], "recurse": true, "replace": true, "merge-dev": true, "merge-extra": false, "merge-extra-deep": false, "merge-scripts": false } } }

Kindly suggest

ARY112 commented 2 years ago

@nadsella , Did you get a chance to look into this. This package is very critical for our project

nadsella commented 2 years ago

@ARY112 unfortunately our team size means that we can't prioritise this change currently. I'd suggest if you need an urgent solution, fork this repository and do the PHP upgrade for your own projects use. I can't foresee the team getting time to upgrade this repo over the next few days. Apologies for the inconvenience.

IVGraze commented 2 years ago

@ARY112 I'm definitely wanting to give it a go at the version upgrade with rector, but like @nadsella says, it's unlikely to be this week. If you do fork and do it, feel free to do a pull request so we can incorporate your upgrades. Otherwise I might manage it next week. 🤞

ARY112 commented 2 years ago

@IVGraze , Did you get a chance to look into this. Also graze/console-diff-renderer needs to be modified as well because it is the dependent package for parallel-process.

ARY112 commented 2 years ago

@nadsella @IVGraze , I have created below pull request: https://github.com/graze/parallel-process/pull/31

Can you please take a look.