DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:content-type:mime-version:subject:from
	:in-reply-to:date:content-transfer-encoding:message-id
	:references:to; q=dns; s=default; b=NATMN1OIoV+KQIgxfxdHJr2+YToD
	5KXyKY/7RRp6S1LQFrTULHJxUrudQWmrXtvsTCuxPC3hFbwquzMQ1+Nfjjw7MrbC
	At6mUO4jRvRrj1K6H8Xg4Q8taaBZ1PTFOO084x3vkUccosuoWLySLT0u89PXu13o
	NrmbYcyc6V8fLxM=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\))
Subject: Re: Problem with new acl code and cdrtools
From: Warren Young <wyml@etr-usa.com>
In-Reply-To: <CAFo71_58PFDgaNU4-bdj4JoCndXAd=+i57w_eAgeq==RVcyoDw@mail.gmail.com>
Date: Tue, 9 Feb 2016 01:46:32 -0700
Message-Id: <BAF8618B-B333-4391-91B9-17174761484C@etr-usa.com>
References: <CAFo71_4fDB3WkhXTL7eybNzKh2idVF1UHb7oJjaxFshcS9BFUg@mail.gmail.com> <20160208131835.GC12975@calimero.vinschen.de> <CAFo71_5=KT=kJY-wz0EZWZ_LBriCjX2s6=5PSVhHRSxx-A92qg@mail.gmail.com> <56B8C25F.7050206@lysator.liu.se> <CAFo71_4eQ2VMypAnC=Bx_bvbzGvVCs=jj4r9Ax6GDbgaH1zGOQ@mail.gmail.com> <CA0C06F1-A50D-4DBF-AB58-AADCE3D9F9F4@etr-usa.com> <CAFo71_58PFDgaNU4-bdj4JoCndXAd=+i57w_eAgeq==RVcyoDw@mail.gmail.com>
To: The Cygwin Mailing List <cygwin@cygwin.com>
Content-Transfer-Encoding: 8bit

On Feb 9, 2016, at 12:56 AM, Ismail Donmez <ismail@i10z.com> wrote:
> 
> On Tue, Feb 9, 2016 at 6:14 AM, Warren Young <wyml@etr-usa.com> wrote:
>> On Feb 8, 2016, at 9:59 AM, Ismail Donmez <ismail@i10z.com> wrote:
>>> 
>>> Thats a hack :)
>> 
>> Peter’s proposal is as far from a hack as it gets.
> 
> What Peter suggests is telling autoconf to assume aclfromtext() does
> not exist on the system which is a hack and I already have a hack I
> could use.

No.  Once again, he’s suggesting that you write a *test* that checks whether aclfromtext() exists here.  Hypothesis -> test -> decision is science, not assumption or hackery.

>>   #include <config.h>
>>   #if HAVE_FUNC_ACLFROMTEXT
>>   #  include <cygwin/acl.h>
>>   #endif
> 
> I could check for __cygwin__ instead of that which would do the same thing.

Not if it’s Cygwin 1.7.34.
--
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