From owner-dev-commits-src-all@freebsd.org Thu Jan 7 15:24:15 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 9DCB64D3C30 for ; Thu, 7 Jan 2021 15:24:15 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-qk1-x72b.google.com (mail-qk1-x72b.google.com [IPv6:2607:f8b0:4864:20::72b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4DBVQ73rwzz3CZ0 for ; Thu, 7 Jan 2021 15:24:15 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-qk1-x72b.google.com with SMTP id d14so5644215qkc.13 for ; Thu, 07 Jan 2021 07:24:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=1o1hoIJDBeMuM1JVaY93JMg7XKoBTrLcsaCIqR+8ymU=; b=umnUEAanxSgg4ZNmCO/ORa3aLURj6yBl67efBUEH/lvsPB4oB5QlhIDHxJ/A8C2jlI lfWigvtv48GCHrhO9aXCmFrFpZ2w/zwHU+YZso7lOBVxe0WL5NP/lvON7xK34V2ojhej 4L9xd1fR3WRGRFUndHsBWE+Z+/Mdkbu8GYtr8Sop9LB49IBRnYmG0fjLM4SJFH40XYfr NFdke1C0zVGvw0IfuJBnI5MD4heTpVZaTr5G4k7bjghMf8M2QJg0kVuxuCN/1EOxViJy k1OAojlfR9LS9a8rcoNBH7C0gLLWzTHuXCMGjDE3cbqPaX/q1FQzz7fAvVEde3mO9tn6 kjeA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=1o1hoIJDBeMuM1JVaY93JMg7XKoBTrLcsaCIqR+8ymU=; b=oClRdSleYnxDXIk11Ytio+FgeiN/7NLlk2YKblThWvj5ZtLCiSfQbOXJDEa7FmBauO ljvR28k5HU9RzJ/yi9tLe7Cuzf0WDj4JIXmkvzt17RGhmA26JPXtVUYK/3khBnx7XKKL n2uL1/ep/OLrMBzqv6GK5fz9RGvfU4xvKd4SBwe+betC/802OY5JF9E5qdn0e6hisRxv 4HuFR+MQ9chFB7T8ZxnR7P3Qr6n2GuctBTrIyzr0N2Sk7+UhGR1ee/QjEx50HYNVDmaI NjBG0skX4kYakreIU8iKeenEWOKeLzizq1IGLPP3OXgA0FyX4BtUae1ICU+aNEUY7Vw+ DFKA== X-Gm-Message-State: AOAM533KMzkk6GlOSLmQd3qJG1K1ZiPQ/lrwLiZ+m4pMcjipw1Utlc08 4fYvPqEWsoubTSm4r6u4eT4+N7M4t7iP9qf4pWuHJg== X-Google-Smtp-Source: ABdhPJz34h3upQWrTh/vgGzhVgWeW0ljD7Lt4Ka+DbHQM/MARBXt+t+0w8MfpVi0ItdrcWdxkr3tVrJIZYB/rs08rRU= X-Received: by 2002:ae9:ebd5:: with SMTP id b204mr9296724qkg.195.1610033054456; Thu, 07 Jan 2021 07:24:14 -0800 (PST) MIME-Version: 1.0 References: <202012311717.0BVHHp4G092948@gitrepo.freebsd.org> In-Reply-To: From: Warner Losh Date: Thu, 7 Jan 2021 08:24:02 -0700 Message-ID: Subject: Re: git: ac5f382a9d0a - main - stand: properly declare subdir deps or .WAIT, do parallel build To: Kyle Evans Cc: Peter Jeremy , src-committers , dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org X-Rspamd-Queue-Id: 4DBVQ73rwzz3CZ0 X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.34 X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 Jan 2021 15:24:15 -0000 On Thu, Jan 7, 2021, 6:34 AM Kyle Evans wrote: > On Thu, Jan 7, 2021 at 1:50 AM Peter Jeremy wrote: > > > > On 2020-Dec-31 17:17:51 +0000, Kyle Evans wrote: > > >The branch main has been updated by kevans: > > > > > >URL: > https://cgit.FreeBSD.org/src/commit/?id=ac5f382a9d0a26685b92b49abb845d3b30ea5f91 > > > > > >commit ac5f382a9d0a26685b92b49abb845d3b30ea5f91 > > >Author: Kyle Evans > > >AuthorDate: 2020-12-31 17:15:45 +0000 > > >Commit: Kyle Evans > > >CommitDate: 2020-12-31 17:15:45 +0000 > > > > This commit adds fdt as a hard dependency for efi but leaves fdt as > > optionally built based on WITH_FDT. As a result, adding WITHOUT_FDT=yes > > causes the build to fail: > > make[3]: don't know how to make all_subdir_stand/fdt. Stop > > > > Either MK_FDT needs to be forced on by MK_EFI or the dependency needs to > > be made conditional. > > > > Ah, yes, sorry- will fix. It looks like that should be the only other > conditional dep that I missed here. The latter option is what we opt > for here- > We should add the missing items in our test matrix for tools/boot/universe.sh Warner Thanks, > > Kyle Evans >