I believe this mklib call was added recently due to a race condition and it was not built before it was needed - any chance to fix the race properly instead of all the workarounds?
On Tue, Oct 02, 2018 at 02:38:00PM -0400, Jacob Stiffler wrote: > > On 10/2/2018 8:29 AM, Jacob Stiffler wrote: > > > >I recently ran into this error, so I sent this out in a hurry to get it > >fixed. But now I am thinking that it may be better to remove this library > >in do_configure, similar to base class calls make clean > > > > After thinking about it some more, I would prefer to remove the library in > do_configure() to be safe in the unlikely scenario that there are > dependencies involved. > > > As it is now, this is hardcoded to a single library. While I am making > changes, should I enhance this to take a list of libraries so that it can be > easily extended in the future? > > > >Any suggestions? > > > > > >Thanks, > > > >Jake > > > >On 10/2/2018 7:36 AM, Jacob Stiffler wrote: > >>* Prevent a potential error as mklib will return an error if the > >> library already exists. > >>* The error encountered is the following: > >> > >>| >> ERROR: mklib: destination library > >>.../work/x86_64-linux/ti-cgt-arm-native/16.9.2-r1/ti-cgt-arm-16.9.2/ti-cgt-arm_16.9.2.LTS/lib/rtsv7R4_A_le_v3D16_eabi.lib > >> already exists > >>| WARNING: > >>.../work/x86_64-linux/ti-cgt-arm-native/16.9.2-r1/temp/run.do_compile.13758:1 > >> exit 1 from 'lib/mklib --pattern=rtsv7R4_A_le_v3D16_eabi.lib' > >> > >>Signed-off-by: Jacob Stiffler<[email protected]> > >>--- > >> recipes-ti/devtools/ti-cgt-arm.inc | 10 +++++++--- > >> 1 file changed, 7 insertions(+), 3 deletions(-) > >> > >>diff --git a/recipes-ti/devtools/ti-cgt-arm.inc > >>b/recipes-ti/devtools/ti-cgt-arm.inc > >>index b9f7566..e83973b 100644 > >>--- a/recipes-ti/devtools/ti-cgt-arm.inc > >>+++ b/recipes-ti/devtools/ti-cgt-arm.inc > >>@@ -9,7 +9,7 @@ require recipes-ti/includes/ti-paths.inc > >> S = "${WORKDIR}/ti-cgt-arm-${PV}" > >>-PR = "r1" > >>+PR = "r2" > >> SRC_URI > >> ="http://software-dl.ti.com/codegen/esd/cgt_public_sw/TMS470/${PV}.LTS/${BINFILE};name=${BINFILE_NAME}"; > >>@@ -19,8 +19,12 @@ TI_BIN_UNPK_CMDS = "" > >> do_compile_append() { > >> cd ${S}/ti-cgt-arm_${PV}.LTS > >>- export PATH=${PATH}:${S}/ti-cgt-arm_${PV}.LTS/bin > >>- lib/mklib --pattern=rtsv7R4_A_le_v3D16_eabi.lib > >>+ > >>+ if [ ! -f "./lib/rtsv7R4_A_le_v3D16_eabi.lib" ] > >>+ then > >>+ export PATH=${PATH}:${S}/ti-cgt-arm_${PV}.LTS/bin > >>+ lib/mklib --pattern=rtsv7R4_A_le_v3D16_eabi.lib > >>+ fi > >> } > >> do_install() { > -- > _______________________________________________ > meta-ti mailing list > [email protected] > https://lists.yoctoproject.org/listinfo/meta-ti -- _______________________________________________ meta-ti mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-ti
