drupal/openai dev-1.0.x requires drupal/core ^10

Created on 3 March 2023, about 2 years ago
Updated 27 May 2023, almost 2 years ago

Hi, is there a specific reason why the dev branch is only compatible with Drupal 10? I would like to use the latest module changes, but we are still using Drupal 9.

✨ Feature request
Status

Closed: works as designed

Version

1.0

Component

Code

Created by

πŸ‡¨πŸ‡ΏCzech Republic siva01

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @siva01
  • πŸ‡¨πŸ‡ΏCzech Republic siva01

    I tried clone dev module to our Drupal 9 project. I changed required core. For now I don't find any core related issue, so it looks, that dev can work with Drupal 9 too.

  • πŸ‡ΊπŸ‡ΈUnited States kevinquillen

    We can support it by adding the following to info files:

    php: 8.1

    This ensures that as long as you have 8.1, ^9 || ^10 will work.

  • @kevinquillen opened merge request.
  • πŸ‡ΊπŸ‡ΈUnited States kevinquillen

    How did you get it to be okay with the following?

    --- Errors ---
    You are using the deprecated option "--no-suggest". It has no effect and will break in Composer 3.
    ./composer.json has been updated
    Running composer update drupal/openai
    > Drupal\Composer\Composer::ensureComposerVersion
    Loading composer repositories with package information
    Updating dependencies
    Your requirements could not be resolved to an installable set of packages.
    
      Problem 1
        - openai-php/client[v0.3.0, ..., v0.3.4] require guzzlehttp/guzzle ^7.5.0 -> found guzzlehttp/guzzle[dev-master, 7.5.0, 7.5.x-dev (alias of dev-master)] but the package is fixed to 6.5.8 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
        - drupal/openai 1.0.x-dev is an alias of drupal/openai dev-1.0.x and thus requires it to be installed too.
        - drupal/openai dev-1.0.x requires openai-php/client ^v0.3.0 -> satisfiable by openai-php/client[v0.3.0, ..., v0.3.4].
        - Root composer.json requires drupal/openai 1.0.x-dev -> satisfiable by drupal/openai[1.0.x-dev (alias of dev-1.0.x)].
    
    Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
  • πŸ‡ΊπŸ‡ΈUnited States kevinquillen

    I reverted the commit. Here is the patch file for the work.

    We are using the OpenAI PHP Client which requires 8.1, and Guzzle 7.5+. That is why it is flagged as Drupal 10.

    https://github.com/openai-php/client/blob/main/composer.json#L14

  • Issue was unassigned.
  • Status changed to Needs work about 2 years ago
  • Status changed to Closed: works as designed almost 2 years ago
  • πŸ‡ΊπŸ‡ΈUnited States SocialNicheGuru

    I suppose that if you are a site builder/dev you can upgrade to guzzle 7.5+ on Drupal 9 at your own risk here: https://www.drupal.org/node/3268032#core β†’

Production build 0.71.5 2024