Re: [PATCH 2/3] config.sub: Factor out `invalid_config` function

2023-09-25 Thread Jacob Bachmeyer
John Cotton Ericson wrote: On 9/25/23 10:24, Zack Weinberg wrote: It is ultimately ldv's call, but Jacob's experiment is not good enough for me to approve. So I saw Jacob's research as not demonstrating using functions is *definitely* OK, but that has indicating it *might* be OK. Does

Re: [PATCH] config.sub: Accept LLVM-style $cpu-$vendor-windows-cygnus

2023-09-25 Thread Jacob Bachmeyer
Zack Weinberg wrote: On Thu, Sep 21, 2023, at 9:49 PM, Jacob Bachmeyer wrote: I further argue that alternate output modes should only be supported at config.sub; config.guess should always produce the current hyphen- delimited tuple format. That would preclude autoconf using the new fo

Re: [PATCH 2/3] config.sub: Factor out `invalid_config` function

2023-09-25 Thread Zack Weinberg
>> On Mon, Sep 25, 2023, at 10:57 AM, John Cotton Ericson wrote: >>> Sure. If they show up, we can definitely revert it. But what happens >>> if they *don't* show up? What do you think in that case? >>> >> How long are you prepared to stick to your position? How many angry >> e- mails are you prepa

Re: [PATCH 2/3] config.sub: Factor out `invalid_config` function

2023-09-25 Thread John Cotton Ericson
On 9/25/23 11:06, Zack Weinberg wrote: For clarity, I was talking about the overhead of _AS_DETECT_BETTER_SHELL, not the overhead of functions themselves. Great! It is unclear to me whether bourne-ish shells that were new in March 13 1992 that still didn't support shell functions, or whether

Re: [PATCH 2/3] config.sub: Factor out `invalid_config` function

2023-09-25 Thread Zack Weinberg
On Mon, Sep 25, 2023, at 10:57 AM, John Cotton Ericson wrote: > On 9/25/23 10:24, Zack Weinberg wrote: > >> It is ultimately ldv's call, but Jacob's experiment is not good >> enough for me to approve. >> > So I saw Jacob's research as not demonstrating using functions is > *definitely* OK, but that

Re: [PATCH 2/3] config.sub: Factor out `invalid_config` function

2023-09-25 Thread John Cotton Ericson
On 9/25/23 10:24, Zack Weinberg wrote: It is ultimately ldv's call, but Jacob's experiment is not good enough for me to approve. So I saw Jacob's research as not demonstrating using functions is *definitely* OK, but that has indicating it *might* be OK. Does that sound better? The thing i

Re: [PATCH] config.sub: Accept LLVM-style $cpu-$vendor-windows-cygnus

2023-09-25 Thread Zack Weinberg
On Thu, Sep 21, 2023, at 9:49 PM, Jacob Bachmeyer wrote: > I further argue that alternate output modes should only be supported > at config.sub; config.guess should always produce the current hyphen- > delimited tuple format. That would preclude autoconf using the new format. We don't want to hav

Re: [PATCH 2/3] config.sub: Factor out `invalid_config` function

2023-09-25 Thread Zack Weinberg
On Fri, Sep 22, 2023, at 5:14 PM, John Cotton Ericson wrote: > On 9/21/23 22:31, Jacob Bachmeyer wrote: >> Zack Weinberg wrote: >>> >>> Um. I don't think you can use shell functions in this file (or in >>> config.guess), for the same reason you can't use $(...). >> >> It may be worth noting that th