Re: [Xen-devel] [PATCH] build: fix dependencies for files compiled from their parent directory

[Jonathan Creekmore <jonathan@xxxxxxxxxxxxxxxxx>]

On Wed, Nov 25, 2015 at 10:16 AM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
> The use of $(basename ...) here was wrong (yet I'm sure I tested it).
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
>
> --- a/xen/Rules.mk
> +++ b/xen/Rules.mk
> @@ -105,7 +105,7 @@ include Makefile
>  DEPS = .*.d
>  define gendep
>      ifneq ($(1),$(subst /,:,$(1)))
> -        DEPS += $(dir $(1)).$(basename $(notdir $(1))).d
> +        DEPS += $(dir $(1)).$(notdir $(1)).d
>      endif
>  endef
>  $(foreach o,$(filter-out %/,$(obj-y)),$(eval $(call gendep,$(o))))

I finally got a chance to take a look at this patch today and it does fix the
dependency tracking for files in the xen/common/compat directory. However,
it still doesn't do anything to fix the failure to clean the object
files that are
left behind in that directory by 'make clean'. After running it, I still have:

$ find . -name "*.o*"
./common/compat/kernel.o
./common/compat/domain.o
./common/compat/tmem_xen.o
./common/compat/memory.o
./common/compat/multicall.o
./common/compat/xlat.o

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel