nuttx icon indicating copy to clipboard operation
nuttx copied to clipboard

greenhills: add cmake support

Open extinguish opened this issue 1 year ago • 12 comments

Summary

  1. refactor the toolchain management in CMake
  2. unify the cmake toolchain name style
  3. support greenhills build procedure with CMake
  4. add protect build for greenhills and gnu with CMake

Impact

  1. refactor the cmake toolchain management, each toolchain has its own cmake file, make the toolchain specific feature modification constrain in its own cmake file, which is more clear

Testing

  1. has pass the os test,
  2. has test on gcc/ghs compiler

extinguish avatar Sep 26 '24 06:09 extinguish

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 26 '24 06:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 26 '24 09:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 26 '24 11:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 26 '24 12:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 27 '24 12:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 28 '24 07:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 28 '24 08:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 29 '24 01:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 29 '24 02:09 github-actions[bot]

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 29 '24 03:09 github-actions[bot]

the current build break is fixed by: https://github.com/apache/nuttx/pull/13717 which is still under review

extinguish avatar Sep 29 '24 06:09 extinguish

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

github-actions[bot] avatar Sep 29 '24 08:09 github-actions[bot]

@extinguish @xiaoxiang781216 hi, after this PR the colored output from compilation with cmake doesn't work.

Before this PR:

image

after this PR compiler output on terminal looks like normal text also for warnings and errors:

image

raiden00pl avatar Oct 18 '24 10:10 raiden00pl

@extinguish @xuxin930 please take a look.

xiaoxiang781216 avatar Oct 18 '24 11:10 xiaoxiang781216

@extinguish @xuxin930 please take a look.

this is fixed is https://github.com/apache/nuttx/pull/14404, and is under review

extinguish avatar Oct 18 '24 12:10 extinguish