From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dan.rpsys.net (5751f4a1.skybroadband.com [87.81.244.161]) by mail.openembedded.org (Postfix) with ESMTP id 211CA65E3C for ; Fri, 13 Jun 2014 08:07:00 +0000 (UTC) Received: from localhost (dan.rpsys.net [127.0.0.1]) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu4) with ESMTP id s5D86nif003226; Fri, 13 Jun 2014 09:06:49 +0100 X-Virus-Scanned: Debian amavisd-new at dan.rpsys.net Received: from dan.rpsys.net ([127.0.0.1]) by localhost (dan.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id D45rr8ssA-99; Fri, 13 Jun 2014 09:06:48 +0100 (BST) Received: from [192.168.3.10] (rpvlan0 [192.168.3.10]) (authenticated bits=0) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id s5D86gaR003222 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT); Fri, 13 Jun 2014 09:06:44 +0100 Message-ID: <1402646797.12440.477.camel@ted> From: Richard Purdie To: Otavio Salvador Date: Fri, 13 Jun 2014 09:06:37 +0100 In-Reply-To: References: <37280175766b34129a22d2602f057511e90263b1.1402552609.git.sgw@linux.intel.com> X-Mailer: Evolution 3.8.4-0ubuntu1 Mime-Version: 1.0 Cc: Patches and discussions about the oe-core layer Subject: Re: [RFC - WIP v2 01/10] conf-files: New recipe to create single recipe for config files X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 13 Jun 2014 08:07:06 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Thu, 2014-06-12 at 10:57 -0300, Otavio Salvador wrote: > On Thu, Jun 12, 2014 at 2:56 AM, Saul Wold wrote: > > This recipe will create 1 package for config files, we could optionally add > > a bbclass file to ensure consistency with RRECOMMENDS_ = =conf > > > > This is a work in progress, the do_install might even beable to automagically > > generated. We don't want to create a bbclass for these since it will cause > > the actual recipe/packaging to become machine specific, using this recipe will > > ioslate that. > > > > [YOCTO #4011] > > > > Signed-off-by: Saul Wold > > I think the configuration file, nowadays, already made those machine > specific in every BSP which has those overriden so I don't see why use > a single recipe to provide several configuration files. > > I think it will be confusing and this recipe will fast grow. There are a few good reasons to do this. Machine customisation is spread around a whole load of different recipes at the moment and its hard to obtain a good view of what files are available and which ones a BSP author may need to provide. Its rather ugly to have to provide and maintain multiple bbappend files with rather ugly syntax within them. Its also rather inefficient from a build process standpoint to have 15-20 recipes just packaging configuration files. The intent isn't to mandate *every* config file should be in this recipe, you will as now be able to add additional ones. Where we see the same files being added in many layers, adding something common and shared makes sense though. It should in some cases also allow the "core" recipe to stop being machine specific and shared, improving build efficiency. There is little point in a recipe becomming machine specific over a config file. So I'd consider this move a consolation which we can improve over time. For new users I'd suggest that one more common place for the majority of machine specific files would be more understandable too. Cheers, Richard