[PATCH vkd3d v4 2/7] build: Make the sed(1) command more portable.
cdavis at codeweavers.com
Fri Aug 10 16:01:29 CDT 2018
BSD sed(1) requires an argument after -i giving the suffix to put on the
backup it makes. Also, the regular expression given doesn't seem to work
for me. This command should work everywhere.
Signed-off-by: Chip Davis <cdavis at codeweavers.com>
Try 3: Remove possibly redundant and/or unsupported '-f' option. The
definition of RM might already have it, or even be something (like
the DEL command on Windows) which doesn't support it.
(As if anyone would want to build this on Windows... :)
Try 4: We're not specifying a recognized extension to widl(1) anymore,
so we need to explicitly tell it to generate a header now.
Makefile.am | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 14f2d51..770b6e0 100644
@@ -150,8 +150,9 @@ vkd3d_v_widl_1 =
EXTRA_DIST += $(widl_headers) $(widl_headers:.h=.idl)
$(widl_headers): %.h: %.idl
- $(VKD3D_V_WIDL)$(WIDL) -o $@ $<
- sed -e '/#include <\(rpc.h\|rpcndr.h\)>/d' -i $@
+ $(VKD3D_V_WIDL)$(WIDL) -h -o $@.tmp $<
+ $(SED) -e '/#include <rpc.h>/d' -e '/#include <rpcndr.h>/d' <$@.tmp >$@
+ $(RM) $@.tmp
echo "widl is required to generate $<"
More information about the wine-devel