[oe] [PATCH] Angstrom: weak-assign DEPLOY_DIR

Andrea Adami andrea.adami at gmail.com
Thu Mar 26 08:47:24 UTC 2009


was: DEPLOY_DIR out of TMPDIR

>Sounds like Angstrom needs:
>USER_DEPLOY_DIR ?= "${TMPDIR}/deploy/${ANGSTROMLIBC}"
>DEPLOY_DIR = "${USER_DEPLOY_DIR}" if users are going to further override
>what the distro thinks is best.

As pointed out by Denix and me, we need changes to base.bbclass too:

 package_stagefile_shell()
 ...
-destfile=`echo $srcfile | sed s#${TMPDIR}#${PSTAGE_TMPDIR_STAGE}#`
+destfile=`echo $srcfile | sed s#${DEPLOY_DIR}#${PSTAGE_TMPDIR_STAGE}#`
...

I tested both patches building from scratch and using packaged staging.

Issues: the 5-6 recipes using package_stagefile_shell() are possibly
broken...DEPLOY_DIR get sed out and the files are not found.
I don't have a list here offhand, just grep...two of them are
zaurus-updater and git.

I fixed locally zaurus-updater just avoiding to call the a.m. function

-        package_stagefile_shell ${DEPLOY_DIR_IMAGE}/updater.sh.${MACHINE}
+       cp -dp updater.sh ${DEPLOY_DIR_IMAGE}/updater.sh.${MACHINE}

-        package_stagefile_shell ${DEPLOY_DIR_IMAGE}/gnu-tar
+       cp -dp gnu-tar ${DEPLOY_DIR_IMAGE}/gnu-tar

Can I proceed and send the set of patches to the mailing list? Better ideas?


Regards

Andrea




More information about the Openembedded-devel mailing list