Re: [lustre-devel] [PATCH 1/2] staging: lustre: Add include path to Makefile

James Simmons
2016-08-15T21:08:57Z

> On Mon, Aug 15, 2016 at 12:33:23PM -0700, Joe Perches wrote:
> > Start to rationalize include paths in source code files.
> >
> > Signed-off-by: Joe Perches <joe@perches.com>
> > ---
> > drivers/staging/lustre/Makefile | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/staging/lustre/Makefile b/drivers/staging/lustre/Makefile
> > index 95ffe33..9d00237 100644
> > --- a/drivers/staging/lustre/Makefile
> > +++ b/drivers/staging/lustre/Makefile
> > @@ -1,2 +1,5 @@
> > +subdir-ccflags-y += -I$(srctree)/drivers/staging/lustre/include/
> > +subdir-ccflags-y += -I$(srctree)/drivers/staging/lustre/lustre/include/
> > +
> > obj-$(CONFIG_LNET) += lnet/
> > obj-$(CONFIG_LUSTRE_FS) += lustre/
>
> This is good, but does this break the subdir make command:
> make M=drivers/staging/lustre/foo_dir/
> ?
>
> I remember the last time I tried to clean this up, it took a while...

It breaks make M=drivers/staging/lustre/foo_dir