[Pkg-linaro-lava-devel] Bug#934269: lavacli: default_flow_style changed in newer pyyaml

[Remi Duraffort]

On Thu, 08 Aug 2019 17:44:40 -0400 Mathieu Trudel-Lapierre <
cyphermox at ubuntu.com> wrote:
> Package: lavacli
> Version: 0.9.7-1
> Severity: normal
> Tags: patch
> User: ubuntu-devel at 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