config-patches
[Top][All Lists]
Advanced

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

Re: * config.sub: Properly parse the KERNEL-OS case.


From: John Ericson
Subject: Re: * config.sub: Properly parse the KERNEL-OS case.
Date: Sat, 27 Jun 2020 13:04:05 -0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0

Sorry! It is indeed incomprehensible---I must have been more tired than I thought.

Here is a new version with a ChangeLog entry that's hopefully actually understandable.

Thanks,

John

On 6/26/20 8:39 AM, Ben Elliston wrote:
John,

I guess I am getting back to the refactoring I was doing before for one last
round, so now finally no two components are ever conflated:

     * config.sub: Properly parse the KERNEL-OS case.

     I previous parsed basic_machine into separate `cpu` and `kernel`
     variables, I parse `basic_os` (a rename of `os` when used early on) into
     `kernel` and `os`.

     Like before, this does make thins a bit longer, but I think it makes the
     code understand and more robust, and so is worth the cost.
Can you please re-write the above? It makes no sense to me.

Thanks,
Ben

Attachment: 0001-config.sub-Properly-parse-the-KERNEL-OS-case.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]