You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "yamt (via GitHub)" <gi...@apache.org> on 2024/05/24 07:53:45 UTC
[PR] esp32s2/esp32s3 textheap: do not require RTC heap [nuttx]
yamt opened a new pull request, #12399:
URL: https://github.com/apache/nuttx/pull/12399
## Summary
Unlike esp32, kmm memory is executable.
## Impact
## Testing
Tested wamr aot on esp32s3-devkitc.
esp32s2 part is not tested.
--
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
Re: [PR] esp32s2/esp32s3 textheap: do not require RTC heap [nuttx]
Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 merged PR #12399:
URL: https://github.com/apache/nuttx/pull/12399
--
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
Re: [PR] esp32s2/esp32s3 textheap: do not require RTC heap [nuttx]
Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis commented on PR #12399:
URL: https://github.com/apache/nuttx/pull/12399#issuecomment-2129523925
There are tons of this error message:
```
====================================================================================
Configuration/Tool: rv-virt/citest64
------------------------------------------------------------------------------------
Cleaning...
Configuring...
Building NuttX...
usrsocktest_basic_daemon.c: In function 'usrsocktest_group_basic_daemon_teardown':
Error: usrsocktest_basic_daemon.c:505:7: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]
505 | int ret;
| ^~~
cc1: all warnings being treated as errors
make[2]: *** [/github/workspace/sources/apps/Application.mk:200: usrsocktest_basic_daemon.c.github.workspace.sources.apps.examples.usrsocktest.o] Error 1
usrsocktest_basic_getsockname.c: In function 'usrsocktest_group_basic_getsockname_teardown':
Error: usrsocktest_basic_getsockname.c:238:7: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]
238 | int ret;
| ^~~
cc1: all warnings being treated as errors
make[2]: *** [/github/workspace/sources/apps/Application.mk:200: usrsocktest_basic_getsockname.c.github.workspace.sources.apps.examples.usrsocktest.o] Error 1
usrsocktest_basic_getsockopt.c: In function 'usrsocktest_group_basic_getsockopt_teardown':
Error: usrsocktest_basic_getsockopt.c:217:7: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]
217 | int ret;
| ^~~
```
--
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
Re: [PR] esp32s2/esp32s3 textheap: do not require RTC heap [nuttx]
Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 commented on PR #12399:
URL: https://github.com/apache/nuttx/pull/12399#issuecomment-2129545817
@acassis fix here: https://github.com/apache/nuttx-apps/pull/2406
--
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