Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
We currently ship with composer 1.0.0
, which is pretty dated, and causes version parsing problems with modern packages.
Example:
[UnexpectedValueException]
Could not parse version constraint ^v1.0: Invalid version string "^v1.0"
Proposed resolution
Currently the way to work around this is to edit the composer.json
yourself and update. However there is no reason that I know to stay on 1.0.0
, so let's use the latest version 1.7.3
so users do not need to struggle with this when they install.
Comment | File | Size | Author |
---|---|---|---|
#6 | 3013044-6.patch | 7.11 MB | suzymasri |
#2 | 3013044-2.patch | 6.74 MB | Manuel Garcia |
Comments
Comment #2
Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer and at Appnovation for Pfizer, Inc. commentedComment #3
jribeiro CreditAttribution: jribeiro for Pfizer, Inc. commented+1 RTBC
Comment #4
geoffreyr CreditAttribution: geoffreyr commented+1. I've hit issues with other packages by being stuck on 1.0.0, because of its out of date version constraint parsing. The sooner the version of Composer bundled with this gets bumped, the better.
Might also put in a patch to bump to 1.8.0.
Comment #5
bburg+1 RTBC.
Yes, and it doesn't seem that the extension without this patch will run in PHP 7.
Comment #6
suzymasriPatch to upgrade Composer to 1.9.3, which is currently the latest stable release.
Comment #7
suzymasriComment #8
RobLoachThanks so much, all!
Applied the patch, and had some issues with the binary files. Ended up touching the patch a bit, and re-installing the dependencies. Let me know if this commit is good:
https://git.drupalcode.org/project/composer/-/commit/b63117119b7955ffe2f...
I've also added suzy and tim as maintainers. Feel free to push any patches forwards you find appropriate!
Comment #10
Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer and at Appnovation for Pfizer, Inc. commentedfantastic news, thanks all!