Re: [PATCH v1 1/1] selinux: use absolute path to include directory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Dec 4, 2015 at 3:07 PM, Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> 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 <andriy.shevchenko@xxxxxxxxxxxxxxx>
> ---
>  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?

>  $(addprefix $(obj)/,$(selinux-y)): $(obj)/flask.h
>
> --
> 2.6.2
>

-- 
paul moore
www.paul-moore.com
_______________________________________________
Selinux mailing list
Selinux@xxxxxxxxxxxxx
To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx.
To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.



[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux