Skip to content

disable the combination of st-iar-mesh example #4964

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

disable the combination of st-iar-mesh example #4964

wants to merge 1 commit into from

Conversation

studavekar
Copy link
Contributor

We have seen inconsistent failure for above combination disabling it for now.
will enable when we update to IAR 8.XX

Todos

  • Tests

We have seen inconsistent failure for above combination disabling it for now.
will enable when we update to IAR 8.XX
Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok with me as long as we communicate clearly that this is a temporary measure until a fix/root cause can be found.

Perhaps we should update this issue with the current workaround: #4879

@LMESTM
Copy link
Contributor

LMESTM commented Aug 24, 2017

Hi - wouldn't you rather investigate the root cause of #4879 ?
Are those failures visible and reproduced anywhere else than in the ci environment ?
You are reporting here "inconsistent failure" - could you be more precise : is this always the same build error or sometimes different ones ?

@studavekar
Copy link
Contributor Author

studavekar commented Aug 24, 2017

@LMESTM after further investigation looks like we may see failures with other platforms as well

iasmarm --cpu Cortex-M4F -D__CORTEX_M4 -DARM_MATH_CM4 -D__FPU_PRESENT=1 -D__MBED_CMSIS_RTOS_CM -D__CMSIS_RTOS -DFSL_RTOS_MBED -DCPU_MK66FN2M0VMD18 -f ./BUILD/K66F/IAR/.includes_d5bd6946ec713c5c47ef4b6c7c6f9a4c.txt -o ./BUILD/K66F/IAR/mbed-os/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_K66F/device/TOOLCHAIN_IAR/startup_MK66F18.o ./mbed-os/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_K66F/device/TOOLCHAIN_IAR/startup_MK66F18.S

Before this we had always seen failure only with ST family devices. May because CI scheduler, runs in this pattern causing above combination to fail.

I will probably close this PR after root causing the problem.

@LMESTM
Copy link
Contributor

LMESTM commented Aug 24, 2017

@studavekar thx

@deepikabhavnani
Copy link

Seems we had this issue earlier as well in CI. #3934

@bridadan - Did you came to conclusion that it was related to working directory path (including project name) > 65 characters.
IAR follows _MAX_PATH limitation of windows and will error out if path exceeds _MAX_PATH character which is 256.

@bridadan
Copy link
Contributor

@deepikabhavnani I didn't really come to any conclusion unfortunately, but yes there did seem to be a correlation between path length and failure rate. Sounds similar to what @studavekar has found as well.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 31, 2017

@studavekar Can we close this or it's still relevant?

@studavekar
Copy link
Contributor Author

Closing this a issue PR, as fix for IAR been added #4977

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants