Re: [PATCH] bootconfig: Fix to terminate value search if it hits a newline

[Steven Rostedt <rostedt@goodmis.org>]

On Fri, 30 Jan 2026 13:41:45 +0900
"Masami Hiramatsu (Google)" <mhiramat@kernel.org> wrote:

> Note that if the line has a comment, it still keep searching the
> value to the next line::
> 
>   foo = # comment
>   value

Hmm, so the above is the same as: foo = "value";

But what is:

    foo = value # comment
    bar

 ?

Would that be foo = value bar;

Or simply an error. Or would both be an error?

OK, this isn't related to the patch, but as comment processing was brought
to my attention, I'm wondering how it works for the above.

> 
> This is interpreted as `foo = "value"`.
> This also updates the test script so that it can check the above cases.
> 
> Fixes: 76db5a27a827 ("bootconfig: Add Extra Boot Config support")
> Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
> ---
>  tools/bootconfig/test-bootconfig.sh |   18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)

As for the patch itself:

Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>

-- Steve