Problem/Motivation

Currently the module is not installable with composer. Composer returns the following message

Problem 1
    - drupal/subrequests 2.0.0-beta1 requires skyscanner/jsonpath dev-master -> no matching package found.
    - drupal/subrequests 2.0.0-beta2 requires skyscanner/jsonpath dev-master -> no matching package found.
    - drupal/subrequests 2.0.0-beta3 requires skyscanner/jsonpath dev-master -> no matching package found.
    - drupal/subrequests 2.0.0-rc1 requires skyscanner/jsonpath dev-master -> no matching package found.
    - drupal/subrequests 2.x-dev requires skyscanner/jsonpath dev-master -> no matching package found.

It also makes the contentacms project setup failing at the moment.

When I looked into the skyscanner/jsonpath library on packagist, I saw that the dev-master release is not published anymore.

Proposed resolution

It might be the responsibility of the maintainer of the library to promote the dev-master again, but maybe should make use of the tagged release, since the library has a stable 1.0 tag now.

CommentFileSizeAuthor
#2 subrequests-use_json_path_stable_release-2948057-2.patch398 bytesszeidler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

szeidler created an issue. See original summary.

szeidler’s picture

szeidler CreditAttribution: szeidler at Ramsalt Lab commented
Status: Active » Needs review
FileSize
subrequests-use_json_path_stable_release-2948057-2.patch398 bytes

Here's a patch for the version constraints.

Status: Needs review » Needs work

The last submitted patch, 2: subrequests-use_json_path_stable_release-2948057-2.patch, failed testing. View results

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented
Status: Needs work » Fixed

Thanks for the patch!

murat_halici’s picture

murat_halici CreditAttribution: murat_halici commented

I'm also running into contenta install failures due to this issue. Just wondering how you were able to apply the patch and get the install to work. Thanks!

szeidler’s picture

szeidler CreditAttribution: szeidler at Ramsalt Lab commented

Unfortunately it's not possible to patch composer.json definitions with composer-patches. What I did as a workaround in an existing contentacms installation, was to add a new dependency in the root composer.json "require" section, like this.

"skyscanner/jsonpath": "1.0 as dev-master"

It's an inline alias, that will load the release 1.0 when `dev-master` (which is missing) is required for the jsonpath library.

  • e0ipso committed 8c688a3 on 8.x-2.x authored by szeidler 
    Issue #2948057 by szeidler, e0ipso: skyscanner/jsonpath dependency - no...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented

I was unable to push this for a while. It should be available in the latest release.

Status: Fixed » Closed (fixed)

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