You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by GitBox <gi...@apache.org> on 2021/02/22 15:12:52 UTC
[GitHub] [mynewt-mcumgr] niclas-ats opened a new issue #116: fs mgmt not working under zephyr 2.5
niclas-ats opened a new issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116
It seems like the `fs_mgmt_impl_read` and `fs_mgmt_impl_write` functions are missing a proper initialization of the local `struct fs_file_t file` variable. The attached patch fixes the up-/download of files for me.
[0001-zephyr-fix-initialization-of-file-struct.txt](https://github.com/apache/mynewt-mcumgr/files/6022866/0001-zephyr-fix-initialization-of-file-struct.txt)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] niclas-ats closed issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
niclas-ats closed issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] de-nordic commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
de-nordic commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-855682090
@niclas-ats Can you close the issue?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] de-nordic commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
de-nordic commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-784908103
>
>
> The fix didn't make it into yesterdays snapshot update. Once it's there I can test it and then close this issue.
Yes, it was little late to the party and the snapshot update has been already waiting for some time due to Zephyr release.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] de-nordic commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
de-nordic commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-855749085
Thanks!
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] de-nordic commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
de-nordic commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-783972866
Yes, I will do the snapshot update.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] de-nordic commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
de-nordic commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-783511213
Can you post PR with the fix?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] niclas-ats commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
niclas-ats commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-784931329
Nevermind. I'll use my fork in the meantime. Thx for the quick approval!
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] niclas-ats commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
niclas-ats commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-783955612
Will this be picked up by https://github.com/zephyrproject-rtos/mcumgr as well? Or do I need to create a seperate pull request in that repo?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-mcumgr] niclas-ats commented on issue #116: fs mgmt not working under zephyr 2.5
Posted by GitBox <gi...@apache.org>.
niclas-ats commented on issue #116:
URL: https://github.com/apache/mynewt-mcumgr/issues/116#issuecomment-784889835
The fix didn't make it into yesterdays snapshot update. Once it's there I can test it and then close this issue.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org