X-Recipient: archive-cygwin@delorie.com
X-SWARE-Spam-Status: No, hits=-0.7 required=5.0	tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,T_TO_NO_BRKTS_FREEMAIL
X-Spam-Check-By: sourceware.org
MIME-Version: 1.0
In-Reply-To: <20100817090816.GB16404@calimero.vinschen.de>
References: <AANLkTimQfeALLK=x_UvJYLjusd6ygCtjUKDrdA91e7Ej@mail.gmail.com>	<20100812225838.GB15724@ednor.casa.cgf.cx>	<4C647D32.1090907@redhat.com>	<AANLkTikCc0gdoVTWfhp=xoNY_m=Up4XQiApLmm9K9da+@mail.gmail.com>	<AANLkTin_VGPC-qfVROBhyFqcU0DBD5B8RAxLCF9RsNDC@mail.gmail.com>	<20100817090816.GB16404@calimero.vinschen.de>
Date: Tue, 17 Aug 2010 22:44:07 -0300
Message-ID: <AANLkTimnEpcPfYmV0WcM4RGp=wP6pPXp68bECSW4eeX2@mail.gmail.com>
Subject: Re: diff /usr/include/endian.orig.h /usr/include/endian.h > endian.h.diff
From: Pedro Izecksohn <pedro.izecksohn@gmail.com>
To: cygwin@cygwin.com
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie.com@cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com

--- Corinna Vinschen wrote:
>
> For this patch, given that it is just a bunch of rather obvious
> defines, I don't think we have to treat the patch as significant.

  I do not think that these macros are obvious. I think that I was
there when these macros were first implemented at 1987: I talked with
the programmer who was paid by the Correios to port everything from
the mainframe to the Apple II. I was a stenchy kid with his father,
that asked the employee what he was doing. When I asked about other
possibilities of endianness, the programmer answered about the PDP.
The programmer explained about BSD being a flavor of Unix. That talk
changed my life.

> However, please don't use _BSD_SOURCE. =C2=A0The newlib/Cygwin headers are
> not set up like the Linux headers and the big bunch of definitions from
> /usr/include/features.h are not available so far. =C2=A0The _BSD_SOURCE
> define is not honored or set anywhere right now.
>
> What you can do is to use _POSIX_SOURCE. =C2=A0So, if you're going to use
> #ifndef _POSIX_SOURCE instead, and if you're creating a ChangeLog entry
> in the correct ChangeLog entry format (see the ChangeLog file), then
> I guess we can check it in.

  Would you agree with:

#if defined _BSD_SOURCE || ! defined _POSIX_SOURCE

  ?

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

