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

Drop packages defined by Drupal itself from composer.json #50

Open
theMusician opened this issue Apr 22, 2022 · 0 comments
Open

Drop packages defined by Drupal itself from composer.json #50

theMusician opened this issue Apr 22, 2022 · 0 comments
Assignees
Labels
Composer Dancing to the tune of dependencies Release Duties Things nobody wants to do
Milestone

Comments

@theMusician
Copy link
Contributor

theMusician commented Apr 22, 2022

Upon trying to upgrade to Drupal core 9.3.12 composer tossed out a few errors.
docker exec -ti esmero-php bash -c "composer require drupal/core:9.3.12 drupal/core-dev:9.3.12 drupal/core-composer-scaffold:9.3.12 drupal/core-project-message:9.3.12 drupal/core-recommended:9.3.12 -W"

First,
drupal/core-recommended 9.3.11 requires egulias/email-validator 3.1.2 -> found egulias/email-validator[3.1.2] but it conflicts with your root composer.json require (^2.0).

and then after updating that to be ^3.0 manually in composer.json and rerunning the command

drupal/core-recommended 9.3.11 requires typo3/phar-stream-wrapper v3.1.7 -> found typo3/phar-stream-wrapper[v3.1.7] but it conflicts with your root composer.json require (3.1.6).

Fixing those both manually let the command succeed and the upgrade worked.

In conferring on Slack with Diego it seems Drupal itself now declares these packages as dependencies so they are no longer needed in the default composer.json file.


docker exec -ti esmero-php bash -c "composer why egulias/email-validator"

drupal/core           9.3.11     requires  egulias/email-validator (^2.1.22|^3.0)  
symfony/mime          v5.4.7     conflicts egulias/email-validator (~3.0.0)

docker exec -ti esmero-php bash -c "composer why typo3/phar-stream-wrapper"

drupal-composer/drupal-project  -       requires  typo3/phar-stream-wrapper (3.1.7)
drupal/core                     9.3.12  requires  typo3/phar-stream-wrapper (^3.1.3)
drupal/core-recommended         9.3.12  requires  typo3/phar-stream-wrapper (v3.1.7)
@DiegoPino DiegoPino self-assigned this Apr 22, 2022
@DiegoPino DiegoPino added Release Duties Things nobody wants to do Composer Dancing to the tune of dependencies labels Apr 22, 2022
@DiegoPino DiegoPino added this to the 1.0.0 milestone Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Composer Dancing to the tune of dependencies Release Duties Things nobody wants to do
Projects
None yet
Development

No branches or pull requests

2 participants