Received: by 2002:a05:6a10:2726:0:0:0:0 with SMTP id ib38csp1955875pxb; Fri, 25 Mar 2022 08:35:16 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyC2OS/uyK46VwUTnE8VPs94S0DQl3s1VKsAUJDMLnOFZR9hTEa5nwfQKtqg5b9auYKh2MZ X-Received: by 2002:a17:907:6d82:b0:6d6:da31:e542 with SMTP id sb2-20020a1709076d8200b006d6da31e542mr11913038ejc.135.1648222516015; Fri, 25 Mar 2022 08:35:16 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1648222516; cv=none; d=google.com; s=arc-20160816; b=WEXIG3hk8s/zSStPFq0veegPiNASvVcBAf8gkj+3rY8awjrUftwlJ/KBbZ8nlioTTm hgf6YD/Sx0rhr9V/qqDMTGeNiHde+uwA+Z3ZxD+7Nvf4kTbZTLEYiBmQdyh16B/HNiWX n96MvKcVC0hRTsWq2H8rG+eLpVVh4ChE4MwXR8+Lugw8UqIWktVjxRoyKRER3aCzNeDg HDR1F2c9YyRCF/crlAyJi/pbqKaCS75LX0DEB2aLmNMqpUieyt2aBwWQ5fHc/mu3LQ9r Z0+Ss6Q9sJURiyuluJFI8fjRalKUU/iuFeoas4Efxpf9oxx9exGIA/zvY5fp4cPoDCCX S0ig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=96OPHRtjpLUkzMKOmZHTFwbJz5Jyl9hyIas7RLYSw4Q=; b=hXqptLAhbDzTSJ81/gzThKSKLcowCrx4WhuYdpSkolbqO4Pk5LL2cnBZV1xzomGpgq vGENjZ1mKtnJZ9MNyi22jN/awpGiZHcI4IQPs5QQUAWJwEpNZDxJqUyNIE2WdvyRYrmC uWhGLIegB/FZ8pamPnAS+HXflXihp3b1bt/u8wkNx/mHdUXPhN3OAg9XKU+aU52pesmS VIlXW4JCslHgwTnrc33j+F6PD0O+dPqnyhJm0zusZVLhI24UN2dYSV6o+lx3Zxjf+X0r Kv4PZV7yOjaLVe5+7K4ENRtZgZ1jOHnNENA2eRsuuZSv9LhmBBGJziFlMT7CJ+pv32Zs XuLQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=Y031F6hf; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w23-20020a1709061f1700b006df76385bb2si2468869ejj.82.2022.03.25.08.34.47; Fri, 25 Mar 2022 08:35:16 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=Y031F6hf; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240972AbiCWVFv (ORCPT + 99 others); Wed, 23 Mar 2022 17:05:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58686 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240854AbiCWVFu (ORCPT ); Wed, 23 Mar 2022 17:05:50 -0400 Received: from mail-ej1-x62d.google.com (mail-ej1-x62d.google.com [IPv6:2a00:1450:4864:20::62d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 74CF412AF8 for ; Wed, 23 Mar 2022 14:04:18 -0700 (PDT) Received: by mail-ej1-x62d.google.com with SMTP id o10so5373442ejd.1 for ; Wed, 23 Mar 2022 14:04:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=96OPHRtjpLUkzMKOmZHTFwbJz5Jyl9hyIas7RLYSw4Q=; b=Y031F6hf2sWxJzVczsI6DwN15K/fScPKXLhbWHFfYBi5upYSFMRNec8Mbc4WSNARgG 4sqVpLnuoG2DGIOHvH8mL3diB5NIeN+mwoDHLMhIHicZa7/WmLnePbKJOMCYQeyBnksJ zXeVrIYCMBR8bHLbJfOgZ3nIXkjF0DkkmiA+K8Lbyw2184VsnpUDPLqIUWX3fJN8OIpw oLIOpj707wDyyb05BnFgblOUt6Qy7S5VXF01Au9UWWc+bnH/xymFEue+2j8Yms2aRMAp gseQ5FsJr12DGqdxxWBNrjOJbAYM+NK4NRYB1e2bbFfxyA0StQlKn9rt7pBAoOqeH4zq nHYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=96OPHRtjpLUkzMKOmZHTFwbJz5Jyl9hyIas7RLYSw4Q=; b=pzkvxy+ZkOY1SPKS+FvujUZHgmklA7SdSTIVu92bqZNXHkJr3xksgflFuLHN71ATnI 09cy1H8dXJks2gn9ICugY4lmAmf6KtIzJNPYSMI5nRtsv3IwXIe31mrnkBZaws3Y702m TiSDAF7ie7YeaoXRpzJY9ptfhGcqoTTbTnk1XPP2iwLzqmLPEduewN7ms42WzqlXCBBo wGd0wdpDmRqy87s5fBv/wlzHaHg3oz5lYYfonWROVW29R1ndtaA4rXNVlkIx+dDx6Eaz CBnulapCuk6EtRppWTFWqPwkavLzHXuS2u3x+0UfqxRBPJ1a1dEfK36dZZs8YHVAMNBA JjEA== X-Gm-Message-State: AOAM532FxDmBc1viioN8yc4L3ozeH9+mIK2hCeQXVKpMyxKPslTgF1xG JvoUxCzpYqyjmTgbz90urCaspwujg1qDLK5rDRuhCQ== X-Received: by 2002:a17:907:c018:b0:6df:e31b:d912 with SMTP id ss24-20020a170907c01800b006dfe31bd912mr2204489ejc.196.1648069456760; Wed, 23 Mar 2022 14:04:16 -0700 (PDT) MIME-Version: 1.0 References: <20220118190922.1557074-1-dlatypov@google.com> <20220118190922.1557074-5-dlatypov@google.com> In-Reply-To: <20220118190922.1557074-5-dlatypov@google.com> From: Brendan Higgins Date: Wed, 23 Mar 2022 17:04:05 -0400 Message-ID: Subject: Re: [PATCH 5/5] kunit: tool: simplify code since build_dir can't be None To: Daniel Latypov Cc: davidgow@google.com, linux-kernel@vger.kernel.org, kunit-dev@googlegroups.com, linux-kselftest@vger.kernel.org, skhan@linuxfoundation.org Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-17.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, ENV_AND_HDR_SPF_MATCH,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE,USER_IN_DEF_DKIM_WL,USER_IN_DEF_SPF_WL autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 18, 2022 at 2:09 PM Daniel Latypov wrote: > > --build_dir is set to a default of '.kunit' since commit ddbd60c779b4 > ("kunit: use --build_dir=.kunit as default"), but even before then it > was explicitly set to ''. > > So outside of one unit test, there was no way for the build_dir to be > ever be None, and we can simplify code by fixing the unit test and > enforcing that via updated type annotations. > > E.g. this lets us drop `get_file_path()` since it's now exactly > equivalent to os.path.join(). > > Note: there's some `if build_dir` checks that also fail if build_dir is > explicitly set to '' that just guard against passing "O=" to make. > But running `make O=` works just fine, so drop these checks. > > Signed-off-by: Daniel Latypov Reviewed-by: Brendan Higgins