All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Cooper Jr., Franklin" <fcooper@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [PATCH v2] tisdk-image: Fix manifest generation for nativesdk.
Date: Fri, 6 Mar 2015 17:48:18 -0500	[thread overview]
Message-ID: <20150306224818.GZ6052@edge> (raw)
In-Reply-To: <8F29D6B095ED194EA1980491A5E029710C8390E1@DFLE08.ent.ti.com>

Jake,

Can you please refresh my mind with more details of the problem? Thanks.


On Fri, Mar 06, 2015 at 08:49:10PM +0000, Cooper Jr., Franklin wrote:
> Ah I'm not aware of that since I never ran into the issue.
> 
> > -----Original Message-----
> > From: Stiffler, Jacob
> > Sent: Friday, March 06, 2015 1:45 PM
> > To: Cooper Jr., Franklin; meta-arago@arago-project.org
> > Subject: RE: [meta-arago] [PATCH v2] tisdk-image: Fix manifest generation
> > for nativesdk.
> > 
> > I ran into this issue in December. Denys also reported seeing the same issue.
> > 
> > -----Original Message-----
> > From: Cooper Jr., Franklin
> > Sent: Friday, March 06, 2015 2:38 PM
> > To: Stiffler, Jacob; meta-arago@arago-project.org
> > Subject: RE: [meta-arago] [PATCH v2] tisdk-image: Fix manifest generation
> > for nativesdk.
> > 
> > Don't know what the issue is especially since your saying that the problem is
> > on the build machine and not your local setup.
> > 
> > I just made a release a little less than a month ago and haven't had a problem
> > on my local or build machine. Nothing important has changed within oe-core
> > in the last several months and nothing has changed within oe-core within the
> > last several months. Denys did change the version of bitbake used by default
> > but that wouldn't explain the different behavior between your machines.
> > 
> > So it seems like the issue points to your build machine.
> > 
> > > -----Original Message-----
> > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> > > Sent: Friday, March 06, 2015 1:26 PM
> > > To: meta-arago@arago-project.org
> > > Subject: Re: [meta-arago] [PATCH v2] tisdk-image: Fix manifest
> > > generation for nativesdk.
> > >
> > > I also need to revoke this patch. I forgot I am not able to recreate
> > > this issue on my local setup, and I have only seen it on the build machine.
> > >
> > > This patch will cause a build failure if any of the oe* package
> > > indexes are empty, as the " grep -n "^Package" $pkg_idx" will return an
> > error.
> > >
> > > Are there any other comments which I may address in the v3?
> > >
> > > Thanks,
> > > Jake
> > >
> > > -----Original Message-----
> > > From: Stiffler, Jacob
> > > Sent: Friday, March 06, 2015 9:34 AM
> > > To: meta-arago@arago-project.org
> > > Cc: Stiffler, Jacob
> > > Subject: [PATCH v2] tisdk-image: Fix manifest generation for nativesdk.
> > >
> > > * The manifest is automatically generated based on *.control
> > >   files found within the various images.
> > > * The control files for the nativesdk image have gone missing.
> > > * Work around this by regenerating the control files from the
> > >   package index.
> > >
> > > Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> > > ---
> > > v2 changes:
> > > Parse all "oe*" package indexes instead of only "oe-*-nativesdk" indexes.
> > >
> > >
> > >  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..b9d86af 100644
> > > --- a/meta-arago-distro/classes/tisdk-image.bbclass
> > > +++ b/meta-arago-distro/classes/tisdk-image.bbclass
> > > @@ -353,6 +353,43 @@ echo "
> > >  </tr>
> > >  " >> ${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`
> > > +
> > > +            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
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


  reply	other threads:[~2015-03-06 22:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-06 14:34 [PATCH v2] tisdk-image: Fix manifest generation for nativesdk Jacob Stiffler
2015-03-06 19:26 ` Stiffler, Jacob
2015-03-06 19:38   ` Cooper Jr., Franklin
2015-03-06 19:45     ` Stiffler, Jacob
2015-03-06 20:49       ` Cooper Jr., Franklin
2015-03-06 22:48         ` Denys Dmytriyenko [this message]
2015-03-09 13:49           ` Stiffler, Jacob
  -- strict thread matches above, loose matches on Subject: below --
2015-03-09 16:03 Dmytriyenko, Denys

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150306224818.GZ6052@edge \
    --to=denys@ti.com \
    --cc=fcooper@ti.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.