Re: [PATCH] config.sub: Add support for SerenityOS

2021-03-10 Thread Dmitry V. Levin
On Wed, Mar 10, 2021 at 05:27:49PM +0200, Idan Horo wrote: > I noticed i was missing a test case in my earlier patch, so attached is a new > patch that includes an appropriate test case. Applied, thanks. -- ldv

Re: config.sub/config.guess using nonportable $(...) substitutions

2021-03-10 Thread Karl Berry
1. $(...) breaks Solaris 10 /bin/sh. 2. Solaris 10 is still supported by the vendor, and people still use it with GNU tools. 3. There is no technical benefit to $(...) in config.*. What's the harm in using `...` a few more years in config.*? Answer, as far as I can see: none. -k

Re: [PATCH] config.sub: Add support for SerenityOS

2021-03-10 Thread Idan Horo
I noticed i was missing a test case in my earlier patch, so attached is a new patch that includes an appropriate test case. --- config.sub| 2 +- testsuite/config-sub.data | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/config.sub b/config.sub index 63c1f1c.