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? > > > 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
