Skip to content

Conversation

@gpsimenos
Copy link
Contributor

No description provided.

Copy link
Contributor

@evedon evedon left a comment

Choose a reason for hiding this comment

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

Looks good

@evedon
Copy link
Contributor

evedon commented Jun 22, 2020

@mikter Any idea why CI fails as this PR does not introduce a code change?

@mikter
Copy link

mikter commented Jun 22, 2020

ERROR] /usr/local/gcc-arm-none-eabi-9-2019-q4-major/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: out/KW41Z_GCC_ARM_internal_6lp/mbed-os/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/fsl_common.o: in function InstallIRQHandler': /builds/ws/mal_PR-308-2G222WDVXU56XCMT2PRK7@3/mbed-os-example-mesh-minimal/./mbed-os/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/fsl_common.c:61: undefined reference to __Vectors'
collect2: error: ld returned 1 exit status

@evedon This seems to be the error, but I have no idea what is causing it @artokin any ideas

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 22, 2020

what version of Mbed OS are you using it? The error is one we had when we updated cmsis, but was fixed in Mbed OS.

@artokin
Copy link

artokin commented Jun 22, 2020

Here is the issue Martin is referencing: ARMmbed/mbed-os#13062
It was fixed by PR ARMmbed/mbed-os#13064 but I believe the fix is not available in the Mbed OS 6.0.0 that is used by this application mbed-os.lib.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 22, 2020

Correct, the fix will be in 6.1 (as I can see was merged post 6.0), unfortunately.

@evedon
Copy link
Contributor

evedon commented Jun 22, 2020

@0xc0170 I think we should go ahead wit merging this PR as there is no code change.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 22, 2020

@mikter Please merge

@artokin artokin merged commit e93eeda into ARMmbed:master Jun 22, 2020
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