Re: [PATCH] libc: fix _dummy_signal_syscalls name in signalr.c
Corinna Vinschen <[email protected]> Tue, 20 Jan 2026 19:42:32 +0100
| Newsgroups | gmane.comp.lib.newlib |
|---|---|
| Message-ID | <[email protected]> |
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 > > > >