tmedicci commented on PR #15099: URL: https://github.com/apache/nuttx/pull/15099#issuecomment-2531820233
> > > About the CI, It's failing because of the flags used to build Python. I can fix them, but I'm afraid that it'd take too long to build the test because it requires building python for the host too. @lupyuen , do you have any suggestions on how to skip this test (or at least running it only from time to time)? > > > > > > but should we fix the build issue first before disabling it from ci? I saw the real build problem from the previous build, it isn't good to merge the unworkable defconfig. > > It's usable. The problem here is that ci sets `EXTRAFLAGS="-Wno-cpp -Werror"`. I'm fixing it by disabling such warnings in the nuttx-apps (as other apps do) and I'll submit it soon. > > However, although we have this "issue", the Python build requires building the whole Python for the host system (yes, we need Python to build Python), so it'd be a huge load for the CI and that's why I think we should skip it on CI. Ideally, we should exclude it from CI but run it on a scheduled basis (from time to time). I don't know if our CI allows it somehow. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
