From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id C2B061382C5 for ; Thu, 21 May 2020 22:08:54 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id A4A3FE0921; Thu, 21 May 2020 22:08:53 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 880C0E0921 for ; Thu, 21 May 2020 22:08:53 +0000 (UTC) Date: Thu, 21 May 2020 15:08:50 -0700 From: Brian Dolbec To: gentoo-catalyst@lists.gentoo.org Subject: Re: [gentoo-catalyst] [PATCH 1/5] embedded: remove actions that are broken by default Message-ID: <20200521150850.5743f856@storm> In-Reply-To: References: <20200521173248.2297765-1-gentoo.catalyst@xxoo.ws> X-Mailer: Claws Mail 3.17.5 (GTK+ 2.24.32; x86_64-pc-linux-gnu) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-catalyst@lists.gentoo.org Reply-to: gentoo-catalyst@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Archives-Salt: 877772da-458a-4961-828e-202610e7c848 X-Archives-Hash: def418d7439b9152606c00ca8ef0ab47 On Thu, 21 May 2020 13:41:18 -0700 Matt Turner wrote: > On Thu, May 21, 2020 at 10:34 AM Daniel Cordero > wrote: > > > > From: Daniel Cordero > > > > dir_setup() doesn't exist, bootloader() exists but requires specific > > tools to be installed in the seed stage and doesn't check that they > > are, causing the build to fail. > > --- > > If I have misconstrued the purpose of bootloader, then documentation > > needs to be written. > > I don't actually see any documentation about > > catalyst/targets/embedded.py | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/catalyst/targets/embedded.py > > b/catalyst/targets/embedded.py index aa23f5b3..1b4ad9d6 100644 > > --- a/catalyst/targets/embedded.py > > +++ b/catalyst/targets/embedded.py > > @@ -41,7 +41,6 @@ class embedded(StageBase): > > > > def set_action_sequence(self): > > self.settings['action_sequence'] = [ > > - "dir_setup", > > Nice. This function was removed in 2005, so the embedded target has > been broken ever since. That certainly answers my question as to > whether anyone uses it. > > Fixes: 1dafb5fa06d2 (Add locking support. ...) > > So, you must use the embedded target. Could you tell me how you use > it, for what device, etc? > > > "unpack", > > "config_profile_link", > > "setup_confdir", > > @@ -51,7 +50,6 @@ class embedded(StageBase): > > "setup_environment", > > "build_kernel", > > "build_packages", > > - "bootloader", > > It's not obvious to me what specific tools this requires to be > installed in the seed stage. Presumably you're referring to this? > > cmd([self.settings['controller_file'], 'bootloader', > self.settings['target_path'].rstrip('/')] > > which eventually runs bootloader-setup.sh. > > I'd assume it's generally sensible to install a bootloader in the > embedded target, and I don't think we have mechanism for customizing > the action_sequence, so if we remove bootloader from the list then I > think the python bootloader() function is just dead code, isn't it? I > doubt that's the end result we want. > No, if you notice, this is removing the bootloader action sequence only for the embedded target. Each target subclasses stagebase, so can override what is defined in stagebase as needed. Customizing the action_sequence is something that pretty much all the targets do.