RTEMS | Draft: cpukit: Remove obsolete RTEMS_NETWORKING macro (!1563)

[Saahil Shaw (@Kernel)]

Saahil Shaw commented on a discussion on cpukit/include/rtems/shellconfig.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1563#note_162146

 >  extern rtems_shell_cmd_t rtems_shell_MALLOC_INFO_Command;
 >  extern rtems_shell_cmd_t rtems_shell_RTRACE_Command;
 >  extern rtems_shell_cmd_t rtems_shell_PRIO_Command;
 > -#if RTEMS_NETWORKING
 > -  extern rtems_shell_cmd_t rtems_shell_IFCONFIG_Command;
 > -  extern rtems_shell_cmd_t rtems_shell_ROUTE_Command;
 > -  extern rtems_shell_cmd_t rtems_shell_NETSTATS_Command;
 > -  extern rtems_shell_cmd_t rtems_shell_PING_Command;
 > -#endif

@vijay @joel 

As part of removing RTEMS_NETWORKING, the shell command "extern" declarations need to be moved in rtems-net-legacy.

Vijay suggested moving them into a common header. The most common header that I could find is rtems_bsdnet.h. I would like to propose to move them into a new header explicitly for the shell commands.

My reasons are as follows:

1. **Separation :** Keeps shell integration objects separate from the networking stack API (rtems_bsdnet.h).
2. **Clean for writing application:** Developers only pull in shell symbols when configuring the shell without needing to pull the whole API.

If this doesn't look good, we can go with simply adding the "extern" declarations in the rtems_bsdnet.h

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1563#note_162146
You're receiving this email because of your account on gitlab.rtems.org. Unsubscribe from this thread: https://gitlab.rtems.org/-/namespace/49/sent_notifications/6-5d937uf6m70l19cdvqj0p26ax-1d/unsubscribe | Manage all notifications: https://gitlab.rtems.org/-/profile/notifications | Help: https://gitlab.rtems.org/help


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20261109/7d5d659c/attachment-0001.htm>