[PATCH] qemu: Do not include file if not exists

Oleksiy Obitotskyy posted 1 patch 2 weeks ago
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test FreeBSD passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200325085246.21118-1-oobitots@cisco.com
Maintainers: Riku Voipio <riku.voipio@iki.fi>, Laurent Vivier <laurent@vivier.eu>
linux-user/syscall.c | 2 ++
1 file changed, 2 insertions(+)

[PATCH] qemu: Do not include file if not exists

Posted by Oleksiy Obitotskyy 2 weeks ago
Script configure checks for if_alg.h and check failed but
if_alg.h still included.

Signed-off-by: Oleksiy Obitotskyy <oobitots@cisco.com>
---
 linux-user/syscall.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 35f4146662..819774af0d 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -109,7 +109,9 @@
 #include <linux/blkpg.h>
 #include <netpacket/packet.h>
 #include <linux/netlink.h>
+#if defined(CONFIG_AF_ALG)
 #include <linux/if_alg.h>
+#endif
 #include <linux/rtc.h>
 #include <sound/asound.h>
 #include "linux_loop.h"
-- 
2.19.1


Re: [PATCH] qemu: Do not include file if not exists

Posted by Markus Armbruster 1 week ago
Cc'ing maintainers.

Oleksiy, this looks like a respin of your patch.  Since it is buried in
the original thread, our patch processing automation won't pick it up,
and humans may well miss it, too.  Next time, start a new thread, with
proper [PATCH v2] versioning in the subject, and cc: the maintainers
(use scripts/get_maintainer.pl to find them).  Also collect the
Reviewed-by tags, if any, and explain what changed since the previous
version.

https://wiki.qemu.org/Contribute/SubmitAPatch

I know getting everything just right seems overwhelming at first.  Don't
worry about making innocent mistakes, just learn from them :)

Thanks for your fix!