You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "pkarashchenko (via GitHub)" <gi...@apache.org> on 2023/03/17 05:24:45 UTC
[GitHub] [nuttx] pkarashchenko opened a new pull request, #8836: tools/ci: upgrade to the latest ESP32 toolchains
pkarashchenko opened a new pull request, #8836:
URL: https://github.com/apache/nuttx/pull/8836
## Summary
Upgrade ESP32 toolchains on CI
## Impact
Hope to be none
## Testing
Pass CI
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] tmedicci commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "tmedicci (via GitHub)" <gi...@apache.org>.
tmedicci commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1473627268
Just a nitpick: the more appropriate title would be: "tools/ci: upgrade to the latest ESP32 **xtensa** toolchains"
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] pkarashchenko commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "pkarashchenko (via GitHub)" <gi...@apache.org>.
pkarashchenko commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1474001631
Yes, of course. I'm still traveling from EW2023, so can't even find the "concert to draft" option in the mobile version of the GitHub :(
I will take a look when I arrive
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] tmedicci commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "tmedicci (via GitHub)" <gi...@apache.org>.
tmedicci commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1473625834
Hi @pkarashchenko , could you test for thrown warnings during compilation? Just to make sure: our ci does not compile with `-Werror`, right?
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] pkarashchenko commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "pkarashchenko (via GitHub)" <gi...@apache.org>.
pkarashchenko commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1473981985
I think we already upgraded to GCC 12 for ARM, so most of the common issues should be already fixed.
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] tmedicci commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "tmedicci (via GitHub)" <gi...@apache.org>.
tmedicci commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1473998082
Hi @pkarashchenko !
I've caught some warnings regarding the defconfigs `esp32-devkitc:audio` and `:cxx`. Could you please take a look?
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] gustavonihei merged pull request #8836: tools/ci: upgrade to the latest Xtensa ESP32 toolchains
Posted by "gustavonihei (via GitHub)" <gi...@apache.org>.
gustavonihei merged PR #8836:
URL: https://github.com/apache/nuttx/pull/8836
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] tmedicci commented on pull request #8836: tools/ci: upgrade to the latest Xtensa ESP32 toolchains
Posted by "tmedicci (via GitHub)" <gi...@apache.org>.
tmedicci commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1477782997
This PR may be reopened, @pkarashchenko !
@gustavonihei , Do you want to take a look at it?
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] tmedicci commented on pull request #8836: tools/ci: upgrade to the latest Xtensa ESP32 toolchains
Posted by "tmedicci (via GitHub)" <gi...@apache.org>.
tmedicci commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1477748095
> @tmedicci I've tested many Xtensa configurations today (definitely not all available). Could you please try this PR? The issues in `esp32-devkitc:audio` and `:cxx` are fixed now
I'll try it right now. Just a moment.
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] pkarashchenko commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "pkarashchenko (via GitHub)" <gi...@apache.org>.
pkarashchenko commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1473628235
> Hi @pkarashchenko , could you test for thrown warnings during compilation? Just to make sure: our ci does not compile with `-Werror`, right?
Yes. I will rebuild locally and test configurations. Actually I'm using GCC 11 on MAC for ESP32 and some of the configurations that I use are compiled without errors.
I will upgrade to GCC 12 and run build for all ESP32 configurations. Let's mark this as Draft fro now
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] tmedicci commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "tmedicci (via GitHub)" <gi...@apache.org>.
tmedicci commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1474004499
> Yes, of course. I'm still traveling from EW2023, so can't even find the "concert to draft" option in the mobile version of the GitHub :( I will take a look when I arrive
No problems at all! Have a nice conference and say hello to our community colleagues!
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] pkarashchenko commented on pull request #8836: tools/ci: upgrade to the latest Xtensa ESP32 toolchains
Posted by "pkarashchenko (via GitHub)" <gi...@apache.org>.
pkarashchenko commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1476339863
@tmedicci I've tested many Xtensa configurations today (definitely not all available). Could you please try this PR? The issues in `esp32-devkitc:audio` and `:cxx` are fixed now
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] acassis commented on pull request #8836: tools/ci: upgrade to the latest ESP32 toolchains
Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis commented on PR #8836:
URL: https://github.com/apache/nuttx/pull/8836#issuecomment-1473956704
not only CI will fail when -Werror is enabled, but many configurations where #warning flag is used in the source code. i.e. bluetooth, flash, etc
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org