> -----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. > > -- > Denys > > > > 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
