From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from comal.ext.ti.com (comal.ext.ti.com [198.47.26.152]) by arago-project.org (Postfix) with ESMTPS id 3CB0B52A59 for ; Fri, 13 Mar 2015 20:20:22 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id t2DKKLuk011745 for ; Fri, 13 Mar 2015 15:20:21 -0500 Received: from DLEE71.ent.ti.com (dlee71.ent.ti.com [157.170.170.114]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id t2DKKKDp024890 for ; Fri, 13 Mar 2015 15:20:20 -0500 Received: from dlep32.itg.ti.com (157.170.170.100) by DLEE71.ent.ti.com (157.170.170.114) with Microsoft SMTP Server id 14.3.224.2; Fri, 13 Mar 2015 15:20:19 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dlep32.itg.ti.com (8.14.3/8.13.8) with ESMTP id t2DKKJJi014433; Fri, 13 Mar 2015 15:20:20 -0500 Date: Fri, 13 Mar 2015 16:20:19 -0400 From: Denys Dmytriyenko To: "Stiffler, Jacob" Message-ID: <20150313202019.GE26701@edge> References: <1425923703-6810-1-git-send-email-j-stiffler@ti.com> <4B26FB5A64EBA14C87266869E037F958141088E7@DFLE09.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <4B26FB5A64EBA14C87266869E037F958141088E7@DFLE09.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" Subject: Re: [PATCH v3] tisdk-image: Fix manifest generation for nativesdk. 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, 13 Mar 2015 20:20:22 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline I don't have any issues with the patch, per se, but wouldn't the main logic to break up package index be better written in awk? :) On Fri, Mar 13, 2015 at 10:47:28AM +0000, Stiffler, Jacob wrote: > Ping. > > Anything preventing this from being committed? I have another patch on top > of this to also support the text format. > > Thanks, > Jake > > -----Original Message----- > From: Stiffler, Jacob > Sent: Monday, March 09, 2015 1:55 PM > To: meta-arago@arago-project.org > Cc: Stiffler, Jacob > Subject: [PATCH v3] tisdk-image: Fix manifest generation for nativesdk. > > The software manifest is created by parsing the *.control files in the images /var/lib/opkg directory. For the nativesdk sysroot, there are some cases where the control files are not present. The reason for this and the specific circumstances of when this occurs has not yet been root caused. > > When this does happen, the "Development Host Content" tables of the manifest are not valid. However, the package indexes are present. These contain all of the information in the control files, only concatenated together in a large file. This patch will split up these package indexes into individual control files which will allow the previous parsing to produce a valid output. > > Here is an example of the invalid output when the control files are not present: > >

Development Host Content

This table describes any software being delivered that is expected to run on a Development Host, instead of the target device. Some of this software may be licensed under GPLv3 but it is not expected to be shipped as a product.

> > > > > > > > > > > > > > > > > > > > > > Signed-off-by: Jacob Stiffler > --- > meta-arago-distro/classes/tisdk-image.bbclass | 37 +++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass > index 3030a38..747670f 100644 > --- a/meta-arago-distro/classes/tisdk-image.bbclass > +++ b/meta-arago-distro/classes/tisdk-image.bbclass > @@ -353,6 +353,43 @@ echo " > > " >> ${SW_MANIFEST_FILE} > > + control_files_there=0 > + for possible_control_file in $control_dir/*.control > + do > + if [ -f $possible_control_file ] > + then > + control_files_there=1 > + break > + fi > + done > + > + if [ $control_files_there -eq 0 ] > + then > + for pkg_idx in $control_dir/oe*; do > + package_start=`grep -n "^Package" $pkg_idx || true` > + > + IFS_OLD=${IFS} > + IFS=" > +" > + > + for pkg in ${package_start}; do > + end=`echo $pkg | cut -d: -f1` > + > + if [ -z $begin ]; then > + pkg_name=`echo $pkg | cut -d: -f3` > + begin=`echo $pkg | cut -d: -f1` > + continue > + fi > + cnt=$[$cnt+1] > + head -n $[$end - 1] $pkg_idx | tail -n $[$end - $begin] > + > ${control_dir}/${pkg_name// /}.control > + > + pkg_name=`echo $pkg | cut -d: -f3` > + begin=$end > + done > + IFS=${IFS_OLD} > + done > + fi > + > for i in $control_dir/*.control > do > package="`cat $i | grep Package: | awk {'print $2'}`" > -- > 1.7.9.5 > > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
Software NameVersionLicenseLocationDelivered AsModified by TIObtained from
__.ipkBinaryNo