Drupal.org's Composer endpoint uses a different version schema than Drupal-Packagist's. Page Manager uses the Drupal-Packagist version scheme in its composer.json which does not work with Drupal.org.

CommentFileSizeAuthor
#7 make_page_manager-2746149-7.patch278 bytesmglaman
#2 2746149-2.patch403 bytesDevin Carlson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Devin Carlson created an issue. See original summary.

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Title: Make pathauto compatible with Drupal Packagist and Drupal.org » Make Page Manager compatible with Drupal Packagist and Drupal.org
Status: Active » Needs review
Related issues: +#2727559: Make pathauto compatible with Drupal Packagist and Drupal.org
FileSize
2746149-2.patch403 bytes

This was also an issue with Pathauto #2727559: Make pathauto compatible with Drupal Packagist and Drupal.org which was addressed by removing the dependencies.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented

But Page Manager does depend on CTools. Shouldn't it need to be in the composer.json file?

mvdve’s picture

mvdve CreditAttribution: mvdve commented

The dependency should be in the composer.json file, but with the current ~8.3.0-alpha21 dependency, it breaks the page_manager module in combination with panels because of #2769405: AjaxError when trying to Add new block

Only the dev branch of ctools is up-to-date.

sdstyles’s picture

sdstyles CreditAttribution: sdstyles at FFW commented
Category: Task » Bug report
Priority: Normal » Major

Same thing for me, I'm using Composer as package manager for modules dependencies, I have ctools 8.x-3.x-dev already installed, installing page_manager via composer requires dependency compatible with 8.x-3.0-alphaX so I'm not able to install the module without update ctools.

I updated the issue because this is a blocker which doesn't allow you to install latest ctools 8.x-3.x-dev branch.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman as a volunteer commented 
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for drupal/page_manager 1.x-dev -> satisfiable by drupal/page_manager[1.x-dev].
    - drupal/page_manager 1.x-dev requires drupal/ctools ~3.0.0-alpha21 -> satisfiable by drupal/ctools[3.0.0-alpha27, 3.0.0-alpha26, 3.0.0-alpha25, 3.0.0-alpha24, 3.0.0-alpha23, 3.0.0-alpha22, 3.0.0-alpha21].
    - Can only install one of: drupal/ctools[3.0.0-alpha27, 8.3.0-alpha27].
    - Can only install one of: drupal/ctools[3.0.0-alpha26, 8.3.0-alpha27].
    - Can only install one of: drupal/ctools[3.0.0-alpha25, 8.3.0-alpha27].
    - Can only install one of: drupal/ctools[3.0.0-alpha24, 8.3.0-alpha27].
    - Can only install one of: drupal/ctools[3.0.0-alpha23, 8.3.0-alpha27].
    - Can only install one of: drupal/ctools[3.0.0-alpha22, 8.3.0-alpha27].
    - Can only install one of: drupal/ctools[3.0.0-alpha21, 8.3.0-alpha27].
    - Installation request for drupal/ctools (locked at 8.3.0-alpha27) -> satisfiable by drupal/ctools[8.3.0-alpha27].


Installation failed, reverting ./composer.json to its original content.

Should change "drupal/ctools": "~8.3.0-alpha21" to "drupal/ctools": "~3"

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman as a volunteer commented
FileSize
make_page_manager-2746149-7.patch278 bytes

Here is updated patch to make resolving dependencies a lot simpler and prevent conflicts.

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
CreditAttribution: Mixologic at Drupal Association commented

Can you update https://www.drupal.org/node/2746737 when this one gets fixed?

  • EclipseGc committed 0859721 on 8.x-1.x authored by mglaman 
    Issue #2746149 by Devin Carlson, mglaman: Make Page Manager compatible...
EclipseGc’s picture

EclipseGc CreditAttribution: EclipseGc commented
Status: Needs review » Fixed

Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Status: Closed (fixed) » Needs work

The last submitted patch, 7: make_page_manager-2746149-7.patch, failed testing.

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
CreditAttribution: Mixologic at Drupal Association commented
Status: Needs work » Fixed

Weird. Why did that retest?

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.