Re: [PATCH 0/2] CI: don't fail OSX tests due to brew v.s. perforce.com mis-matchy
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Apr 22, 2022, 09:21 UTC
- Message-ID
- <220422.86h76l5v7t.gmgdl@evledraar.gmail.com>
- In-Reply-To
- <20220421213024.xzkzuy3l7q5assxo@carlos-mbp.lan>
On Thu, Apr 21 2022, Carlo Marcelo Arenas Belón wrote:
Show 13 quoted lines
> On Thu, Apr 21, 2022 at 02:53:50PM +0200, Ævar Arnfjörð Bjarmason wrote: >> For the past days we've again had CI failures due to "brew install" >> detecting a SHA-256 mismatch when trying to install the perforce > > Since the only reason why that is a concern is because it aborts the > rest of the run and is a recurring problem, wouldn't it be better to > tell the script to continue regardless and therefore skip all perforce > tests? > > Sure, there is a window where that integration could be broken which > will be only visible once the perforce cask gets fixed and perforce > installs again, but wouldn't that be less intrusive and overall safer > than the currently proposed change?
I tried to answer all of this in the updated v3 CL. Thanks!: https://lore.kernel.org/git/cover-v2-0.3-00000000000-20220422T085958Z-avarab@gmail.com/