Re: [Xen-devel] [XEN PATCH v3 2/6] xen: Have Kconfig check $(CC)'s version

[Jan Beulich <jbeulich@xxxxxxxx>]

On 17.01.2020 17:23, Anthony PERARD wrote:
> On Thu, Jan 16, 2020 at 01:40:39PM +0100, Jan Beulich wrote:
>> On 16.01.2020 13:29, Anthony PERARD wrote:
>> Indeed, hence also my "as suggested before". I remain unconvinced
>> that is it useful to have e.g.
>>
>> CONFIG_GCC_VERSION=80300
>> CONFIG_CLANG_VERSION=0
>>
>> in xen/.config. This is at best confusing, because it may not
>> represent what the system actually has installed (which I realize
>> is also not the intention, but the variable naming suggests that
>> this is what was found on the system; I have no better naming
>> suggestion, to preempt a possible question to this effect).
> 
> After a talk on #xendevel yesterday, I have George agreeing that we
> should keep the same behavior as the one Linux have. And Ian saying that
> we should copy entire files where we can. If we modify the behavior of
> %_VERSION, it would make it more difficult to copy entire files from
> %Linux later.
> 
> So, now, can we finally commit the patch series, with both %_VERSION
> set, and let this bikeshedding rest, and move on?

Well, someone feel free to go ahead then. Every time I'll get to see
this I'll be tempted to clean up the redundancy, but so be it. (I
don't view this as bike shedding at all, btw. While every individual
instance may not matter much, it is the sum of them that I'm worried
about.)

Jan

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel