Skip to content
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

Targets: NXP: IMXRT: Fixed GCC_ARM lds syntax. #15355

Merged
merged 1 commit into from
Jan 17, 2023

Conversation

imi415
Copy link
Contributor

@imi415 imi415 commented Dec 7, 2022

Signed-off-by: Yilin Sun [email protected]

Summary of changes

Target: MIMXRT1170_EVK: A space is missing from the GCC_ARM linker script, fixed
Target: MIMXRT1050_EVK: A space is missing from the GCC_ARM linker script, fixed

Impact of changes

Migration actions required

Documentation


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers


@0xc0170 0xc0170 added the release-type: patch Indentifies a PR as containing just a patch label Dec 8, 2022
0xc0170
0xc0170 previously approved these changes Dec 8, 2022
@0xc0170
Copy link
Contributor

0xc0170 commented Dec 8, 2022

CI started

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 16, 2022

I accidentally run the incorrect test job for continuous-integration/jenkins/pr-head. I'll restart but first we need to understand the license check failing.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 2, 2023

@imi415 I fixed the scancode license issue on master, can you rebase your pull request to the latest master branch?

@mergify mergify bot dismissed 0xc0170’s stale review January 2, 2023 15:25

Pull request has been modified.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 3, 2023

We found another issue with the license check (another fix is now on master). Please rebase again to get the fix in.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 17, 2023

CI started

@mbed-ci
Copy link

mbed-ci commented Jan 17, 2023

Jenkins CI Test : ✔️ SUCCESS

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

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

Successfully merging this pull request may close these issues.

None yet

5 participants