> -----Original Message----- > From: Dmytriyenko, Denys > Sent: Wednesday, April 10, 2013 5:02 PM > To: Maupin, Chase > Cc: [email protected] > Subject: Re: [meta-arago] [PATCH] tisdk-image: add support for > DTB files > > On Wed, Apr 10, 2013 at 05:58:47PM -0400, Maupin, Chase wrote: > > > -----Original Message----- > > > From: Dmytriyenko, Denys > > > Sent: Wednesday, April 10, 2013 4:48 PM > > > To: Maupin, Chase > > > Cc: [email protected] > > > Subject: Re: [meta-arago] [PATCH] tisdk-image: add support > for > > > DTB files > > > > > > On Fri, Apr 05, 2013 at 09:37:04PM -0400, Maupin, Chase > wrote: > > > > I was thinking that in the case you do want all dtb files > you > > > could set no > > > > filter and get them all. > > > > > > You can also set filter to "*", which would result in 3 stars > > > filter :) but > > > still work as a shell wildcard and pick up all dtbs. > Otherwise we > > > would need > > > to check if the filter is "unknown" to skip calling "find" > > > uselessly... > > > > Are you saying this is an issue? Not sure if you are expecting > some update > > from me here. > > I'm just trying to avoid running find if it's not needed. Are you > Ok keying it > off empty/unset DTB_FILTER as I suggested below? Or if not, do > you want to > check it against "unknown" instead?
i.e. if DTB_FILTER = "unknown" don't run the for loop? > > > > > > On Apr 5, 2013, at 6:57 PM, "Dmytriyenko, Denys" > <[email protected]> > > > wrote: > > > > > > > > > On Wed, Apr 03, 2013 at 11:48:30AM -0500, Chase Maupin > wrote: > > > > >> * If DTB files exist add support for copying those files > > > into > > > > >> the SDK prebuilt-images directory. > > > > >> * The files to be copied can be controlled using the > > > DTB_FILTER > > > > >> variable. > > > > >> > > > > >> Signed-off-by: Chase Maupin <[email protected]> > > > > >> --- > > > > >> meta-arago-distro/classes/tisdk-image.bbclass | 10 > > > ++++++++++ > > > > >> 1 files changed, 10 insertions(+), 0 deletions(-) > > > > >> > > > > >> diff --git a/meta-arago-distro/classes/tisdk- > image.bbclass > > > b/meta-arago-distro/classes/tisdk-image.bbclass > > > > >> index a1ce1ad..4e3ed52 100644 > > > > >> --- a/meta-arago-distro/classes/tisdk-image.bbclass > > > > >> +++ b/meta-arago-distro/classes/tisdk-image.bbclass > > > > >> @@ -528,6 +528,16 @@ do_sdk_image () { > > > > >> return 1 > > > > >> fi > > > > > > > > > > I was thinking of adding DTB_FILTER ?= "unknown" here, > but > > > then realized it > > > > > would just waste few cycles trying to find something in > the > > > deploy directory, > > > > > which we can avoid completely by testing if it set or > not... > > > > > > > > > > > > > > >> + # Copy the DTB files if they exist. > > > > >> + # NOTE: For simplicity remove the uImage- prefix on > the > > > dtb files. Get just the symlink > > > > >> + # files for a cleaner name. Use the > DTB_FILTER > > > variable to allow finding the > > > > >> + # dtb files for only that MACHINE type > > > > > > > > > > Wrap the below for loop in "if [ -z "${DTB_FILTER}" ]; > then" > > > here? > > > > > > > > > > > > > > >> + for f in `find ${DEPLOY_DIR_IMAGE} -type l -name > > > "*${DTB_FILTER}*.dtb"` > > > > >> + do > > > > >> + dtb_file=`basename $f | sed s/uImage-//` > > > > >> + cp $f ${prebuilt_dir}/${dtb_file} > > > > >> + done > > > > >> + > > > > >> if [ "${SECONDARY_BOOTLOADER_NAME}" != "" ] > > > > >> then > > > > >> # Copy the secondary bootloader image if it > exists > > > > >> -- > > > > >> 1.7.0.4 > > > > >> > > > > >> _______________________________________________ > > > > >> meta-arago mailing list > > > > >> [email protected] > > > > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta- > arago > > > > >> _______________________________________________ meta-arago mailing list [email protected] http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
