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.

-- 
Andy Shevchenko <[email protected]>
Intel Finland Oy

_______________________________________________
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