errno.h: ESTRPIPE

[Ralf Corsepius]

Corinna Vinschen wrote:
> On Mar 16 08:29, Ralf Corsepius wrote:
>> Corinna Vinschen wrote:
>>> This error code is used in sound-related code which Yaakov is trying to
>>> port to Cygwin.  After a discussion on the cygwin-patches mailing list,
>>> we decided to add this error code.
>> Ouch ... bad decision.
>>
>>> Patch applied.
>> Please ifdef __CYGWIN__ it or revert it.
>>
>> newlib should stay with well established standards and not be polluted  
>> with underdocumented, proprietary extensions.
> 
> The error code is used at least in Linux and Solaris. 
It's not part of any POSIX or other standard I am aware about.

If there should be one, please point me to it.

> If we generate
> our own Cygwin-specific error codes we will at one point have a errno
> list where the same error code is used for different purposes on
> different systems.  I don't see that this gains anybody anything.

I don't have much of a problem with Cygwin wanting to adopt other OS's 
proprietary extensions. But this is not what your patch is doing.

Your patch is infecting newlib with Cygwin's wish to generalize ESTRIPE. 
This is wrong. ESTRIPE is a non-standardized and OS-specific extension, 
and therefore should be treated as such (i.e. #ifdef'ed).

C.f. the patch below.

Ralf


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: newlib-ESTRPIPE.diff
URL: <http://sourceware.org/pipermail/newlib/attachments/20090316/3a4d3315/attachment.ksh>