(buildstream) 01/01: _yaml: set a large 'width' value in the roundtrip yaml config

[ak...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

akitouni pushed a commit to branch abderrahim/roundtrip
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 6258d8936f44c12940be586d93e7c6c409cd6582
Author: Abderrahim Kitouni <ak...@gnome.org>
AuthorDate: Mon Aug 25 14:32:58 2025 +0100

    _yaml: set a large 'width' value in the roundtrip yaml config
    
    At some point, ruamel-yaml started line wrapping at 80 columns, breaking our
    'roundtrip' promise
---
 src/buildstream/_yaml.pyx                | 2 ++
 tests/internals/yaml/roundtrip-test.yaml | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/src/buildstream/_yaml.pyx b/src/buildstream/_yaml.pyx
index cc8da6aa7..c962bb881 100644
--- a/src/buildstream/_yaml.pyx
+++ b/src/buildstream/_yaml.pyx
@@ -389,6 +389,8 @@ yaml.RoundTripConstructor.add_constructor(u'tag:yaml.org,2002:timestamp',
 def prepare_roundtrip_yaml():
     yml = yaml.YAML()
     yml.preserve_quotes=True
+    # defaults to 80 if we don't set it
+    yml.width=9999
 
     # For each of YAML 1.1 and 1.2, force everything to be a plain string
 
diff --git a/tests/internals/yaml/roundtrip-test.yaml b/tests/internals/yaml/roundtrip-test.yaml
index 2e3f5c721..c81b86020 100644
--- a/tests/internals/yaml/roundtrip-test.yaml
+++ b/tests/internals/yaml/roundtrip-test.yaml
@@ -51,5 +51,7 @@ nullcheck: null
 
 timestamp: 2019-03-14
 
+long-string: Sometimes very long strings get truncated by ruamel-yaml because the default width is set to 80.
+
 # That is all