On Tue, Jan 19, 2016 at 11:11 AM, Andy Shevchenko
<[email protected]> wrote:
> On Tue, 2016-01-19 at 10:53 -0500, Paul Moore wrote:
>> On Fri, Dec 4, 2015 at 3:07 PM, Andy Shevchenko
>> <[email protected]> wrote:
>> > Compiler warns us a lot that it can't find include folder because
>> > it's provided
>> > in relative form.
>> >
>> >   CC      security/selinux/netlabel.o
>> > cc1: warning: security/selinux/include: No such file or directory
>> > [-Wmissing-include-dirs]
>> > cc1: warning: security/selinux/include: No such file or directory
>> > [-Wmissing-include-dirs]
>> > cc1: warning: security/selinux/include: No such file or directory
>> > [-Wmissing-include-dirs]
>> > cc1: warning: security/selinux/include: No such file or directory
>> > [-Wmissing-include-dirs]
>> >
>> > Add $(srctree) prefix to the path.
>> >
>> > Signed-off-by: Andy Shevchenko <[email protected]>
>> > ---
>> >  security/selinux/Makefile | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/security/selinux/Makefile b/security/selinux/Makefile
>> > index ad5cd76..90d1adb 100644
>> > --- a/security/selinux/Makefile
>> > +++ b/security/selinux/Makefile
>> > @@ -13,7 +13,7 @@ selinux-$(CONFIG_SECURITY_NETWORK_XFRM) += xfrm.o
>> >
>> >  selinux-$(CONFIG_NETLABEL) += netlabel.o
>> >
>> > -ccflags-y := -Isecurity/selinux -Isecurity/selinux/include
>> > +ccflags-y := -Isecurity/selinux
>> > -I$(srctree)/security/selinux/include
>>
>> Why not add the $(srctree) prefix to both paths?
>
> I don't care as long as it fixes the annoying messages. Though, I can
> re-do the patch if it looks better that way.

Please do.

-- 
paul moore
www.paul-moore.com
_______________________________________________
Selinux mailing list
[email protected]
To unsubscribe, send email to [email protected].
To get help, send an email containing "help" to [email protected].

Reply via email to