From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by arago-project.org (Postfix) with ESMTPS id 60862529A0 for ; Fri, 20 Sep 2013 21:14:27 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id r8KLEQU1026466 for ; Fri, 20 Sep 2013 16:14:26 -0500 Received: from DFLE73.ent.ti.com (dfle73.ent.ti.com [128.247.5.110]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id r8KLEQWL004120 for ; Fri, 20 Sep 2013 16:14:26 -0500 Received: from dflp33.itg.ti.com (10.64.6.16) by DFLE73.ent.ti.com (128.247.5.110) with Microsoft SMTP Server id 14.2.342.3; Fri, 20 Sep 2013 16:14:26 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id r8KLEQnp000852; Fri, 20 Sep 2013 16:14:26 -0500 Date: Fri, 20 Sep 2013 17:14:26 -0400 From: Denys Dmytriyenko To: "Cooper Jr., Franklin" Message-ID: <20130920211425.GF30673@edge> References: <1379445202-31364-1-git-send-email-fcooper@ti.com> <8F29D6B095ED194EA1980491A5E029710C553A50@DFLE08.ent.ti.com> <20130920193051.GB30673@edge> <8F29D6B095ED194EA1980491A5E029710C55817B@DFLE08.ent.ti.com> <20130920201855.GD30673@edge> <8F29D6B095ED194EA1980491A5E029710C558370@DFLE08.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <8F29D6B095ED194EA1980491A5E029710C558370@DFLE08.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" Subject: Re: [PATCH 1/2] tisdk-rootfs-base-image: Rename base file system image X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 20 Sep 2013 21:14:27 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Fri, Sep 20, 2013 at 04:55:07PM -0400, Cooper Jr., Franklin wrote: > > > > -----Original Message----- > > From: Dmytriyenko, Denys > > Sent: Friday, September 20, 2013 3:19 PM > > To: Cooper Jr., Franklin > > Cc: meta-arago@arago-project.org > > Subject: Re: [meta-arago] [PATCH 1/2] tisdk-rootfs-base-image: Rename base > > file system image > > > > On Fri, Sep 20, 2013 at 03:38:06PM -0400, Cooper Jr., Franklin wrote: > > > > > > > > > > -----Original Message----- > > > > From: Dmytriyenko, Denys > > > > Sent: Friday, September 20, 2013 2:31 PM > > > > To: Cooper Jr., Franklin > > > > Cc: meta-arago@arago-project.org > > > > Subject: Re: [meta-arago] [PATCH 1/2] tisdk-rootfs-base-image: > > > > Rename base file system image > > > > > > > > On Tue, Sep 17, 2013 at 04:05:10PM -0400, Cooper Jr., Franklin wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: Dmytriyenko, Denys > > > > > > Sent: Tuesday, September 17, 2013 2:52 PM > > > > > > To: Cooper Jr., Franklin; meta-arago@arago-project.org > > > > > > Subject: RE: [meta-arago] [PATCH 1/2] tisdk-rootfs-base-image: > > > > > > Rename base file system image > > > > > > > > > > > > What was the initial reason to start naming rootfs images with > > > > > > tisdk-rootfs- prefix? I remember it was something forced by the SDK > > setup... > > > > > [Franklin] > > > > > This was before my time so I am not sure what the original reasoning was. > > > > > > > > > > I do assume there might be instances where some script somewhere > > > > > will need to be updated if it refers to the full image as "tisdk-rootfs*" > > > > > > > > So, I located the change I was talking about: > > > > http://arago-project.org/git/?p=meta- > > > > arago.git;a=commitdiff;h=91f2be34e301deebbc75c54bbad6a5a27e25bf2c > > > > > > > > The "tisdk-image class had issues" is a rather vague description, so > > > > I'm not sure or don't remember what was the problem that you had to > > > > rename the original arago-tisdk-image into tisdk-rootfs-image... > > > > > > > > > [Franklin] Ah I remember. The TARGET_IMAGES variable is used in two > > > ways in that class. 1 to add a build dependency and 2 to reference the > > > image tarball. So the assumption was that the recipe/package name was > > > the same as the IMAGE_BASENAME. So the recipe and IMAGE_BASENAME > > must > > > match. Otherwise the image would be built but the class wouldn't know > > > what tarball to look for. > > > > Ah, OK, this is not exactly what I was looking for. Tracing even further back > > shows that you added the recipe to have IMAGE_BASENAME = "tisdk-rootfs": > > > > http://arago-project.org/git/?p=meta- > > arago.git;a=commitdiff;h=f27b1f34aee2c2cbee25a0fb5b21cdb04f16ec4a > > > > Do you remember the reason for that? This image recipe was in Classic arago, > > but there it had the basename matching the recipe name. Wondering why the > > change? > [Franklin] I think I was simply trying to match the filesystem tarball with > the name used in previous sdks. I am assuming that the SDK backend script > that built the sdk renamed those filesystem files. Looking at Sitara AMSDK > 05.05.01 (last oe-classic based sdk) the two filesystem tarballs were named > base-rootfs-am335x-evm.tar.gz and tisdk-rootfs-am335x-evm.tar.gz. During > that time I was focused on just getting the filesystem working and wanted it > to look like the output from previous sdks. I don't think I had any other > reason. Thanks! Just wanted to check if there was some sort of requirement for that name change. If there is not, then we can look for a better names for full image as well as base image. Should we look into unifying the names for other image recipes? Is tisdk-rootfs- the best base we can think of? -- Denys > > > > > > -----Original Message----- > > > > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > > > > > > bounces@arago-project.org] On Behalf Of Cooper Jr., Franklin > > > > > > Sent: Tuesday, September 17, 2013 3:13 PM > > > > > > To: meta-arago@arago-project.org > > > > > > Cc: Cooper Jr., Franklin > > > > > > Subject: [meta-arago] [PATCH 1/2] tisdk-rootfs-base-image: Rename > > base > > > > file > > > > > > system image > > > > > > > > > > > > * People are frequently confused by the difference in naming between > > the > > > > > > minimal > > > > > > file system and the larger file system. > > > > > > * Rename the minimal file system to mimic the naming convention of the > > > > larger > > > > > > file system. > > > > > > > > > > > > Signed-off-by: Franklin S. Cooper Jr > > > > > > --- > > > > > > ...e-tisdk-image.bb => tisdk-rootfs-base-image.bb} | 2 +- > > > > > > 1 files changed, 1 insertions(+), 1 deletions(-) rename meta-arago- > > > > > > distro/recipes-core/images/{arago-base-tisdk-image.bb => tisdk-rootfs- > > > > base- > > > > > > image.bb} (87%) > > > > > > > > > > > > diff --git a/meta-arago-distro/recipes-core/images/arago-base-tisdk- > > > > image.bb > > > > > > b/meta-arago-distro/recipes-core/images/tisdk-rootfs-base-image.bb > > > > > > similarity index 87% > > > > > > rename from meta-arago-distro/recipes-core/images/arago-base-tisdk- > > > > > > image.bb > > > > > > rename to meta-arago-distro/recipes-core/images/tisdk-rootfs-base- > > > > image.bb > > > > > > index 2c0e217..a3a9123 100644 > > > > > > --- a/meta-arago-distro/recipes-core/images/arago-base-tisdk-image.bb > > > > > > +++ b/meta-arago-distro/recipes-core/images/tisdk-rootfs-base- > > image.bb > > > > > > @@ -12,4 +12,4 @@ IMAGE_INSTALL += "\ > > > > > > packagegroup-arago-base-tisdk \ > > > > > > " > > > > > > > > > > > > -export IMAGE_BASENAME = "arago-base-tisdk-image" > > > > > > +export IMAGE_BASENAME = "tisdk-rootfs-base-image" > > > > > > -- > > > > > > 1.7.0.4 > > > > > > > > > > > > _______________________________________________ > > > > > > meta-arago mailing list > > > > > > meta-arago@arago-project.org > > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago