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

Re: [Xen-devel] [PATCH ARM v3 1/7] mini-os: build fixes

Thomas Leonard, le Wed 11 Jun 2014 11:30:12 +0100, a écrit :
> Make .o rules depend on the includes. Before, only the final link step
> depended on setting up the includes directory, making parallel builds
> unreliable.
> Make symlinks use explicit make rules instead of using a phony target.
> Avoids unnecessary rebuilds.
> Signed-off-by: Thomas Leonard <talex5@xxxxxxxxx>

This makes sense, just one thing:

> +LINKS = include/list.h $(ARCH_LINKS) include/mini-os include/xen 
> include/$(TARGET_ARCH_FAM)/mini-os
> +
> +EXTRA_DEPS += $(LINKS) include/list.h
> +

include/list.h thus appears twice in EXTRA_DEPS.  I guess there is no
need for an intermediate LINKS variable.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.