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 smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B4ED4C433EF for ; Thu, 28 Apr 2022 21:50:22 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 6DBA683F28; Thu, 28 Apr 2022 21:50:22 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id bmNFQaAH5rhH; Thu, 28 Apr 2022 21:50:21 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp1.osuosl.org (Postfix) with ESMTP id 7B9C683E65; Thu, 28 Apr 2022 21:50:20 +0000 (UTC) Received: from smtp4.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by ash.osuosl.org (Postfix) with ESMTP id DCDCC1BF2E4 for ; Thu, 28 Apr 2022 21:50:18 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp4.osuosl.org (Postfix) with ESMTP id C673441BF4 for ; Thu, 28 Apr 2022 21:50:18 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Authentication-Results: smtp4.osuosl.org (amavisd-new); dkim=pass (2048-bit key) header.d=free.fr Received: from smtp4.osuosl.org ([127.0.0.1]) by localhost (smtp4.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id woIzpZCqFvKq for ; Thu, 28 Apr 2022 21:50:17 +0000 (UTC) X-Greylist: whitelisted by SQLgrey-1.8.0 Received: from smtp4-g21.free.fr (smtp4-g21.free.fr [212.27.42.4]) by smtp4.osuosl.org (Postfix) with ESMTPS id 1CFA041BF2 for ; Thu, 28 Apr 2022 21:50:16 +0000 (UTC) Received: from ymorin.is-a-geek.org (unknown [IPv6:2a01:cb19:8b51:cb00:dcb4:7861:786a:5693]) (Authenticated sender: yann.morin.1998@free.fr) by smtp4-g21.free.fr (Postfix) with ESMTPSA id 1DDFD19F58A; Thu, 28 Apr 2022 23:50:04 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=free.fr; s=smtp-20201208; t=1651182614; bh=dSET5zQMh2rFEwzWuFPxWyAkiPzIEmq7p8+6ufOKbeA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=LtOGG0rvD+8UMKcDXU10365PUncbPgTu/ssvJs1U8O4VvK0JNC/OBSjs6aBKHCo/0 qlFBJ9J9qC0t4VAO7MrVlV0WcVrwANelNRaw0mFDZi7aQv99bFv/bHTXbZw2aSvVtF FvG5VsHWxL+s9oltkdnp+OD9d7QgLYSf/dIBMZ7vX5lnfIGK7f+4EMk7AZsDiXBL4Q aMSG2dELtR/BXw0mgo/f8DOBAbDkIL4hzN9inuOxO0Jiy39IyhJkGCPAczu34clXfe M/zs9yeujktYgOEuDruGoTa8hBZfiYfXMS/L/dzAglDmhjU6qwLEGpc4Bik+NSWFAB IKAsZv/WXHJZQ== Received: by ymorin.is-a-geek.org (sSMTP sendmail emulation); Thu, 28 Apr 2022 23:50:03 +0200 Date: Thu, 28 Apr 2022 23:50:03 +0200 From: "Yann E. MORIN" To: Thomas Petazzoni Message-ID: <20220428215003.GG3624965@scaer> References: <20220428212011.534725-1-thomas.petazzoni@bootlin.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220428212011.534725-1-thomas.petazzoni@bootlin.com> User-Agent: Mutt/1.5.22 (2013-10-16) Subject: Re: [Buildroot] [PATCH] Revert "toolchain/toolchain-external: error if BR2_TOOLCHAIN_EXTERNAL_PATH is not set" X-BeenThere: buildroot@buildroot.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion and development of buildroot List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: James Hilliard , Kristof Havasi , Buildroot List , Giulio Benetti , Romain Naour , Thomas De Schampheleire Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: buildroot-bounces@buildroot.org Sender: "buildroot" Thomas, All, On 2022-04-28 23:20 +0200, Thomas Petazzoni via buildroot spake thusly: > This reverts commit 8945ba49480a7885d104445ca855ef7681a374ff. This is a commit on the 2022.02.x branch; it is a backport of 8e91385a2cb81c5f8114354f4494230e2aebb93a on master. Did you meant this patch to be applied only to 2022.02.x? I would believe it should be applied to master and backported to 2022.02 instead, no? Regards, Yann E. MORIN. > This commit is incorrect, as it is perfectly valid for > BR2_TOOLCHAIN_EXTERNAL_PATH to be empty. The help text of > BR2_TOOLCHAIN_EXTERNAL_PATH even documents it as a supported case: > > If empty, the compiler will be searched in $PATH. > > Commit 392b0a26f5fd8d3aedce7e0c15f9762a79312f01 ("toolchain-external: > default BR2_TOOLCHAIN_EXTERNAL_PATH to empty") even made that the > default saying "In addition, it in fact works correctly when it is > empty. In that case, the toolchain will be searched in PATH." > > A user has reported that commit > 8945ba49480a7885d104445ca855ef7681a374ff breaks his use-case: > > https://lore.kernel.org/buildroot/CADBnMvhgaozAgZgy3njckjL1i0U6bZ0fLrq-kdFF-qpGhFWgmw@mail.gmail.com/ > > Reported-by: Kristof Havasi > Signed-off-by: Thomas Petazzoni > --- > This should be backported to 2022.02.x > --- > toolchain/toolchain-external/pkg-toolchain-external.mk | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/toolchain/toolchain-external/pkg-toolchain-external.mk b/toolchain/toolchain-external/pkg-toolchain-external.mk > index c7f4175c9e..299b6008aa 100644 > --- a/toolchain/toolchain-external/pkg-toolchain-external.mk > +++ b/toolchain/toolchain-external/pkg-toolchain-external.mk > @@ -69,12 +69,7 @@ ifeq ($(BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD),y) > TOOLCHAIN_EXTERNAL_INSTALL_DIR = $(TOOLCHAIN_EXTERNAL_DOWNLOAD_INSTALL_DIR) > else > TOOLCHAIN_EXTERNAL_INSTALL_DIR = $(abspath $(call qstrip,$(BR2_TOOLCHAIN_EXTERNAL_PATH))) > -ifeq ($(BR2_TOOLCHAIN_EXTERNAL)$(BR_BUILDING),yy) > -ifeq ($(TOOLCHAIN_EXTERNAL_INSTALL_DIR),) > -$(error No external toolchain path set, check your BR2_TOOLCHAIN_EXTERNAL_PATH setting) > -endif > endif > -endif # BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD > > ifeq ($(TOOLCHAIN_EXTERNAL_INSTALL_DIR),) > ifneq ($(TOOLCHAIN_EXTERNAL_PREFIX),) > -- > 2.35.1 > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot