ffbuild: Avoid using the --preprocessor argument to windres
authorMartin Storsjö <martin@martin.st>
Fri, 14 May 2021 21:42:38 +0000 (00:42 +0300)
committerMartin Storsjö <martin@martin.st>
Fri, 18 Jun 2021 08:19:02 +0000 (11:19 +0300)
commitf9626d1065c43f1d51afe66bdf988b9f33729440
tree5728dd1aed23dd1269673c6f4b6ad5d9b13e4d18
parent604924a069735f3f1fd56b5dd125e68d402f15ba
ffbuild: Avoid using the --preprocessor argument to windres

Instead use --preprocessor-arg; in binutils 2.36, the --preprocessor
flag was changed so that it no longer accepts a string containing
multiple arguments, but the whole --preprocessor argument is
treated as the path to the preprocessor executable (where the path
can contain spaces).

It's currently unclear whether this behaviour will stay or if it
is going to be reverted in the future, see discussion at [1]. Just
to be safe, avoid using the --preprocessor argument. Don't redeclare
the full preprocessing command, but just add the $(CC_DEPFLAGS) options.

Based on a patch by Kyle Schwartz.

[1] https://sourceware.org/bugzilla/show_bug.cgi?id=27594

Signed-off-by: Martin Storsjö <martin@martin.st>
configure
ffbuild/common.mak