Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
#1920902: Add a Drupal Yaml wrapper so we can default to PECL Yaml component if it is available would like to add support for the PECL YAML parser which is significantly faster then the in-php implementation from Symfony. To test this and make sure the included tests catch broken YAML testbot need to have the YAML extension installed.
Proposed resolution
Include PECL YAML in the php containers.
Remaining tasks
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | yaml.patch | 1.13 KB | neclimdul |
Comments
Comment #2
neclimdulComment #3
neclimdulDon't really know how to test this but this seems like the appropriate change.
Comment #4
dawehnerWhat about running all core tests inside a patched test runner, just to be sure?
Comment #7
alan evans commentedI was just wondering why the YAML extension is only added here for 5.5 and 5.6 and the conclusion I came to was because 1) it only makes sense to add for versions that are supported on Drupal 8 and 2) Maybe there is no version of the extension available yet for PHP 7? Is that about right?
For future reference, it seems like some work has been done on it ( https://github.com/php/pecl-file_formats-yaml/tree/php7 ) but I doubt there's any easy package for install.
Oh and hi btw @neclimdul :) Long time no see ... hope you're well.
Comment #8
MixologicMostly #2. There doesnt yet exist a stable yaml extention for php7.
Comment #9
neclimdulYeah that about sums it up.
Hey Alan! Good to "see" you. ;)
Comment #11
yesct commentedAnd... now we need one without it also.
#2597762: Add a new testbot with (only) Symfony yaml parser (not the pecl one)
Comment #12
MixologicComment #13
elachlan commentedMoved to DrupalCI Environments.
Comment #14
MixologicThis now exists at the container level, and should be removed from the testrunner.