[PATCH 2/2] rcmd.c: Removed references to unsupported functions

Joel Sherrill joel at rtems.org
Sat May 4 14:21:37 UTC 2019


Sorry for missing that it actually was referenced

Rcmd can and should work. Rcmdsh will not ever and should be removed from
libbsd.

Does rcmd need testing? It clearly had no test since it wouldn't have
linked.

--joel

On Sat, May 4, 2019, 4:40 AM Peter Dufault <dufault at hda.com> wrote:

>
>
> On May 4, 2019, at 02:37 , Chris Johns <chrisj at rtems.org> wrote:
>
> On 4/5/19 2:30 pm, Joel Sherrill wrote:
>
> On Fri, May 3, 2019, 8:18 PM Chris Johns <chrisj at rtems.org <
> mailto:chrisj at rtems.org <chrisj at rtems.org>>> wrote:
>    On 4/5/19 9:16 am, Joel Sherrill wrote:
>     > On Fri, May 3, 2019, 6:04 PM Chris Johns <chrisj at rtems.org
>    <mailto:chrisj at rtems.org <chrisj at rtems.org>>
>     > <mailto:chrisj at rtems.org <chrisj at rtems.org> <mailto:chrisj at rtems.org
> <chrisj at rtems.org>>>> wrote:
>     >
>     >     On 4/5/19 7:44 am, Lou Woods wrote:
>     >      > From: Lou Woods <Lou.Woods at OARCorp.com>
>     >      >
>     >      > rcmdsh() and innetgr() are referenced by rcmd, but aren’t
>    available
>     >      > in libbsd and are unresolvable external symbols when
>    libbsd.a is
>     >      > processed by the RTL.
>     >
>     >     Why does this not show up when we statically link?
>     >
>     >     I wonder if this is related to separate sections where a
>    function in a
>     >     file is present but no used and it not linked but is with the
>    RTL?
>     >
>     > Statically linking an exe normally wouldn't have hit this unless you
>     > referenced rcmd. Ld wouldn't have pulled it out of the library.
>    Oh your app is referencing `rcmd`?
> No. Lou tracked down the symbols that the RTL showed as undefined. That
> led us to look into rcmd and rcmdsh. I spotted the fork/exec and realized
> some chunk had to be removed. Rcmdsh in whole and parts of rcmd.
>
>
> What do the module dependencies look like? See ...
>
> https://docs.rtems.org/branches/master/shell/dl_commands.html#rtl-list
>
> I am sorry I am not following why the rcmd.c file is being loaded. If
> nothing references any symbols in it it should not be loaded it.
>
> In the code Lou is looking at it is referenced.
>
>
>     > FWIW rcmdsh.* needs to be dropped out of the build. It requires
>     > fork/exec so will never work.
>    Should this change be a build system change or is this needed plus the
>    build system change?
> Both. Rcmdsh.* should not be in libbsd.
>
>
> Does the `#if __rtems__` formatting conform to CONTRIBUTING.md ? It looks
> to me like some extra lines and `#else` with out `__rtems__`.
>
> Chris
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
>
> Peter
> -----------------
> Peter Dufault
> HD Associates, Inc.      Software and System Engineering
>
> This email is delivered through the public internet using protocols
> subject to interception and tampering.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20190504/5168ba3f/attachment-0002.html>


More information about the devel mailing list