Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
As 8.5 is now officially unsupported, I just created one last release that requires it so we can now increase our dependency version and tackle our associated tasks.
Comment | File | Size | Author |
---|---|---|---|
#7 | 3053200-7--drupal__8_6_dependency--without_composer_change.patch | 1.86 KB | drunken monkey |
| |||
#7 | 3053200-7--drupal__8_6_dependency.patch | 2.2 KB | drunken monkey |
|
Comments
Comment #2
drunken monkeyTrivial patch attached.
Comment #3
borisson_Not sure what I can add as a comment here. The patch looks good :)
Comment #4
mkalkbrennerPeople who manage their environment via composer should not be surprised after they "successfully" run
composer update
!Comment #5
borisson_Ah yes, that should help indeed. Good idea @mkalkbrenner!
Comment #6
drunken monkeyCan you explain in detail which scenario you’re trying to prevent?
Drupal’s packagist repository will already use data from
.info.yml
files to enrich the Composer information of its packages. So, it will already be impossible to install the latest Search API dev version (or 1.14, once that’s out) together with Drupal 8.5. (You can try with Drupal 8.4 and Search API 1.9+.)If adding this explicit
conflict
directive would indeed help for some cases, I’d find it a lot better if Drupal’s Packagist engine already added that automatically. Doesn’t seem sensible to do this manually for all packages with a minimum Core version.Comment #7
drunken monkeyRevised patch to also include clean-up for #2881689: Tracker-Sync Task does not scale well with a lot of content.
Comment #8
mkalkbrennerYou're right that you can't install it. But you since the requirements and conflicts are not auto-generated . This is the generated composer.json of Search API 1.13:
it misses the conflict for drupal cores before 8.5 as declared in the generated search_api.info.yml:
So if you install any outdated core, composer installs drupal/search_api:^1.13 perfectly. But you'll run into issues when you try to install search_api afterwards or run drush updb.
Comment #9
drunken monkeyAs said, I cannot reproduce this:
Comment #10
borisson_I cannot reproduce this either.
Comment #12
drunken monkeyGreat to hear, thanks a lot for verifying, Joris!
Committed.
Markus: Also note that we didn’t use this until now, either, and never had a complaint like that (as far as I remember).
Anyways, thanks again, everyone!