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

Re: [Xen-devel] [RFC PATCH 6/8] Config.mk: expand cc-ver a bit



>>> On 06.10.16 at 16:37, <wei.liu2@xxxxxxxxxx> wrote:
> --- a/Config.mk
> +++ b/Config.mk
> @@ -112,17 +112,17 @@ endef
>  
>  cc-options-add = $(foreach o,$(3),$(call cc-option-add,$(1),$(2),$(o)))
>  
> -# cc-ver: Check compiler is at least specified version. Return boolean 
> 'y'/'n'.
> -# Usage: ifeq ($(call cc-ver,$(CC),0x030400),y)
> +# cc-ver: Check compiler against the version requirement. Return boolean 
> 'y'/'n'.
> +# Usage: ifeq ($(call cc-ver,$(CC),-ge,0x030400),y)
>  cc-ver = $(shell if [ $$((`$(1) -dumpversion | awk -F. \
> -           '{ printf "0x%02x%02x%02x", $$1, $$2, $$3}'`)) -ge $$(($(2))) ]; \
> +           '{ printf "0x%02x%02x%02x", $$1, $$2, $$3}'`)) $(2) $$(($(3))) ]; 
> \
>             then echo y; else echo n; fi ;)
>  
>  # cc-ver-check: Check compiler is at least specified version, else fail.
>  # Usage: $(call cc-ver-check,CC,0x030400,"Require at least gcc-3.4")
>  cc-ver-check = $(eval $(call cc-ver-check-closure,$(1),$(2),$(3)))
>  define cc-ver-check-closure
> -    ifeq ($$(call cc-ver,$$($(1)),$(2)),n)
> +    ifeq ($$(call cc-ver,$$($(1)),-ge,$(2)),n)

Could you keep the dash in the macro, making such invocations look
a little more "normal"?

Jan


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

 


Rackspace

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