[dpdk-dev] [PATCH] mk: fix command-line dependency check for quoted strings
Thomas Monjalon
thomas.monjalon at 6wind.com
Fri Nov 15 15:55:37 CET 2013
15/11/2013 15:38, Adrien Mazarguil :
> Before recompiling a file, rte.compile-pre.mk checks whether the command
> line is different from the previous one.
>
> This is done by storing for each object file the entire command line in a
> kind of dependency file with a .cmd extension (see obj2cmd). If that file
> exists, the line is retrieved first and compared against $(C_TO_O_STR).
> The object file gets recompiled if the file doesn't exist or if the line
> is different.
>
> The problem is that sometimes, files are recompiled for no apparent reason.
>
> The check doesn't work properly when a command line contains double-quoted
> strings such as -DFOO='"bar"' because the shell interprets and strips them.
> This is fixed by protecting C_TO_O_CMD with simple quotes, knowing that
> such quotes are already escaped in C_TO_O_STR.
>
> Moreover, because simple quotes are escaped in C_TO_O_STR, the retrieved
> command should be compared against C_TO_O instead.
>
> Signed-off-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
Acked and applied.
Thanks
--
Thomas
More information about the dev
mailing list