On Thu, 08 Aug 2019 17:44:40 -0400 Mathieu Trudel-Lapierre < cypher...@ubuntu.com> wrote: > Package: lavacli > Version: 0.9.7-1 > Severity: normal > Tags: patch > User: ubuntu-de...@lists.ubuntu.com > Usertags: origin-ubuntu eoan ubuntu-patch > > Dear Maintainer, > > pyyaml 5.2.1 appears to have some changes in the default flow style used. > Rather than representing sequences and dicts similary to python, it now > defaults to printing them in longform [1]: > > - item 1 > - item 2 > > etc. > > I've written a quick and heavyhanded patch to add default_flow_style=None > to all of the calls to yaml.dump() that otherwise don't include options, but > that might not be the best solution. Maybe upstream would rather fix the tests? > > [1] https://github.com/yaml/pyyaml/issues/265 > > > *** /tmp/tmpton7Uz/bug_body > > In Ubuntu, the attached patch was applied to achieve the following: > > * debian/patches/fix_pyyaml_default_flow_style.patch: Make sure all calls > to yaml.dump() include default_flow_style=None, to retain previous behavior > that is checked for in tests, unless it's already otherwise specified. > > > Thanks for considering the patch.
The patch has been integrated upstream with https://git.lavasoftware.org/lava/lavacli/commit/1d374ccba0dc291e8d745ec90ffa8b4a32fb11af Thanks for your patch. Rgds -- Rémi Duraffort
