-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix macOS build error. #15774
base: master
Are you sure you want to change the base?
Fix macOS build error. #15774
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a proper commit message describing the fix
1. fix apache#14774. Change the defconfig of libcxxtest and select libcxxabi by default to fix sim:libcxxtest build in macOS CI can't find cxxabi.h. 2. fix apache#15491. libcxxabi cannot be used with uclibc, so when compiling macos sim, libcxxabi cannot be selected by default in Kconfig. Signed-off-by: cuiziwei <[email protected]>
43ac2bf
to
20a9a81
Compare
ok |
@@ -71,7 +71,6 @@ if !LIBCXXMINI | |||
|
|||
choice | |||
prompt "C++ low level library select" | |||
default LIBCXXABI if ARCH_SIM && HOST_MACOS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not enable LIBCXXABI by default if ARCH_SIM ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cause select LIBCXXMINI by default. if select LIBCXXABI, we need to select LIBCXX by default if ARCH_SIM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see line 70, this choice snippet is no LIBCXXMINI case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean why not just remove HOST_MACOS:
default LIBCXXABI if ARCH_SIM && HOST_MACOS
->
default LIBCXXABI if ARCH_SIM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some sim defconfig(like https://github.com/apache/nuttx/blob/master/boards/sim/sim/sim/configs/cxxtest/defconfig) enable uclibcxx by default. If libcxxabi is selected by default, there may also be compilation problems.
Summary
fix [BUG] macOS CI:
sim:libcxxtest
can't find cxxabi.h #14774.Change the defconfig of libcxxtest and select libcxxabi by default.
fix [BUG] Mirror of NuttX: All macOS Jobs are broken #15491.
libcxxabi cannot be used with uclibc, so when compiling macos sim, libcxxabi cannot be selected by default in Kconfig.
Impact
macos sim
Testing
ci test