On Wed, Jul 07, 2021 at 05:15:44PM +0200, Jan Beulich wrote:
> On 01.07.2021 16:09, Anthony PERARD wrote:
> > The make variable $(subdir-y) isn't used yet but will be in a
> > following patch. Anything in $(subdir-y) doesn't to have a '/' as
> > suffix as we already now it's a directory.
> >
> > Rework the rules so that it doesn't matter whether there is a '/' or
> > not. It also mimic more closely to the way Linux's Kbuild descend in
> > subdirectories.
> >
> > FORCE phony target isn't needed anymore running clean, so it can be
> > removed.
> >
> > Signed-off-by: Anthony PERARD <anthony.per...@citrix.com>
>
> Reviewed-by: Jan Beulich <jbeul...@suse.com>
Thanks.
> > --- a/xen/scripts/Makefile.clean
> > +++ b/xen/scripts/Makefile.clean
> > @@ -12,19 +12,18 @@ include Makefile
> > # Figure out what we need to clean from the various variables
> > #
> > ==========================================================================
> > subdir-all := $(subdir-y) $(subdir-n) $(subdir-) \
> > - $(filter %/, $(obj-y) $(obj-n) $(obj-))
> > + $(patsubst %/,%, $(filter %/, $(obj-y) $(obj-n) $(obj-)))
>
> Isn't this a normalization which also wants doing in xen/Rules.mk for
> subdir-y? Or perhaps this is part of one of the subsequent patches
> already?
Indeed, a similar change is done as part of
build: build everything from the root dir, use obj=$subdir
Cheers,
--
Anthony PERARD
