* [U-Boot] [PATCH 1/1] buildman: Improve [make-flags] section parser to allow quoted strings
@ 2019-11-24 20:30 Cristian Ciocaltea
2019-12-10 13:05 ` Simon Glass
0 siblings, 1 reply; 2+ messages in thread
From: Cristian Ciocaltea @ 2019-11-24 20:30 UTC (permalink / raw)
To: u-boot
The parser responsible for the '[make-flags]' section in
the '.buildman' settings file is currently not able to
handle quoted strings, as given in the sample bellow:
[make-flags]
qemu_arm=HOSTCC="cc -isystem /add/include" HOSTLDFLAGS="-L/add/lib"
This patch replaces the simple string splitter based on the <space>
delimiter with a regex tokenizer that preserves spaces inside double
quoted strings.
Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>
---
tools/buildman/toolchain.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py
index cc26e2ede5..086a4d47cd 100644
--- a/tools/buildman/toolchain.py
+++ b/tools/buildman/toolchain.py
@@ -435,9 +435,10 @@ class Toolchains:
self._make_flags['target'] = board.target
arg_str = self.ResolveReferences(self._make_flags,
self._make_flags.get(board.target, ''))
- args = arg_str.split(' ')
+ args = re.findall("(?:\".*?\"|\S)+", arg_str)
i = 0
while i < len(args):
+ args[i] = args[i].replace('"', '')
if not args[i]:
del args[i]
else:
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH 1/1] buildman: Improve [make-flags] section parser to allow quoted strings
2019-11-24 20:30 [U-Boot] [PATCH 1/1] buildman: Improve [make-flags] section parser to allow quoted strings Cristian Ciocaltea
@ 2019-12-10 13:05 ` Simon Glass
0 siblings, 0 replies; 2+ messages in thread
From: Simon Glass @ 2019-12-10 13:05 UTC (permalink / raw)
To: u-boot
On Sun, 24 Nov 2019 at 13:30, Cristian Ciocaltea
<cristian.ciocaltea@gmail.com> wrote:
>
> The parser responsible for the '[make-flags]' section in
> the '.buildman' settings file is currently not able to
> handle quoted strings, as given in the sample bellow:
>
> [make-flags]
> qemu_arm=HOSTCC="cc -isystem /add/include" HOSTLDFLAGS="-L/add/lib"
>
> This patch replaces the simple string splitter based on the <space>
> delimiter with a regex tokenizer that preserves spaces inside double
> quoted strings.
>
> Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com>
> ---
> tools/buildman/toolchain.py | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-12-10 13:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-24 20:30 [U-Boot] [PATCH 1/1] buildman: Improve [make-flags] section parser to allow quoted strings Cristian Ciocaltea
2019-12-10 13:05 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.