From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1C963C5479D for ; Wed, 4 Jan 2023 01:09:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234330AbjADBJn (ORCPT ); Tue, 3 Jan 2023 20:09:43 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55800 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230167AbjADBJl (ORCPT ); Tue, 3 Jan 2023 20:09:41 -0500 Received: from mail-pl1-x633.google.com (mail-pl1-x633.google.com [IPv6:2607:f8b0:4864:20::633]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6497D15732 for ; Tue, 3 Jan 2023 17:09:40 -0800 (PST) Received: by mail-pl1-x633.google.com with SMTP id y19so15409994plb.2 for ; Tue, 03 Jan 2023 17:09:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=2VT5QZhcZZeDbsiEz/fllfSXONvy51XdiSS8QokpwQg=; b=Q4DEfeoNUsJnujmG5WDcxlCAMaEfhBmMWAS7aUzEC2GW3ofbkRwZ22ZYFjITsxmo6K bb0vjmVJXZL0G/l4TGKr3KMTIDcdwQUeKoeFnx6I8MLcodY9TVH6AGEpkz9x9GWI5hwx k7R0UjHzbUt2Ai1wJRy2IFQHhF6DWYLx8cxQAFMWDWI6Xgrug2FARlR3RiNRVi4rCbo9 AP/OLiZNnuV7UvZFm8qEsAphMEm93ULbKnV9d+A7rgWdhM6aiDUP6a5tWQx2bF/ggq6A f7JPgRzWEeYZPXHV5Lm4ZgFljKv8WtzXvID2mUUKGwqdRm31BuBbvKfhkRXpKAhdOylW Zj3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=2VT5QZhcZZeDbsiEz/fllfSXONvy51XdiSS8QokpwQg=; b=KXOMbg+vqFeH7ADPN2TxZJhhDiOSB4ucp0v/Yqr15M2rfzaIk1uA2OeN6xtpJDw6ZE 3SZTM+Y6Y3GlWDxYOlz7rES5KDQs0RZvEbFhkAhPuf82uniPtqPriTKJYSeQjRMgir6D X8PH6P0XMPj0p8Lob8APOO3BHPN/PWeE8M83DNbe3OOL8atIjq3Ob+l/coz6Tb0xxory BFKYTir2z4O7u4TH6stcMTCPJ8Wsrpuws3kUG6nX9g49QJ0HNySgs1hlGbCJiGRuR1HG gdeZw2H3/f3uxG3XvpDi1L2e5tFAKI51eBOldeA2DiHtQ5/ePgG1HTJTRXoapVH7xJSo 5NFQ== X-Gm-Message-State: AFqh2ko7Ko3bkapXw5GaIdqiQx5eJvN8momt+RNIEj331Fi1NGLyRkjF nkriVZ3Vc9/jWgRs4b1c9nijeg== X-Google-Smtp-Source: AMrXdXt4xHOajqscKlKaEpBiTyEw5HpxcS2+BmkjFSTy1AWvaTA2O2HtoOh9s0+BdQR2dE8QR1delA== X-Received: by 2002:a17:90a:9912:b0:219:f970:5119 with SMTP id b18-20020a17090a991200b00219f9705119mr3677621pjp.1.1672794579775; Tue, 03 Jan 2023 17:09:39 -0800 (PST) Received: from google.com (7.104.168.34.bc.googleusercontent.com. [34.168.104.7]) by smtp.gmail.com with ESMTPSA id dw6-20020a17090b094600b00223f495dc28sm19707506pjb.14.2023.01.03.17.09.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Jan 2023 17:09:39 -0800 (PST) Date: Wed, 4 Jan 2023 01:09:35 +0000 From: Sean Christopherson To: Paolo Bonzini Cc: Marc Zyngier , Paul Walmsley , Palmer Dabbelt , Albert Ou , Nathan Chancellor , Nick Desaulniers , James Morse , Alexandru Elisei , Suzuki K Poulose , Oliver Upton , Tom Rix , kvm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, kvmarm@lists.cs.columbia.edu, linux-riscv@lists.infradead.org, llvm@lists.linux.dev, linux-kernel@vger.kernel.org, Ricardo Koller , Aaron Lewis , Raghavendra Rao Ananta Subject: Re: [PATCH 12/14] KVM: selftests: Use wildcards to find library source files Message-ID: References: <20221213001653.3852042-1-seanjc@google.com> <20221213001653.3852042-13-seanjc@google.com> <363f4713-6105-82d1-351e-423d07470cdf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <363f4713-6105-82d1-351e-423d07470cdf@redhat.com> Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org On Sat, Dec 24, 2022, Paolo Bonzini wrote: > On 12/13/22 01:16, Sean Christopherson wrote: > > Use $(wildcard ...) to find the library source files instead of manually > > defining the inputs, which is a maintenance burden and error prone. > > No, please don't. This leads to weird errors, for example when "git > checkout" is interrupted with ^C. Well, don't do that ;-) Are there concerns beyond having an incomplete and/or modified git working tree? E.g. could we do something crazy like fail the build by default if the working tree isn't pristine? The library files aren't terrible, but the number of tests is getting unwieldy, to say the least.