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

Unify the definition of GCCVER and remove duplicate code. #14159

Merged
merged 1 commit into from
Oct 13, 2024

Conversation

cuiziwei1
Copy link
Contributor

Summary

Impact

Testing

@github-actions github-actions bot added Area: Tooling Area: Build system Arch: arm Issues related to ARM (32-bit) architecture Area: OS Components OS Components issues Size: S The size of the change in this PR is small labels Oct 12, 2024
@nuttxpr
Copy link

nuttxpr commented Oct 12, 2024

[Experimental Bot, please feedback here]

I need the content of the PR to assess if it meets the NuttX requirements. Please provide the following:

  • The actual PR content: This includes the PR description, code changes, and any related files.
  • Links to the PR: If the PR is already open on GitHub, please share the link.

With this information, I can review the PR against the NuttX requirements and provide a concise assessment.

tools/Config.mk Outdated Show resolved Hide resolved
arch/arm/src/cmake/gcc.cmake Outdated Show resolved Hide resolved
@cuiziwei1
Copy link
Contributor Author

@anchao @xiaoxiang781216 please review this patch to resolve #14130 (comment)

@xuxin930
Copy link
Contributor

hi @anchao please review whether this location is reasonable.

Copy link
Contributor

@GUIDINGLI GUIDINGLI left a comment

Choose a reason for hiding this comment

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

LGTM

@GUIDINGLI GUIDINGLI merged commit 7246533 into apache:master Oct 13, 2024
37 checks passed
arch/arm/src/cmake/gcc.cmake Show resolved Hide resolved
arch/arm/src/common/Toolchain.defs Show resolved Hide resolved
libs/libxx/libcxx.cmake Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: arm Issues related to ARM (32-bit) architecture Area: Build system Area: OS Components OS Components issues Area: Tooling Size: S The size of the change in this PR is small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants