[PATCH] tools: binman: ti_board_cfg: improve error message

[Neha Malcom Francis]

Hi Michael,

On 26/03/24 15:09, Michael Walle wrote:
> When there is a lint error the user gets the following cryptic message:
> 
>    binman: Node '/path/to/some/node': Yamllint error: 18: comments
> 
> This isn't very helpful. Improve the message to tell the user that the
> number is actually a line number and also tell the user in which file
> they have to look.
> 
> Signed-off-by: Michael Walle <mwalle at kernel.org>
> ---
>   tools/binman/etype/ti_board_config.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/binman/etype/ti_board_config.py b/tools/binman/etype/ti_board_config.py
> index 2c3bb8f7b56..c10d66edcb1 100644
> --- a/tools/binman/etype/ti_board_config.py
> +++ b/tools/binman/etype/ti_board_config.py
> @@ -248,7 +248,7 @@ class Entry_ti_board_config(Entry_section):
>   
>               yaml_config = config.YamlLintConfig("extends: default")
>               for p in yamllint.linter.run(open(self._config_file, "r"), yaml_config):
> -                self.Raise(f"Yamllint error: {p.line}: {p.rule}")
> +                self.Raise(f"Yamllint error: Line {p.line} in {self._config_file}: {p.rule}")
>               try:
>                   validate(self.file_yaml, self.schema_yaml)
>               except Exception as e:

Reviewed-by: Neha Malcom Francis <n-francis at ti.com>

-- 
Thanking You
Neha Malcom Francis