[Buildroot] [PATCH 1/1] package/tio: fix termio redefinition

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sun Dec 29 13:51:46 UTC 2019


Hello Fabrice,

On Fri, 27 Dec 2019 21:45:20 +0100
Fabrice Fontaine <fontaine.fabrice at gmail.com> wrote:

> + #include <sys/ioctl.h>
> +-#include <asm/ioctls.h>
> +-#include <asm/termbits.h>
> ++#include <asm-generic/ioctls.h>
> ++#include <asm-generic/termbits.h>

Thanks for proposing a patch. While I haven't had the chance to look
for a better solution, I wanted to state that directly including an
<asm-generic/foo.h> header from user-space is generally consider a very
bad thing. Not all architectures use the asm-generic headers for all
headers, and the fact that some use asm-generic headers is an internal
implementation detail.

So, this patch feels like a (potentially incorrect) workaround more
than a proper fix.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


More information about the buildroot mailing list