Re: [PATCH 2/7] Cygwin: Enable backward binary compatibility

Sebastian Huber <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
On 13/05/2022 22:15, Corinna Vinschen wrote:
> On May 13 15:46, Sebastian Huber wrote:
>> ---
>>   winsup/cygwin/include/cygwin/config.h | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/winsup/cygwin/include/cygwin/config.h b/winsup/cygwin/include/cygwin/config.h
>> index 71a216fbd..2d410a2e7 100644
>> --- a/winsup/cygwin/include/cygwin/config.h
>> +++ b/winsup/cygwin/include/cygwin/config.h
>> @@ -80,6 +80,7 @@ extern inline struct _reent *__getreent (void)
>>   #define __TM_GMTOFF tm_gmtoff
>>   #define __TM_ZONE   tm_zone
>>   #define _USE_LONG_TIME_T 1
>> +#define _NEWLIB_BACKWARD_BINARY_COMPAT 1
> Why?
> 
> AFAICS we don't expose _REENT to user space.  In theory (knocking
> on wood here), we shouldn't need _REENT binary compat.

Ok good, I always thought the unused members in struct _reent were 
mainly for Cygwin.

Maybe the option should be named --enable-newlib-reent-binary-compat and 
restricted to struct _reent.

-- 
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: [email protected]
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.