[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] m4sh: prefer bash over sh
From: |
Ross Burton |
Subject: |
Re: [PATCH] m4sh: prefer bash over sh |
Date: |
Mon, 23 Mar 2020 16:11:13 +0000 |
On Mon, 23 Mar 2020 at 15:12, Zack Weinberg <address@hidden> wrote:
> Ignoring what I just said in the other thread, a patch like this would
> be a lot more compelling if it came with some performance
> measurements. Do you have time to find out whether bash is, in fact,
> faster at executing long configure scripts than dash?
You're absolutely right that I should provide evidence. We didn't
make this change for no real reason but the underlying assumptions may
be historical. gettext is my favourite pathological test as it runs
several nested configure scripts.
$ export CONFIG_SHELL=/bin/dash
$ hyperfine '$CONFIG_SHELL configure'
Benchmark #1: $CONFIG_SHELL configure
Time (mean ± σ): 50.530 s ± 0.188 s [User: 28.922 s, System: 13.621 s]
Range (min … max): 50.207 s … 50.868 s 10 runs
$ export CONFIG_SHELL=/bin/bash
$ hyperfine '$CONFIG_SHELL configure'
Benchmark #1: $CONFIG_SHELL configure
Time (mean ± σ): 55.853 s ± 0.214 s [User: 32.428 s, System: 16.291 s]
Range (min … max): 55.578 s … 56.276 s 10 runs
Consider this patch retracted.
Ross