From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS53758 23.128.96.0/24 X-Spam-Status: No, score=-4.5 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A, RCVD_IN_DNSWL_LOW,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by dcvr.yhbt.net (Postfix) with ESMTP id 7CE181F5AE for ; Fri, 18 Jun 2021 14:03:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232136AbhFROFX (ORCPT ); Fri, 18 Jun 2021 10:05:23 -0400 Received: from smtp.hosts.co.uk ([85.233.160.19]:40880 "EHLO smtp.hosts.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230387AbhFROFW (ORCPT ); Fri, 18 Jun 2021 10:05:22 -0400 Received: from host-78-147-180-220.as13285.net ([78.147.180.220] helo=[192.168.1.37]) by smtp.hosts.co.uk with esmtpa (Exim) (envelope-from ) id 1luF5U-000C6z-D2; Fri, 18 Jun 2021 15:03:12 +0100 Subject: Re: [PATCH 0/3] Make CMake work out of the box To: Johannes Schindelin Cc: Bagas Sanjaya , Matthew Rogers via GitGitGadget , git@vger.kernel.org, Sibi Siddharthan , Danh Doan , Matthew Rogers References: <6a9150ca-5c1a-1874-5f8b-35187f197d47@gmail.com> From: Philip Oakley Message-ID: Date: Fri, 18 Jun 2021 15:03:12 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Content-Language: en-GB Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Hi Dscho On 18/06/2021 14:42, Johannes Schindelin wrote: >>> We already exercise the plain Makefile plenty, and the CMake-based build >>> using Windows (in the `vs-build` job in `.github/workflows/main.yml`). >> There is one 'gotcha' in the yml (probably historical) in that it >> doesn't actually test the approach/changes that Matt addresses regarding >> my [1]. >> >> That is, I'm looking at the 'out of the box' view, while the yml test >> _preloads_ the vcpkg artefacts. > We need to "pre-load" them because building them would add another > whopping 20 minutes to each CI run. And I am not talking total time, but > wall-clock time. > > And we're not in the business of testing vcpkg's build. > > So I am really not in favor of even thinking about changing this > "pre-loading" strategy. > > I can see the common sense in that, however I was trying to highlight that the approach in patch series could go stale, as did the previous method. Making the entry ramp to investigating the code for the wide variety windows users should have _some_ testing.. I don't have any good ideas about how to get out of that 20 minute Catch-22 issue at the moment. Maybe it needs an independent, on-demand (i.e. infrequent;-) test. Maybe there is a way of adding a `--CI-test` option that at least exercises the logic without needing the vcpkg to be built again (IIRC, and I may well be wrong, we build once, remember the artefacts, and then re-used them, but .. dunno). Philip