Re: [PATCH] libc: fix _dummy_signal_syscalls name in signalr.c

On Jan 21 12:40, Jeff Johnston wrote:
> Agreed.  I have modified the patch to also change the fstat64 name to be
> unique and pushed to main.

Great!  For the newlib snapshot, let's wait a bit for a reply to the
patch from https://sourceware.org/pipermail/newlib/2026/022236.html


Corinna

> 
> -- Jeff J.
> 
> On Wed, Jan 21, 2026 at 4:21 AM Corinna Vinschen <[email protected]>
> wrote:
> 
> > On Jan 20 15:48, Jeff Johnston wrote:
> > > Likely, but I would rather this be tested first on multiple platforms.
> >
> > Alternatively we just use Andrew's patch.  It's the safe way forward,
> > I guess.
> >
> >
> > Corinna
> >
> >
> > >
> > > -- Jeff J.
> > >
> > > On Tue, Jan 20, 2026 at 1:43 PM Corinna Vinschen <[email protected]>
> > > wrote:
> > >
> > > > On Jan 20 11:55, Jeff Johnston wrote:
> > > > > IIRC, the dummy variables were to prevent the object files from being
> > > > empty
> > > > > and causing issues with one of the build tools we were using (likely
> > > > > libtool).
> > > >
> > > > Meaning, we don't need them anymore?
> > > >
> > > >
> > > > Corinna
> > > >
> > > >
> > > > >
> > > > > -- Jeff J.
> > > > >
> > > > > On Tue, Jan 20, 2026 at 4:47 AM Corinna Vinschen <
> > [email protected]>
> > > > > wrote:
> > > > >
> > > > > > On Jan 19 12:11, [email protected] wrote:
> > > > > > > From: Andrew Oates <[email protected]>
> > > > > > >
> > > > > > > I don't understand the purpose of these dummy variables, but they
> > > > date
> > > > > > > back to 2000 and the import of the 2000-02-17 snapshot.
> > > > > >
> > > > > > Me neither.  I defer to Jeff here.
> > > > > >
> > > > > >
> > > > > > Corinna
> > > > > >
> > > > > > > _dummy_link_syscalls is defined in two places, here and linkr.c.
> > > > This
> > > > > > > updates the symbol name to match the file, so we don't define the
> > > > same
> > > > > > > symbol in two places.
> > > > > > > ---
> > > > > > >  newlib/libc/reent/signalr.c | 2 +-
> > > > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > > > >
> > > > > > > diff --git a/newlib/libc/reent/signalr.c
> > > > b/newlib/libc/reent/signalr.c
> > > > > > > index 863ae7400..e4945102f 100644
> > > > > > > --- a/newlib/libc/reent/signalr.c
> > > > > > > +++ b/newlib/libc/reent/signalr.c
> > > > > > > @@ -17,7 +17,7 @@
> > > > > > >
> > > > > > >  #ifdef REENTRANT_SYSCALLS_PROVIDED
> > > > > > >
> > > > > > > -int _dummy_link_syscalls = 1;
> > > > > > > +int _dummy_signal_syscalls = 1;
> > > > > > >
> > > > > > >  #else
> > > > > > >
> > > > > > > --
> > > > > > > 2.52.0
> > > > > >
> > > > > >
> > > >
> > > >
> >
> >