Hello again NuttX maintainers, I have created a second PR to revert my previous changes: https://github.com/apache/nuttx/pull/15331. First, I want to apologize for the changes I introduced yesterday. I was expecting them to not break anything but it seems the user and kernel space code is tightly coupled.
The change ideally should have only affected user-space code, as syslog, vsprintf, and friends are user-space libraries. However, some NuttX code in the kernel-space area uses those functions instead of having kernel ones. For the moment, I'm proposing reverting this change, and in the future work on a better approach to address this. Thanks in advance for all of your support. Merry Christmas, Javier Alonso Silva (he, him, his) Geotab Embedded Systems Developer | GEUR *Quickly schedule a meeting <https://calendar.app.google/DRoGm4sLw89JC8At6>* Toll-free Visit +34 900 535 371 www.geotab.com/es Twitter <https://twitter.com/geotab> | Facebook <https://www.facebook.com/Geotab> | YouTube <https://www.youtube.com/user/MyGeotab> | LinkedIn <https://www.linkedin.com/company/geotab/> On Mon, Dec 23, 2024 at 3:54 PM Javier Alonso <javieralo...@geotab.com> wrote: > Good afternoon NuttX devs, > > I hope this email finds you well. First of all, Merry Christmas! I have > just submitted a PR to the NuttX repo regarding a missing feature in the > syslog module. The IEEE Std 1003.1-2024 states that the syslog() call shall > accept the %m modifier, which causes the current strerror to be printed - > it is a shorthand to "%s", strerror(errno). > > The PR is available at https://github.com/apache/nuttx/pull/15320. > > Have a nice day, > > Javier Alonso Silva (he, him, his) > Geotab > > Embedded Systems Developer | GEUR > > > *Quickly schedule a meeting > <https://calendar.app.google/DRoGm4sLw89JC8At6>* > > Toll-free > > Visit > > +34 900 535 371 > www.geotab.com/es > > Twitter <https://twitter.com/geotab> | Facebook > <https://www.facebook.com/Geotab> | YouTube > <https://www.youtube.com/user/MyGeotab> | LinkedIn > <https://www.linkedin.com/company/geotab/> >
